Description of problem: At the moment, Web console (cockpit) is opening link with private address instead of public one. That makes cockpit seem to be unreachable. Version-Release number of selected component (if applicable): How reproducible: 100% Steps to Reproduce: 1.Add vm with cockpit enabled on AWS 2.press Web Console button on Instance detail in CFME 3. Actual results: We need a way to determine which ip to use in case vm has more then one. Expected results: Additional info:
https://github.com/ManageIQ/manageiq/pull/15303
https://github.com/ManageIQ/manageiq/pull/15901
https://github.com/ManageIQ/manageiq-ui-classic/pull/2039
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/0ade15960d6b0f448194f0ff198c877393740dd9 commit 0ade15960d6b0f448194f0ff198c877393740dd9 Author: Brian McLaughlin <bmclaugh> AuthorDate: Tue Aug 29 09:19:42 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Fri Sep 1 13:10:46 2017 -0400 Fix missing nil check https://bugzilla.redhat.com/show_bug.cgi?id=1450109 app/models/container_node.rb | 11 ++++---- app/models/vm/operations.rb | 10 +++++-- spec/models/vm/operations_spec.rb | 59 +++++++++++++++++++++++++++++++-------- 3 files changed, 61 insertions(+), 19 deletions(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/41b564d98adf5ad2317dec129a1910b0f6634d82 commit 41b564d98adf5ad2317dec129a1910b0f6634d82 Author: Brian McLaughlin <bmclaugh> AuthorDate: Tue Aug 29 09:22:52 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Fri Sep 1 13:10:46 2017 -0400 Make AWS and GCE connect on public ips for Web Console https://bugzilla.redhat.com/show_bug.cgi?id=1450109 app/models/vm/operations.rb | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/dc1731fc020d0947f47e30db1bac741071a489ca commit dc1731fc020d0947f47e30db1bac741071a489ca Author: Brian McLaughlin <bmclaugh> AuthorDate: Wed Aug 30 13:52:46 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Fri Sep 1 13:15:04 2017 -0400 Refactoring https://bugzilla.redhat.com/show_bug.cgi?id=1450109 https://bugzilla.redhat.com/show_bug.cgi?id=1451655 app/models/mixins/cockpit_mixin.rb | 10 ++++++++++ app/models/vm/operations.rb | 20 +++++--------------- 2 files changed, 15 insertions(+), 15 deletions(-) create mode 100644 app/models/mixins/cockpit_mixin.rb
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/1ca5c494b9c79cd5a7ebb209b862cd752a867f85 commit 1ca5c494b9c79cd5a7ebb209b862cd752a867f85 Author: Brian McLaughlin <bmclaugh> AuthorDate: Wed Aug 30 14:42:29 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Fri Sep 1 13:15:04 2017 -0400 Fix rubocop error https://bugzilla.redhat.com/show_bug.cgi?id=1450109 https://bugzilla.redhat.com/show_bug.cgi?id=1451655 app/models/mixins/cockpit_mixin.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/47bcea1e6bcba474142c48313f349ed0ee69f423 commit 47bcea1e6bcba474142c48313f349ed0ee69f423 Author: Brian McLaughlin <bmclaugh> AuthorDate: Thu Aug 31 14:25:27 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Fri Sep 1 13:15:04 2017 -0400 Refactoring https://bugzilla.redhat.com/show_bug.cgi?id=1450109 https://bugzilla.redhat.com/show_bug.cgi?id=1451655 app/models/vm/operations.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/106f7c597bac160e043fb5a4ccdc39e7b7c08bce commit 106f7c597bac160e043fb5a4ccdc39e7b7c08bce Author: Brian McLaughlin <bmclaugh> AuthorDate: Thu Aug 31 09:35:18 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Fri Sep 1 13:15:04 2017 -0400 Fix travis failures https://bugzilla.redhat.com/show_bug.cgi?id=1450109 https://bugzilla.redhat.com/show_bug.cgi?id=1451655 app/models/vm/operations.rb | 2 +- spec/models/vm/operations_spec.rb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-)
New commit detected on ManageIQ/manageiq/master: https://github.com/ManageIQ/manageiq/commit/ae845c60ef371312ce1d1172995a1bc8ebf4f8d0 commit ae845c60ef371312ce1d1172995a1bc8ebf4f8d0 Author: Brian McLaughlin <bmclaugh> AuthorDate: Fri Sep 1 13:13:42 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Fri Sep 1 13:13:42 2017 -0400 Conflict resolution https://bugzilla.redhat.com/show_bug.cgi?id=1450109 https://bugzilla.redhat.com/show_bug.cgi?id=1451655 app/models/container_node.rb | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-)
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/8deca8e75cba29cdff83d5003ebe1c557e46ccc8 commit 8deca8e75cba29cdff83d5003ebe1c557e46ccc8 Author: Brian McLaughlin <bmclaugh> AuthorDate: Wed Aug 30 13:54:52 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Wed Aug 30 13:54:52 2017 -0400 Refactoring https://bugzilla.redhat.com/show_bug.cgi?id=1450109 https://bugzilla.redhat.com/show_bug.cgi?id=1451655 app/controllers/vm_remote.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
It works fine in AWS and GCE. Verified in Version 5.9.0.20.20180206200727_d1873b9