diff options
| author | melanie witt <melwittt@gmail.com> | 2023-02-15 22:37:40 +0000 |
|---|---|---|
| committer | Dan Smith <dansmith@redhat.com> | 2023-05-10 07:09:05 -0700 |
| commit | db455548a12beac1153ce04eca5e728d7b773901 (patch) | |
| tree | 076581e7ff37283eb046ca3d19d14246def6652f /nova/cmd/status.py | |
| parent | 105afb338b7dd25916fe601c2a226c2a552c44c7 (diff) | |
| download | nova-db455548a12beac1153ce04eca5e728d7b773901.tar.gz | |
Use force=True for os-brick disconnect during delete
The 'force' parameter of os-brick's disconnect_volume() method allows
callers to ignore flushing errors and ensure that devices are being
removed from the host.
We should use force=True when we are going to delete an instance to
avoid leaving leftover devices connected to the compute host which
could then potentially be reused to map to volumes to an instance that
should not have access to those volumes.
We can use force=True even when disconnecting a volume that will not be
deleted on termination because os-brick will always attempt to flush
and disconnect gracefully before forcefully removing devices.
Closes-Bug: #2004555
Change-Id: I3629b84d3255a8fe9d8a7cea8c6131d7c40899e8
Diffstat (limited to 'nova/cmd/status.py')
| -rw-r--r-- | nova/cmd/status.py | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/nova/cmd/status.py b/nova/cmd/status.py index 29e4a5d01e..4a4e28d7e8 100644 --- a/nova/cmd/status.py +++ b/nova/cmd/status.py @@ -271,6 +271,15 @@ https://docs.openstack.org/latest/nova/admin/hw_machine_type.html""")) return upgradecheck.Result(upgradecheck.Code.SUCCESS) + def _check_service_user_token(self): + if not CONF.service_user.send_service_user_token: + msg = (_(""" +Service user token configuration is required for all Nova services. +For more details see the following: +https://docs.openstack.org/latest/nova/admin/configuration/service-user-token.html""")) # noqa + return upgradecheck.Result(upgradecheck.Code.FAILURE, msg) + return upgradecheck.Result(upgradecheck.Code.SUCCESS) + # The format of the check functions is to return an upgradecheck.Result # object with the appropriate upgradecheck.Code and details set. If the # check hits warnings or failures then those should be stored in the @@ -294,6 +303,8 @@ https://docs.openstack.org/latest/nova/admin/hw_machine_type.html""")) (_('Older than N-1 computes'), _check_old_computes), # Added in Wallaby (_('hw_machine_type unset'), _check_machine_type_set), + # Added in Bobcat + (_('Service User Token Configuration'), _check_service_user_token), ) |