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

Enable uploading a public ssh key into the user account #43

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

FlorinAndrei
Copy link

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 the pgp_key variable.

why

I need ssh keys attached to my IAM user accounts.

references

@FlorinAndrei FlorinAndrei requested review from a team as code owners September 10, 2024 22:04
@mergify mergify bot added the triage Needs triage label Sep 10, 2024
Copy link
Member

@Gowiem Gowiem left a 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!

@FlorinAndrei
Copy link
Author

Done, and pushed to the branch.

@Gowiem
Copy link
Member

Gowiem commented Sep 10, 2024

/terratest

@Gowiem Gowiem added the minor New features that do not break anything label Sep 10, 2024
@FlorinAndrei
Copy link
Author

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.

Copy link

mergify bot commented Sep 19, 2024

💥 This pull request now has conflicts. Could you fix it @FlorinAndrei? 🙏

@mergify mergify bot added the conflict This PR has conflicts label Sep 19, 2024
@mergify mergify bot removed the conflict This PR has conflicts label Sep 19, 2024
@Gowiem
Copy link
Member

Gowiem commented Sep 24, 2024

/terratest

@Gowiem
Copy link
Member

Gowiem commented Sep 24, 2024

@FlorinAndrei we're getting this failure in our tests:

TestExamplesComplete 2024-09-24T14:41:19Z retry.go:72: terraform [output -no-color user_name]
TestExamplesComplete 2024-09-24T14:41:19Z command.go:53: Running command terraform with args [output -no-color user_name]
TestExamplesComplete 2024-09-24T14:41:20Z command.go:121: "cp-testing-user"
    examples_complete_test.go:33: 
        	Error Trace:	examples_complete_test.go:33
        	Error:      	Not equal: 
        	            	expected: "cp-testing-user"
        	            	actual  : "\"cp-testing-user\""
        	            	
        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-cp-testing-user
        	            	+"cp-testing-user"
        	Test:       	TestExamplesComplete

Can you look into that? It seems we're double quoting or doing something odd with the username?

@FlorinAndrei
Copy link
Author

@Gowiem That doesn't seem related. My changes only refer to the ssh key resource.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor New features that do not break anything triage Needs triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants