Bug 1953925 - Web console displays slash ('/') as '%2f' in source VM name
Summary: Web console displays slash ('/') as '%2f' in source VM name
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Migration Toolkit for Virtualization
Classification: Red Hat
Component: Inventory
Version: 2.0.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 2.1.0
Assignee: Jeff Ortel
QA Contact: Maayan Hadasi
Avital Pinnick
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-04-27 08:13 UTC by Maayan Hadasi
Modified: 2021-08-26 07:09 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2021-08-26 07:09:08 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
mini-rhel7⁄mguetta (94.37 KB, image/png)
2021-04-27 08:13 UTC, Maayan Hadasi
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Github konveyor forklift-controller pull 314 0 None open Un-URL-encode vSphere object names. 2021-07-13 22:06:50 UTC
Red Hat Product Errata RHEA-2021:3278 0 None None None 2021-08-26 07:09:19 UTC

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


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