Bug 1952312 - Replace VM ID with VM name in migration status message
Summary: Replace VM ID with VM name in migration status message
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Migration Toolkit for Virtualization
Classification: Red Hat
Component: Controller
Version: 2.0.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 2.0.0
Assignee: Fabien Dupont
QA Contact: Nandini Chandra
Avital Pinnick
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-04-22 03:45 UTC by Nandini Chandra
Modified: 2023-09-15 01:05 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-06-10 17:11:46 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
screenshot of a warm migration where a VM without CBT is skipped (51.05 KB, image/png)
2021-04-22 03:45 UTC, Nandini Chandra 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
Github konveyor forklift-controller pull 236 0 None open BZ 1952312 - Add VM name to builder error messages 2021-04-27 07:38:51 UTC
Red Hat Product Errata RHEA-2021:2381 0 None None None 2021-06-10 17:11:57 UTC

Description Nandini Chandra 2021-04-22 03:45:23 UTC 
Created attachment 1774238 [details]
screenshot of a warm migration where a VM without CBT is skipped

Description of problem:
-----------------------
The status message when a warm migration is skipped for a VM without CBT shows this:
Changed Block Tracking (CBT) is disabled for VM vm-647

The VM ID should be replaced with VM name.See attached screenshot.


Version-Release number of selected component (if applicable):
-------------------------------------------------------------


How reproducible:
-----------------
Always


Steps to Reproduce:
-------------------
1.
2.
3.


Actual results:
---------------


Expected results:
----------------


Additional info:
----------------
Comment 1 Miguel Perez Colino 2021-04-22 12:28:15 UTC 
Could we display VM name *and* VM ID?
Comment 2 Mike Turley 2021-04-22 13:03:39 UTC 
This message is provided by the controller, in the error object's `reasons` property. @jortel is this an easy change in there?
Comment 3 Fabien Dupont 2021-04-27 07:38:51 UTC 
This validation has been recently added and followed the existing message format. I agree thatr it is not user friendly. Let's add the VM name and keep the MoRef as well.
Comment 4 Fabien Dupont 2021-05-03 12:09:05 UTC 
The fix should be part of build mtv-operator-bundle-container-2.0.0-4 / iib:72115.
Comment 5 Nandini Chandra 2021-05-03 16:48:29 UTC 
Verified on MTV 2.0.0-4
Comment 8 errata-xmlrpc 2021-06-10 17:11:46 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.0.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:2381
Comment 9 Red Hat Bugzilla 2023-09-15 01:05:28 UTC 
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 500 days
Note You need to log in before you can comment on or make changes to this bug.