Skip to content
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

feat: enable multi-arch fbc verification #1743

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

arewm
Copy link
Member

@arewm arewm commented Dec 10, 2024

While the FBC builder pipeline has been changed to be multi-arch aware, the validate-fbc task will cause EC failures since it does not report that the results are valid for all of the OCI image manifest digests. Even if we are not performing additional verification on the architecture specific images, we need to set the IMAGES_PROCESSED result so that EC will pass.

@arewm arewm requested a review from a team as a code owner December 10, 2024 16:22
@arewm arewm force-pushed the validate-fbc-return-images-processed branch 4 times, most recently from f468540 to ec94719 Compare December 10, 2024 18:14
While the FBC builder pipeline has been changed to be multi-arch aware,
the validate-fbc task will cause EC failures since it does not report
that the results are valid for all of the OCI image manifest digests.
Even if we are not performing additional verification on the
architecture specific images, we need to set the IMAGES_PROCESSED result
so that EC will pass.

Signed-off-by: arewm <[email protected]>
@arewm arewm force-pushed the validate-fbc-return-images-processed branch from ec94719 to 6e9fb2f Compare December 10, 2024 18:17
@arewm arewm enabled auto-merge December 10, 2024 18:19
@arewm arewm added this pull request to the merge queue Dec 10, 2024
Merged via the queue into konflux-ci:main with commit 3462d3e Dec 10, 2024
16 checks passed
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.

2 participants