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

rewrite for hiera 5? #27

Open
davealden opened this issue Mar 14, 2017 · 12 comments
Open

rewrite for hiera 5? #27

davealden opened this issue Mar 14, 2017 · 12 comments

Comments

@davealden
Copy link
Contributor

Hi,
Have you had any thoughts about rewriting this for hiera 5? It looks like it would it make it much simpler (and eliminate the need for the additional patches to keep it from going through the entire vault backend for every class call).
...dave

@davealden
Copy link
Contributor Author

@jsok Any thoughts on this? :)

@davealden
Copy link
Contributor Author

Is anyone using this with the latest hiera (that comes with puppet-agent 1.10)? I've been stuck at 1.8.3 for a few months now and I'm pretty sure it's due to this module.

@oogs
Copy link

oogs commented May 2, 2017

This would be pretty nice to have in hiera 5.

@davealden
Copy link
Contributor Author

@jsok I've rewritten this to be a hiera 5 backend. I believe this eliminates the need for hiera-router (and any other solutions suggested in Issue #22 ) since you can easily specify a regex for which keys should be looked up in the backend. This is my first ruby code, so it's probably not 100% - any interest in this?

@jsok
Copy link
Owner

jsok commented May 10, 2017

@davealden I'm fairly out of the loop on hiera these days, but I'd be happy to keep supporting and maintaining this project from community contributions.
We could mint a new major version which supports hiera 5 and newer based on your contribution.

@hlindberg
Copy link

@davealden and @jsok - If you need help with review of the code, or need any help regarding hiera 5, please ping me.

@blackophelia
Copy link

Hey @davealden @jsok - just came across this thread... I'm interested in using this with hiera 5.... any updates as to where/when you'll go with this?

@jsok
Copy link
Owner

jsok commented Jul 31, 2017

Again I'm not involved in Puppet/Hiera enough these days to be able to make a change like this.

The project in its current state is pretty poor. There is 0 unit test coverage and I'm very hesitant to accept any changes as is. Any major refactor would need to be accompanied with a substantial test siute.

@davealden
Copy link
Contributor Author

@blackophelia - I rewrote this to work with Hiera 5 (and only 5, it is not compatible with any previous versions). I did not finish testing it (and looking through the readme, I have not given proper credit to @jsok for his writing of the initial module -- I definitely want to do that before I make any official type of release). I hope to be able to finish it up sometime in the next 2-3 months, but I have been temporarily pulled off that project for now. Feel free to take a look at it:

https://github.com/davealden/hiera-vault

Note that in the install instructions, I tell you to do a "puppet module install ...", but as I haven't finished it, I have not submitted it to the forge. :) So for now you'll have to manually install the hiera_vault function.

@blackophelia
Copy link

@davealden thanks for the update! We've implemented our vault access in a different way for now, so the need for this functionality isn't so urgent.
Thanks again for keeping all this going!

@keith-miller
Copy link

So I'm a bit confused. Is Heira 5 support in? Thanks!

@petems
Copy link

petems commented Mar 25, 2018

Using @davealden's Hiera 5 version, I built a Vagrant repo as a proof of concept to play with.

It worked perfectly! 👍 https://github.com/petems/puppet-hiera-vault-vagrant

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

7 participants