{"action":"created","issue":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/6685","repository_url":"https://api.github.com/repos/ManageIQ/manageiq","labels_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/6685/labels{/name}","comments_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/6685/comments","events_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/6685/events","html_url":"https://github.com/ManageIQ/manageiq/pull/6685","id":133950147,"number":6685,"title":"Generalize topology entity ids","user":{"login":"abonas","id":6277245,"avatar_url":"https://avatars.githubusercontent.com/u/6277245?v=3","gravatar_id":"","url":"https://api.github.com/users/abonas","html_url":"https://github.com/abonas","followers_url":"https://api.github.com/users/abonas/followers","following_url":"https://api.github.com/users/abonas/following{/other_user}","gists_url":"https://api.github.com/users/abonas/gists{/gist_id}","starred_url":"https://api.github.com/users/abonas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/abonas/subscriptions","organizations_url":"https://api.github.com/users/abonas/orgs","repos_url":"https://api.github.com/users/abonas/repos","events_url":"https://api.github.com/users/abonas/events{/privacy}","received_events_url":"https://api.github.com/users/abonas/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ManageIQ/manageiq/labels/providers/containers","name":"providers/containers","color":"5319e7"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-02-16T11:03:39Z","updated_at":"2016-02-16T13:48:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ManageIQ/manageiq/pulls/6685","html_url":"https://github.com/ManageIQ/manageiq/pull/6685","diff_url":"https://github.com/ManageIQ/manageiq/pull/6685.diff","patch_url":"https://github.com/ManageIQ/manageiq/pull/6685.patch"},"body":"queued behind #6661 \r\n\r\n@miq-bot add_label providers/containers\r\n\r\n@Fryguy this is per our discussion yesterday on different ids."},"comment":{"url":"https://api.github.com/repos/ManageIQ/manageiq/issues/comments/184688531","html_url":"https://github.com/ManageIQ/manageiq/pull/6685#issuecomment-184688531","issue_url":"https://api.github.com/repos/ManageIQ/manageiq/issues/6685","id":184688531,"user":{"login":"miq-bot","id":7806257,"avatar_url":"https://avatars.githubusercontent.com/u/7806257?v=3","gravatar_id":"","url":"https://api.github.com/users/miq-bot","html_url":"https://github.com/miq-bot","followers_url":"https://api.github.com/users/miq-bot/followers","following_url":"https://api.github.com/users/miq-bot/following{/other_user}","gists_url":"https://api.github.com/users/miq-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/miq-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miq-bot/subscriptions","organizations_url":"https://api.github.com/users/miq-bot/orgs","repos_url":"https://api.github.com/users/miq-bot/repos","events_url":"https://api.github.com/users/miq-bot/events{/privacy}","received_events_url":"https://api.github.com/users/miq-bot/received_events","type":"User","site_admin":false},"created_at":"2016-02-16T13:48:21Z","updated_at":"2016-02-16T13:48:21Z","body":"<rubocop />Checked commits https://github.com/abonas/manageiq/compare/7053aa21989cfdce99f9fa9de0f2fce2b7676b18~...095c914728d9d9bae6a22bfc1c98d70c8321e779 with ruby 2.2.3, rubocop 0.34.2, and haml-lint 0.13.0\n3 files checked, 49 offenses detected\n\n**app/services/container_topology_service.rb**\n- [ ] :small_blue_diamond: - [Line 1](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/app/services/container_topology_service.rb#L1), Col 1 - [Metrics/ClassLength](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/ClassLength) - Class has too many lines. [116/100]\n\n**app/services/topology_service.rb**\n- [ ] :large_orange_diamond: - [Line 5](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/app/services/topology_service.rb#L5), Col 9 - [Style/ExtraSpacing](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/ExtraSpacing) - Unnecessary spacing detected.\n- [ ] :large_orange_diamond: - [Line 23](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/app/services/topology_service.rb#L23), Col 24 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n\n**spec/services/container_topology_service_spec.rb**\n- [ ] :large_orange_diamond: - [Line 38](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L38), Col 51 - [Style/ExtraSpacing](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/ExtraSpacing) - Unnecessary spacing detected.\n- [ ] :large_orange_diamond: - [Line 70](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L70), Col 66 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 70](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L70), Col 71 - [Style/ExtraSpacing](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/ExtraSpacing) - Unnecessary spacing detected.\n- [ ] :large_orange_diamond: - [Line 112](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L112), Col 66 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 112](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L112), Col 71 - [Style/ExtraSpacing](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/ExtraSpacing) - Unnecessary spacing detected.\n- [ ] :large_orange_diamond: - [Line 129](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L129), Col 42 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 129](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L129), Col 100 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 130](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L130), Col 39 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 130](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L130), Col 94 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 132](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L132), Col 25 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 132](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L132), Col 60 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 133](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L133), Col 36 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 133](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L133), Col 76 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 134](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L134), Col 9 - [Style/BracesAroundHashParameters](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/BracesAroundHashParameters) - Redundant curly braces around a hash parameter.\n- [ ] :large_orange_diamond: - [Line 134](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L134), Col 39 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 134](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L134), Col 85 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 152](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L152), Col 9 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 152](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L152), Col 24 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 152](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L152), Col 47 - [Style/ExtraSpacing](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/ExtraSpacing) - Unnecessary spacing detected.\n- [ ] :large_orange_diamond: - [Line 159](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L159), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 160](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L160), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 161](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L161), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 162](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L162), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 165](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L165), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 166](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L166), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 167](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L167), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 168](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L168), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 171](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L171), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 172](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L172), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 173](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L173), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 174](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L174), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 177](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L177), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 178](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L178), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 179](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L179), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 180](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L180), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 181](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L181), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 184](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L184), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 185](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L185), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 186](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L186), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 187](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L187), Col 58 - [Style/AlignHash](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/AlignHash) - Align the elements of a hash literal if they span more than one line.\n- [ ] :large_orange_diamond: - [Line 192](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L192), Col 39 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 192](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L192), Col 94 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 194](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L194), Col 9 - [Style/BracesAroundHashParameters](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/BracesAroundHashParameters) - Redundant curly braces around a hash parameter.\n- [ ] :large_orange_diamond: - [Line 194](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L194), Col 36 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :large_orange_diamond: - [Line 194](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L194), Col 76 - [Style/SpaceAroundOperators](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Style/SpaceAroundOperators) - Surrounding space missing for operator `+`.\n- [ ] :small_blue_diamond: - [Line 129](https://github.com/abonas/manageiq/blob/095c914728d9d9bae6a22bfc1c98d70c8321e779/spec/services/container_topology_service_spec.rb#L129), Col 121 - [Metrics/LineLength](http://rubydoc.info/gems/rubocop/0.34.2/RuboCop/Cop/Metrics/LineLength) - Line is too long. [125/120]\n"}}