Skip to content

Conversation

@lsm5
Copy link
Contributor

@lsm5 lsm5 commented Nov 4, 2025

Copy link
Contributor

@eddie-knight eddie-knight left a comment

Choose a reason for hiding this comment

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

Looks great to me, thanks for addressing the previous feedback!

Copy link
Contributor

@JustinCappos JustinCappos left a comment

Choose a reason for hiding this comment

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

Thanks for addressing the comments in cncf/tag-security#1498

@JustinCappos JustinCappos merged commit 09554f1 into cncf:main Nov 4, 2025
2 checks passed
@lsm5
Copy link
Contributor Author

lsm5 commented Nov 5, 2025

Thanks all!!

@lsm5 lsm5 deleted the podman-containers-self-assessment branch November 5, 2025 13:29
Copy link

@flouthoc flouthoc left a comment

Choose a reason for hiding this comment

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

I think actors should incude buildah as well since podman uses buildah as embedded library.

Maybe Actions can also have another section Building OCI images since podman allows that.

@lsm5
Copy link
Contributor Author

lsm5 commented Nov 5, 2025

I think actors should incude buildah as well since podman uses buildah as embedded library.

Maybe Actions can also have another section Building OCI images since podman allows that.

@flouthoc I'll make additional PRs for buildah and skopeo. Will incorporate your suggestions in there.

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.

4 participants