feat: added support for ssl_config for postgresql_profile

[pete-layerhealth]

Added support for ssl_config for postgresql_profile

shamelessly copied https://github.com/GoogleCloudPlatform/magic-modules/pull/12771 and incorporated feedback from @roaks3

Fixes https://github.com/hashicorp/terraform-provider-google/issues/20929

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.

datastream: added `postgresql_profile.ssl_config` to `google_datastream_connection_profile`

[google-cla[bot]]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[github-actions[bot]]

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.

@slevenick, 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 ( 4 files changed, 486 insertions(+), 5 deletions(-)) google-beta provider: Diff ( 4 files changed, 486 insertions(+), 5 deletions(-)) terraform-google-conversion: Diff ( 5 files changed, 372 insertions(+)) Open in Cloud Shell: Diff ( 4 files changed, 170 insertions(+))

Missing test report

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

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

resource "google_datastream_connection_profile" "primary" {
  postgresql_profile {
    ssl_config {
      server_verification {
        ca_certificate = # value needed
      }
    }
  }
}

[modular-magician]

Non-exercised tests

🔴 Tests were added that are skipped in VCR:

• TestAccDatastreamConnectionProfile_datastreamStreamPostgresqlSslconfigServerAndClientVerificationExample

Tests analytics

Total tests: 19 Passed tests: 9 Skipped tests: 10 Affected tests: 0

Click here to see the affected service packages

• datastream

🟢 All tests passed!

View the build log

[pete-layerhealth]

@slevenick would you be able to take a look? the previous PR was nearly approved but the author went MIA so it ended up staling

[github-actions[bot]]

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

[slevenick]

It still sees the commit as belonging to another contributor

[pete-layerhealth]

should be good now! sorry bout that

[pete-layerhealth]

@slevenick would love another check when you get a chance 🙏

[github-actions[bot]]

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

[github-actions[bot]]

@GoogleCloudPlatform/terraform-team @slevenick This PR has been waiting for review for 1 week. Please take a look! Use the label disable-review-reminders to disable these notifications.

[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 ( 4 files changed, 486 insertions(+), 5 deletions(-)) google-beta provider: Diff ( 4 files changed, 486 insertions(+), 5 deletions(-)) terraform-google-conversion: Diff ( 5 files changed, 372 insertions(+)) Open in Cloud Shell: Diff ( 4 files changed, 170 insertions(+))

Missing test report

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

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

resource "google_datastream_connection_profile" "primary" {
  postgresql_profile {
    ssl_config {
      server_verification {
        ca_certificate = # value needed
      }
    }
  }
}

[modular-magician]

Non-exercised tests

🔴 Tests were added that are skipped in VCR:

• TestAccDatastreamConnectionProfile_datastreamStreamPostgresqlSslconfigServerAndClientVerificationExample

Tests analytics

Total tests: 19 Passed tests: 9 Skipped tests: 10 Affected tests: 0

Click here to see the affected service packages

• datastream

🟢 All tests passed!

View the build log

[pete-layerhealth]

@slevenick any chance youd have a moment to take a look today? im hoping itll be quick given the previous attempt was already reviewed

[github-actions[bot]]

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

[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 ( 4 files changed, 486 insertions(+), 5 deletions(-)) google-beta provider: Diff ( 4 files changed, 486 insertions(+), 5 deletions(-)) terraform-google-conversion: Diff ( 5 files changed, 371 insertions(+)) Open in Cloud Shell: Diff ( 4 files changed, 170 insertions(+))

Missing test report

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

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

resource "google_datastream_connection_profile" "primary" {
  postgresql_profile {
    ssl_config {
      server_verification {
        ca_certificate = # value needed
      }
    }
  }
}

[modular-magician]

Non-exercised tests

🔴 Tests were added that are skipped in VCR:

• TestAccDatastreamConnectionProfile_datastreamStreamPostgresqlSslconfigServerAndClientVerificationExample

Tests analytics

Total tests: 19 Passed tests: 9 Skipped tests: 10 Affected tests: 0

Click here to see the affected service packages

• datastream

🟢 All tests passed!

View the build log

[github-actions[bot]]

@GoogleCloudPlatform/terraform-team @slevenick This PR has been waiting for review for 1 week. Please take a look! Use the label disable-review-reminders to disable these notifications.