Bug 1331498 - [Trackers][RFE] - Improve HE setup indications in UI
Summary: [Trackers][RFE] - Improve HE setup indications in UI
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-engine
Classification: oVirt
Component: RFEs
Version: future
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ovirt-4.1.0-beta
: 4.1.0
Assignee: Andrej Krejcir
QA Contact: meital avital
URL:
Whiteboard: PM-09
Depends On: 1369827 1392389 1392407 1392412 1392418
Blocks: 902971 1367924
TreeView+ depends on / blocked
 
Reported: 2016-04-28 16:57 UTC by Marina Kalinin
Modified: 2017-03-30 01:14 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Feature: This is a tracker feature to improve various indicators about the hosted engine in the UI. Reason: The information shown by this feature was hard to get otherwise. Result: New icons were added: - for VMs, indicating if it is the hosted engine - for hosts, showing if it can run the hosted engine - for storage domains, indicating if it contains the hosted engine disk Buttons for enabling and disabling hosted engine global maintenance mode were changed: - they were moved to the context menu of a host that can run the hosted engine - only the relevant button is enabled
Clone Of:
: 1367924 (view as bug list)
Environment:
Last Closed: 2017-03-16 14:48:26 UTC
oVirt Team: SLA
Embargoed:
rule-engine: ovirt-4.1+
mavital: testing_plan_complete-
ylavi: planning_ack+
rule-engine: devel_ack+
mavital: testing_ack+


Attachments (Terms of Use)

Description Marina Kalinin 2016-04-28 16:57:10 UTC
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.

Comment 1 Marina Kalinin 2016-04-28 16:59:10 UTC
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.

