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

Bug 2018983

Summary: [MTV] [VMWARE] Cannot select migration network for providers with long names
Product: Migration Toolkit for Virtualization Reporter: Amos Mastbaum <amastbau>
Component: User ExperienceAssignee: Mike Turley <mturley>
Status: CLOSED ERRATA QA Contact: Amos Mastbaum <amastbau>
Severity: medium Docs Contact:
Priority: medium    
Version: 2.2.0CC: fbladilo, fdupont, mturley, rhoch
Target Milestone: ---   
Target Release: 2.3.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: 2022-04-04 18:03:40 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
ui none

Description Amos Mastbaum 2021-11-01 12:34:02 UTC
Created attachment 1838841 [details]
ui

Description of problem:

If the Provider Name is too long, Selectioning Migration Network Fails:


"Request failed with status code 422
Secret "vsphere-6.5-rhev-node-05.rdu2.scalelab.redhat.com-host-732l8hg6" is invalid: metadata.labels: Invalid value: "vsphere-6.5-rhev-node-05.rdu2.scalelab.redhat.com-host-732-config": must be no more than 63 characters"

(screenshot attached)

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


How reproducible:
100%

Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 Fabien Dupont 2021-11-02 10:02:56 UTC
It means that the names are "too user friendly" and will likely exceed the 63 characters limit from k8s.
Would it be possible to change the naming convention to always ensure a name shorter than 63 chars ?

Comment 2 Sam Lucidi 2022-01-27 18:40:09 UTC
I think this might be a UI issue. To my knowledge the UI is creating the host secret and setting the labels on it.[1] Mike, what do you think?

[1] https://github.com/konveyor/forklift-ui/blob/7825ff19d5b9cc59d2fe95fdb0b8b325c0f4235d/pkg/web/src/app/queries/hosts.ts#L67

Comment 3 Mike Turley 2022-01-27 22:57:11 UTC
That makes sense, the UI is just creating that name based on the provider name and host id. We could just use a random string or hash for the secret name... I'm not sure how important it is that it is human-readable on the CLI?

Either way, Sam is right. Self-assigning.

Comment 4 Mike Turley 2022-03-01 16:44:28 UTC
Moving to MODIFIED, this was backported as part of a fast-forward merge from main to release-v2.3.0 (d040082)

Comment 5 Amos Mastbaum 2022-03-06 15:56:59 UTC
OCP 4.10.0-rc.3	
CNV 4.10.0-686
MTV-2.3.0-32
Verified

Comment 8 errata-xmlrpc 2022-04-04 18:03:40 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.3.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-2022:1183