Bug 1367924

Summary: [RFE] Improve HE setup indications in UI
Product: Red Hat Enterprise Virtualization Manager Reporter: Marina Kalinin <mkalinin>
Component: RFEsAssignee: Andrej Krejcir <akrejcir>
Status: CLOSED ERRATA QA Contact: Nikolai Sednev <nsednev>
Severity: high Docs Contact:
Priority: high    
Version: 3.6.8CC: bgraveno, bugs, cshao, dfediuck, dguo, gklein, lsurette, mavital, michal.skrivanek, mkalinin, phbailey, rbalakri, sherold, srevivo, ycui, ykaul, ylavi, yzhao
Target Milestone: ovirt-4.1.0-betaKeywords: FutureFeature, Improvement, Triaged
Target Release: ---Flags: gklein: testing_plan_complete+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: PM-09
Fixed In Version: Doc Type: Enhancement
Doc Text:
Multiple updates were made to the UI for Self-hosted Engine. New icons have been added: - To virtual machines to indicate whether it is the Manager virtual machine. - To hosts to indicate whether it can run the Manager virtual machine. - To storage domains to indicate whether it contains the Manager virtual machine. Buttons to enable and disable global maintenance mode have been moved to the context menu of a host that can run the Manager virtual machine. Depending on the current status of global maintenance mode, either the enable or disable option will be enabled.
 Story Points: ---
Clone Of: 1331498 Environment:
Last Closed: 2017-04-25 00:46:30 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: SLA RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1331498    
Bug Blocks: 902971    
Attachments:
Description Flags
hosts - can run he vm.png
none
hosts - running he vm.png
none
storage crown contains he vm data.png
none
VMs is HE-vm crown.png
none
enable global maintenance for active host.png
none
disable global maintenance when its enabled.png
none
he ha is in global maintenance.png
none
he host with active score 3400.png
none
sowtware tab lacks for HE components.png
none
unavailable due to ha score.png none

Description Marina Kalinin 2016-08-17 19:56:08 UTC 
+++ This bug was initially created as a clone of Bug #1331498 +++

Provide an indication to HE VM that it is HE VM. 

Right now, there is no indication on HE VM that it is the manager. But having it would be extremely useful. Especially for the support team, working with customer backups and understanding customers' setups.

--- Additional comment from Marina on 2016-04-28 12:59:10 EDT ---

Another, less optimal, option is to make sure rhevm appears in the list of applications of that VM. But this will also require running guest agent on the manager. And if the agent is not installed, that information would be unavailable.

--- Additional comment from Michal Skrivanek on 2016-04-29 02:57:08 EDT ---

(In reply to Marina from comment #1)

what kind of indication do you have in mind for the ideal case?

> Another, less optimal, option is to make sure rhevm appears in the list of
> applications of that VM.

that would be easy to add

> But this will also require running guest agent on
> the manager. And if the agent is not installed, that information would be
> unavailable.

the guest agent should be a mandatory running service for HE, if it is not already

--- Additional comment from Marina on 2016-04-29 16:40:23 EDT ---

(In reply to Michal Skrivanek from comment #2)
> (In reply to Marina from comment #1)
> 
> what kind of indication do you have in mind for the ideal case?
Something in the UI, that once I look into it, I'd be able knowing right away which vm is the hosted engine. Maybe like the crown we have for the mgmt logical network?

> 
> > Another, less optimal, option is to make sure rhevm appears in the list of
> > applications of that VM.
> 
> that would be easy to add
> 
> > But this will also require running guest agent on
> > the manager. And if the agent is not installed, that information would be
> > unavailable.
> 
> the guest agent should be a mandatory running service for HE, if it is not
> already
I don't know.
I went through the HE guide[1] and I didn't find how to configure non-appliance RHEV-M. So, I could not see anywhere that the agent is required.


[1] https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Virtualization/3.6/html/Self-Hosted_Engine_Guide/chap-Introduction.html

--- Additional comment from Marina on 2016-05-17 14:51:13 EDT ---

Another scenario when such indication is needed: many times in the guides we suggest to put HE VM into maintenance through the admin portal. But if the VM does not have the default name "HostedEngine", which is quite possible, and if it is a new sys admin managing that environment, that person would have no clue which VM to use to set the maintenance through UI.

--- Additional comment from Marina on 2016-05-18 17:18:36 EDT ---

On the same note, please tell me if I should open another RFE for this or you can handle this here:
It should also indicate if HE setup in global maintenance or not. Right now there is no indication and both options are available:
- Enable Global HA maintenance
- Disable Global HA maintenance.
And seems like we can click on them both multiple times.

I think we should gray out the option that is not available.

--- Additional comment from Marina on 2016-05-18 17:19:28 EDT ---

(In reply to Marina from comment #5)
> On the same note, please tell me if I should open another RFE for this or
> you can handle this here:
> It should also indicate if HE setup in global maintenance or not. Right now
> there is no indication and both options are available:
> - Enable Global HA maintenance
> - Disable Global HA maintenance.
> And seems like we can click on them both multiple times.
> 
> I think we should gray out the option that is not available.

And add indication of the status on the portal, together with the indication of the HE VM.

--- Additional comment from Marina on 2016-05-23 11:18:24 EDT ---

We need to extend this RFE to the hosts and cluster as well.


--- Additional comment from Marina on 2016-06-01 17:41:42 EDT ---

Another scenario - in case of RHEV-H, there is no way to know if it is a hosted engine setup or not, since no UI indication + ha packages are installed on RHEV-H by default.

--- Additional comment from Marina on 2016-06-21 10:33:57 EDT ---

Changing bug title to a more global name.

Currently, UI is lacking any notification on HE setup.
- HE VM should have indicator
- HE hosts/cluster should have indicators
- [HE storage?]
- HA packages should be presented under software information on the host
- Global/local maintenance indication is missing
and more.

--- Additional comment from Michal Skrivanek on 2016-06-24 06:23:24 EDT ---

moving to Integration since the RFE is about improving global HE experience
Comment 4 Nikolai Sednev 2016-12-28 13:52:40 UTC 
Works for me on these components on hosts:
ovirt-vmconsole-host-1.0.4-1.el7ev.noarch
mom-0.5.8-1.el7ev.noarch
ovirt-hosted-engine-setup-2.1.0-0.0.master.20161221071755.git46cacd3.el7.centos.noarch
ovirt-setup-lib-1.1.0-1.el7.centos.noarch
libvirt-client-2.0.0-10.el7_3.2.x86_64
ovirt-release41-pre-4.1.0-0.6.beta2.20161221025826.gitc487776.el7.centos.noarch
ovirt-vmconsole-1.0.4-1.el7ev.noarch
qemu-kvm-rhev-2.6.0-28.el7_3.2.x86_64
ovirt-hosted-engine-ha-2.1.0-0.0.master.20161221070856.20161221070854.git387fa53.el7.centos.noarch
rhevm-appliance-20161116.0-1.el7ev.noarch
sanlock-3.4.0-1.el7.x86_64
ovirt-host-deploy-1.6.0-0.0.master.20161215101008.gitb76ad50.el7.centos.noarch
ovirt-engine-sdk-python-3.6.9.1-1.el7ev.noarch
ovirt-imageio-common-0.5.0-0.201611201242.gitb02532b.el7.centos.noarch
vdsm-4.18.999-1218.gitd36143e.el7.centos.x86_64
ovirt-imageio-daemon-0.5.0-0.201611201242.gitb02532b.el7.centos.noarch
Linux version 3.10.0-514.2.2.el7.x86_64 (mockbuild.eng.bos.redhat.com) (gcc version 4.8.5 20150623 (Red Hat 4.8.5-11) (GCC) ) #1 SMP Wed Nov 16 13:15:13 EST 2016
Linux 3.10.0-514.2.2.el7.x86_64 #1 SMP Wed Nov 16 13:15:13 EST 2016 x86_64 x86_64 x86_64 GNU/Linux
Red Hat Enterprise Linux Server release 7.3 (Maipo)

On engine:
ovirt-engine-setup-plugin-ovirt-engine-common-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-imageio-proxy-0.5.0-0.201611201242.gitb02532b.el7.centos.noarch
ovirt-iso-uploader-4.1.0-0.0.master.20160909154152.git14502bd.el7.centos.noarch
ovirt-engine-userportal-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-dbscripts-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-setup-plugin-vmconsole-proxy-helper-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-extensions-api-impl-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-imageio-common-0.5.0-0.201611201242.gitb02532b.el7.centos.noarch
ovirt-host-deploy-1.6.0-0.0.master.20161215101008.gitb76ad50.el7.centos.noarch
python-ovirt-engine-sdk4-4.1.0-0.1.a0.20161215git77fce51.el7.centos.x86_64
ovirt-host-deploy-java-1.6.0-0.0.master.20161215101008.gitb76ad50.el7.centos.noarch
ovirt-release41-pre-4.1.0-0.6.beta2.20161221025826.gitc487776.el7.centos.noarch
ovirt-setup-lib-1.1.0-1.el7.centos.noarch
ovirt-engine-extension-aaa-jdbc-1.1.2-1.el7.noarch
ovirt-engine-dwh-setup-4.1.0-0.0.master.20161129154019.el7.centos.noarch
ovirt-imageio-proxy-setup-0.5.0-0.201611201242.gitb02532b.el7.centos.noarch
ovirt-engine-tools-backup-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-websocket-proxy-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-setup-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-backend-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-tools-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-webadmin-portal-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-restapi-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-vmconsole-proxy-helper-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-setup-plugin-ovirt-engine-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-wildfly-overlay-10.0.0-1.el7.noarch
ovirt-engine-cli-3.6.9.2-1.el7.centos.noarch
ovirt-web-ui-0.1.1-2.el7.centos.x86_64
ovirt-engine-setup-base-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-vmconsole-1.0.4-1.el7.centos.noarch
ovirt-engine-dwh-4.1.0-0.0.master.20161129154019.el7.centos.noarch
ovirt-engine-setup-plugin-websocket-proxy-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-hosts-ansible-inventory-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-engine-dashboard-1.1.0-0.4.20161128git5ed6f96.el7.centos.noarch
ovirt-engine-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-guest-agent-common-1.0.13-1.20161220085008.git165fff1.el7.centos.noarch
ovirt-engine-sdk-python-3.6.9.1-1.el7.centos.noarch
ovirt-engine-wildfly-10.1.0-1.el7.x86_64
ovirt-engine-lib-4.1.0-0.3.beta2.20161221085908.el7.centos.noarch
ovirt-vmconsole-proxy-1.0.4-1.el7.centos.noarch
Linux version 3.10.0-514.2.2.el7.x86_64 (builder.centos.org) (gcc version 4.8.5 20150623 (Red Hat 4.8.5-11) (GCC) ) #1 SMP Tue Dec 6 23:06:41 UTC 2016
Linux 3.10.0-514.2.2.el7.x86_64 #1 SMP Tue Dec 6 23:06:41 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
CentOS Linux release 7.3.1611 (Core)
Comment 5 Nikolai Sednev 2016-12-28 13:53:30 UTC 
Created attachment 1235641 [details]
hosts - can run he vm.png
Comment 6 Nikolai Sednev 2016-12-28 13:54:07 UTC 
Created attachment 1235642 [details]
hosts - running he vm.png
Comment 7 Nikolai Sednev 2016-12-28 13:54:56 UTC 
Created attachment 1235643 [details]
storage crown contains he vm data.png
Comment 8 Nikolai Sednev 2016-12-28 13:55:48 UTC 
Created attachment 1235653 [details]
VMs is HE-vm crown.png
Comment 9 Nikolai Sednev 2016-12-28 14:11:03 UTC 
Currently, UI capabilities were improved to notify that:
1)- HE VM has an indicator.
2)- HE hosts have indicators (HE is not limited on cluster level).
3)- HE storage domain has an indicator.
4)- HA packages should be presented under software information on the host - Still missing and not yet implemented.
5)- Global/local maintenance indication is missing-Partially implemented.
Please see the attachments, in which you still can't clearly see that host is in local maintenance, because regular maintenance functions as local maintenance for HE hosts, but global maintenance can be clearly seen now, please see the attachments.

Please clarify regarding requirements 4 and 5.
Comment 10 Nikolai Sednev 2016-12-28 14:11:36 UTC 
Created attachment 1235659 [details]
enable global maintenance for active host.png
Comment 11 Nikolai Sednev 2016-12-28 14:12:08 UTC 
Created attachment 1235660 [details]
disable global maintenance when its enabled.png
Comment 12 Nikolai Sednev 2016-12-28 14:12:56 UTC 
Created attachment 1235661 [details]
he ha is in global maintenance.png
Comment 13 Nikolai Sednev 2016-12-28 14:13:29 UTC 
Created attachment 1235662 [details]
he host with active score 3400.png
Comment 14 Nikolai Sednev 2016-12-28 14:14:36 UTC 
Created attachment 1235663 [details]
sowtware tab lacks for HE components.png
Comment 15 Nikolai Sednev 2016-12-28 14:15:15 UTC 
Created attachment 1235664 [details]
unavailable due to ha score.png
Comment 16 Nikolai Sednev 2016-12-28 14:19:59 UTC 
Please provide your input regarding Comment #9.
I see that requirements #4 and 5 were not implemented in UI as required.
Please see the attachments from screenshots from UI.
Comment 17 Yaniv Lavi 2016-12-28 16:05:55 UTC 
(In reply to Nikolai Sednev from comment #16)
> Please provide your input regarding Comment #9.
> I see that requirements #4 and 5 were not implemented in UI as required.
> Please see the attachments from screenshots from UI.

4- will be dealt like any other host package with upgrade manager with RHEL-H\RHV-H in 4.2. So not a bug issue, you can verify without this.

5- If you are only using the UI\API local maintenance is when the host in general maintenance for both VM and HE VM. Global maintenance is indeed much improved with the state aware buttons and the field in each HE host that is now clearly marked.
Comment 19 Marina Kalinin 2016-12-28 20:12:55 UTC 
Beautiful job, Phillip and Nikolai.
Thank you!

I think I am fine with #5 - indeed so many improvements.
I am not sure what Yaniv meant for #4 for though.
I do think it is important information to have, however with the other fixes in this bug, I can survive without it.
Comment 20 Nikolai Sednev 2016-12-29 08:55:52 UTC 
Moving to verified then, forth to comment #4 to #19.
Comment 21 Phillip Bailey 2017-01-12 13:26:20 UTC 
(In reply to Marina from comment #19)
> Beautiful job, Phillip and Nikolai.
> Thank you!
> 
Thanks Marina, but I can't take credit. I helped a very small amount, but this was Andrej's work. He deserves all the credit. =)