Commit Graph

60787 Commits

Author SHA1 Message Date
Zuul bded279a00 Merge "docs: Add more information about unified limits" 2024-05-23 19:48:39 +00:00
Zuul d7d2fb1edd Merge "scheduler: AggregateMultitenancyIsolation to support unlimited tenant" 2024-05-23 15:35:54 +00:00
Zuul 4e3a41f0a4 Merge "Stop using split UEC image (mostly)" 2024-05-22 19:03:32 +00:00
Zuul 3a53d715cd Merge "[doc] Improve description for nova-manage db purge" 2024-05-22 02:44:50 +00:00
Alexey Stupnikov ac8729ac87 [doc] Improve description for nova-manage db purge
--before argument is currently described in ambiguous way: it
is not actually used to filter entries ARCHIVED before specified
date. Instead, it compares provided data with "deleted_at" value
for most rows and "updated_at" or "created_at" for remaining ones.

Since we already talk about time of deletion when describing
--before argument of "nova-manage db archive_deleted_rows" rows,
it make sense to not provide extra details here as well.

Change-Id: Ib5940e88a52dc8d32303e27237e567c3481fc3dc
2024-05-21 20:19:18 +02:00
Zuul 4bc5ff1c99 Merge "fix py312 tox definitions" 2024-05-21 17:43:27 +00:00
Zuul c60b81fa4b Merge "tox: Drop envdir" 2024-05-21 15:17:45 +00:00
Sean Mooney 7ff24958ee fix py312 tox definitions
change I6de86f3e3e283ba404f927ea4c8164f791df3989
added the py312 funtional job definition but did not
update the tox.ini to define it.
As a result it is runing the unit tests not the functional tests.
This change simpley corrects that.

Change-Id: Id6ee76e0190469ac09baf0bc56a9022317c6f881
2024-05-21 12:03:41 +01:00
Zuul 61ad4f1f27 Merge "Enable virtio-scsi in nova-next" 2024-05-20 21:41:24 +00:00
Dan Smith eed3e2b47f Stop using split UEC image (mostly)
This reverts us back to using the standard disk image for most of our
tests, which is more representative of how people actually use nova.
This leaves the UEC image on a few jobs for the sake of comparison
data for the time being, and because we should actually test that
code path if we're going to say we support it.

Change-Id: I16ed92d342464325d4bef33c1e22b328bcfbe7d6
2024-05-20 08:24:18 -07:00
Dan Smith 84b0a481fe Enable OCaaS for several nova jobs
This uses the OCaaS feature in devstack which saves *minutes* of time
running devstack by effectively caching the openstackclient startup
instead of taking that hit for every invocation.

Change-Id: I78308128c6249f7f871e0231ce717b1ec0f88509
2024-05-20 08:24:18 -07:00
melanie witt d45379c6b4 docs: Add more information about unified limits
The admin docs are missing some details about enabling unified limits,
like oslo.limit configuration and Keystone roles. This adds more
information about what roles are needed for what actions, how to set
quota limits, quota enforcement, and unified limits in general.

This also removes a couple of tables from the user docs that show
obsolete/deprecated quota limits because they may be more confusing
than helpful considering we don't want new deployments to use them and
they add more clutter to the page.

More info is also added regarding the CLI commands for unified limits
and makes it consistent between the user and admin docs.

Change-Id: Id93f9997d1b217e0c2151c88323564f7a7fefc02
2024-05-17 19:17:05 +00:00
Zuul ab3ca1e205 Merge "Make python 3.12 unit and functional voting" 2024-05-16 15:42:38 +00:00
Zuul b6a846c6c8 Merge "Fix hacking test with syntax error" 2024-05-16 14:26:26 +00:00
Zuul fa44978141 Merge "Fix notification object hashes for python 3.12" 2024-05-16 14:26:18 +00:00
Pierre Riteau bfd3525863 Fix formatting issues in extra-specs docs
Change-Id: I693f70ffb7630fe99336fef52783ea55c492624d
2024-05-16 11:08:19 +02:00
Zuul d467edac49 Merge "do not use str(url) to stringify a URL for subsequent use" 2024-05-15 20:05:28 +00:00
Dan Smith 50b180023f Make python 3.12 unit and functional voting
Change-Id: I6de86f3e3e283ba404f927ea4c8164f791df3989
2024-05-15 12:29:14 -07:00
Dan Smith 3f0879ccc3 Fix hacking test with syntax error
This hacking test has a syntax error in it. On older pythons, this
does not prevent us from finding the second popen() use, but on
python 3.12 it does.

