Skip to content

Add support for regional secret resource google_secret_manager_regional_secret #2792

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

modular-magician
Copy link
Collaborator

Add support for new regional secret resource google_secret_manager_regional_secret. I have also defined the iam_policy field in the RegionalSecret.yaml file to include the IAM resources and IAM datasources as well.

More info about regional secrets: https://cloud.google.com/secret-manager/docs/regional-secrets-overview

Notes:

  1. In RegionalSecret.yaml file, the versionAliases field is commented and the respective test cases covered in the resource_secret_manager_regional_secret_test.go.erb file are also commented because this field depends on creating regional_secret_version_resource. We intend to uncomment once this PR is merged and dependent tests can pass.
  2. As of now, the API reference of the regional secrets are being documented on GCP docs side. Hence, the link mentioned in the ResourceSecret.yaml file will be updated later.

Release Note Template for Downstream PRs (will be copied)

`google_secret_manager_regional_secret`

Derived from GoogleCloudPlatform/magic-modules#11678

…nal_secret` (#11678)

[upstream:3084e72fbb69daba66bf7164f10a7b6af3a82d4a]

Signed-off-by: Modular Magician <[email protected]>
@modular-magician modular-magician requested a review from a team as a code owner September 12, 2024 20:26
@modular-magician modular-magician requested review from shuyama1 and removed request for a team September 12, 2024 20:26
Copy link

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@modular-magician modular-magician merged commit e7c6c59 into GoogleCloudPlatform:main Sep 12, 2024
3 checks passed
@modular-magician modular-magician deleted the downstream-pr-3084e72fbb69daba66bf7164f10a7b6af3a82d4a branch November 18, 2024 06:12
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.

1 participant