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

Support for Puppet 4 #26

Closed
genebean opened this issue Feb 25, 2017 · 8 comments
Closed

Support for Puppet 4 #26

genebean opened this issue Feb 25, 2017 · 8 comments

Comments

@genebean
Copy link

I see that in master the metadata.json says it supports Puppet 4 but the latest release does not... how is the Puppet 4 support coming along?

@raphink
Copy link
Member

raphink commented Feb 25, 2017

I don't see a reason why it wouldn't work on Puppet 4. Are you seeing a specific failure?

@genebean
Copy link
Author

Before switching from the latest release to master I did. Was hoping that if the Puppet 4 stuff was stable you'd release a new version so I could get off of the git version. Thanks for the quick reply!

@raphink
Copy link
Member

raphink commented Feb 26, 2017

There's no changes in types & providers API in Puppet 4. Most of the changes are in the Puppet DSL, so really types & providers for Puppet 3 should just work in Puppet 4 without changes. As far as I know, they just work.

@trevor-vaughan
Copy link
Contributor

So far all of the ones that we're using "just work" with Puppet 4 AIO and PE

@genebean
Copy link
Author

The one that broke for me before switching to master was kernel_parameter. It broke the ability to run the agent. The issue disappeared when switching to master.

@nmesstorff
Copy link

Same here after changing stringify_facts = false in the puppet.conf on the agent (as suggested by puppetlabs as preparation to puppet 4 migration).

Error message on the puppet node:
Error: /Stage[main]/Main/Node[default]/Kernel_parameter[console]: Could not evaluate: Augeas didn't load /etc/default/grub with Shellvars_list.lns: Iterated lens matched less than it should (line:8, character:0)

The affected puppet code:
kernel_parameter { "console": ensure => present, value => "ttyS0,115200n8", }

@raphink
Copy link
Member

raphink commented Jun 13, 2017

@nmesstorff I'm not sure I see how stringify_facts = false could play a role in this really. The error you're reporting says that Augeas was not able to parse the file at /etc/default/grub and failed to parse on line 8.

Could you paste that file (or at least line 8) please?

@jhoblitt
Copy link
Member

puppet 4 is long EOL.

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

No branches or pull requests

5 participants