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 Terraform Random provider to version 3.4.3 #66

Conversation

kwilczynski
Copy link
Contributor

There have been several important bug fixes since the release 3.1.0, which is now almost two years old, as per:

Additionally, HashiCorp released a version of the Terraform Random provider updated to use the new Terraform Plugin Framework.

This is a maintenance update, and no changes to the underlying infrastructure are intended. Note: some deprecation warnings that need to be addressed in follow-up changes might be issued.

Signed-off-by: Krzysztof Wilczyński [email protected]

@rporres
Copy link
Contributor

rporres commented Jan 5, 2023

How do you plan to test this?

@kwilczynski
Copy link
Contributor Author

How do you plan to test this?

@rporres, I have been testing things manually locally as much as possible. That said, this is not a very involved upgrade as the feature set this provider has is small, and hopefully, if something breaks, it will be easy to fix.

Unless you have some recommendations on how to test this change better? I am happy to do whatever is needed to ensure that everything works.

@kwilczynski kwilczynski requested a review from rporres January 6, 2023 01:24
@rporres
Copy link
Contributor

rporres commented Jan 9, 2023

Let's discuss how you have been testing this, as we're in the same situation as in #65

@kwilczynski kwilczynski force-pushed the feature/update-terraform-random-provider-to-3.4.3 branch from ed9d7e0 to 159cea3 Compare March 1, 2023 06:35
@kwilczynski
Copy link
Contributor Author

Re-based against master.

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