Add external catalog table options to google_bigquery_table beta

[wj-chen]

Add external catalog table options to google_bigquery_table beta.

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.

bigquery: added `external_catalog_table_options` field to `google_bigquery_table` resource (beta)

[github-actions]

Hello! I am a robot. Tests will require approval from a repository maintainer to run.

@zli82016, a repository maintainer, has been assigned to review your changes. If you have not received review feedback within 2 business days, please leave a comment on this PR asking them to take a look.

You can help make sure that review is quick by doing a self-review and by running impacted tests locally.

[modular-magician]

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.

google provider: Diff ( 3 files changed, 72 insertions(+), 10 deletions(-))
google-beta provider: Diff ( 3 files changed, 431 insertions(+), 7 deletions(-))

Missing test report

Your PR includes resource fields which are not covered by any test.

Resource: google_bigquery_table (146 total tests)
Please add an acceptance test which includes these fields. The test should include the following:

resource "google_bigquery_table" "primary" {
  external_catalog_table_options {
    connection_id = # value needed
  }
}

[modular-magician]

Tests analytics

Total tests: 127
Passed tests: 116
Skipped tests: 10
Affected tests: 1

Click here to see the affected service packages

• bigquery

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

• TestAccBigQueryTable_externalCatalogTableOptions

Get to know how VCR tests work

[modular-magician]

🟢 Tests passed during RECORDING mode:
TestAccBigQueryTable_externalCatalogTableOptions [Debug log]

🟢 No issues found for passed tests after REPLAYING rerun.

---

🟢 All tests passed!

View the build log or the debug log for each test

[zli82016]

Missing test report

Your PR includes resource fields which are not covered by any test.

Resource: google_bigquery_table (146 total tests) Please add an acceptance test which includes these fields. The test should include the following:

resource "google_bigquery_table" "primary" {
  external_catalog_table_options {
    connection_id = # value needed
  }
}

Is it possible to add the missing test for connection_id?

[zli82016]

add the missing test for connection_id

[modular-magician]

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.

google provider: Diff ( 3 files changed, 72 insertions(+), 10 deletions(-))
google-beta provider: Diff ( 3 files changed, 447 insertions(+), 7 deletions(-))

[modular-magician]

Tests analytics

Total tests: 128
Passed tests: 117
Skipped tests: 10
Affected tests: 1

Click here to see the affected service packages

• bigquery

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

• TestAccBigQueryTable_externalCatalogTableOptions

Get to know how VCR tests work

[modular-magician]

🔴 Tests failed during RECORDING mode:
TestAccBigQueryTable_externalCatalogTableOptions [Error message] [Debug log]

🔴 Errors occurred during RECORDING mode. Please fix them to complete your PR.

View the build log or the debug log for each test

[modular-magician]

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.

google provider: Diff ( 3 files changed, 72 insertions(+), 10 deletions(-))
google-beta provider: Diff ( 3 files changed, 451 insertions(+), 7 deletions(-))

[modular-magician]

Tests analytics

Total tests: 128
Passed tests: 117
Skipped tests: 10
Affected tests: 1

Click here to see the affected service packages

• bigquery

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

• TestAccBigQueryTable_externalCatalogTableOptions

Get to know how VCR tests work

[modular-magician]

🟢 Tests passed during RECORDING mode:
TestAccBigQueryTable_externalCatalogTableOptions [Debug log]

🟢 No issues found for passed tests after REPLAYING rerun.

---

🟢 All tests passed!

View the build log or the debug log for each test

[wj-chen]

add the missing test for connection_id

Added connection_id as a field to the same new test

[zli82016]

LGTM

[zli82016]

@obada-ab, can you review this PR when you have time? Thanks.

[zli82016]

@wj-chen, is there a Github issue for this PR?

[obada-ab]

LGTM