-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
Enable uploading a public ssh key into the user account #43
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FlorinAndrei this is looking good, but unfortunately we need to run some automation to pass our tests. Mind doing the following locally, adding + committing the result, and pushing to your branch?
make init
make readme
Thanks!
Done, and pushed to the branch. |
/terratest |
AFAICT, now it's the automated tests themselves that are failing for some reason (old test environment maybe?). Please let me know if there's anything I need to do. |
💥 This pull request now has conflicts. Could you fix it @FlorinAndrei? 🙏 |
/terratest |
@FlorinAndrei we're getting this failure in our tests:
Can you look into that? It seems we're double quoting or doing something odd with the username? |
@Gowiem That doesn't seem related. My changes only refer to the ssh key resource. |
what
This PR enables uploading of a public ssh key into the IAM user account. By default the ssh key uploading is disabled.
It also fixes a minor bug where, even when
login_profile_enabled = false
, the module would still ask for a value for thepgp_key
variable.why
I need ssh keys attached to my IAM user accounts.
references