{"action":"created","issue":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/14709","repository_url":"https://api.github.com/repos/hashicorp/terraform","labels_url":"https://api.github.com/repos/hashicorp/terraform/issues/14709/labels{/name}","comments_url":"https://api.github.com/repos/hashicorp/terraform/issues/14709/comments","events_url":"https://api.github.com/repos/hashicorp/terraform/issues/14709/events","html_url":"https://github.com/hashicorp/terraform/pull/14709","id":230217551,"number":14709,"title":"provider/vcd: Add port mapping to DNAT rule","user":{"login":"ewypych","id":25039172,"avatar_url":"https://avatars3.githubusercontent.com/u/25039172?v=4","gravatar_id":"","url":"https://api.github.com/users/ewypych","html_url":"https://github.com/ewypych","followers_url":"https://api.github.com/users/ewypych/followers","following_url":"https://api.github.com/users/ewypych/following{/other_user}","gists_url":"https://api.github.com/users/ewypych/gists{/gist_id}","starred_url":"https://api.github.com/users/ewypych/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ewypych/subscriptions","organizations_url":"https://api.github.com/users/ewypych/orgs","repos_url":"https://api.github.com/users/ewypych/repos","events_url":"https://api.github.com/users/ewypych/events{/privacy}","received_events_url":"https://api.github.com/users/ewypych/received_events","type":"User","site_admin":false},"labels":[{"id":85946959,"url":"https://api.github.com/repos/hashicorp/terraform/labels/enhancement","name":"enhancement","color":"d4c5f9","default":true},{"id":310797860,"url":"https://api.github.com/repos/hashicorp/terraform/labels/provider/vcloud","name":"provider/vcloud","color":"bfdadc","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-05-21T12:29:41Z","updated_at":"2017-11-01T19:29:31Z","closed_at":"2017-11-01T19:29:31Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/hashicorp/terraform/pulls/14709","html_url":"https://github.com/hashicorp/terraform/pull/14709","diff_url":"https://github.com/hashicorp/terraform/pull/14709.diff","patch_url":"https://github.com/hashicorp/terraform/pull/14709.patch"},"body":"Hello!\r\nProvider vDC allows to create a DNAT rule on the Edge Gateway, but now one can make only one to one port mapping. As a service provider employee, I know that people often use only one public IP address with many local subnets.  For example:\r\n\r\n```\r\nexternal_ip:3040 <-> 10.0.0.10:80\r\nexternal_ip:3041 <-> 10.0.0.11:80\r\nexternal_ip:3042 <-> 10.0.1.10:80\r\n```\r\nvCloud Air API bindings for Golang allows to create a full port mapping, so I made some changes and replaced one to one mapping with the full DNAT port mapping. Even if someone wants to create one to one mapping, it is still possible - one just needs to type two variables instead of one, but with the same value.\r\n\r\nI think it's a lot more useful with a full support for DNAT port mapping. :smile: "},"comment":{"url":"https://api.github.com/repos/hashicorp/terraform/issues/comments/341213766","html_url":"https://github.com/hashicorp/terraform/pull/14709#issuecomment-341213766","issue_url":"https://api.github.com/repos/hashicorp/terraform/issues/14709","id":341213766,"user":{"login":"apparentlymart","id":20180,"avatar_url":"https://avatars1.githubusercontent.com/u/20180?v=4","gravatar_id":"","url":"https://api.github.com/users/apparentlymart","html_url":"https://github.com/apparentlymart","followers_url":"https://api.github.com/users/apparentlymart/followers","following_url":"https://api.github.com/users/apparentlymart/following{/other_user}","gists_url":"https://api.github.com/users/apparentlymart/gists{/gist_id}","starred_url":"https://api.github.com/users/apparentlymart/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apparentlymart/subscriptions","organizations_url":"https://api.github.com/users/apparentlymart/orgs","repos_url":"https://api.github.com/users/apparentlymart/repos","events_url":"https://api.github.com/users/apparentlymart/events{/privacy}","received_events_url":"https://api.github.com/users/apparentlymart/received_events","type":"User","site_admin":false},"created_at":"2017-11-01T19:29:31Z","updated_at":"2017-11-01T19:29:31Z","author_association":"CONTRIBUTOR","body":"Hi @ewypych! Thanks for working on this.\r\n\r\nIt looks like the other PR you mentioned was eventually migrated to terraform-providers/terraform-provider-vcd#19 as part of separating this and other providers from Terraform Core, and that change was eventually merged.\r\n\r\nSince this change can no longer apply to this repository, I'm going to close it. Thanks again for working on this!"}}
{"actor":{"display_login":"apparentlymart"}}