Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Load puppet versions from shared file & Add missing factsets #334

Merged
merged 3 commits into from
May 16, 2024

Conversation

yakatz
Copy link
Member

@yakatz yakatz commented May 15, 2024

This will allow scripts for different operating systems to be separated and still use the same versions

This also helps keep track of what facter versions are being used with each version of Puppet Agent.

Tested on RHEL 9, openSUSE 15.4, Ubuntu 20.04, Windows 10/11, Windows Server 2019/2022

Inspired by #330

yakatz added 2 commits May 15, 2024 17:20
This will allow scripts for different operating systems to be separated and still use the same versions

This also helps keep track of what facter versions are being used with each version of Puppet Agent.

Tested on RHEL 9, openSUSE 15.4, Ubuntu 20.04, Windows 10, Windows Server 2019

Inspired by voxpupuli#330
@yakatz yakatz force-pushed the windows_puppet_versions branch from 578b94a to cb32548 Compare May 15, 2024 21:22
Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I like this idea. Will need some refactoring once Puppet 9 comes out, but we'll cross that bridge when we get there.

Since this also adds some facts, I'm not sure the skip-changelog label is entirely correct.

@yakatz
Copy link
Member Author

yakatz commented May 16, 2024

I like this idea. Will need some refactoring once Puppet 9 comes out, but we'll cross that bridge when we get there.

For Windows, it doesn't matter because the download links are probably the same.

For OSes that need different repos, the change to loading the file is minimal:

puppet8_agent_versions=$(grep --only-matching --perl-regexp '^8\.\d+\.\d+' $puppetAgentVersionList)
puppet9_agent_versions=$(grep --only-matching --perl-regexp '^9\.\d+\.\d+' $puppetAgentVersionList)

@bastelfreak bastelfreak changed the title Load puppet versions from shared file Load puppet versions from shared file & Add missing factsets May 16, 2024
@bastelfreak
Copy link
Member

I adjusted the PR title and removed the label.

@bastelfreak bastelfreak merged commit 9b9c648 into voxpupuli:master May 16, 2024
7 checks passed
@yakatz yakatz deleted the windows_puppet_versions branch May 16, 2024 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants