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: graceful handling of 404 in github_actions_repository_oidc_subject_claim_customization_template #2483

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

acouvreur
Copy link

The resource github_actions_repository_oidc_subject_claim_customization_template would error when the associated repository was deleted.

This change will gracefully remove the resource on 404 not found.

Resolves #2482


Before the change?

Currently when running terraform destroy execution fails if the repository that has oidc customization outside terraform. The 404 is handled when querying the oidc customization itself, but not in the prior request to get the repository id.

After the change?

If the repository no longer exists then the github_actions_repository_oidc_subject_claim_customization_template resource will be removed from the state (if the repo no longer exists then the oidc customization cannot either).

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

…te): graceful handling of 404 on read

The resource github_actions_repository_oidc_subject_claim_customization_template would error when the associated repository was deleted.

This change will gracefully remove the resource on 404 not found.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant