Project

General

Profile

action #7086

add some worker version check

Added by lnussel over 6 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Feature requests
Target version:
Start date:
2018-02-26
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Difficulty:

Description

the worker should have some kind of version that the webui can check. the webui should then be able to reject workers that are not sufficiently new. that helps in case of protocol changes or new mandatory features.


Subtasks

action #32260: Improve worker version checking.Resolveddasantiago


Related issues

Related to openQA Project - action #26890: Warn users about version incompatibilityRejected2017-10-19

Related to openQA Project - action #29250: [tools] Error on journalctl - [worker:error] got unknown command infoRejected2017-12-11

Related to openQA Project - action #30002: Provide currently installed os-autoinst version in openQA worker page and logResolved2018-01-05

History

#1 Updated by okurz almost 6 years ago

  • Category set to 132

#2 Updated by coolo almost 6 years ago

  • Priority changed from High to Normal

this is not actually of high priority as we just have to be careful when deploying new versions to deploy all machines. Annoying, but not critical

#3 Updated by coolo almost 6 years ago

I think we also need a stricter check on using the same os-autoinst on the workers. But possibly if the worker required the os-autoinst version it was built against, we could sync this issue with that other problem.

#4 Updated by okurz about 4 years ago

  • Related to action #26890: Warn users about version incompatibility added

#5 Updated by coolo about 4 years ago

  • Target version set to Ready

#6 Updated by coolo almost 4 years ago

  • Related to action #29250: [tools] Error on journalctl - [worker:error] got unknown command info added

#7 Updated by EDiGiacinto almost 4 years ago

Just my 2c on this:

Aside from displaying workers versions: i think for a communication protocol perspective (especially wrt websocket server/worker) is a good thing to have also a variable/constant identifying only the protocol version - inside the codebase - instead of the packaging version.
That allows us with a bit of more control and we can bump the protocol version number inside the code when we know we are breaking compatibility with our changes and let the worker immediately die in such case.

#9 Updated by EDiGiacinto almost 4 years ago

  • Related to action #30002: Provide currently installed os-autoinst version in openQA worker page and log added

#10 Updated by EDiGiacinto almost 4 years ago

okurz wrote:

like https://github.com/os-autoinst/os-autoinst/blob/master/isotovideo#L55 ?

yup, i think we should do the same for the websocket/internal commands

#12 Updated by dasantiago almost 4 years ago

  • Assignee set to dasantiago

#13 Updated by dasantiago almost 4 years ago

  • Target version changed from Ready to Current Sprint

#14 Updated by dasantiago almost 4 years ago

  • Status changed from New to In Progress

#15 Updated by dasantiago almost 4 years ago

  • Status changed from In Progress to Resolved

This has been into production since 16/02/2018.

#16 Updated by szarate almost 4 years ago

  • Target version changed from Current Sprint to Done

Also available in: Atom PDF