{"action":"created","issue":{"url":"https://api.github.com/repos/jclouds/jclouds-labs-google/issues/138","labels_url":"https://api.github.com/repos/jclouds/jclouds-labs-google/issues/138/labels{/name}","comments_url":"https://api.github.com/repos/jclouds/jclouds-labs-google/issues/138/comments","events_url":"https://api.github.com/repos/jclouds/jclouds-labs-google/issues/138/events","html_url":"https://github.com/jclouds/jclouds-labs-google/pull/138","id":70836700,"number":138,"title":"WIP: JCLOUDS-892 Compute Abstraction Network and Firewall changes","user":{"login":"danbroudy","id":8703125,"avatar_url":"https://avatars.githubusercontent.com/u/8703125?v=3","gravatar_id":"","url":"https://api.github.com/users/danbroudy","html_url":"https://github.com/danbroudy","followers_url":"https://api.github.com/users/danbroudy/followers","following_url":"https://api.github.com/users/danbroudy/following{/other_user}","gists_url":"https://api.github.com/users/danbroudy/gists{/gist_id}","starred_url":"https://api.github.com/users/danbroudy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danbroudy/subscriptions","organizations_url":"https://api.github.com/users/danbroudy/orgs","repos_url":"https://api.github.com/users/danbroudy/repos","events_url":"https://api.github.com/users/danbroudy/events{/privacy}","received_events_url":"https://api.github.com/users/danbroudy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-04-25T01:31:34Z","updated_at":"2015-04-30T21:52:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jclouds/jclouds-labs-google/pulls/138","html_url":"https://github.com/jclouds/jclouds-labs-google/pull/138","diff_url":"https://github.com/jclouds/jclouds-labs-google/pull/138.diff","patch_url":"https://github.com/jclouds/jclouds-labs-google/pull/138.patch"},"body":"This is a WIP branch. \r\n\r\nThis branch changes the compute abstraction to use the default network when creating nodes in a group. A user can still create nodes in a different network by creating a network and then passing in the URI through `GoogleComputeEngineTemplateOptions`.\r\n\r\nNode deletion and clean up makes no attempt to delete networks. This change encourages users to use the default network. If users want to use a different network then they are in charge of creating and destroying the network.\r\n\r\nThis PR also addresses the issue of creating one firewall rule per inbound port. \r\n\r\nThere is still work to be done. \r\nWDYT?"},"comment":{"url":"https://api.github.com/repos/jclouds/jclouds-labs-google/issues/comments/97981986","html_url":"https://github.com/jclouds/jclouds-labs-google/pull/138#issuecomment-97981986","issue_url":"https://api.github.com/repos/jclouds/jclouds-labs-google/issues/138","id":97981986,"user":{"login":"danbroudy","id":8703125,"avatar_url":"https://avatars.githubusercontent.com/u/8703125?v=3","gravatar_id":"","url":"https://api.github.com/users/danbroudy","html_url":"https://github.com/danbroudy","followers_url":"https://api.github.com/users/danbroudy/followers","following_url":"https://api.github.com/users/danbroudy/following{/other_user}","gists_url":"https://api.github.com/users/danbroudy/gists{/gist_id}","starred_url":"https://api.github.com/users/danbroudy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danbroudy/subscriptions","organizations_url":"https://api.github.com/users/danbroudy/orgs","repos_url":"https://api.github.com/users/danbroudy/repos","events_url":"https://api.github.com/users/danbroudy/events{/privacy}","received_events_url":"https://api.github.com/users/danbroudy/received_events","type":"User","site_admin":false},"created_at":"2015-04-30T21:52:52Z","updated_at":"2015-04-30T21:52:52Z","body":"Turns out there was a problem extracting the group name from firewall rules with names like `<group-name>-port-<hash> `. I removed port from the name and that should fix the problem. I also updated the mock test that was failing for a similar reason. \r\n\r\nI will update this again later this afternoon with a few more fixes. "}}