Delete old CV versions by maximiliankolb · Pull Request #4869 · theforeman/foreman-documentation

maximiliankolb · 2026-05-21T12:59:51Z

What changes are you introducing?

Modules to delete old CV versions.

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

Response to https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian which I consider valuable feedback so I tested it (minus the Ansible stuff in a Playbook) and wrote some docs.

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

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.19/Katello 4.21
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, 7.7, and 7.8)
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 · 2026-05-21T13:04:02Z

The PR preview for 7f4dbd3 is available at theforeman-foreman-documentation-preview-pr-4869.surge.sh

The following output files are affected by this PR:

show diff

show diff as HTML

maximiliankolb

@jeremylenz Could you please check if this makes any sense to you? Are my assumptions about Katello orphan cleanup correct?

jafiala

Other than the Additional resources thing, LGTM

jeremylenz · 2026-05-21T14:02:47Z

Are my assumptions about Katello orphan cleanup correct?

Tagging @sjha4 for review here, as he is more of a subject matter expert.

Lennonka

Good draft. I have a couple of suggestions.

maximiliankolb

Rebased to HEAD of "master" and applied all suggestions.

maximiliankolb

Rebased to HEAD of "master" and applied both suggestions.

Lennonka

Two optional nitpicks, LGTM otherwise.

This patch is a prerequisite to adding a module for Hammer CLI. $ rg -i "Deleting_Multiple_Content_View_Versions_" Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian

Tested on Foreman 3.18/Katello 4.20 with Hammer CLI katello 1.20. Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian

Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian Refs https://theforeman.github.io/foreman-ansible-modules/latest/plugins/content_view_version_module.html#ansible-collections-theforeman-foreman-content-view-version-module

Tested on Foreman 3.18/Katello 4.20 with Hammer CLI katello 1.20. Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian

Suggested by Lena during review.

maximiliankolb · 2026-06-03T07:14:50Z

@sjha4 Can you please do a tech review? I already tested the commands locally on Foreman 3.18/Katello 4.20.

sjha4 · 2026-06-03T14:02:38Z

Ack...Commands are correct.. 👍🏼

Lennonka · 2026-06-03T20:53:02Z

I'll review the abstracts in the scope of #4897, but we're good to merge.

* Rename module to delete CV versions via WebUI This patch is a prerequisite to adding a module for Hammer CLI. $ rg -i "Deleting_Multiple_Content_View_Versions_" Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian * Add module to delete CV versions via CLI Tested on Foreman 3.18/Katello 4.20 with Hammer CLI katello 1.20. Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian * Add module to delete CV versions via Ansible Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian Refs https://theforeman.github.io/foreman-ansible-modules/latest/plugins/content_view_version_module.html#ansible-collections-theforeman-foreman-content-view-version-module * Add module to purge content view versions Tested on Foreman 3.18/Katello 4.20 with Hammer CLI katello 1.20. Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian * Move deleting CV versions to end of assembly Suggested by Lena during review. (cherry picked from commit 8f70ca4)

maximiliankolb · 2026-06-05T06:01:04Z

Merged to "master" and cherry-picked:
bb44685..02c884b 3.19 -> 3.19
ce5b5d4..5c8ec59 3.18 -> 3.18

pr-processor Bot added the Not yet reviewed label May 21, 2026

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 May 21, 2026

maximiliankolb commented May 21, 2026

View reviewed changes

Comment thread guides/common/modules/con_content-view-version-cleanups.adoc Outdated

Comment thread guides/common/modules/proc_cleaning-up-content-view-versions-by-using-cli.adoc Outdated

Comment thread guides/common/modules/proc_deleting-multiple-content-view-versions-by-using-ansible.adoc

pr-processor Bot removed the Not yet reviewed label May 21, 2026

jafiala reviewed May 21, 2026

View reviewed changes

Comment thread guides/common/modules/proc_cleaning-up-content-view-versions-by-using-cli.adoc Outdated

jafiala reviewed May 21, 2026

View reviewed changes

Lennonka requested changes May 25, 2026

View reviewed changes

pr-processor Bot added the Waiting on contributor Requires an action from the author label May 25, 2026

maximiliankolb force-pushed the delete_old_cv_versions branch from 50d8cf4 to 0b6836e Compare June 1, 2026 13:33

pr-processor Bot added Needs re-review and removed Waiting on contributor Requires an action from the author labels Jun 1, 2026

maximiliankolb commented Jun 1, 2026

View reviewed changes

pr-processor Bot removed the Needs re-review label Jun 1, 2026

Lennonka reviewed Jun 1, 2026

View reviewed changes

Comment thread guides/common/modules/proc_purging-content-view-versions-by-using-cli.adoc Outdated

Lennonka reviewed Jun 1, 2026

View reviewed changes

Comment thread guides/common/modules/proc_purging-content-view-versions-by-using-cli.adoc Outdated

maximiliankolb force-pushed the delete_old_cv_versions branch from 0b6836e to 7edb9a5 Compare June 2, 2026 06:05

maximiliankolb commented Jun 2, 2026

View reviewed changes

Comment thread guides/common/modules/proc_purging-content-view-versions-by-using-cli.adoc Outdated

Comment thread guides/common/modules/proc_purging-content-view-versions-by-using-cli.adoc Outdated

jeremylenz reviewed Jun 2, 2026

View reviewed changes

Comment thread guides/common/modules/proc_purging-content-view-versions-by-using-cli.adoc Outdated

maximiliankolb force-pushed the delete_old_cv_versions branch from 7edb9a5 to 78aa8ec Compare June 2, 2026 13:26

Lennonka approved these changes Jun 2, 2026

View reviewed changes

Comment thread guides/common/modules/proc_purging-content-view-versions-by-using-cli.adoc Outdated

Comment thread guides/common/modules/proc_purging-content-view-versions-by-using-cli.adoc

Lennonka 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 Jun 2, 2026

maximiliankolb added 4 commits June 3, 2026 09:09

Rename module to delete CV versions via WebUI

dc35524

This patch is a prerequisite to adding a module for Hammer CLI. $ rg -i "Deleting_Multiple_Content_View_Versions_" Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian

Add module to delete CV versions via CLI

5cef9e4

Tested on Foreman 3.18/Katello 4.20 with Hammer CLI katello 1.20. Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian

Add module to purge content view versions

a4df1c8

Tested on Foreman 3.18/Katello 4.20 with Hammer CLI katello 1.20. Refs https://community.theforeman.org/t/quick-question-on-automating-the-removal-of-old-content-versions/46402?u=maximilian

Move deleting CV versions to end of assembly

7f4dbd3

Suggested by Lena during review.

maximiliankolb force-pushed the delete_old_cv_versions branch from 78aa8ec to 7f4dbd3 Compare June 3, 2026 07:10

maximiliankolb added testing done No issues from the functional perspective and removed Needs testing Requires functional testing labels Jun 3, 2026

Lennonka added tech review done No issues from the technical perspective and removed Needs tech review Requires a review from the technical perspective labels Jun 3, 2026

maximiliankolb merged commit 8f70ca4 into theforeman:master Jun 5, 2026
11 checks passed

maximiliankolb deleted the delete_old_cv_versions branch June 5, 2026 05:56

Conversation

maximiliankolb commented May 21, 2026

What changes are you introducing?

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

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

Contributor checklists

Uh oh!

github-actions Bot commented May 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

maximiliankolb left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jafiala left a comment

Choose a reason for hiding this comment

Uh oh!

jeremylenz commented May 21, 2026

Uh oh!

Lennonka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

maximiliankolb left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

maximiliankolb left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Lennonka left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

maximiliankolb commented Jun 3, 2026

Uh oh!

sjha4 commented Jun 3, 2026

Uh oh!

Lennonka commented Jun 3, 2026

Uh oh!

Uh oh!

maximiliankolb commented Jun 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

github-actions Bot commented May 21, 2026 •

edited

Loading