Skip to content

(CAT-2375) puppetcore update#554

Merged
gavindidrichsen merged 5 commits intomainfrom
CAT-2375-Puppetcore_update
Aug 29, 2025
Merged

(CAT-2375) puppetcore update#554
gavindidrichsen merged 5 commits intomainfrom
CAT-2375-Puppetcore_update

Conversation

@LukasAud
Copy link
Copy Markdown
Contributor

@LukasAud LukasAud commented Aug 28, 2025

A set of changes dedicated to implementing puppetcore into our modules.
This update, amongst other changes, removes Puppet 7 support.

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.

A set of changes dedicated to implementing puppetcore into our modules.
This update, amongst other changes, removes Puppet 7 support.
@LukasAud LukasAud requested review from a team, bastelfreak, ekohl and smortex as code owners August 28, 2025 10:16
@LukasAud LukasAud changed the title Cat 2375 puppetcore update (CAT-2375) puppetcore update Aug 28, 2025
@LukasAud LukasAud force-pushed the CAT-2375-Puppetcore_update branch from d251659 to a86fe9f Compare August 28, 2025 10:17
Copy link
Copy Markdown
Contributor

@gavindidrichsen gavindidrichsen left a comment

Choose a reason for hiding this comment

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

LGTM

@gavindidrichsen gavindidrichsen merged commit 2f77497 into main Aug 29, 2025
29 of 57 checks passed
@gavindidrichsen gavindidrichsen deleted the CAT-2375-Puppetcore_update branch August 29, 2025 08:18
new(
name: namevar(section_name, setting),
value: value,
value:,
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.

Other than making it really hard for people to upgrade, what is this sort of change really improving!?

This module is the sort used across all your roles/profiles, so you can't upgrade it if you even have just 2% of your estate using an older puppet agent version.

And this has been released in a minor version.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The latest version of this module reverts this change.

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