Bug 1258397 - Should disable RHEV-M page after succeed setup Hosted Engine or provide a warning info
Summary: Should disable RHEV-M page after succeed setup Hosted Engine or provide a war...
Keywords:
Status: CLOSED EOL
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: ovirt-node-plugin-vdsm
Version: 3.5.4
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: ---
Assignee: Douglas Schilling Landgraf
QA Contact: wanghui
URL:
Whiteboard:
Depends On:
Blocks: 1316697
TreeView+ depends on / blocked
 
Reported: 2015-08-31 09:43 UTC by wanghui
Modified: 2022-04-16 09:05 UTC (History)
16 users (show)

Fixed In Version: ovirt-node-plugin-vdsm-0.6.1-6.el7ev
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1316697 (view as bug list)
Environment:
Last Closed: 2016-06-08 12:59:11 UTC
oVirt Team: Node
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHV-45721 0 None None None 2022-04-16 09:05:34 UTC
oVirt gerrit 49052 0 master MERGED Disable engine registration if hosted engine is configured 2016-01-13 16:16:33 UTC
oVirt gerrit 51800 0 ovirt-3.6 MERGED Disable engine registration if hosted engine is configured 2016-01-13 16:29:21 UTC
oVirt gerrit 51825 0 master MERGED Check for Hosted Engine in self.application.plugins 2016-01-15 13:04:47 UTC
oVirt gerrit 52370 0 ovirt-3.6 MERGED Check for Hosted Engine in self.application.plugins 2016-01-18 15:34:23 UTC
oVirt gerrit 53324 0 master MERGED ui: Hide registration if HE was run 2016-03-10 19:27:44 UTC
oVirt gerrit 54613 0 ovirt-3.6 MERGED ui: Hide registration if HE was run 2016-03-10 19:30:13 UTC

Description wanghui 2015-08-31 09:43:32 UTC
Description of problem:
It should disabled the RHEV-M page after succeed setup Hosted Engine or at least provide a warning info when people want to register to another rhevm.. Since rhevh can only be managed by one rhevm. So after setup Hosted Engine succeed, it should not allow to register to other RHEV-M or provide a warning info.

Version-Release number of selected component (if applicable):
rhev-hypervisor7-7.1-20150827.1.el7ev
ovirt-node-3.2.3-20.el7.noarch
ovirt-node-plugin-hosted-engine-0.2.0-18.0.el7ev.noarch

How reproducible:
100%

Steps to Reproduce:
1. Install rhev-hypervisor7-7.1-20150827.1.el7ev
2. Enabled network with dhcp type
3. Setup Hosted Engine with ova type
4. Check the RHEV-M page

Actual results:
1. RHEVM page still can be used as normal.

Expected results:
1. It should disabled the rhevm page in TUI or provide a warning infor when people want to register to another rhevm.

Additional info:

Comment 1 Fabian Deutsch 2015-09-01 09:55:05 UTC
I would not disable the page. But maybe we can set a flag or query a file if the setup completed successful and show that information.

But the solution also needs to handle failed attempts - IMO we should just ignore them, or hint that a cleanup is needed.

Comment 2 Ryan Barry 2015-09-02 00:40:07 UTC
For the RHEV-M page, it's easy to check the ha client to get enough information.

I would prefer not to handle any failed attempts of any kind in the TUI, including hinting that a cleanup is needed. There are many possible edge cases and failures, and reliably determining whether a cleanup is or isn't needed requires the TUI to know too much about the internals of ovirt-hosted-engine-setup.

Comment 4 wanghui 2016-02-03 07:56:27 UTC
Hi Fabian,

I notice node already has disabled the RHEVM page after HE is registered. But due to the bug#1301486, the change has been reverted.

So this issue is still exists.

Here is my suggestion, since we can not just disabled the RHEVM page simply, we can provide a warning info when people want to register to another rhevm.

Thanks,
Hui Wang

Comment 8 Mike McCune 2016-03-28 23:24:30 UTC
This bug was accidentally moved from POST to MODIFIED via an error in automation, please see mmccune with any questions

Comment 9 Fabian Deutsch 2016-03-30 14:49:43 UTC
Moving back to assigned, because the patches were incorrect and got reverted.

Comment 13 Fabian Deutsch 2016-06-08 12:59:11 UTC
There were no complaints so far on this item.

Closing this bug, please reopen if necessary.

However, this will be tackled in a different way in RHEV 4.


Note You need to log in before you can comment on or make changes to this bug.