Actions
action #161327
closedcoordination #155485: [saga][epic] Efficient openQA worker pool resource handling in datacenters
coordination #162041: [epic] Improved worker behaviour
Prevent remote workers registering as "localhost" size:S
Description
Motivation¶
In #160424 we first estimated with quite some effort what could and should be done but in the end the ticket was resolved after "just" setting static hostnames which we AFAIK have not even mentioned as a practice in openQA documentation. Are we ok with that or can we do better?
Acceptance criteria¶
- AC1: Remote worker hosts can only register with a proper hostname and not localhost without needing to hardcode the hostname in workers.ini
- AC2: There is a clear error message with suggestion on invalid remote worker connection attempts as "localhost"
- AC3: It is still possible to register as "localhost", e.g. for single instance container setup, when using the loopback interface
Suggestions¶
- Consider only allowing "localhost" when the connection is using the local loopback interface. Start with extending t/24-worker-webui-connection.t . The according implementation code is likely in lib/OpenQA/Worker/WebUIConnection.pm in the "register" function.
- Provide a useful suggestion in any error message preventing the connection attempt.
- Use the
is_host_local
utility function: https://github.com/os-autoinst/openQA/blob/21904f825ca0a24ecd6499b3d5e7e96710d4858e/lib/OpenQA/Utils.pm#L950
Updated by okurz about 2 months ago
- Copied from action #160424: o3: arm21 & arm22 show up as "localhost", (almost) all jobs incomplete size:M added
Updated by okurz about 1 month ago
- Subject changed from Follow-up to "o3: arm21 & arm22 show up as "localhost", (almost) all jobs incomplete" - are we fine to have invested quite some effort for estimating but then just set static hostnames? to Prevent remote workers registering as "localhost" size:S
- Description updated (diff)
- Status changed from New to Workable
Updated by okurz about 1 month ago
- Status changed from Workable to In Progress
- Assignee set to okurz
Updated by okurz about 1 month ago
- Due date set to 2024-06-24
- Status changed from In Progress to Feedback
Updated by okurz about 1 month ago
- Project changed from openQA Infrastructure to openQA Project
- Category changed from Feature requests to Feature requests
Updated by okurz about 1 month ago
- Due date deleted (
2024-06-24) - Status changed from Feedback to Resolved
https://github.com/os-autoinst/openQA/pull/5685 merged. All done.
Actions