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

source-build: read base image from SBOM #1152

Merged

Conversation

chmeliik
Copy link
Contributor

@chmeliik chmeliik commented Jul 11, 2024

STONEBLD-2607

When the BASE_IMAGES param is empty, fall back to reading the base image from the SBOM of the BINARY_IMAGE.

This removes the source-build task's dependency on the BASE_IMAGES_DIGESTS result, finally allowing us to stop returning this result from build tasks.

Before you complete this pull request ...

Look for any open pull requests in the repository with the title "e2e-tests update" and
see if there are recent e2e-tests updates that will be applicable to your change.

@chmeliik chmeliik force-pushed the sourcebuild-get-base-images-from-sboms branch 2 times, most recently from 8839358 to e3ced4f Compare July 11, 2024 12:35
@chmeliik chmeliik force-pushed the sourcebuild-get-base-images-from-sboms branch from e3ced4f to 75adfb1 Compare July 11, 2024 12:42
@chmeliik
Copy link
Contributor Author

timed out when waiting for image-controller annotations to be updated on component test-comp-sgjs in namespace build-templates-e2e. component: {

/retest

@chmeliik
Copy link
Contributor Author

timed out when waiting for image-controller annotations to be updated on component test-comp-ilms

again 😞
looking

@chmeliik
Copy link
Contributor Author

no idea
/retest

@chmeliik chmeliik force-pushed the sourcebuild-get-base-images-from-sboms branch from 75adfb1 to 03822ff Compare July 11, 2024 13:52
@chmeliik
Copy link
Contributor Author

Rebased on main. That should fix it 🤦

@chmeliik chmeliik force-pushed the sourcebuild-get-base-images-from-sboms branch from 03822ff to f00fdae Compare July 12, 2024 11:14
@chmeliik
Copy link
Contributor Author

/retest

Copy link
Contributor

@tkdchen tkdchen left a comment

Choose a reason for hiding this comment

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

LGTM. There are a few minor comments.

task/source-build/0.1/source-build.yaml Outdated Show resolved Hide resolved
task/source-build/0.1/source-build.yaml Show resolved Hide resolved
task/source-build/0.1/source-build.yaml Show resolved Hide resolved
@chmeliik chmeliik force-pushed the sourcebuild-get-base-images-from-sboms branch from f00fdae to c9c3918 Compare July 15, 2024 08:33
STONEBLD-2607

When the BASE_IMAGES param is empty, fall back to reading the base image
from the SBOM of the BINARY_IMAGE.

This removes the source-build task's dependency on the
BASE_IMAGES_DIGESTS result, finally allowing us to stop returning this
result from build tasks.

Signed-off-by: Adam Cmiel <[email protected]>
@chmeliik chmeliik force-pushed the sourcebuild-get-base-images-from-sboms branch from c9c3918 to 89ed04c Compare July 15, 2024 08:36
@chmeliik
Copy link
Contributor Author

Re-generated the oci-ta variant

Copy link
Contributor

@tkdchen tkdchen left a comment

Choose a reason for hiding this comment

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

👍

@chmeliik chmeliik added this pull request to the merge queue Jul 15, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 15, 2024
@chmeliik chmeliik added this pull request to the merge queue Jul 15, 2024
Merged via the queue into konflux-ci:main with commit 5fd4314 Jul 15, 2024
7 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.

3 participants