Skip to content

Backport konflux changes to 3.20 (stackrox 4.6) #2176

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 66 commits into
base: release-3.20
Choose a base branch
from

Conversation

Molter73
Copy link
Collaborator

@Molter73 Molter73 commented Jun 16, 2025

Description

The list of commits to add into the branch was retrieved with the following command:

git log --oneline release-3.20...master -- rpms.* .tekton .konflux .github/workflows/konflux.yml collector/container/konflux.Dockerfile .github/CODEOWNERS .syft.yaml

The commits were then cherry-picked by reverting the list in neovim like this:

.! git log --oneline release-3.20...master -- rpms.* .tekton .konflux .github/workflows/konflux.yml collector/container/konflux.Dockerfile .github/CODEOWNERS .syft.yaml
g/^/m0

Had to manually remove some commits from the command because they either had already been cherry-picked on a previous PR and were producing empty commits or the changed themselves were not needed for konflux (i.e updating a dependency that shouldn't be updated in this release needed changes on the konflux dockerfile).

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

CI builds konflux images and runs tests on them correctly.

@Molter73 Molter73 force-pushed the mauro/konflux-backport-3.20 branch from 207eaf2 to 255ce9a Compare June 16, 2025 15:24
@Molter73 Molter73 marked this pull request as ready for review June 17, 2025 09:30
@Molter73 Molter73 requested review from a team as code owners June 17, 2025 09:30
@Molter73 Molter73 force-pushed the mauro/konflux-backport-3.20 branch from 255ce9a to 59c1e4a Compare June 17, 2025 11:17
Copy link
Contributor

@msugakov msugakov left a comment

Choose a reason for hiding this comment

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

red-hat-konflux bot and others added 21 commits June 20, 2025 11:04
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Since konflux always builds multiarch images and we always try to test
them, we should force the test container to build in multiarch mode. If
we don't, then we get test failures on the konflux integration tests for
PRs due to the tester image not existing.
…ux Dockerfiles (#2002)

* Add comment not to delete source-location LABEL from Konflux Dockerfiles

* Update collector/container/konflux.Dockerfile

Co-authored-by: Misha Sugakov <[email protected]>

---------

Co-authored-by: Misha Sugakov <[email protected]>
red-hat-konflux bot and others added 26 commits June 20, 2025 11:04
…est to fa86065 (#2119)

Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com>
* ROX-29078: switch to per component service account

* move SA def to PipelineRun
…est to b4f8de3 (#2136)

Signed-off-by: red-hat-konflux <126015336+red-hat-konflux[bot]@users.noreply.github.com>
Co-authored-by: red-hat-konflux[bot] <126015336+red-hat-konflux[bot]@users.noreply.github.com>
@Molter73 Molter73 force-pushed the mauro/konflux-backport-3.20 branch from 59c1e4a to 8d9cc13 Compare June 20, 2025 09:04
@Molter73 Molter73 requested a review from msugakov June 23, 2025 08:44
Copy link
Contributor

@msugakov msugakov left a comment

Choose a reason for hiding this comment

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

Looks good Konflux-wise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants