Skip to content

Users cannot push containers to docker type repositories with upstream URL synced to Foreman+Katello (SAT-44672)#4792

Open
jsvarova wants to merge 3 commits intotheforeman:masterfrom
jsvarova:SAT-44672-update-considerations
Open

Users cannot push containers to docker type repositories with upstream URL synced to Foreman+Katello (SAT-44672)#4792
jsvarova wants to merge 3 commits intotheforeman:masterfrom
jsvarova:SAT-44672-update-considerations

Conversation

@jsvarova
Copy link
Copy Markdown

Implementing feedback from issue #4776.

  • I am okay with my commits getting squashed when you merge this PR.
  • I am familiar with the contributing guidelines.

Please cherry-pick my commits into:

  • Foreman 3.18/Katello 4.20 (Satellite 6.19)
  • Foreman 3.17/Katello 4.19
  • Foreman 3.16/Katello 4.18 (Satellite 6.18; orcharhino 7.6 and 7.7)
  • Foreman 3.15/Katello 4.17
  • Foreman 3.14/Katello 4.16 (Satellite 6.17; orcharhino 7.4; orcharhino 7.5)
  • Foreman 3.13/Katello 4.15 (EL9 only)
  • Foreman 3.12/Katello 4.14 (Satellite 6.16; orcharhino 7.2 on EL9 only; orcharhino 7.3)
  • We do not accept PRs for Foreman older than 3.12.

@github-actions github-actions Bot added Needs tech review Requires a review from the technical perspective Needs style review Requires a review from docs style/grammar perspective Needs testing Requires functional testing labels Apr 26, 2026
@maximiliankolb maximiliankolb changed the title Sat 44672 update considerations Users cannot push containers to docker type repositories with upstream URL synced to Foreman+Katello (SAT-44672) Apr 27, 2026
Copy link
Copy Markdown
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

one small suggestion. About git commit messages: we have a link to a blog article somewhere in CONTRIBUTING.md that explains how to phrase them.

I have adjusted the title of your PR to make it more descriptive too.

Comment thread guides/common/modules/proc_using-container-registries.adoc Outdated
@aneta-petrova aneta-petrova requested a review from ianballou April 27, 2026 07:26
@aneta-petrova aneta-petrova removed the Needs testing Requires functional testing label Apr 27, 2026
Co-authored-by: Maximilian Kolb <mail@maximilian-kolb.de>
Copy link
Copy Markdown
Contributor

@maximiliankolb maximiliankolb left a comment

Choose a reason for hiding this comment

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

style-wise LGTM

@maximiliankolb maximiliankolb added style review done No issues from docs style/grammar perspective and removed Needs style review Requires a review from docs style/grammar perspective labels Apr 28, 2026
@jsvarova
Copy link
Copy Markdown
Author

jsvarova commented May 4, 2026

Hello Max, I read through https://cbea.ms/git-commit/#seven-rules (as pointed out by Lennonka in my other PR as well). I'll keep in in mind!

@ianballou Could you please provide the technical review when time allows? Thank you!

endif::[]

Considerations for pushing content to the {Project} container registry::
* You cannot push containers to repositories that are synchronized from an upstream URL.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The behavior is a little more nuanced, you can only push containers to repositories created by the container push workflow. To be technical, any repository created by using the container registry endpoints can be pushed to, and repositories created by the general repositories endpoints cannot be pushed to. Pretty much this means that users dealing with "container push" repositories should interact with them only via podman (or docker), unless the publish them into a content view.

So while the above gets to the gist of the situation, it omits that you cannot create an empty repo with no upstream URL and push content to it, which users may try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs tech review Requires a review from the technical perspective style review done No issues from docs style/grammar perspective

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants