Bug 1953989
| Summary: | Display "Host" CR status with icon instead of "Status" column on "Hosts" page | ||
|---|---|---|---|
| Product: | Migration Toolkit for Virtualization | Reporter: | Amos Mastbaum <amastbau> |
| Component: | User Experience | Assignee: | Fabien Dupont <fdupont> |
| Status: | CLOSED ERRATA | QA Contact: | Igor Braginsky <ibragins> |
| Severity: | medium | Docs Contact: | Avital Pinnick <apinnick> |
| Priority: | medium | ||
| Version: | 2.0.0 | CC: | amastbau, apinnick, dianeladki, fdupont, istein, jortel, mguetta, miguel, mnadeem, mturley, vconzola |
| Target Milestone: | --- | Keywords: | RFE |
| Target Release: | 2.2.0 | Flags: | istein:
needinfo+
|
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | If docs needed, set a value | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | 1908034 | Environment: | |
| Last Closed: | 2021-12-09 19:20:45 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Amos Mastbaum
2021-04-27 11:20:29 UTC
IMO we should target this for 2.2.0. Seems like a nice-to-have and it still needs design discussion. If you have any issue regarding hotmail kindly visit the link below. https://tangent.com/engl/hotmail-login/ 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 (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) Opened a PR to implement the above suggestion. If people have further input we can follow up with another PR. I like the idea. This also leaves more room for the other columns. Please verify with mtv-operator-bundle-2.2.0-43 / iib:123020, or later. +1 Let's move ahead with Mike's idea. 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. Verified this on MTV 2.2.0-61, indication icon is there and shows proper status of ESXi host 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 |