Skip to content

backport: Use P and Z GHA runners (#2215)#2326

Closed
Stringy wants to merge 1 commit intorelease-3.20from
giles/backport-pz-runners
Closed

backport: Use P and Z GHA runners (#2215)#2326
Stringy wants to merge 1 commit intorelease-3.20from
giles/backport-pz-runners

Conversation

@Stringy
Copy link
Collaborator

@Stringy Stringy commented Aug 5, 2025

Backport P&Z runners

@Stringy Stringy requested a review from a team as a code owner August 5, 2025 16:55
Comment on lines +177 to +178
- build-builder-image
- create-multiarch-manifest
Copy link
Contributor

Choose a reason for hiding this comment

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

I am not sure about the indentation here

Suggested change
- build-builder-image
- create-multiarch-manifest
- build-builder-image
- create-multiarch-manifest

@@ -208,15 +174,8 @@ jobs:

retag-x86-image:
Copy link
Contributor

Choose a reason for hiding this comment

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

Just guessing from the error in the last run

Suggested change
retag-x86-image:
retag-x86-image:
runs-on: ubuntu-24.04

Copy link
Collaborator

@Molter73 Molter73 left a comment

Choose a reason for hiding this comment

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

This will not be as straightforward, on top of #2215 you will need #2106 and #2210, as seen in the backport to 3.21: https://github.com/stackrox/collector/pull/2254/commits

Those other commits probably need even more cherry-picks for them to work. Unless you desperately need this on 3.20, I would just give up and deal with the 8 hour builds for a couple more weeks/months until it goes EOL (heck, that's what I did).

@ovalenti
Copy link
Contributor

ovalenti commented Aug 7, 2025

@Stringy, @Molter73 found out why the Z VM was failing to checkout (something to do with annotated tags), and so far the build is going smoothly 🤞

@Stringy Stringy closed this Aug 19, 2025
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