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 policy #88

Merged
merged 11 commits into from
Jul 17, 2024
Merged

fix policy #88

merged 11 commits into from
Jul 17, 2024

Conversation

leiicamundi
Copy link
Contributor

@leiicamundi leiicamundi commented Jul 16, 2024

Following the upgrade of the EKS module to v20, we encounter a lot of permission denied on the storageclass policy

        	            	�[31m│�[0m �[0m�[1m�[31mError: �[0m�[0m�[1mstorageclasses.storage.k8s.io is forbidden: User "arn:aws:iam::444804106854:user/tf-automation-user" cannot create resource "storageclasses" in API group "storage.k8s.io" at the cluster scope�[0m

e.g. https://github.com/camunda/camunda-tf-eks-module/actions/runs/9957809597

The error is due to a missing dependency between the creation of the cluster and the associated updated of the storage class. This PR solves this issue by moving the storage class definition in the same file.

Also, starting EKS 1.30, gp2 is not the default anymore, I removed it.

This PR also update EKS to v1.30

@leiicamundi leiicamundi added bug Something isn't working and removed feature labels Jul 16, 2024
@leiicamundi leiicamundi enabled auto-merge (squash) July 16, 2024 15:44
@leiicamundi leiicamundi requested a review from Langleu July 16, 2024 15:44
@leiicamundi leiicamundi self-assigned this Jul 16, 2024
@leiicamundi
Copy link
Contributor Author

leiicamundi commented Jul 16, 2024

@Langleu once it's merged, it will unlock #87

Also, I'll tag a new version for this module 2.1.0, then we will be able to merge camunda/camunda-docs#4004

@leiicamundi leiicamundi disabled auto-merge July 16, 2024 21:09
@leiicamundi leiicamundi enabled auto-merge (squash) July 17, 2024 10:46
@leiicamundi leiicamundi merged commit 57884dc into main Jul 17, 2024
10 checks passed
@leiicamundi leiicamundi deleted the feature/gp3-disk branch July 17, 2024 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants