Comment 1 for bug 2039597

Revision history for this message
sean mooney (sean-k-mooney) wrote :

just leaving some context before i finish for tonight.
when i originally asked for this check it was with the intent to detect the case where we were going to rely on this in the next release and notify the operator that one of the compute nodes was not upgaged.

chatting about this on irc that logic was flawed.
in the current release we don't depend on this being set.

before upgrading the compute nodes and restarting them it will always not be set and the help text for this
command says it should be run before the service are restarted to execute the new code.

the check as written will also not support clouds that have ironic deployed as tey will not have a compute service id set in the compute node record (and cant until we remove the hash ring code)

for those reasons i think we should likely revert this status check.

alternatively we can modify it to filter out ironic compute nodes and add a min compute service version check a the start. i.e. only run it if all comptue services are at least upgraded to bobcat.

if they are 2023.2+ and we filter out ironic that menast that you have db corruption as something removed the compute service id form the compute node record.