Skip to content

Conversation

github-actions[bot]
Copy link

This PR contains a snapshot of wallaby from upstream unmaintained/wallaby.

Elod Illes and others added 4 commits August 16, 2024 18:11
Latest Zuul drops the following warnings:

  All regular expressions must conform to RE2 syntax, but an
  expression using the deprecated Perl-style syntax has been detected.
  Adjust the configuration to conform to RE2 syntax.

  The RE2 syntax error is: invalid perl operator: (?!

This patch replaces the 'irrelevant-files' to 'files' with explicitly
listing the pattern which files should be the tests run against.

Conflicts:
  .zuul.yaml

NOTE(elod.illes): parts of the Xena version of this patch were not
needed as they were not introduced in Wallaby yet.
Also, devstack-gate based base job is dropped as it is not used in
zuul.yaml at all and devstack-gate is retired and removed completely.

Change-Id: If287e800fb9ff428dbe6f9c4c046627f22afe3df
(cherry picked from commit 9b77bae)
(cherry picked from commit 8223e6a)
(cherry picked from commit 510a27b)
(cherry picked from commit 197b14d)
(cherry picked from commit ffc252e)
(cherry picked from commit 23fc1b9)
(cherry picked from commit 6fadd99)
This patch concerns the time when a VM is being unshelved and the
compute manager set the task_state to spawning, claimed resources of
the VM and then called driver.spawn(). So the instance is in vm_state
SHELVED_OFFLOADED, task_state spawning.

If at this point a new update_available_resource periodic job is
started that collects all the instances assigned to the node to
calculate resource usage. However the calculation assumed that a
VM in SHELVED_OFFLOADED state does not need resource allocation on
the node (probably being removed from the node as it is offloaded)
and deleted the resource claim.

Given all this we ended up with the VM spawned successfully but having
lost the resource claim on the node.

This patch changes what we do in vm_state SHELVED_OFFLOADED, task_state
spawning. We no longer delete the resource claim in this state and
keep tracking the resource in stats.

Change-Id: I8c9944810c09d501a6d3f60f095d9817b756872d
Closes-Bug: #2025480
(cherry picked from commit f1dc4ec)
(cherry picked from commit 4239d1f)
(cherry picked from commit 683ecc0)
(cherry picked from commit e41962f)
(cherry picked from commit 22dfd1f)
(cherry picked from commit 26a9cae)
@github-actions github-actions bot added automated Automated action performed by GitHub Actions synchronisation labels Jan 20, 2025
@github-actions github-actions bot requested a review from a team as a code owner January 20, 2025 06:25
@priteau priteau closed this Jan 20, 2025
@priteau priteau reopened this Jan 20, 2025
@priteau priteau merged commit 20b64ed into stackhpc/wallaby Jan 20, 2025
0 of 3 checks passed
@priteau priteau deleted the upstream/wallaby-2025-01-20 branch January 20, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated Automated action performed by GitHub Actions synchronisation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants