Skip to content

Fixes #39653 - Remove Puppet 6 references - #11180

Open
jakduch wants to merge 1 commit into
theforeman:developfrom
jakduch:fix/remove-puppet6-references
Open

Fixes #39653 - Remove Puppet 6 references#11180
jakduch wants to merge 1 commit into
theforeman:developfrom
jakduch:fix/remove-puppet6-references

Conversation

@jakduch

@jakduch jakduch commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Puppet 6 reached EOL in 2023. Remove its version-specific host parameters, repository setup, package references, and documentation from provisioning templates.

This also updates the documented minimum version to Puppet 7 and makes Puppet 8 the unconditional FreeBSD package after the Puppet 6 option is removed.

Stack

Depends on #10928. Until that PR is merged, Assert Single Commit is expected to report the preceding Puppet 5 commit as well.

Verification

  • compiled every changed ERB template
  • checked the complete active provisioning-template tree for remaining Puppet 6 references
  • verified the resulting Puppet 7 and Puppet 8 repository paths
  • verified Puppet 8 is present in the supported FreeBSD 14.4 and 15.1 release ports branches
  • full test suite runs in CI

AI assistance

This pull request was prepared with assistance from Codex 5.6 Sol High.

@jakduch

jakduch commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

This is stacked on #10928, so the single-commit check is expected to fail until #10928 is merged. After that, this PR will contain only the #39653 commit.

@adamruzicka

Copy link
Copy Markdown
Contributor

This is stacked on

pr processor will be confused from this :)

Now that p5 removal is in, could you please rebase?

@jakduch
jakduch force-pushed the fix/remove-puppet6-references branch from 7f75bb3 to 3e25e8d Compare August 19, 2026 13:11
@jakduch

jakduch commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

Rebased, thanks!

adamruzicka
adamruzicka previously approved these changes Aug 19, 2026

@adamruzicka adamruzicka left a comment

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.

I opened a community post to lay out the broader picture and what I was aiming for with the 6 & 7 removal issues.

I'm fine with this, but I'll let it sit until tomorrow to give other people a chance to chime in.

@d1nuc0m

d1nuc0m commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Please wait until #10816 is merged, I'm reviewing it

@adamruzicka

Copy link
Copy Markdown
Contributor

This now needs a rebase

@jakduch
jakduch force-pushed the fix/remove-puppet6-references branch from 3e25e8d to 32df733 Compare August 22, 2026 09:41
@jakduch
jakduch force-pushed the fix/remove-puppet6-references branch 2 times, most recently from 1b968d9 to a1b58f4 Compare September 8, 2026 17:22
@jakduch

jakduch commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

The container build failed while downloading CentOS Stream 9 BaseOS metadata (a checksum mismatch followed by mirror connection failures), before reaching the Foreman changes. This looks like a transient mirror/infrastructure issue.

@jakduch

jakduch commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

All other checks have passed, including the Ruby, JavaScript, Katello, RPM, RuboCop, and single-commit checks. Only the infrastructure-related container build remains. Could someone with permission please rerun that check?

@d1nuc0m

d1nuc0m commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

@jakduch just amend/force push again (ik, it's the whole pipeline but it's faster)

@jakduch
jakduch force-pushed the fix/remove-puppet6-references branch from a1b58f4 to 576cd53 Compare September 9, 2026 06:26
Puppet 6 reached EOL in 2023. Remove its host parameters, repository setup, package references, and documentation from provisioning templates. Preserve the current Puppet and OpenVox package selection and update the documented minimum Puppet version to 7.

Assisted-By: Codex 5.6 Sol High
@jakduch
jakduch force-pushed the fix/remove-puppet6-references branch from 576cd53 to bdeb53a Compare September 9, 2026 07:08
@jakduch
jakduch requested a review from adamruzicka September 9, 2026 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants