Skip to content

Fix TestAccVPCAccessConnectorDatasource_basic#8766

Merged
ScottSuarez merged 2 commits into
GoogleCloudPlatform:mainfrom
ScottSuarez:TestAccVPCAccessConnectorDatasource_basic
Aug 29, 2023
Merged

Fix TestAccVPCAccessConnectorDatasource_basic#8766
ScottSuarez merged 2 commits into
GoogleCloudPlatform:mainfrom
ScottSuarez:TestAccVPCAccessConnectorDatasource_basic

Conversation

@ScottSuarez

Copy link
Copy Markdown
Contributor

Clashing due to subnet... changes ranges to be unique

=== RUN   TestAccVPCAccessConnectorDatasource_basic
=== PAUSE TestAccVPCAccessConnectorDatasource_basic
=== CONT  TestAccVPCAccessConnectorDatasource_basic
    vcr_utils.go:152: Step 1/1 error: Error running apply: exit status 1
        Error: Error waiting to create Connector: Error waiting for Creating Connector: Error code 3, message: Operation failed: Invalid IP CIDR range was provided. It conflicts with an existing subnetwork. Please delete the connector manually.
          with google_vpc_access_connector.connector,
          on terraform_plugin_test.tf line 2, in resource "google_vpc_access_connector" "connector":
           2: resource "google_vpc_access_connector" "connector" {
--- FAIL: TestAccVPCAccessConnectorDatasource_basic (37.30s)
FAIL

Release Note Template for Downstream PRs (will be copied)


@ScottSuarez ScottSuarez requested review from a team and trodge and removed request for a team August 25, 2023 02:10
@modular-magician

Copy link
Copy Markdown
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, 2 insertions(+), 2 deletions(-))
Terraform Beta: Diff ( 2 files changed, 2 insertions(+), 2 deletions(-))

@modular-magician

Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 2988
Passed tests 2689
Skipped tests: 296
Affected tests: 3

Action taken

Found 3 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
TestAccAppEngineStandardAppVersion_update|TestAccDataprocJobIamPolicy|TestAccVPCAccessConnectorDatasource_basic

Get to know how VCR tests work

@modular-magician

Copy link
Copy Markdown
Collaborator

$\textcolor{green}{\textsf{Tests passed during RECORDING mode:}}$
TestAccAppEngineStandardAppVersion_update[Debug log]
TestAccDataprocJobIamPolicy[Debug log]

Rerun these tests in REPLAYING mode to catch issues

$\textcolor{green}{\textsf{No issues found for passed tests after REPLAYING rerun.}}$


$\textcolor{red}{\textsf{Tests failed during RECORDING mode:}}$
TestAccVPCAccessConnectorDatasource_basic[Error message] [Debug log]

$\textcolor{red}{\textsf{Please fix these to complete your PR.}}$
View the build log or the debug log for each test

@modular-magician

Copy link
Copy Markdown
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, 2 insertions(+), 2 deletions(-))
Terraform Beta: Diff ( 2 files changed, 2 insertions(+), 2 deletions(-))

@modular-magician

Copy link
Copy Markdown
Collaborator

Tests analytics

Total tests: 2994
Passed tests 2697
Skipped tests: 296
Affected tests: 1

Action taken

Found 1 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected tests
TestAccVPCAccessConnectorDatasource_basic

Get to know how VCR tests work

@modular-magician

Copy link
Copy Markdown
Collaborator

$\textcolor{green}{\textsf{Tests passed during RECORDING mode:}}$
TestAccVPCAccessConnectorDatasource_basic[Debug log]

Rerun these tests in REPLAYING mode to catch issues

$\textcolor{green}{\textsf{No issues found for passed tests after REPLAYING rerun.}}$


$\textcolor{green}{\textsf{All tests passed!}}$
View the build log or the debug log for each test

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