Bug 1953925

Summary: Web console displays slash ('/') as '%2f' in source VM name
Product: Migration Toolkit for Virtualization Reporter: Maayan Hadasi <mguetta>
Component: InventoryAssignee: Jeff Ortel <jortel>
Status: CLOSED ERRATA QA Contact: Maayan Hadasi <mguetta>
Severity: medium Docs Contact: Avital Pinnick <apinnick>
Priority: medium    
Version: 2.0.0CC: apinnick, fdupont, istein, jortel, mturley
Target Milestone: ---   
Target Release: 2.1.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-08-26 07:09:08 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
mini-rhel7⁄mguetta none

Description Maayan Hadasi 2021-04-27 08:13:21 UTC
Created attachment 1775875 [details]
mini-rhel7⁄mguetta

Description of problem:
In Create migration plan wizard, the VM name contains '%2f' instead of '/' character of the original source VM name (please see attached image)


Version-Release number of selected component (if applicable):
MTV-2.0.0-20 (iib 69034)


How reproducible:
100%

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Mike Turley 2021-05-11 16:33:31 UTC
Fabien, do we still need this fixed in 2.1.0? Do you think it needs a fix in the UI or should the inventory API pass the properly rendered name?

Comment 2 Fabien Dupont 2021-05-11 17:16:14 UTC
That's a good question. I think the API should pass the properly rendered name. If it marshals the string when storing it in the DB, it should unmarshal it, too.

@jortel, what do you think?

Comment 3 Jeff Ortel 2021-07-13 22:06:51 UTC
The VM.name is being returned by the vmware govmomi property collector as URL-encoded. Makes sense to decode before storing in the DB.

Comment 4 Maayan Hadasi 2021-08-11 12:04:14 UTC
Verified as fixed.

Versions:
MTV 2.1.0-44 iib 96817
CNV 4.8.1-17 iib 97584
OCP 4.8.4
(PSI)

Comment 7 errata-xmlrpc 2021-08-26 07:09:08 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 (Migration Toolkit for Virtualization 2.1.0), 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:3278