Consider making upgrade check for old computes a failure

Bug #1956983 reported by Pierre Riteau
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Unassigned

Bug Description

Currently, the upgrade check for older than N-1 computes only produces a warning.
For example:

Check: Older than N-1 computes
Result: Warning
Details: Current Nova version does not support computes older than
  Victoria but the minimum compute service level in your
  system is 30 and the oldest supported service level is 52.

If this is overlooked, Nova services will fail to start after upgrade. With Nova API down, the old services cannot be removed without database edits.

Is there a specific reason to keep this check as a warning rather than a failure?

Revision history for this message
Sylvain Bauza (sylvain-bauza) wrote :

I'm not sure I'd classify it as a bug. Probably a good thought so, so marking it Invalid/Wishlist but I'm open to thoughts.

To answer your question, now that we have an hardstop blocking nova services to restart if there are old enough, this sound a legit blueprint to address.

Changed in nova:
importance: Undecided → Wishlist
status: New → Invalid
Revision history for this message
Balazs Gibizer (balazs-gibizer) wrote :

This is already an error since Wallaby https://review.opendev.org/c/openstack/nova/+/738482

Revision history for this message
Pierre Riteau (priteau) wrote :

@gibi: which is why I am suggesting the upgrade check should be fatal, since presence of old computes is fatal for nova

Pierre Riteau (priteau)
Changed in nova:
status: Invalid → New
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/nova/+/845262

Changed in nova:
status: New → In Progress
Revision history for this message
Balazs Gibizer (balazs-gibizer) wrote :

I misunderstood this issue originally. Now I see that it is targeting the upgrade check. I agree that it should be changed from WARNING to FAILURE

Changed in nova:
importance: Wishlist → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.opendev.org/c/openstack/nova/+/845262
Committed: https://opendev.org/openstack/nova/commit/9af4c6115fcfce9c76a1e7fd5b93bb43cec66a4d
Submitter: "Zuul (22348)"
Branch: master

commit 9af4c6115fcfce9c76a1e7fd5b93bb43cec66a4d
Author: Pierre Riteau <email address hidden>
Date: Thu Jun 9 20:56:26 2022 +0200

    Change TooOldComputeService upgrade check code to failure

    The TooOldComputeService upgrade check currently produces a warning,
    which may be missed if the upgrade process only checks the exit code of
    the upgrade check command.

    Since this can lead to Nova control services failing to start, make the
    upgrade check a failure instead, so it results in a non-zero exit code.

    Closes-Bug: #1956983
    Change-Id: Ia3ce6a0b0b810667ac0a66502a43038fe43c5aed

Changed in nova:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers