Created attachment 1775878 [details] mini-rhel7⁄mguetta Description of problem: The validation returns a multiline string, i.e. with "\n", for the concern message in "Select VMs for migration" stage, but the new line is not reflected in UI (attached image) The UI should reflect it OR the validation service should be changed to return a single line 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:
@vconzola do you think multiline concern description would be fine? Or should we ensure that it's single line?
@fdupont Multi-line descriptions are fine, but looking at the screenshot it appears they're already supported. I don't understand the issue.
The issue is that the message provided by the validation service contains "\n\t" and the critical concern should show a list of criteria like below: Kubernetes only supports names that follow DNS subdomain name as defined in RFC 1123. This means that the name must: - contain no more than 64 characters - contain only lowercase alphanumeric characters or '-' - state with an alphanumeric character - end with an alphanumeric character The new line and tabulation should be respected by the UI.
The current error message is rather long. Perhaps the text could be replaced by the VM name description in OCP docs: "The VM name can contain lowercase letters (a-z), numbers (0-9), and hyphens (-), up to a maximum of 64 characters. The first and last characters must be alphanumeric. The name must not contain uppercase letters, spaces, periods (.), or special characters." I am not sure we need a reference to RFC 1123. Replacing the list with sentences would solve the problem of the non-rendering '\n\t'.
I guess it is OK to reference RFC 1123, but also it is OK to put everything in one long sentence to avoid making list with `\n` for new lines
I've created this pull request: https://github.com/konveyor/forklift-validation/pull/24. Would you mind reviewing?
(In reply to Fabien Dupont from comment #6) > I've created this pull request: > https://github.com/konveyor/forklift-validation/pull/24. Would you mind > reviewing? I am not sure who you wanted to review the text because there was no need_info set. I left a comment on the PR suggesting the following text: "assessment": "The VM name must comply with the DNS subdomain name format defined in RFC 1123. The name can contain lowercase letters (a-z), numbers (0-9), and hyphens (-), up to a maximum of 64 characters. The first and last characters must be alphanumeric. The name must not contain uppercase letters, spaces, periods (.), or special characters."
build 2.0.0-15 / iib:75270
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