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

provider/aws Implement username updates properly for aws-sdk-go #3227

Merged
merged 4 commits into from
Nov 10, 2015

Conversation

pgray
Copy link
Contributor

@pgray pgray commented Sep 13, 2015

This should solve #2818

@pgray pgray changed the title Implement username updates properly provider/aws Implement username updates properly for aws-sdk-go Sep 15, 2015
@catsby
Copy link
Contributor

catsby commented Oct 28, 2015

Does this also fix #2761 ?

@catsby catsby added the waiting-response An issue/pull request is waiting for a response from the community label Oct 28, 2015
@catsby
Copy link
Contributor

catsby commented Oct 28, 2015

Any thoughts on that, @graycoder ?

@pgray
Copy link
Contributor Author

pgray commented Oct 28, 2015

@catsby the 2 PRs ( This one and #3237 ) related to this that I have open solve #2761 for groups and users, but do not solve it for roles unfortunately. The AWS api doesn't seem to provide the same hooks to update roles... so I haven't found a fix that would close #2761

@catsby
Copy link
Contributor

catsby commented Nov 9, 2015

@graycoder hey sorry for the delay here. Do you by chance have a config that demonstrates for me what this is fixing? I believe that it's good and necessary, but I'm not following when this situation pops up

@pgray
Copy link
Contributor Author

pgray commented Nov 10, 2015

@catsby This situation pops up when you rename a user.
This is because the API requires that you specifically call the Update functions on users that are in groups instead of just deleting and recreating the user, which is what the current code does.

Without proper update support, one must manually remove the user from any groups they're in when they want to change the user's name (this often happens unexpectedly during an apply). One must also remove any managed policies attached to the user otherwise the same set of events (api errors) happen.

@catsby
Copy link
Contributor

catsby commented Nov 10, 2015

Ah, got it. Thanks @graycoder , this looks good to me!

catsby added a commit that referenced this pull request Nov 10, 2015
provider/aws Implement username updates properly for aws-sdk-go
@catsby catsby merged commit 8b1f749 into hashicorp:master Nov 10, 2015
bmcustodio pushed a commit to bmcustodio/terraform that referenced this pull request Sep 26, 2017
@ghost
Copy link

ghost commented Apr 30, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators Apr 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug provider/aws waiting-response An issue/pull request is waiting for a response from the community
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants