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

Update github provider version. #325

Merged
merged 2 commits into from
Oct 7, 2023
Merged

Conversation

nuclearsandwich
Copy link
Collaborator

In addition to updating this provider version I have also run

terraform state replace-provider hashicorp/github integrations/github

in order to move all state to the canonical provider. This also protects from unintentional provider version mismatch.

To complete the transition and fully remove the other provider, I'll need to complete a deployment so that no current state or current configuration uses the old provider unintentionally.

I may also need to update data sources manually which may not be covered by the provider-replacement, I can't quite tell.

In addition to updating this provider version I have also run

    terraform state replace-provider hashicorp/github integrations/github

in order to move all state to the canonical provider. This also protects from unintentional provider version mismatch.
Without this resources created in the module will implicitly use the
hashicorp provider without a version restriction.
@nuclearsandwich
Copy link
Collaborator Author

This has been vexing me for weeks and I finally worked out what was missing and causing the provider to regress.

With the latest iteration of this we're finally sticking to one terraform provider for all resources and state has been updated to match.

@nuclearsandwich nuclearsandwich merged commit 1bf402a into latest Oct 7, 2023
@nuclearsandwich nuclearsandwich deleted the github-provider-updates branch October 7, 2023 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant