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

Revert "Revert "fix: Update config.go and framework_config.go to use transport.Creds for all cases"" #7955

Merged
merged 1 commit into from
May 16, 2023

Conversation

shuyama1
Copy link
Member

Reverts #7948

The original PR #7875 does not cause the error in hashicorp/terraform-provider-google#14602. Therefore, bring back the feature.

provider: fixed an issue where mtls transports were not used consistently

…transport.Creds for all cases" (#7948)"

This reverts commit b2d535b.
@modular-magician
Copy link
Collaborator

Hi there, I'm the Modular magician. I've detected the following information about your changes:

Diff report

Your PR generated some diffs in downstreams - here they are.

Terraform GA: Diff ( 2 files changed, 6 insertions(+), 10 deletions(-))
Terraform Beta: Diff ( 2 files changed, 6 insertions(+), 10 deletions(-))
TF Conversion: Diff ( 3 files changed, 6 insertions(+), 8 deletions(-))

@shuyama1 shuyama1 requested a review from rileykarson May 16, 2023 17:07
Copy link
Member

@rileykarson rileykarson left a comment

Choose a reason for hiding this comment

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

You've checked that unreverting this + applying your other change has the intended effect, right? Just to make sure haha

@shuyama1
Copy link
Member Author

You've checked that unreverting this + applying your other change has the intended effect, right? Just to make sure haha

Yeah, I tested it locally with the other fix on top of the provider prior to the original PR revert, and it worked as intended.

@modular-magician
Copy link
Collaborator

Tests analytics

Total tests: 2740
Passed tests 2448
Skipped tests: 283
Affected tests: 9

Action taken

Found 9 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
TestAccDataSourceGoogleFirebaseAndroidAppConfig|TestAccComputeFirewallPolicyRule_multipleRules|TestAccBigtableAppProfile_bigtableAppProfileMulticlusterExample|TestAccBigtableAppProfile_bigtableAppProfileAnyclusterExample|TestAccAlloydbCluster_missingLocation|TestAccApigeeKeystoresAliasesPkcs12_ApigeeKeystoresAliasesPkcs12Example|TestAccDataSourceAlloydbLocations_basic|TestAccApigeeKeystoresAliasesKeyCertFile_apigeeKeystoresAliasesKeyCertFileTestExample|TestAccAlloydbBackup_missingLocation

Get to know how VCR tests work

@modular-magician
Copy link
Collaborator

Tests passed during RECORDING mode:
TestAccDataSourceGoogleFirebaseAndroidAppConfig[Debug log]
TestAccBigtableAppProfile_bigtableAppProfileMulticlusterExample[Debug log]
TestAccBigtableAppProfile_bigtableAppProfileAnyclusterExample[Debug log]
TestAccAlloydbCluster_missingLocation[Debug log]
TestAccDataSourceAlloydbLocations_basic[Debug log]
TestAccAlloydbBackup_missingLocation[Debug log]

Tests failed during RECORDING mode:
TestAccComputeFirewallPolicyRule_multipleRules[Error message] [Debug log]
TestAccApigeeKeystoresAliasesPkcs12_ApigeeKeystoresAliasesPkcs12Example[Error message] [Debug log]
TestAccApigeeKeystoresAliasesKeyCertFile_apigeeKeystoresAliasesKeyCertFileTestExample[Error message] [Debug log]

Please fix these to complete your PR
View the build log or the debug log for each test

@rileykarson rileykarson mentioned this pull request May 16, 2023
5 tasks
@shuyama1 shuyama1 merged commit 3e26f18 into main May 16, 2023
@shuyama1 shuyama1 deleted the revert-7948-revert-7875-main branch May 16, 2023 20:50
shourya116 pushed a commit to shourya116/magic-modules that referenced this pull request May 25, 2023
ericayyliu pushed a commit to ericayyliu/magic-modules that referenced this pull request Jul 26, 2023
wj-chen pushed a commit to wj-chen/magic-modules that referenced this pull request Aug 1, 2023
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