Skip to content

MLE-32962: Upgrade image pull is skipped for ARM images and inconsistently skipped for rootless images - #486

Merged
vitalykorolev merged 3 commits into
developfrom
MLE-32962_fix-arm-rootless-upgrade-images
Sep 29, 2026
Merged

vitalykorolev merged 3 commits into
developfrom
MLE-32962_fix-arm-rootless-upgrade-images

Conversation

@vitalykorolev

Copy link
Copy Markdown
Collaborator

Summary

Fixes two issues in the upgrade image handling logic: ARM image types were always skipping the upgrade image pull/test entirely instead of pulling the correct architecture-matched image, and the rootless skip condition was inconsistent across rootless variants. Also removes an upgrade test skip in Robot Framework keywords that is no longer needed now that ARM upgrade images are pulled correctly.

Root cause

pullUpgradeDockerImage() in the Jenkinsfile unconditionally skipped ARM image types, and its rootless skip check only matched the literal string "ubi-rootless" (missing "ubi9-rootless" and "ubi9-rootless-arm"), while also relying on a string comparison against params.DOCKER_TESTS instead of the resolved DOCKER_TESTS boolean.

Fix

Updated pullUpgradeDockerImage() to pull the matching ARM image type's latest-<majorVersion> tag instead of skipping, changed the rootless check to dockerImageType.contains("rootless") so it applies uniformly across all rootless variants, gated the skip on the DOCKER_TESTS boolean, and removed the now-unnecessary Skip If 'arm' in '${IMAGE_TYPE}' line from the Create upgrade container with keyword in test/keywords.resource.

Validation

This comment was marked as outdated.

This comment was marked as outdated.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

A critical ARM platform mismatch remains in the upgrade-image pull path.

Review effort: Lite
Findings: 1 High severity

Open (1)
Resolved since last review (1)

Comment thread Jenkinsfile
@vitalykorolev
vitalykorolev merged commit 7f9ec40 into develop Sep 29, 2026
6 checks passed
@vitalykorolev
vitalykorolev deleted the MLE-32962_fix-arm-rootless-upgrade-images branch September 29, 2026 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants