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 minio_iam_user_policy_attachment: Error: Provider produced inconsistent result after apply (fix #494) #495

Merged
merged 1 commit into from
Jul 29, 2023

Conversation

Adphi
Copy link
Contributor

@Adphi Adphi commented Jul 29, 2023

Fix minio_iam_user_policy_attachment: Error: Provider produced inconsistent result after apply

This PR implements the following changes:

  • Use locks to prevent concurrent policy attachments on same user/group

Reference

Closing issues

…istent result after apply (fix #494)

Signed-off-by: Adphi <philippe.adrien.nousse@gmail.com>
@felladrin felladrin merged commit 39e7b92 into aminueza:master Jul 29, 2023
2 checks passed
@Adphi
Copy link
Contributor Author

Adphi commented Jul 29, 2023

Thank you so much for your impressive responsiveness!

@Adphi Adphi deleted the fix-494 branch July 29, 2023 15:23
doonga pushed a commit to doonga/greyrock-ops that referenced this pull request Jul 30, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [minio](https://registry.terraform.io/providers/aminueza/minio)
([source](https://github.com/aminueza/terraform-provider-minio)) |
required_provider | patch | `1.17.0` -> `1.17.1` |

---

### Release Notes

<details>
<summary>aminueza/terraform-provider-minio (minio)</summary>

###
[`v1.17.1`](https://github.com/aminueza/terraform-provider-minio/releases/tag/v1.17.1)

[Compare
Source](https://github.com/aminueza/terraform-provider-minio/compare/v1.17.0...v1.17.1)

#### What's Changed

- Fix minio_iam_user_policy_attachment error: Provider produced
inconsistent result after apply by
[@&#8203;Adphi](https://github.com/Adphi) in
[aminueza/terraform-provider-minio#495

**Full Changelog**:
aminueza/terraform-provider-minio@v1.17.0...v1.17.1

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://github.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4yNS41IiwidXBkYXRlZEluVmVyIjoiMzYuMjUuNSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: greyrock-bot <112729691+greyrock-bot[bot]@users.noreply.github.com>
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.

minio_iam_user_policy_attachment: Error: Provider produced inconsistent result after apply
2 participants