Skip to content

Conversation

@jbtrystram
Copy link
Member

Adding the container signature in the pipeline [1] added an extra argument to the container push call.

Cherry-pick 677fe26 to add the option to unbreak the pipeline for RHCOS.

[1] coreos/fedora-coreos-pipeline@b4df2de

- This maps to Podman's --digestfile option, allowing the manifest
  digest to be saved for future reference or use.

Signed-off-by: Renata Ravanelli <[email protected]>
@jbtrystram jbtrystram requested a review from jlebon September 10, 2025 11:37
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request backports the --write-digest-to-file option to the cmd-push-container-manifest command. The changes correctly add the new command-line argument and propagate it through the function calls to podman manifest push. The implementation is sound, but I've identified a couple of minor documentation issues where the docstrings for modified functions were not updated to include the new parameter. Addressing these will improve code maintainability.

@dustymabe
Copy link
Member

Sorry things are changing so quickly here.

I think once coreos/fedora-coreos-pipeline#1216 merges we'll be back safe again?

@jbtrystram
Copy link
Member Author

Sorry things are changing so quickly here.

I think once coreos/fedora-coreos-pipeline#1216 merges we'll be back safe again?

ah yes, thanks !

@jbtrystram jbtrystram closed this Sep 10, 2025
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