Description of problem: The server role of Cockpit under Configuration has not taken into count, even if its not enabled, 'Web Console' is still accessible for VMs Version-Release number of selected component (if applicable): Version master.20171002075455_8547972 also seeing this in 5.9 nightly build we got How reproducible: 100% Steps to Reproduce: 1. Don't enable Cockpit Server role in configuration 2. Compute->Infra->Virtual Machine 3. 'Access'->'Web Console' Actual results: Web console option showed and working even if server role is disabled Expected results: Should require to first enable the Server role Additional info:
https://github.com/ManageIQ/manageiq-ui-classic/pull/2285
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/a6e0e68a8918704a40455727865c899528820c14 commit a6e0e68a8918704a40455727865c899528820c14 Author: Brian McLaughlin <bmclaugh> AuthorDate: Tue Oct 3 15:56:12 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Thu Oct 5 16:51:02 2017 -0400 Check for cockpit_ws server role to enable Web Console button https://bugzilla.redhat.com/show_bug.cgi?id=1497684 .../application_helper/button/cockpit_console.rb | 6 +- .../buttons/cockpit_console_spec.rb | 68 ++++++++++++++++------ 2 files changed, 53 insertions(+), 21 deletions(-)
New commit detected on ManageIQ/manageiq-ui-classic/master: https://github.com/ManageIQ/manageiq-ui-classic/commit/a48b91d77ff1f5d21dee46b5fec16236f3303c18 commit a48b91d77ff1f5d21dee46b5fec16236f3303c18 Author: Brian McLaughlin <bmclaugh> AuthorDate: Wed Oct 4 13:50:52 2017 -0400 Commit: Brian McLaughlin <bmclaugh> CommitDate: Thu Oct 5 16:51:45 2017 -0400 Seed MiqRegion to support checking for web console role https://bugzilla.redhat.com/show_bug.cgi?id=1497684 spec/controllers/container_node_controller_spec.rb | 1 + spec/controllers/vm_cloud_controller/trees_spec.rb | 1 + spec/controllers/vm_infra_controller/trees_spec.rb | 1 + spec/controllers/vm_infra_controller_spec.rb | 1 + 4 files changed, 4 insertions(+)
Able to verify this on 5.9.0.4
*** Bug 1469966 has been marked as a duplicate of this bug. ***