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

Support expired access policy tokens #1678

Merged
merged 4 commits into from
Jul 11, 2024
Merged

Conversation

malcolmholmes
Copy link
Contributor

When a Cloud access policy token expires, the Cloud API deletes it, breaking the Terraform provider resource.

This PR correctly detects the 404 response and fixes Terraform.

@malcolmholmes malcolmholmes requested review from a team as code owners July 11, 2024 13:46
Copy link

In order to lower resource usage and have a faster runtime, PRs will not run Cloud tests automatically.
To do so, a Grafana Labs employee must trigger the cloud acceptance tests workflow manually.

@julienduchesne
Copy link
Member

Just a small lint fix needed

@malcolmholmes malcolmholmes merged commit f39fd6f into main Jul 11, 2024
26 of 27 checks passed
@malcolmholmes malcolmholmes deleted the malcolmholmes/token-rotation branch July 11, 2024 17:13
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.

2 participants