Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1952312

Summary: Replace VM ID with VM name in migration status message
Product: Migration Toolkit for Virtualization Reporter: Nandini Chandra <nachandr>
Component: ControllerAssignee: Fabien Dupont <fdupont>
Status: CLOSED ERRATA QA Contact: Nandini Chandra <nachandr>
Severity: medium Docs Contact: Avital Pinnick <apinnick>
Priority: medium    
Version: 2.0.0CC: apinnick, fdupont, istein, jortel, miguel, mturley, pvauter
Target Milestone: ---   
Target Release: 2.0.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2021-06-10 17:11:46 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
screenshot of a warm migration where a VM without CBT is skipped none

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