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 defined type and facts to generate and export/collect keys across nodes #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

edestecd
Copy link
Contributor

@edestecd edestecd commented Sep 2, 2016

This creates a fact for each found public ssh key for all users with uid above 500.
The defined type then creates an ssh_key, uses the fact to scoop up the detail and then
exports/collects it on another node.

@edestecd edestecd changed the title Add defined type to generate and export/collect keys across nodes Add defined type and facts to generate and export/collect keys across nodes Sep 2, 2016
… nodes

This creates a fact for each found public ssh key for all users with uid above 500.
The defined type then creates an ssh_key, uses the fact to scoop up the detail and then
exports/collects it on another node.
@edestecd
Copy link
Contributor Author

edestecd commented Sep 2, 2016

Hmm, lots of build errors. I guess, some features require the future parser or Puppet 4.
Any thoughts on that, since this is just an extra feature....

@oxc
Copy link

oxc commented Jun 6, 2017

I guess, some features require the future parser or Puppet 4.

That would be foremost the iterations. The checks on newer puppet versions seem to fail only on the undeclared $bits_opt variable.

I would greatly appreciate this feature, it's especially useful for deploying generated keys with forced commands, for example for backup commands.

@edestecd
Copy link
Contributor Author

edestecd commented Jun 6, 2017

We probably need to update this module and drop puppet 3 support in a major version release.
Then this could be merged.

@wyardley
Copy link
Contributor

wyardley commented Sep 8, 2017

@edestecd Puppet 3 support will be dropped as part of the Voxpupuli cutover.

@edestecd
Copy link
Contributor Author

edestecd commented Sep 9, 2017

OK cool. I'll look into updating this with our latest code and adding some tests.

@wyardley
Copy link
Contributor

wyardley commented Sep 9, 2017

@edestecd Sounds good. Things will change a lot once #17 and #16 are merged

@bastelfreak
Copy link
Member

Hi @edestecd, thanks for this PR. Are you able to rebase?

@runejuhl
Copy link

I've got a rebased version at https://github.com/runejuhl/puppet-ssh_keygen/tree/pr/8 if it has any interest.

@bastelfreak
Copy link
Member

@runejuhl are you interested in creating a PR with the changes?

@runejuhl
Copy link

runejuhl commented Mar 9, 2019

@bastelfreak sure! I had forgotten about this one. I'll submit a PR sometime soon

@runejuhl
Copy link

New PR available at #49

@vox-pupuli-tasks
Copy link

Dear @edestecd, thanks for the PR!

This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@vox-pupuli-tasks
Copy link

Dear @edestecd, thanks for the PR!

This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@vox-pupuli-tasks
Copy link

Dear @edestecd, thanks for the PR!

This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@vox-pupuli-tasks
Copy link

Dear @edestecd, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli Github Bot. I noticed that your pull request has CI failures. Can you please have a look at the failing CI jobs?
If you need any help, you can reach out to us on our IRC channel voxpupuli on Freenode or our Slack channel voxpupuli at slack.puppet.com.
You can find my sourcecode at voxpupuli/vox-pupuli-tasks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants