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

Add Puppet 8 support #297

Merged
merged 1 commit into from
Nov 7, 2023
Merged

Add Puppet 8 support #297

merged 1 commit into from
Nov 7, 2023

Conversation

bastelfreak
Copy link
Member

also contains #296

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.

It fails with:

Error: Undefined variable 'fqdn' (file: /etc/puppetlabs/puppet/hiera.yaml, line: 6) on node centos8-64.example.com

Perhaps #298 will fix that, but the error is vague.

@ekohl
Copy link
Member

ekohl commented Aug 8, 2023

Still the same error. I really wonder where it comes from. I wonder if it's the Hiera 3 format. voxpupuli/beaker-hiera#17 should deal with that.

@ekohl
Copy link
Member

ekohl commented Nov 3, 2023

Trivial rebase now that we've updated the gems to support Puppet 8 testing in CI. I've also opened #312 to test together with voxpupuli/beaker-hiera#17.

@ekohl
Copy link
Member

ekohl commented Nov 3, 2023

Still failing and I'm pretty sure voxpupuli/voxpupuli-acceptance#74 should help.

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.

If CI is green.

@bastelfreak
Copy link
Member Author

@ekohl looks goodm CI passed

@ekohl ekohl merged commit 66fc040 into theforeman:master Nov 7, 2023
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

3 participants