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

Set useful SBOM name for rpm-ostree generated images #1739

Merged
merged 5 commits into from
Dec 11, 2024

Conversation

johnbe11
Copy link
Contributor

@johnbe11 johnbe11 commented Dec 9, 2024

Set SBOM name for rpm-ostree generated images to registry/repository@digestof the pushed image.

JIRA: ISV-5322

…@digest`of the pushed image.

JIRA: ISV-5322

Signed-off-by: John Bell <[email protected]>
@johnbe11 johnbe11 requested a review from cgwalters as a code owner December 9, 2024 21:53
@cgwalters
Copy link
Contributor

CI failure looks unrelated (looks like this repo could use https://github.com/redhat-plumbers-in-action/differential-shellcheck or honestly just less shell script)

I see the task is cloned for a lot of repos, typically changes like this should mirror the buildah task, was that already done?

@chmeliik
Copy link
Contributor

CI failure looks unrelated (looks like this repo could use https://github.com/redhat-plumbers-in-action/differential-shellcheck or honestly just less shell script)

Fun fact: checkton is differential-shellcheck, just for bash-in-yaml. If it wasn't, there would be 1241 shellcheck warnings and 13 errors in this repo. In this case, the change surfaces an existing warning because it changes the line the warning was on.

…@digest`of the pushed image.

(added quotes to prevent word splitting, per shellcheck	warning)

JIRA: ISV-5322

Signed-off-by: John Bell <[email protected]>
…@digest`of the pushed image.

(updated task/rpm-ostree-oci-ta/0.2/rpm-ostree-oci-ta.yaml via hack/generate-ta-tasks.sh)

JIRA: ISV-5322

Signed-off-by: John Bell <[email protected]>
@johnbe11
Copy link
Contributor Author

@cgwalters I've updated and fixed the previous issue, PTAL

Copy link
Contributor

@cgwalters cgwalters left a comment

Choose a reason for hiding this comment

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

Looks sane to me.
Worth noting here we're actively trying to deprecate (and eventually remove) this task, xref https://issues.redhat.com/browse/BIFROST-408

@cgwalters cgwalters enabled auto-merge December 10, 2024 20:56
@cgwalters
Copy link
Contributor

On a procedural note, I doubt anyone else here cares but personally I would avoid merging from main into the branch for trivial patches like this, I'd rebase instead, it'd be a single commit with a single line change.

@chmeliik
Copy link
Contributor

/ok-to-test

@cgwalters cgwalters added this pull request to the merge queue Dec 11, 2024
Merged via the queue into konflux-ci:main with commit 5c1c38b Dec 11, 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.

3 participants