Skip to content

upgrade to Puppet 8 and PDK version to 3.4#823

Merged
fanny-jiang merged 155 commits intoDataDog:mainfrom
xenon8:xenon8/puppet-8-upgrade
Mar 10, 2025
Merged

upgrade to Puppet 8 and PDK version to 3.4#823
fanny-jiang merged 155 commits intoDataDog:mainfrom
xenon8:xenon8/puppet-8-upgrade

Conversation

@xenon8
Copy link
Copy Markdown
Contributor

@xenon8 xenon8 commented Dec 23, 2024

This will update the Puppet syntax for module to v8 of Puppet; I have updated the reference to StdLib to v9 as well (as this is the version that supports Puppet 8).

I used the PDK for this update; resulting me updating the PDK files also, I had to merge your existing code into the PDK 3.4 template, please let me know if I missed something.

@xenon8 xenon8 requested review from a team as code owners December 23, 2024 09:50
… bcrypt_pbkdf 1.1 fails to install with Ruby 3.2.5
@xenon8 xenon8 changed the title upgrade pdk version to 3.4 upgrade to Puppet 8 and PDK version to 3.4 Dec 23, 2024
Comment thread manifests/init.pp Outdated
Comment thread manifests/init.pp Outdated
Comment thread manifests/integrations/elasticsearch.pp Outdated
Comment thread manifests/init.pp Outdated
Comment thread manifests/init.pp Outdated
Copy link
Copy Markdown
Contributor

@nielstholenaar nielstholenaar left a comment

Choose a reason for hiding this comment

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

Thanks for all the work on the Puppet 8 version.

I'm currently in the process of upgrading our Puppet server and this module was one of the modules that lacked support. I have added some fixes for things I experienced while using your version.

Comment thread manifests/init.pp Outdated
Comment thread manifests/reports.pp Outdated
Comment thread manifests/reports.pp Outdated
Comment thread manifests/reports.pp Outdated
Comment thread manifests/ubuntu.pp Outdated
Comment thread manifests/integrations/php_fpm.pp Outdated
Comment thread manifests/integrations/php_fpm.pp Outdated
Comment thread manifests/integrations/disk.pp Outdated
Comment thread manifests/integrations/disk.pp Outdated
Comment thread manifests/integrations/tcp_check.pp Outdated
Comment thread manifests/integrations/tcp_check.pp Outdated
Comment thread manifests/integrations/tcp_check.pp Outdated
Comment thread manifests/integrations/tcp_check.pp Outdated
Comment thread manifests/integrations/tcp_check.pp Outdated
@xenon8 xenon8 requested a review from a team as a code owner March 8, 2025 15:40
Copy link
Copy Markdown
Contributor

@fanny-jiang fanny-jiang left a comment

Choose a reason for hiding this comment

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

🎉

Copy link
Copy Markdown
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Left some feedback for your review!

Comment thread README.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread CHANGELOG.md Outdated
Comment thread manifests/integrations/activemq_xml.pp Outdated
Comment thread manifests/integrations/activemq_xml.pp Outdated
fanny-jiang and others added 2 commits March 10, 2025 15:18
Co-authored-by: Rosa Trieu <107086888+rtrieu@users.noreply.github.com>
@fanny-jiang fanny-jiang requested a review from rtrieu March 10, 2025 19:55
Copy link
Copy Markdown
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

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.

4 participants