feat: optional sql server change tables and transaction logs

[nbugden]

Adds optional blocks for transaction_logs and change_tables to the sql_server_source_config.
Fixes hashicorp/terraform-provider-google#18836

Release Note Template for Downstream PRs (will be copied)

datastream: added `transaction_logs` and `change_tables` to the `datastream_stream` resource

[github-actions]

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

@BBBmau, 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, 93 insertions(+))
google-beta provider: Diff ( 2 files changed, 93 insertions(+))
terraform-google-conversion: Diff ( 1 file changed, 44 insertions(+))

[modular-magician]

Tests analytics

Total tests: 12
Passed tests: 4
Skipped tests: 8
Affected tests: 0

Click here to see the affected service packages

• datastream

$\textcolor{green}{\textsf{All tests passed!}}$

View the build log

[nbugden]

@BBBmau checking in to see if you can review this PR

[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, 201 insertions(+))
google-beta provider: Diff ( 2 files changed, 201 insertions(+))
terraform-google-conversion: Diff ( 1 file changed, 44 insertions(+))

[modular-magician]

Tests analytics

Total tests: 12
Passed tests: 4
Skipped tests: 8
Affected tests: 0

Click here to see the affected service packages

• datastream

$\textcolor{green}{\textsf{All tests passed!}}$

View the build log

[github-actions]

@nbugden, this PR is waiting for action from you. Please address any comments or change requests, or re-request review from a core reviewer if no action is required.

If no action is taken, this PR will be closed in 28 days.

This notification can be disabled with the disable-automatic-closure label.

[nbugden]

@BBBmau any updates here? Want to be able to close this out.

[nbugden]

@BBBmau can we get an eta on your review?

[BBBmau]

@BBBmau can we get an eta on your review?

apologies for the delay, I'll be taking a look at this today.

[BBBmau]

It's worth mentioning also that due to the upcoming 6.0.0 release, I am able to approve but won't be merging until after the major release is done.

[nbugden]

It's worth mentioning also that due to the upcoming 6.0.0 release, I am able to approve but won't be merging until after the major release is done.

Thanks for calling this out. When is the 6.0.0 release going out?

[BBBmau]

It's worth mentioning also that due to the upcoming 6.0.0 release, I am able to approve but won't be merging until after the major release is done.

Thanks for calling this out. When os the 6.0.0 release going out?

6.0.0 is set to be released next monday August 26th

[nbugden]

It's worth mentioning also that due to the upcoming 6.0.0 release, I am able to approve but won't be merging until after the major release is done.

Thanks for calling this out. When os the 6.0.0 release going out?

6.0.0 is set to be released next monday August 26th

Thanks! I made the change to conflicts from exactly_one_of. Can you review in the meantime?
c24b502

[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, 201 insertions(+))
google-beta provider: Diff ( 2 files changed, 201 insertions(+))
terraform-google-conversion: Diff ( 1 file changed, 44 insertions(+))

[modular-magician]

Tests analytics

Total tests: 12
Passed tests: 4
Skipped tests: 8
Affected tests: 0

Click here to see the affected service packages

• datastream

$\textcolor{green}{\textsf{All tests passed!}}$

View the build log

[github-actions]

@BBBmau This PR has been waiting for review for 3 weekdays. Please take a look! Use the label disable-review-reminders to disable these notifications.

[BBBmau]

apologies for the late review, everything looks good. thanks for including your debug logs!