Bug 1953989 - Display "Host" CR status with icon instead of "Status" column on "Hosts" page
Summary: Display "Host" CR status with icon instead of "Status" column on "Hosts" page
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Migration Toolkit for Virtualization
Classification: Red Hat
Component: User Experience
Version: 2.0.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 2.2.0
Assignee: Fabien Dupont
QA Contact: Igor Braginsky
Avital Pinnick
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-04-27 11:20 UTC by Amos Mastbaum
Modified: 2021-12-09 19:20 UTC (History)
11 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1908034
Environment:
Last Closed: 2021-12-09 19:20:45 UTC
Target Upstream Version:
Embargoed:
istein: needinfo+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github konveyor forklift-ui pull 799 0 None open Bug 1953989: Move Host CR conditions into an icon next to the network name, remove Status column 2021-10-07 20:10:26 UTC
Red Hat Product Errata RHEA-2021:5066 0 None None None 2021-12-09 19:20:52 UTC

Description Amos Mastbaum 2021-04-27 11:20:29 UTC
+++ This bug was initially created as a clone of Bug #1908034 +++

Description of problem:

When navigating to the VMware provider host page, There is a blank value for the Status column.
 
Currently, the Status column displays the Host CR status based on its conditions.
However, a host has an intrinsic status on the VMware side and it should be displayed when no Host CR exist.

There is a Host.overallStatus property that can have the "grey", "green", "yellow" and "red" values.

I have read that a change to this property may not trigger an event, so we need to verify that.

Comment 1 Mike Turley 2021-05-11 16:21:34 UTC
IMO we should target this for 2.2.0. Seems like a nice-to-have and it still needs design discussion.

Comment 2 mark angel 2021-06-13 13:03:19 UTC
If you have any issue regarding hotmail kindly visit the link below.

https://tangent.com/engl/hotmail-login/

Comment 3 Mike Turley 2021-09-30 19:18:01 UTC
Some thoughts after discussing this one with Vince:

The original intent of this Status column is simply to have somewhere to show conditions on the Host CR if present (in this case, probably some issue with the credentials the user gave when configuring the transfer network). For this use case it makes sense for the column to be blank on rows with no configured transfer network. The original BZ came up because of confusion as to why this column is blank for those rows.

In previous discussions with Jeff and Fabien, we determined that the host.overallStatus color isn't very meaningful without a message, so that's not necessarily what we want to show. We also considered showing whether the host is in maintenance mode, or some other info. However, these ideas are simply to try and fill in the blank cell here.

Vince and I propose that there is a simpler way to alleviate that confusion: Remove the Status column entirely, and move any CR conditions into an icon+popover next to the configured network name (which is what the conditions are related to anyway). This way we don't have any confusingly blank cells to fill in.

Does anyone object to this solution? Was there another reason we wanted to show more detailed status here other than to fill in the blank cells?

@jortel @fdupont @amastbau

Comment 4 Amos Mastbaum 2021-09-30 22:08:04 UTC
@istein i would like (In reply to Mike Turley from comment #3)
> Some thoughts after discussing this one with Vince:
> 
> The original intent of this Status column is simply to have somewhere to
> show conditions on the Host CR if present (in this case, probably some issue
> with the credentials the user gave when configuring the transfer network).
> For this use case it makes sense for the column to be blank on rows with no
> configured transfer network. The original BZ came up because of confusion as
> to why this column is blank for those rows.
> 
> In previous discussions with Jeff and Fabien, we determined that the
> host.overallStatus color isn't very meaningful without a message, so that's
> not necessarily what we want to show. We also considered showing whether the
> host is in maintenance mode, or some other info. However, these ideas are
> simply to try and fill in the blank cell here.
> 
> Vince and I propose that there is a simpler way to alleviate that confusion:
> Remove the Status column entirely, and move any CR conditions into an
> icon+popover next to the configured network name (which is what the
> conditions are related to anyway). This way we don't have any confusingly
> blank cells to fill in.
> 
> Does anyone object to this solution? Was there another reason we wanted to
> show more detailed status here other than to fill in the blank cells?
> 
> @jortel @fdupont @amastbau

@istein I would like here other people to take on this, Someone from scale maybe (I have note used this part much)

Comment 5 Mike Turley 2021-10-07 20:30:15 UTC
Opened a PR to implement the above suggestion. If people have further input we can follow up with another PR.

Comment 6 Fabien Dupont 2021-10-11 09:45:20 UTC
I like the idea. This also leaves more room for the other columns.

Comment 7 Fabien Dupont 2021-10-13 06:32:55 UTC
Please verify with mtv-operator-bundle-2.2.0-43 / iib:123020, or later.

Comment 8 Miguel Perez Colino 2021-10-13 11:01:49 UTC
+1
Let's move ahead with Mike's idea.

Comment 9 Ilanit Stein 2021-10-13 16:39:19 UTC
To verify this bug (Based on Bug 1908034 comment #9 ),
please check the added icon shows the Host CR conditions.
 
The Host CR exists only for hosts that have a non-default network configured (it's showing whether setting that network was successful). 

When the management network is selected, there is no Host CR, it lets the controller handle network selection and just shows the known details of the default network. So there is no status to show in that case. 
If you select a non-default network for that host, a status should appear.

Comment 10 Igor Braginsky 2021-10-28 10:16:59 UTC
Verified this on MTV 2.2.0-61, indication icon is there and shows proper status of ESXi host

Comment 13 errata-xmlrpc 2021-12-09 19:20:45 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (MTV 2.2.0 Images), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2021:5066


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