Created attachment 1647462 [details] hostname with undercore("_") Description of problem: when we test network configuration for hostname it does not work only if underscore available in the hostname Version-Release number of selected component (if applicable): 5.11.1.2 How reproducible: 100% Steps to Reproduce: 1. Set hostname with an underscore("_"). 2. Test Network Configuration for hostname, Perform the following commands: - ap - space - option 1 - Configure Network - option 4 - Test Network Configuration - Enter hostname which we set on step 1 Actual results: Getting "Please provide a valid Hostname or IP Address." message when entering the correct hostname Expected results: We should get a "success" message while entering the correct hostname. Additional info: Also facing this issue on 5.10.13.1 version
Created attachment 1647463 [details] console where I got error
Per Yuri, "as described in https://tools.ietf.org/html/rfc952: underscore ("_") is not permitted character for host name" Closing because this isn't a bug.
Hi drew, As you are saying we are not permitting hostname with an underscore then we should disallow underscore while setting hostname but we are not. I think there is an issue while setting the hostname with an underscore. Please have a look in attachment Thanks, Devidas Gaikwad
I'd like to see a reproducer, please, since my regex doesn't match underscores.
drew, Do we need to re-open this issue?
No. This isn't an issue. I can't set a hostname with an underscore and from your description, you can't either.
drew, as per my comment I already re-produced this issue with hostname "_test1", re-opening this issue.
Yeah, like I said, I can't reproduce this and I'm on the stated version. Please either give me a reproducer or close this.
Thanks! Yuri, I know it's break, I hope you don't look at this till you get back cause it's low prio but setting the hostname from the options chosen in comment 11 isn't running the regex for validation, should it be?
Hi Drew, yes, I think it should run validation. Looks like bug...
https://github.com/ManageIQ/manageiq-appliance_console/pull/107
New commit detected on ManageIQ/manageiq-appliance_console/master: https://github.com/ManageIQ/manageiq-appliance_console/commit/862ad568abca05cca814ae3595d624c46e9aae9a commit 862ad568abca05cca814ae3595d624c46e9aae9a Author: d-m-u <duhlmann> AuthorDate: Sat Dec 28 11:14:41 2019 -0500 Commit: d-m-u <duhlmann> CommitDate: Sat Dec 28 11:14:41 2019 -0500 Use the regex to validate hostnames when we set them Highline's third arg for this method is the validation Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1786282 bin/appliance_console | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
New commit detected on ManageIQ/manageiq-appliance/master: https://github.com/ManageIQ/manageiq-appliance/commit/8c0dcef26d65468bb3ef3730121f1df83a8867ab commit 8c0dcef26d65468bb3ef3730121f1df83a8867ab Author: Nick Carboni <ncarboni> AuthorDate: Thu Jan 16 10:10:59 2020 -0500 Commit: Nick Carboni <ncarboni> CommitDate: Thu Jan 16 10:10:59 2020 -0500 Update manageiq-appliance_console to version 5.3.1 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1786282 Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1785257 manageiq-appliance-dependencies.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)