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

Update facterdb to 3.x and rspec-puppet-facts to 5.x #593

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

silug
Copy link
Contributor

@silug silug commented Jul 23, 2024

Summary

Update facterdb from 1.x to 3.x and update rspec-puppet-facts from 3.x to 5.x.

Additional Context

Upstream facterdb and rspec-puppet-facts have removed support for legacy facts, which is useful for testing for Puppet 8.

See https://github.com/voxpupuli/facterdb/blob/master/CHANGELOG.md and https://github.com/voxpupuli/rspec-puppet-facts/blob/master/CHANGELOG.md for more details.

Related Issues (if any)

N/A

Checklist

  • 🟢 Spec tests.
  • 🟢 Acceptance tests.
  • Manually verified.

@silug silug requested a review from a team as a code owner July 23, 2024 17:11
@CLAassistant
Copy link

CLAassistant commented Jul 23, 2024

CLA assistant check
All committers have signed the CLA.

@pmcmaw
Copy link
Contributor

pmcmaw commented Aug 9, 2024

Hey @silug for now we will be continuing to support legacy facts for a little while yet, therefore we are unable to merge this PR right now but will revisit in the future when we plan to drop.

@silug
Copy link
Contributor Author

silug commented Oct 8, 2024

I've pushed a change that should resolve the backwards incompatibility.

Copy link
Member

@david22swan david22swan left a comment

Choose a reason for hiding this comment

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

Tested this out as best I can and it looks good.

@david22swan david22swan merged commit d5f5ac1 into puppetlabs:main Dec 19, 2024
2 of 3 checks passed
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.

5 participants