summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Revert "Add logging to find test cases leaking libvirt threads""27.0.0.0rc127.0.0Zuul2023-03-042-44/+0
|\
| * Revert "Add logging to find test cases leaking libvirt threads"Sylvain Bauza2023-02-142-44/+0
| | | | | | | | | | | | | | | | This reverts commit 1778a9c589cf24e17b44f556680b17af9577df11. Reason for revert: We said we wouldn't have it in RC1. Change-Id: Idf0c9a8adeac231f099b312fc24b9cf9726687e0
* | Merge "Add service version for Antelope"Zuul2023-03-032-10/+31
|\ \
| * | Add service version for AntelopeSylvain Bauza2023-03-022-10/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also did a bit of cleanup in the text message to tell *when* we need to bump the min service version. Given 2023.1 is our first SLURP release, we need to clarify the level of support we now have for rolling upgrades. Next cycle, we should update the min version to be Zed. Change-Id: I2dd906f34118da02783bb7755e0d6c2a2b88eb5d
* | | Merge "Add the 2023.1 Antelope prelude section"Zuul2023-03-021-0/+51
|\ \ \
| * | | Add the 2023.1 Antelope prelude sectionSylvain Bauza2023-03-011-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | First time we now use the correct release naming [1]. Shamelessly copied the bullet points from the cycle highlights [2]. [1] https://governance.openstack.org/tc/reference/release-naming.html [2] I02ed58bb5a4ecdc8171d9aa4a150be1bca214528 Change-Id: Id4e2e672e3a1a5aba7e664ba2d2f701b9be988e0
* | | | Merge "Fix logging in MemEncryption-related checks"Zuul2023-02-283-6/+36
|\ \ \ \ | |_|/ / |/| | |
| * | | Fix logging in MemEncryption-related checksAlexey Stupnikov2023-02-113-6/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently Nova produces ambigous error when volume-backed instance is started using flavor with hw:mem_encryption extra_specs flag: ImageMeta doesn't contain name if it represents Cinder volume. This fix sligtly changes steps to get image_meta.name for some MemEncryption-related checks where it could make any difference. Closes-bug: #2006952 Change-Id: Ia69e7cb18cd862f01ecfdbdc358c87af1ab8fbf6
* | | | Merge "Transport context to all threads"Zuul2023-02-274-29/+37
|\ \ \ \
| * | | | Transport context to all threadsFabian Wiesel2022-08-044-29/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The nova.utils.spawn and spawn_n methods transport the context (and profiling information) to the newly created threads. But the same isn't done when submitting work to thread-pools in the ComputeManager. The code doing that for spawn and spawn_n is extracted to a new function and called to submit the work to the thread-pools. Closes-Bug: #1962574 Change-Id: I9085deaa8cf0b167d87db68e4afc4a463c00569c
* | | | | Merge "Doc: update live-migration cmd"Zuul2023-02-271-1/+1
|\ \ \ \ \
| * | | | | Doc: update live-migration cmdArefiev Anton2023-02-241-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The deprecated `--live` option of the `server migrate` command was removed I37ef09eca0db9286544a4b0bb33f845311baa9b2 Update docs with new arguments. Change-Id: Id7a9a7509ca5e7811b6d3ce060390ea23c93d4ce
* | | | | Merge "doc: mark the max microversion for 2023.1 Antelope"Zuul2023-02-271-3/+4
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | doc: mark the max microversion for 2023.1 AntelopeTakashi Natsume2023-02-191-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Just follows the pattern that we do every release. Signed-off-by: Takashi Natsume <takanattie@gmail.com> Change-Id: Iaba463eedf8ec24303a18d5ba63087cd26ca16d3
* | | | | Merge "Use mysql memory reduction flags for ceph job"Zuul2023-02-241-0/+1
|\ \ \ \ \
| * | | | | Use mysql memory reduction flags for ceph jobDan Smith2023-02-211-0/+1
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes the ceph-multistore job use the MYSQL_REDUCE_MEMORY flag in devstack to try to address the frequent OOMs we see in that job. Change-Id: Ibc203bd10dcb530027c2c9f58eb840ccc088280d Closes-Bug: #1961068
* | | | | Add docs for cpu managementSylvain Bauza2023-02-231-0/+91
|/ / / / | | | | | | | | | | | | | | | | | | | | Relates to blueprint libvirt-cpu-state-mgmt Change-Id: Id22018a880b39f694a509c04cc6497ef42343049
* | | | Merge "fup: support evacuate target state"Zuul2023-02-184-6/+5
|\ \ \ \
| * | | | fup: support evacuate target stateSahid Orentino Ferdjaoui2023-02-064-6/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fix minors comments that have been noticed. Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@industrialdiscipline.com> Change-Id: Iee6c224aa0f26d8550b38a8f69b28d8648b1da70
* | | | | Merge "Enable cpus when an instance is spawning"Zuul2023-02-189-0/+625
|\ \ \ \ \
| * | | | | Enable cpus when an instance is spawningSylvain Bauza2023-02-109-0/+625
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By this patch, we now automatically power down or up cores when an instance is either stopped or started. Also, by default, we now powersave or offline dedicated cores when starting the compute service. Implements: blueprint libvirt-cpu-state-mgmt Change-Id: Id645fd1ba909683af903f3b8f11c7f06db3401cb
* | | | | | Merge "libvirt: let CPUs be power managed"Zuul2023-02-185-5/+53
|\ \ \ \ \ \ | |/ / / / /
| * | | | | libvirt: let CPUs be power managedSylvain Bauza2023-02-105-5/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before going further, we need to somehow return the list of CPUs even offline if they are power managed by Nova. Co-Authored-By: Sean Mooney <smooney@redhat.com> Partially-Implements: blueprint libvirt-cpu-state-mgmt Change-Id: I5dca10acde0eff554ed139587aefaf2f5fad2ca5
* | | | | | Merge "libvirt: Add configuration options to set SPICE compression settings"Zuul2023-02-177-1/+204
|\ \ \ \ \ \
| * | | | | | libvirt: Add configuration options to set SPICE compression settingsManuel Bentele2023-01-117-1/+204
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds the following SPICE-related options to the 'spice' configuration group of a Nova configuration: - image_compression - jpeg_compression - zlib_compression - playback_compression - streaming_mode These configuration options can be used to enable and set the SPICE compression settings for libvirt (QEMU/KVM) provisioned instances. Each configuration option is optional and can be set explictly to configure the associated SPICE compression setting for libvirt. If all configuration options are not set, then none of the SPICE compression settings will be configured for libvirt, which corresponds to the behavior before this change. In this case, the built-in defaults from the libvirt backend (e.g. QEMU) are used. Note that those options are only taken into account if SPICE support is enabled (and the VNC support is disabled). Implements: blueprint nova-support-spice-compression-algorithm Change-Id: Ia7efeb1b1a04504721e1a5bdd1b5fa7a87cdb810
* | | | | | | Merge "Factor out a mixin class for candidate aware filters"Zuul2023-02-153-49/+65
|\ \ \ \ \ \ \
| * | | | | | | Factor out a mixin class for candidate aware filtersBalazs Gibizer2023-01-203-49/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | blueprint: pci-device-tracking-in-placement Change-Id: Id044131162e582cef41e424aa751e3cd3618ed0c
* | | | | | | | Merge "cpu: interfaces for managing state and governor"Zuul2023-02-1510-0/+466
|\ \ \ \ \ \ \ \ | | |_|/ / / / / | |/| | | | | |
| * | | | | | | cpu: interfaces for managing state and governorSylvain Bauza2023-02-0910-0/+466
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the first stage of the power management series. In order to be able to switch the CPU state or change the governor, we need a framework to access sysfs. As some bits can be reused, let's create a nova.filesystem helper module that will define read-write mechanisms for accessing sysfs-specific commands. Partially-Implements: blueprint libvirt-cpu-state-mgmt Change-Id: Icb913ed9be8d508de35e755a9c650ba25e45aca2
* | | | | | | | Merge "Add logging to find test cases leaking libvirt threads"Zuul2023-02-142-0/+44
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / |/| | | | | | / | | |_|_|_|_|/ | |/| | | | |
| * | | | | | Add logging to find test cases leaking libvirt threadsSylvain Bauza2023-02-102-0/+44
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We see functional test failures due to leaked libvirt event handling thread weaking up after its original test finished and importing libvirt. If it happens when the libvirt package import is poisoned then the currently executing test will fail. This patch logs the name of the test case that leaked the libvirt event handling thread. We will revert his before RC1. Change-Id: I3146e9afb411056d004fc118ccfa31126a3c6b15 Related-Bug: #1946339
* | | | | | Add docs for stable-compute-uuid behaviorsDan Smith2023-02-083-0/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds some admin guide documentation about the stable compute_id file. It covers upgrade, greenfield generation, and greenfield pre-provisioning by deployment tools. Related to blueprint stable-compute-uuid Change-Id: I078b3f9e1919f2008628dc7b889e8696f1f6159a
* | | | | | Merge "Stable compute uuid functional tests"Zuul2023-02-082-0/+105
|\ \ \ \ \ \
| * | | | | | Stable compute uuid functional testsDan Smith2023-02-012-0/+105
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a number of functional test cases for the stable-compute-uuid error cases. Specifically around checks and aborted startups to make sure we're catching what we expect, and failing in the appropriate ways. Related to blueprint stable-compute-uuid Change-Id: I8bcb93a6887ed06dbd4b7c28c93a20a3705a6077
* | | | | | | Merge "Abort startup if nodename conflict is detected"Zuul2023-02-086-2/+49
|\ \ \ \ \ \ \ | |/ / / / / / | | / / / / / | |/ / / / / |/| | | | |
| * | | | | Abort startup if nodename conflict is detectedDan Smith2023-02-016-2/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We do run update_available_resource() synchronously during service startup, but we only allow certain exceptions to abort startup. This makes us abort for InvalidConfiguration, and makes the resource tracker raise that for the case where the compute node create failed due to a duplicate entry. This also modifies the object to raise a nova-specific error for that condition to avoid the compute node needing to import oslo_db stuff just to be able to catch it. Change-Id: I5de98e6fe52e45996bc2e1014fa8a09a2de53682
* | | | | | Merge "Move comment about _destroy_evacuated_instances()"Zuul2023-02-071-6/+6
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Move comment about _destroy_evacuated_instances()Dan Smith2023-02-011-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Per feedback in I595b27a57516cffe1f172cf2fb736e1b11373a1d, this moves a comment about _destroy_evacuated_instances() further down closer to where that method is called. Change-Id: I72812918b26bf20d6ab5289f5eb855b23cc83881
* | | | | | Merge "Fix 6.2 compute RPC version alias"Zuul2023-02-071-1/+1
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Fix 6.2 compute RPC version aliasTobias Urdin2023-02-061-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The alias for the 6.2 compute RPC version says antilope but that is spelled incorrectly and should be antelope. Change-Id: I0a1e47baf9f8115cf1d02d57659a88b26eae89d0
* | | | | Protect against a deleted node id fileDan Smith2023-02-014-8/+42
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we are starting up for the first time, we expect to generate and write a node_uuid file if one does not exist. If we are upgrading, we expect to do the same. However, if we are starting up not after an upgrade and not for the first time, a missing compute_id file is an error, and we should abort. Because of the additional check that this adds, which is called from a variety of places that don't have the stable compute node singleton stubbed to make it happy, this mocks the check for any test that does not specifically aim to exercise it. Related to blueprint stable-compute-uuid Change-Id: If83ce14b96e7d84ae38eba9d798754557d5abdfd
* | | | Check our nodes for hypervisor_hostname changesDan Smith2023-02-016-24/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When we are loading our ComputeNode objects by UUID according to what the virt driver reported, we can sanity check the DB records against the virt driver's hostname. This covers the case where our CONF.host has not changed but the hostname reported by the virt driver has, assuming we can find the ComputeNode object(s) that match our persistent node uuid. Related to blueprint stable-compute-uuid Change-Id: I41635210d7d6f46b437b06d2570a26a80ed8676a
* | | | Merge "api: extend evacuate instance to support target state"Zuul2023-02-0127-27/+252
|\ \ \ \
| * | | | api: extend evacuate instance to support target stateSahid Orentino Ferdjaoui2023-01-3127-27/+252
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Start to v2.95 any evacuated instances will be stopped a destination Implements: bp/allowing-target-state-for-evacuate Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@industrialdiscipline.com> Change-Id: I141b6f057cc4eb9c541c2bc6eddae27270ede08d
* | | | | Merge "compute: enhance compute evacuate instance to support target state"Zuul2023-02-0115-65/+201
|\ \ \ \ \ | |/ / / /
| * | | | compute: enhance compute evacuate instance to support target stateSahid Orentino Ferdjaoui2023-01-3115-65/+201
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related to the bp/allowing-target-state-for-evacuate. This change is extending compute API to accept a new argument targetState. The targetState argument when set will force state of an evacuated instance to the destination host. Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@industrialdiscipline.com> Change-Id: I9660d42937ad62d647afc6be965f166cc5631392
* | | | | Merge "Detect host renames and abort startup"Zuul2023-02-013-12/+101
|\ \ \ \ \
| * | | | | Detect host renames and abort startupDan Smith2023-01-303-12/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Except on ironic, if we are able to determine that our locally- configured node has a hostname other than what we expect, we should abort startup. Since we currently depend on the loose name-based association of nodes, services, and instances, we need to make sure we do not startup in an inconsistent configuration. Related to blueprint stable-compute-uuid Change-Id: I595b27a57516cffe1f172cf2fb736e1b11373a1d
* | | | | | Merge "Add further workaround features for qemu_monitor_announce_self"Zuul2023-01-314-10/+97
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add further workaround features for qemu_monitor_announce_selfas02023-01-304-10/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some cases on Arista VXLAN fabrics, VMs are inaccessible via network after live migration, despite garps being observed on the fabric itself. This patch builds on the feature ``[workarounds]/enable_qemu_monitor_announce_self`` feature as reported in `bug 1815989 <https://bugs.launchpad.net/nova/+bug/1815989>` This patch adds the ability to config the number of times the QEMU announce_self monitor command is called, and add a new configuration option to specify a delay between calling the announce_self command multiple times, as in some cases, multiple announce_self monitor commands are required for the fabric to honor the garp packets and the VM to become accessible via the network after live migration. Closes-Bug: #1996995 Change-Id: I2f5bf7c9de621bb1dc7fae5b3374629a4fcc1f46