Removed Immutability for log_linked_dataset_query_user_email

[arnavdham]

Subscriber email logging is not an immutable resource. Once enabled, it cannot be turned off (prevented by APIs directly), but if it was initially off, it can be enabled.

bigqueryanalyticshub: supported in-place update for `log_linked_dataset_query_user_email` in `google_bigquery_analytics_hub_listing` and `google_bigquery_analytics_hub_data_exchange` resources. Once enabled, this feature cannot be disabled.

[github-actions]

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

Googlers: For automatic test runs see go/terraform-auto-test-runs.

@shuyama1, 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 ( 2 files changed, 20 insertions(+), 2 deletions(-))
google-beta provider: Diff ( 2 files changed, 20 insertions(+), 2 deletions(-))

[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 ( 2 files changed, 20 insertions(+), 2 deletions(-))
google-beta provider: Diff ( 2 files changed, 20 insertions(+), 2 deletions(-))

[modular-magician]

Tests analytics

Total tests: 17
Passed tests: 17
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages

• bigqueryanalyticshub

🟢 All tests passed!

View the build log

[modular-magician]

Tests analytics

Total tests: 17
Passed tests: 17
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages

• bigqueryanalyticshub

🟢 All tests passed!

View the build log

[shuyama1]

Thanks for the changes! Could you please add a new test or update an existing one to cover the update functionality?

https://googlecloudplatform.github.io/magic-modules/test/test/#add-an-update-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 ( 2 files changed, 20 insertions(+), 2 deletions(-))
google-beta provider: Diff ( 2 files changed, 20 insertions(+), 2 deletions(-))

[modular-magician]

Tests analytics

Total tests: 17
Passed tests: 17
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages

• bigqueryanalyticshub

🟢 All tests passed!

View the build log

[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, 25 insertions(+), 2 deletions(-))
google-beta provider: Diff ( 3 files changed, 25 insertions(+), 2 deletions(-))

[modular-magician]

Tests analytics

Total tests: 17
Passed tests: 16
Skipped tests: 0
Affected tests: 1

Click here to see the affected service packages

• bigqueryanalyticshub

#### 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

• TestAccBigqueryAnalyticsHubListing_bigqueryAnalyticshubListingUpdate

Get to know how VCR tests work

[modular-magician]

🟢 Tests passed during RECORDING mode:
TestAccBigqueryAnalyticsHubListing_bigqueryAnalyticshubListingUpdate [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

[arnavdham]

Thanks for the changes! Could you please add a new test or update an existing one to cover the update functionality?

https://googlecloudplatform.github.io/magic-modules/test/test/#add-an-update-test

Added

[shuyama1]

Thanks for the quick update! Mostly good, just some small comments

[shuyama1]

Your PR description mentions that this field can't be turned off once it's enabled.
Would you mind adding that to the field description?

[shuyama1]

Same here.

[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 ( 5 files changed, 29 insertions(+), 6 deletions(-))
google-beta provider: Diff ( 5 files changed, 29 insertions(+), 6 deletions(-))

[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 ( 5 files changed, 29 insertions(+), 6 deletions(-))
google-beta provider: Diff ( 5 files changed, 29 insertions(+), 6 deletions(-))

[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 ( 5 files changed, 29 insertions(+), 6 deletions(-))
google-beta provider: Diff ( 5 files changed, 29 insertions(+), 6 deletions(-))

[modular-magician]

Tests analytics

Total tests: 17
Passed tests: 17
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages

• bigqueryanalyticshub

🟢 All tests passed!

View the build log

[modular-magician]

Tests analytics

Total tests: 17
Passed tests: 17
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages

• bigqueryanalyticshub

🟢 All tests passed!

View the build log

[modular-magician]

Tests analytics

Total tests: 17
Passed tests: 17
Skipped tests: 0
Affected tests: 0

Click here to see the affected service packages

• bigqueryanalyticshub

🟢 All tests passed!

View the build log