Skip to content

Synchronise 2026.1 with upstream - #941

Merged
Alex-Welsh merged 1 commit into
stackhpc/2026.1from
upstream/2026.1-2026-09-04
Sep 4, 2026
Merged

Alex-Welsh merged 1 commit into
stackhpc/2026.1from
upstream/2026.1-2026-09-04

Conversation

@github-actions

@github-actions github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

This PR contains a snapshot of 2026.1 from upstream stable/2026.1.

check_image() from kolla_docker_worker returns None when the image
doesn't exist locally. This means that when compare_labels tries to read
a value from it, it will error out. This chain happens when comparing an
image that doesn't exist locally. In the past we didn't hit this because
checks ran sequentially and it would exit before hitting this. A recent
change [1] means that now all checks run always.

This patch fixes the issue in two ways. First, it makes the docker
worker return an empty dict from check_image when it doesn't exist.
Second, compare_labels now truthy-checks the output before proceeding,
and returns straight away if it's False.

[1] https://review.opendev.org/c/openstack/kolla-ansible/+/978162

Change-Id: I3c33ac3a474f80bca100005f61c1c0bfedc46170
Signed-off-by: Alex Welsh <alex@stackhpc.com>
Closes-Bug: #2166024
(cherry picked from commit ffc50e8)
@github-actions
github-actions Bot requested a review from a team as a code owner September 4, 2026 09:02
@github-actions github-actions Bot added automated Automated action performed by GitHub Actions synchronisation labels Sep 4, 2026
@Alex-Welsh
Alex-Welsh merged commit 35aa1c2 into stackhpc/2026.1 Sep 4, 2026
3 checks passed
@Alex-Welsh
Alex-Welsh deleted the upstream/2026.1-2026-09-04 branch September 4, 2026 09:05
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.

1 participant