Change-Id: Ib74dc030118e0cb9fab548b112d32ce080969a15
2024-05-15 12:12:55 -07:00
Dan Smith 6ee938fd22 Fix notification object hashes for python 3.12
Python 3.12 changes the repr() syntax for the OrderedDict object,
which causes us to calculate different hashes for the notification
objects across the version gap. The get_extra_data() function wraps
a dict in OrderedDict after sorting the elements of the original dict,
but that is not entirely necessary. The sorted() list of dict.items()
is a perfectly reasonable representation of the things it is capturing,
so we can just eliminate the use of OrderedDict there and thus end
up calculating the same hashes on <3.12 and >3.12.

NOTE: This changes a bunch of the hashes of notification objects
without bumping or changing versions. This is expected purely because
we are changing the hashing method, and thus this is not a violation
or upgrade concern.

Change-Id: I242150138deed7fe74b13d9c44b333293cd24ffa
2024-05-15 12:12:54 -07:00
Zuul 2e6041a76e Merge "Remove SQLAlchemy tips jobs" 2024-05-15 16:51:17 +00:00
Mike Bayer acbe3e28e5 do not use str(url) to stringify a URL for subsequent use
The str(url) function in SQLAlchemy hides the password.
For a URL string that is to be re-used, use
render_as_string(hide_password=False).

Change-Id: I2ab28da5cc2b9ed3a1588259b2e94320662816bb
2024-05-15 16:23:58 +01:00
Zuul 5095336689 Merge "Upload glance image with --file in ceph job" 2024-05-14 16:40:32 +00:00
Zuul 7096423b34 Merge "Reject AZ changes during aggregate add / remove host" 2024-05-09 20:17:32 +00:00
Zuul 5470dedd4d Merge "Fix device_type=lun with boot_index" 2024-05-09 17:32:28 +00:00
Zuul 67119b7de3 Merge "Avoid setting serial on raw LUN devices" 2024-05-08 18:36:11 +00:00
Dan Smith a4e72f71fc Upload glance image with --file in ceph job
This enables our use of the OCaaS devstack feature, which can't support
image upload from stdin.

Change-Id: Idc5646ef6763447e3c1c68de03dea8197c305f6c
2024-05-08 10:57:06 -07:00
Zuul 114b8184e4 Merge "Make overcommit check for pinned instance pagesize aware" 2024-05-08 13:55:26 +00:00
Balazs Gibizer 3c0eadae0b Reject AZ changes during aggregate add / remove host
After this patch nova rejects the add host to aggregate API action
if the host has instances and the new aggregate for the host would
mean that these instances need to move from one AZ (even from the
default one) to another. Such AZ change is not implemented in nova
and currently leads to stuck instances.

Similarly nova will reject remove host from aggregate API action if the
host has instances and the aggregate removal would mean that the
instances need to change AZ.

Depends-On: https://review.opendev.org/c/openstack/tempest/+/821732

Change-Id: I19c4c6d34aa2cc1f32d81e8c1a52762fa3a18580
Closes-Bug: #1907775
2024-05-08 14:56:56 +02:00
Zuul 95bfa492e9 Merge "[ironic] Fix rebooting instance" 2024-05-08 01:10:34 +00:00
Dan Smith 32546d9c1b Enable virtio-scsi in nova-next
This lets us test the direct-lun volume attachment model.

Change-Id: Ibc7bff377cc5b5572e2a11006116401babaac347
Depends-On: https://review.opendev.org/c/openstack/tempest/+/918457
2024-05-07 11:49:21 -07:00
Dan Smith 2f0c340d39 Fix device_type=lun with boot_index
Right now we'll fail to calculate the boot order of a set of BDMs if
one of them is a device_type=lun. This fixes that and teaches us
that it's just a "hd" from qemu's perspective.

Closes-Bug: #2065084
Change-Id: Ic1340918738d503fc797c9373fe2e1dd16b27a09
2024-05-07 11:14:30 -07:00
Dan Smith 575ff86a4f Avoid setting serial on raw LUN devices
Libvirt now enforces that device="lun" (i.e. raw device passthrough)
disks must not have the <serial> property set. We recently enabled
the ability to manage devices by alias instead of serial, but to
fully enable this use-case we need to avoid putting serial in the
XML to appease libvirt.

Related-Bug: #2065084
Change-Id: Ifa2df89f27e58e1e64ce046edeaf6e49a7c89490
2024-05-07 10:39:49 -07:00
Vasyl Saienko 0e766885f6 [ironic] Fix rebooting instance
The correct state for hard and soft reboots are rebooting [0]

[0] https://github.com/openstack/openstacksdk/blob/master/openstack/baremetal/v1/node.py#L44

Closes-Bug: #2064826
Change-Id: I18e0352b3638872e85ce91a3cfcbbfddc812ab67
2024-05-07 20:39:31 +03:00
Zuul 428990e7b7 Merge "Do not close returned image-chunk iterator & get_verifier early" 2024-05-06 17:57:32 +00:00
Zuul 07f05add31 Merge "api: Keep track of action controllers" 2024-05-03 00:52:47 +00:00
Zuul ca5be99837 Merge "Remove old excludes" 2024-05-01 16:59:06 +00:00
Takashi Kajinami b4ff81c329 Remove old excludes
These are detected as errors since the clean up was done[1] in
the requirements repository.

[1] 314734e938f107cbd5ebcc7af4d9167c11347406

Bump the minimum versions to avoid installing these known bad versions.

Change-Id: I5ab0c3a1ac208e3967e65c298573079283a7b6cd
2024-05-01 01:30:04 +09:00
OpenStack Release Bot fa678870b2 reno: Update master for unmaintained/zed
Update the zed release notes configuration to build from
unmaintained/zed.

Change-Id: I2f9a940845996d8870936df81ab35fa3908df82b
2024-04-30 15:00:34 +00:00
Zuul a031921f42 Merge "Note the deleyad address view" 2024-04-29 18:59:37 +00:00
afazekas 25c13220c6 Note the deleyad address view
The addresses field containes delayed (cached) information,
due to internal implementation details.
This used to be considered API contact breakage,
but apperently no plan for fixing it.

Change-Id: I8db2800ede0276d60138369ec2d60c55a1791040
2024-04-29 17:30:10 +00:00
Sahid Orentino Ferdjaoui 3cd26b0a08 scheduler: AggregateMultitenancyIsolation to support unlimited tenant
This commit removes the previous limitation on the number of tenants
that can be filtered using the `filter_tenant_id` aggregate property
in the AggregateMultitenancyIsolation scheduler filter.

The `filter_tenant_id` can now be used as a suffix, allowing for an
unlimited number of tenant ID properties to be set on the
aggregate. This update maintains backward compatibility.

Implements: blueprint aggregatemultitenancyisolation-to-support-unlimited-tenant
Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@industrialdiscipline.com>
Change-Id: Ic87d142647774b62a6af2cc5eb7a3cd66f9afeb7
2024-04-26 07:07:03 +00:00
Fabian Wiesel 198805c7c5 Do not close returned image-chunk iterator & get_verifier early
The GlanceClientWrapper._get_verifier method may fail already on the
metadata, so we better call it early before we open files and start
downloads, which we then abort uncleanly.

This also likely how the bug #1948706 was triggered in the first place:
- The file gets opened
- _get_verifier fails *before* we even iterate over the data
- glance_utils.IterableWithLength won't close the underlying iterator.

The added close statement, now guarded with `may_close_iterator` is
likely superfluous.

If we return the image chunk iterator, then we should rather not
close the underlying iterable, as it will kill the transfer.

Closes-Bug: #2053027
Change-Id: Ia247af39a96fbed90b027ad30158e66dd2f0bd5e
2024-04-25 13:58:12 +02:00
Zuul ca1db54f1b Merge "Fix: migration configuration with cpu_shared_set (libvirt part)" 2024-04-24 20:37:35 +00:00
Zuul 95b4ef6fa4 Merge "Fix: migration configuration with cpu_shared_set (object part)" 2024-04-24 19:58:56 +00:00
René Ribaud 43dadaeb90 Fix: migration configuration with cpu_shared_set (libvirt part)
Live migrating to a host with cpu_shared_set configured will now
update the VM's configuration accordingly.

Example: live migrating a VM from source host with cpu_shared_set=0,1
to destination host with cpu_shared_set=2,3 will now update the
VM configuration.
(<vcpu cpuset="0-1"> will be updated to <vcpu cpuset="2-3">).

Related-Bug: #1869804
Change-Id: I7c717503eba58088094fac05cb99b276af9a3460
2024-04-24 07:59:15 -07:00
René Ribaud 2c3f4f2da5 Fix: migration configuration with cpu_shared_set (object part)
Live migrating to a host with cpu_shared_set configured will now
update the VM's configuration accordingly.

Example: live migrating a VM from source host with cpu_shared_set=0,1
to destination host with cpu_shared_set=2,3 will now update the
VM configuration.
(<vcpu cpuset="0-1"> will be updated to <vcpu cpuset="2-3">).

This update adds a new field, dst_cpu_shared_set_info, to the
LibvirtLiveMigrateData object, which requires an increase in the
object's version. As a result, this patch cannot be backported.

Related-Bug: #1869804
Change-Id: I806da0958fe436c989e09a52ca6b6f1bbd25a865
2024-04-24 07:53:38 -07:00
Zuul e2ef2240b1 Merge "api: Remove FlavorManageController" 2024-04-22 19:58:38 +00:00
Zuul 3abea334d5 Merge "Validate flavor image min ram when resize volume-backed instance" 2024-04-19 23:35:17 +00:00
Zuul eeb69cbd08 Merge "Regression test for bug 2007968" 2024-04-19 22:48:59 +00:00