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

facter -p no longer supported with Puppet 7 #190

Closed
maxadamo opened this issue Jan 19, 2021 · 3 comments · Fixed by #191
Closed

facter -p no longer supported with Puppet 7 #190

maxadamo opened this issue Jan 19, 2021 · 3 comments · Fixed by #191
Assignees
Labels
enhancement New feature or request pending testing

Comments

@maxadamo
Copy link
Contributor

maxadamo commented Jan 19, 2021

Affected Puppet, Ruby, OS and module versions/distributions

  • Puppet: 7
  • Ruby:
  • Distribution: CentOS 7
  • Module version: 0.13

How to reproduce (e.g Puppet code you use)

just use puppet 7 and the cron, will send an email to root with the warning below.

What are you seeing

[2021-01-19 12:21:08.897275 ] WARN Facter::Cli - facter --puppet and facter -p are no longer supported, use puppet facts show instead
[2021-01-19 12:21:08.897321 ] WARN Facter::Cli - the output does not contain puppet facts!

Any additional information you'd like to impart

I wonder if puppet facts show is backward compatible, or you may need to run something as following:

if [ $(puppet --version|cut -d. -f1) -gt 5 ]; then
    ....
fi

with the problem being, puppet facts throws a json or yaml objest instead of the value

@albatrossflavour
Copy link
Owner

Ooops, looks like the tasks aren't working due to changes to the output of puppet facts. Reopening while I look into it.

@albatrossflavour albatrossflavour self-assigned this May 10, 2021
@albatrossflavour albatrossflavour added the enhancement New feature or request label May 10, 2021
@albatrossflavour
Copy link
Owner

Moved all of the facter stuff over to use the native ruby facter integration. Just testing that now but it looks good on Puppet 7.

@albatrossflavour
Copy link
Owner

Testing complete, merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pending testing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants