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

Fix ssh keys #11

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Fix ssh keys #11

wants to merge 3 commits into from

Conversation

tykeal
Copy link

@tykeal tykeal commented May 21, 2016

One of the recent versions of puppet does not like the non-existent Ssh_authorized_keys type that is being used to determine if keys should be created. This resolves this issue and adds in rspec test using current (as of the PR) harness

tykeal added 3 commits May 21, 2016 13:28
* Update to use more modern rspec harness defintions
* Create base hiera for use with current hiera loading
* Cleanup initial tests to remove unneeded definitions

Signed-off-by: Andrew Grimberg <[email protected]>
* Test all module loading interfaces
- direct hash
- hiera any level
- hiera merged all
- hiera top level

Signed-off-by: Andrew Grimberg <[email protected]>
* Fixes the if !defined logic for ssh_authorized_keys to not be looking
  for the non-existent Ssh_authorized_keys type

* Adds in spec tests for users::ssh_authorized_keys and users::setup

Signed-off-by: Andrew Grimberg <[email protected]>
ryanschwartz added a commit to icg-puppet/puppet-users that referenced this pull request Jun 6, 2016
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.

1 participant