Bug 1451655

Summary:	Cockpit web console is not available for RHOS provider
Product:	Red Hat CloudForms Management Engine	Reporter:	Jiri Stefanisin <jistefan>
Component:	Providers	Assignee:	Brian McLaughlin <bmclaugh>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Niyaz Akhtar Ansari <nansari>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	5.8.0	CC:	cpelland, jfrey, jhardy, maufart, nansari, obarenbo, opavlenk, simaishi, vparekh
Target Milestone:	GA	Keywords:	TestOnly, ZStream
Target Release:	5.9.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:	cockpit
Fixed In Version:	5.9.0.1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1496903 (view as bug list)		Environment:
Last Closed:	2018-03-06 14:53:27 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	Openstack	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	1496903

Description Jiri Stefanisin 2017-05-17 08:55:19 UTC

Description of problem:
Even when cockpit is running on instance and it is possible to connect to it directly, CFME does not show Web Console option in Access dropbox on instance details screen.

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.try to access cockpit on instance hosted on RHOS
2.
3.

Actual results:


Expected results:


Additional info:

Comment 3 Dave Johnson 2017-07-14 03:50:32 UTC

Please assess the importance of this issue and update the priority accordingly.  Somewhere it was missed in the bug triage process.  Please refer to https://bugzilla.redhat.com/page.cgi?id=fields.html#priority for a reminder on each priority's definition.

If it's something like a tracker bug where it doesn't matter, please set it to Low/Low.

Comment 5 CFME Bot 2017-08-29 14:00:35 UTC

https://github.com/ManageIQ/manageiq-ui-classic/pull/2039

Comment 6 CFME Bot 2017-08-29 14:07:50 UTC

https://github.com/ManageIQ/manageiq/pull/15901

Comment 7 CFME Bot 2017-09-01 18:21:50 UTC

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/1e1e0b3d0adca5787a38385e2e2facc4fd600777

commit 1e1e0b3d0adca5787a38385e2e2facc4fd600777
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Tue Aug 29 09:25:41 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Fri Sep 1 13:10:46 2017 -0400

    Enable Web Console on RHOS provider
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1451655

 app/models/vm/operations.rb | 2 ++
 1 file changed, 2 insertions(+)

Comment 8 CFME Bot 2017-09-01 18:22:01 UTC

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

Comment 9 CFME Bot 2017-09-01 18:22:10 UTC

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(-)

Comment 10 CFME Bot 2017-09-01 18:22:21 UTC

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(-)

Comment 11 CFME Bot 2017-09-01 18:22:30 UTC

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(-)

Comment 12 CFME Bot 2017-09-01 18:22:41 UTC

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(-)

Comment 13 CFME Bot 2017-09-01 18:38:54 UTC

New commit detected on ManageIQ/manageiq-ui-classic/master:
https://github.com/ManageIQ/manageiq-ui-classic/commit/b94715b8b3c66a05ddddecdc15c6117629b8fab5

commit b94715b8b3c66a05ddddecdc15c6117629b8fab5
Author:     Brian McLaughlin <bmclaugh>
AuthorDate: Tue Aug 29 09:36:12 2017 -0400
Commit:     Brian McLaughlin <bmclaugh>
CommitDate: Wed Aug 30 09:38:20 2017 -0400

    Enable Cockpit console for RHOS provider
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1451655

 .../toolbar/cloud/instance_operations_button_group_mixin.rb      | 9 +++++++++
 1 file changed, 9 insertions(+)

Comment 14 CFME Bot 2017-09-01 18:39:04 UTC

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(-)

Comment 16 Vatsal Parekh 2017-10-10 07:02:45 UTC

Able to verify this on Version 5.9.0.1.20171004215954_13427ce