Skip to content

Use retain version only with additive mirroring#4653

Merged
aneta-petrova merged 2 commits intotheforeman:masterfrom
ianballou:retain-versions-additive
Feb 17, 2026
Merged

Use retain version only with additive mirroring#4653
aneta-petrova merged 2 commits intotheforeman:masterfrom
ianballou:retain-versions-additive

Conversation

@ianballou
Copy link
Contributor

What changes are you introducing?

Adds a note that the retain versions field is only to be used with the additive download policy.

Why are you introducing these changes? (Explanation, links to references, issues, etc.)

The retain versions feature is only possible if using the additive mirroring policy.

Anything else to add? (Considerations, potential downsides, alternative solutions you have explored, etc.)

I wasn't sure if it was worth explicitly mentioning that the retain versions field disappears in the UI if the additive mirroring policy is not selected.

Contributor checklists

  • 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)
  • 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 Feb 13, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
@aneta-petrova aneta-petrova added style review done No issues from docs style/grammar perspective and removed Needs style review Requires a review from docs style/grammar perspective Needs testing Requires functional testing labels Feb 17, 2026
@aneta-petrova
Copy link
Member

Normally a tech review from another engineer would be required but because this update does not significantly change description of product behavior, I believe we can skip this.

@aneta-petrova aneta-petrova added tech review done No issues from the technical perspective and removed Needs tech review Requires a review from the technical perspective labels Feb 17, 2026
@aneta-petrova aneta-petrova merged commit 35fde0d into theforeman:master Feb 17, 2026
9 of 10 checks passed
aneta-petrova added a commit that referenced this pull request Feb 17, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
(cherry picked from commit 35fde0d)
aneta-petrova added a commit that referenced this pull request Feb 17, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
(cherry picked from commit 35fde0d)
aneta-petrova added a commit that referenced this pull request Feb 17, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
(cherry picked from commit 35fde0d)
aneta-petrova added a commit that referenced this pull request Feb 17, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
(cherry picked from commit 35fde0d)
aneta-petrova added a commit that referenced this pull request Feb 17, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
(cherry picked from commit 35fde0d)
aneta-petrova added a commit that referenced this pull request Feb 17, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
(cherry picked from commit 35fde0d)
aneta-petrova added a commit that referenced this pull request Feb 17, 2026
Co-authored-by: Aneta Šteflová Petrová <apetrova@redhat.com>
(cherry picked from commit 35fde0d)
@aneta-petrova
Copy link
Member

Merged to "master" and cherry-picked:

43d2102..787bcd4 3.18 -> 3.18
a31d27a..a5d56f0 3.17 -> 3.17
2bcd03e..30feae9 3.16 -> 3.16
fc1106f..1264515 3.15 -> 3.15
24ea2b5..4ceaf5f 3.14 -> 3.14
0a2f511..64f39f9 3.13 -> 3.13
bfba25b..e3e1323 3.12 -> 3.12

@ianballou ianballou deleted the retain-versions-additive branch February 17, 2026 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments