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

Missing /src/utils/merge_syft_sbom.py in container image built from version 0.9.0 #582

Closed
tkdchen opened this issue Jul 24, 2024 · 0 comments · Fixed by #583
Closed

Missing /src/utils/merge_syft_sbom.py in container image built from version 0.9.0 #582

tkdchen opened this issue Jul 24, 2024 · 0 comments · Fixed by #583
Labels
bug Something isn't working

Comments

@tkdchen
Copy link
Contributor

tkdchen commented Jul 24, 2024

Image quay.io/redhat-appstudio/cachi2:0.9.0@sha256:c5b62c585438a2396018cc8b410694c73e7e9e6c706f64110dd293d6b06b021d does not have script /src/utils/merge_syft_sbom.py. The image does not include that script since commit 49ba69d.

@eskultety eskultety added the bug Something isn't working label Jul 24, 2024
eskultety added a commit to eskultety/cachi2 that referenced this issue Jul 24, 2024
When commit 49ba69d converted the image build to multi-stage it forgot
to install the merge_syft_sbom.py utility script that was previously
installed as part of copying the whole repo into the image to
/src/utils/merge_syft_sbom.py.
This patch takes a bit of a different path by installing the script as
a utility binary to a more standardized location.

Fixes: 49ba69d
Resolves: containerbuildsystem#582

Signed-off-by: Erik Skultety <[email protected]>
github-merge-queue bot pushed a commit that referenced this issue Jul 24, 2024
When commit 49ba69d converted the image build to multi-stage it forgot
to install the merge_syft_sbom.py utility script that was previously
installed as part of copying the whole repo into the image to
/src/utils/merge_syft_sbom.py.
This patch takes a bit of a different path by installing the script as
a utility binary to a more standardized location.

Fixes: 49ba69d
Resolves: #582

Signed-off-by: Erik Skultety <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants