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

Switch back to user-provided IAM key for Floaty #80

Merged
merged 2 commits into from
Jul 7, 2023
Merged

Conversation

simu
Copy link
Member

@simu simu commented Jul 6, 2023

Exoscale's new IAM API (v3) isn't yet supported by the Terraform module, so we switch back to a user-provided IAM key for Floaty

This reverts #69 (commit c693867), reversing changes made to 10e07d4.

This PR also updates the Exoscale Terraform provider to v0.50.0, since we've updated the provider in the vshn-lbaas-exoscale module as well.

See https://github.com/appuio/terraform-modules/releases/tag/v5.0.0

Replaces #81 #76

Checklist

  • Keep pull requests small so they can be easily reviewed.
  • Categorize the PR by setting a good title and adding one of the labels:
    bug, enhancement, documentation, change, breaking, dependency
    as they show up in the changelog
  • Link this PR to related issues.
  • Switch to tagged terraform-modules version

@simu simu force-pushed the revert-floaty-iam branch 3 times, most recently from 1ef9efb to 3d0241d Compare July 6, 2023 14:28
@simu simu added the breaking label Jul 6, 2023
@simu simu marked this pull request as ready for review July 6, 2023 14:29
@simu simu requested a review from a team July 6, 2023 14:29
Exoscale's new IAM API (v3) isn't yet supported by the Terraform module,
so we switch back to a user-provided IAM key for Floaty

This reverts #69 (commit c693867),
reversing changes made to 10e07d4.
@simu simu merged commit 84d431a into master Jul 7, 2023
1 check passed
@simu simu deleted the revert-floaty-iam branch July 7, 2023 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants