{"action":"created","issue":{"url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/issues/805","repository_url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker","labels_url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/issues/805/labels{/name}","comments_url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/issues/805/comments","events_url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/issues/805/events","html_url":"https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/805","id":126941683,"number":805,"title":"openstack: Make AllowPort open not just TCP but UDP ports as well","user":{"login":"meteorfox","id":232300,"avatar_url":"https://avatars.githubusercontent.com/u/232300?v=3","gravatar_id":"","url":"https://api.github.com/users/meteorfox","html_url":"https://github.com/meteorfox","followers_url":"https://api.github.com/users/meteorfox/followers","following_url":"https://api.github.com/users/meteorfox/following{/other_user}","gists_url":"https://api.github.com/users/meteorfox/gists{/gist_id}","starred_url":"https://api.github.com/users/meteorfox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/meteorfox/subscriptions","organizations_url":"https://api.github.com/users/meteorfox/orgs","repos_url":"https://api.github.com/users/meteorfox/repos","events_url":"https://api.github.com/users/meteorfox/events{/privacy}","received_events_url":"https://api.github.com/users/meteorfox/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/labels/cla:%20yes","name":"cla: yes","color":"009800"},{"url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/labels/ready%20for%20review","name":"ready for review","color":"fbca04"}],"state":"open","locked":false,"assignee":{"login":"meteorfox","id":232300,"avatar_url":"https://avatars.githubusercontent.com/u/232300?v=3","gravatar_id":"","url":"https://api.github.com/users/meteorfox","html_url":"https://github.com/meteorfox","followers_url":"https://api.github.com/users/meteorfox/followers","following_url":"https://api.github.com/users/meteorfox/following{/other_user}","gists_url":"https://api.github.com/users/meteorfox/gists{/gist_id}","starred_url":"https://api.github.com/users/meteorfox/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/meteorfox/subscriptions","organizations_url":"https://api.github.com/users/meteorfox/orgs","repos_url":"https://api.github.com/users/meteorfox/repos","events_url":"https://api.github.com/users/meteorfox/events{/privacy}","received_events_url":"https://api.github.com/users/meteorfox/received_events","type":"User","site_admin":false},"milestone":null,"comments":3,"created_at":"2016-01-15T19:36:22Z","updated_at":"2016-02-24T18:43:05Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/pulls/805","html_url":"https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/805","diff_url":"https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/805.diff","patch_url":"https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/805.patch"},"body":"Re-factor AllowPort() to follow the implementations of other provider\r\nthat open both TCP and UDP ports. When static VMs are used, AllowPort()\r\nbecomes a NO-OP.\r\n\r\nIntroduce lock to avoid race condition when checking/creating security\r\ngroup, and security group rules.\r\n\r\nAddresses issue #804"},"comment":{"url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/issues/comments/188400309","html_url":"https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/805#issuecomment-188400309","issue_url":"https://api.github.com/repos/GoogleCloudPlatform/PerfKitBenchmarker/issues/805","id":188400309,"user":{"login":"skschneider","id":11166925,"avatar_url":"https://avatars.githubusercontent.com/u/11166925?v=3","gravatar_id":"","url":"https://api.github.com/users/skschneider","html_url":"https://github.com/skschneider","followers_url":"https://api.github.com/users/skschneider/followers","following_url":"https://api.github.com/users/skschneider/following{/other_user}","gists_url":"https://api.github.com/users/skschneider/gists{/gist_id}","starred_url":"https://api.github.com/users/skschneider/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/skschneider/subscriptions","organizations_url":"https://api.github.com/users/skschneider/orgs","repos_url":"https://api.github.com/users/skschneider/repos","events_url":"https://api.github.com/users/skschneider/events{/privacy}","received_events_url":"https://api.github.com/users/skschneider/received_events","type":"User","site_admin":false},"created_at":"2016-02-24T18:43:05Z","updated_at":"2016-02-24T18:43:05Z","body":"What's the status of this PR?\r\n@cmccoy, I see you've posted your +1, which as I understand,\r\n+1 = `:+1:` = :+1: = LGTM.\r\n@meteorfox, I believe that as a contributor in this github project you have the ability to merge PRs into master, which is usually fine to do after an LGTM.\r\nAre we still waiting for something? Does CONTRIBUTING.md need to be fleshed out regarding the merge process? Or has this PR just slipped through the cracks?"}}