Comment 2 Michal Skrivanek 2016-04-29 06:57:08 UTC
(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

Comment 3 Marina Kalinin 2016-04-29 20:40:23 UTC
(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

Comment 4 Marina Kalinin 2016-05-17 18:51:13 UTC
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.

Comment 5 Marina Kalinin 2016-05-18 21:18:36 UTC
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.

Comment 6 Marina Kalinin 2016-05-18 21:19:28 UTC
(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.

Comment 7 Marina Kalinin 2016-05-23 15:18:24 UTC
We need to extend this RFE to the hosts and cluster as well.

Comment 9 Marina Kalinin 2016-06-01 21:41:42 UTC
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.

Comment 10 Marina Kalinin 2016-06-21 14:33:57 UTC
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.

Comment 12 Michal Skrivanek 2016-06-24 10:23:24 UTC
moving to Integration since the RFE is about improving global HE experience

Comment 17 Yaniv Lavi 2016-09-20 13:25:55 UTC
Action item is to discuss this with integration and UX team to make sure all aspects of the CLI are represented in the UI and that actions completion and state are clear. 
When do you think this will be assigned to some to start this process?

Comment 18 Martin Sivák 2016-10-26 12:21:19 UTC
Andrej, we just need to figure out what we are missing for now, no implementation will be involved just yet.

Comment 19 Andrej Krejcir 2016-10-27 11:43:09 UTC
The CLI has the following commands, some of them can be done in the engine:
  --deploy [options]
    - can be done in engine
    
  --vm-start   
  --vm-start-paused  
  --vm-shutdown
  --vm-poweroff
  
  --vm-status [--json]
    - most of the info can be accessed in engine
    
  --add-console-password [--password=<password>]
  --check-deployed
  --check-liveliness
  --connect-storage
  --console
  
  --set-maintenance --mode=<mode>
    - can be done in engine
  
  --reinitialize-lockspace
  --clean-metadata
  --upgrade-appliance
  --rollback-upgrade
  
The --vm-status command returns this information per host:

  - live-data
    - true if host is running
    
  - hostname
   
  - host-id 
    - sqeuence number of host, probably not exposed to engine
  
  - engine-status
    - status of the engine VM and optional reason
    
  - score 
  - stopped 
    - both are in the webadmin in "general" subtab for a host,
      as "Hosted Engine HA:"
  
  - local maintenance
    - can be seen in the engine as normal maintenance
    
  - crt32
  - host timestamp
  
  - extra
    - more info as a string

It also shows if the HE is in global maintenance. 
In the webadmin, global maintenance can be seen under 
"general" subtab for a host, as "Hosted Engine HA:"

Comment 20 Yaniv Lavi 2016-10-27 15:08:08 UTC
(In reply to Andrej Krejcir from comment #19)
> The CLI has the following commands, some of them can be done in the engine:
>   --deploy [options]
>     - can be done in engine

It currently not complete.
See BZ #1369827.

>     
>   --vm-start   
>   --vm-start-paused  
>   --vm-shutdown
>   --vm-poweroff

I don't expect to be able to do this via UI\REST.

>   
>   --vm-status [--json]
>     - most of the info can be accessed in engine
>     
>   --add-console-password [--password=<password>]
>   --check-deployed
>   --check-liveliness
>   --connect-storage
>   --console

These are probably not relevant after initial deployment or only when troubleshooting from the host.

>   
>   --set-maintenance --mode=<mode>
>     - can be done in engine

The main problem here is that the buttons do not reflect the state for global maintenance. There is no way to know if this action worked or not from the UI.

For local maintenance the host maintenance should trigger this, so probably good there.

>   
>   --reinitialize-lockspace
>   --clean-metadata
>   --upgrade-appliance
>   --rollback-upgrade

All of these are not relevant for when the appliance is up.

>   
> The --vm-status command returns this information per host:
> 
>   - live-data
>     - true if host is running
>     
>   - hostname
>    
>   - host-id 
>     - sqeuence number of host, probably not exposed to engine
>   
>   - engine-status
>     - status of the engine VM and optional reason
>     
>   - score 
>   - stopped 
>     - both are in the webadmin in "general" subtab for a host,
>       as "Hosted Engine HA:"
>   
>   - local maintenance
>     - can be seen in the engine as normal maintenance
>     
>   - crt32
>   - host timestamp
>   
>   - extra
>     - more info as a string

Any info that we might need in the UI?

> 
> It also shows if the HE is in global maintenance. 
> In the webadmin, global maintenance can be seen under 
> "general" subtab for a host, as "Hosted Engine HA:"

We should probably have the engine UI should something more general to note this fact, but this is good enough for now.
I will open RFEs for this and the other issues mentioned above:

- Adding crown to HE VM.
- HE hosts should have indicators and a way to filter them from the rest of the hosts.
- HE storage should have a crown.
- Global maintenance clear indication, set confirmation and stateful button.

Comment 21 Andrej Krejcir 2016-11-07 10:27:23 UTC
(In reply to Yaniv Dary from comment #20)
> >   - extra
> >     - more info as a string
> 
> Any info that we might need in the UI?
> 

No, most of the info is in other fields too.
For example, this 'extra' string is on my setup:

metadata_parse_version=1
metadata_feature_version=1
timestamp=1201239 (Thu Oct 27 11:01:59 2016)
host-id=1
score=3400
maintenance=False
state=EngineUp
stopped=False

Comment 22 Martin Sivák 2016-11-07 11:41:37 UTC
The extra section is a bit tricky though, it is not required to be machine readable, up-to-date or valid. Hosted engine does its best, but the only reliable data block used for synchronization is the one we get the non-extra arguments from. 

We might not need such high guarantees for UI indications though and in that case the state might be something to expose. It needs a translation layer though as it directly reflects the name of the active hosted engine agent state class (the names or the state machine flows can change as they are implementation specific).

Comment 24 Michal Skrivanek 2016-11-22 08:41:45 UTC
btw any update re guest agent being present in appliance? (comment #3)

Comment 25 Yaniv Lavi 2016-11-22 10:02:01 UTC
(In reply to Michal Skrivanek from comment #24)
> btw any update re guest agent being present in appliance? (comment #3)

If it's in the appliance, it always there, since this is the only supported option since 4.0.

Comment 26 meital avital 2017-03-16 07:25:40 UTC
All depends on bug verified - we can close this Tracker.

Comment 27 Lucy Bopf 2017-03-30 01:14:10 UTC
Setting requires_doc_text to -, because the Release Notes text will be covered by bug 1367924.


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