Project

General

Profile

Actions

action #7086

closed

add some worker version check

Added by lnussel over 9 years ago. Updated almost 7 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)

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 1 (0 open1 closed)

action #32260: Improve worker version checking.Resolveddasantiago2018-02-26

Actions

Related issues 3 (0 open3 closed)

Related to openQA Project (public) - action #26890: Warn users about version incompatibilityRejectedokurz2017-10-19

Actions
Related to openQA Project (public) - action #29250: [tools] Error on journalctl - [worker:error] got unknown command infoRejectedokurz2017-12-11

Actions
Related to openQA Project (public) - action #30002: Provide currently installed os-autoinst version in openQA worker page and logResolveddasantiago2018-01-05

Actions
Actions #1

Updated by okurz almost 9 years ago

  • Category set to 132
Actions #2

Updated by coolo almost 9 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

Actions #3

Updated by coolo almost 9 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.

Actions #4

Updated by okurz about 7 years ago

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

Updated by coolo about 7 years ago

  • Target version set to Ready
Actions #6

Updated by coolo almost 7 years ago

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

Updated by EDiGiacinto almost 7 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.

Actions #9

Updated by EDiGiacinto almost 7 years ago

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

Updated by EDiGiacinto almost 7 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

Actions #12

Updated by dasantiago almost 7 years ago

  • Assignee set to dasantiago
Actions #13

Updated by dasantiago almost 7 years ago

  • Target version changed from Ready to Current Sprint
Actions #14

Updated by dasantiago almost 7 years ago

  • Status changed from New to In Progress
Actions #15

Updated by dasantiago almost 7 years ago

  • Status changed from In Progress to Resolved

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

Actions #16

Updated by szarate almost 7 years ago

  • Target version changed from Current Sprint to Done
Actions

Also available in: Atom PDF