openQA webui entry "Assigned worker" shows ip instead of names as formerly
For example https://openqa.opensuse.org/tests/1459575# shows "Assigned worker: 192.168.112.7:4" whereas previously we had the hostname. https://github.com/os-autoinst/openQA/pull/3481 has changed the behaviour.
- AC1: the webUI shows hostnames, not IPv4 addresses
Maybe we can use the hostname. This should be tested, ini file examples should be updated and done at least on o3 as well
- Status changed from New to Feedback
- Assignee set to okurz
- Priority changed from Low to High
- Target version changed from future to Ready
this has been raised as a problem by another user, so it's not just me.
I propose a revert for now with https://github.com/os-autoinst/openQA/pull/3520
DimStar mentioned this problem as well in IRC, merged.
For this ticket: Implement test covering the observations from regression.
For retrospective: Discuss changes done without tickets as been done for the original PR https://github.com/os-autoinst/openQA/pull/3481 . While not necessarily bad we could have still benefit to discuss the idea and potential caveats further and keep the ticket to check ourselves for verification in production and record user feedback.
- Status changed from Feedback to Workable
- Assignee deleted (
- Priority changed from High to Low
revert merged and deployed on o3. Not yet on osd though.
I guess next step would be to try to set real hostnames in our infrastructure and bring back https://github.com/os-autoinst/openQA/commit/1786e30b528cb888fa49474d39cd25b80415d629 but carefully check that where expected only hostnames show up, e.g. webUI on worker page, assigned worker in job details, development mode still works, https://stats.openqa-monitor.qa.suse.de/d/nRDab3Jiz/openqa-jobs-test?orgId=1&fullscreen&panelId=10 shows worker hostnames, etc.
Let me rephrase: Wasn't the point of https://github.com/os-autoinst/openQA/pull/3481 to show the hostname as specified in
workers.ini? When I would have been aware that we specify an IP address there in production I wouldn't have accepted the PR in the first place.
Yes, showing the hostname as specified in workers.ini was the point of https://github.com/os-autoinst/openQA/pull/3481 and yes, we would not have accepted the PR if we knew that we use IPv4 addressed and that would be shown in webUI :)
So what I would prefer is to bring back https://github.com/os-autoinst/openQA/pull/3481 , maybe in a changed form, use hostnames in workers.ini so that we leave the choice to the OS if IPv4 or IPv6 is used and then show that nice hostname in the webUI as well, at best together with an option to override the name in workers.ini as well