Skip to content

bump terraform-plugin-sdk/v2 2.23 -> 2.26 #3412

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

Merged
merged 3 commits into from
Feb 7, 2025

Conversation

BBBmau
Copy link
Contributor

@BBBmau BBBmau commented Feb 6, 2025

this is necessary to get successful tests from passing in terraform-google-conversion as we use sdk/v2 2.26.0 for the latest work on ephemeral write-only support.

We see that the following GHA is due to errors in the tests. https://github.com/GoogleCloudPlatform/magic-modules/actions/runs/13170402493/job/36759965728

Upon more investigation i found that when building the tgc repo locally I found that it's due to version mismatch with sdk/v2
image

upon updating sdk/v2 to the latest version in go.mod and running go mod tidy I resolve the errors.

GoogleCloudPlatform/magic-modules#12800 is dependent on this PR in order for it to be moved into review.

@BBBmau BBBmau requested a review from a team as a code owner February 6, 2025 03:49
@BBBmau BBBmau requested review from shuyama1 and removed request for a team February 6, 2025 03:49
@melinath
Copy link
Member

melinath commented Feb 6, 2025

drive-by /gcbrun

Copy link
Member

@shuyama1 shuyama1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@BBBmau Looks like there're merge conflicts that need to be resolved before merging.

@BBBmau
Copy link
Contributor Author

BBBmau commented Feb 7, 2025

fixed. I need anothe trigger on the GHA @shuyama1

@shuyama1
Copy link
Member

shuyama1 commented Feb 7, 2025

/gcbrun

@BBBmau
Copy link
Contributor Author

BBBmau commented Feb 7, 2025

/gcbrun

any chance you can resolve merge conflicts to test and approve right away? The quick merges to main are causing constant merge conflicts.

@shuyama1
Copy link
Member

shuyama1 commented Feb 7, 2025

/gcbrun

@shuyama1 shuyama1 merged commit 26d9c51 into GoogleCloudPlatform:main Feb 7, 2025
5 of 6 checks passed
@BBBmau
Copy link
Contributor Author

BBBmau commented Feb 7, 2025

@shuyama1 thanks!

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.

3 participants