Skip to content

Sm tags #10119

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

Merged
merged 14 commits into from
Jul 3, 2025
Merged

Sm tags #10119

merged 14 commits into from
Jul 3, 2025

Conversation

vipul7499
Copy link
Contributor

Description

Fixes #

Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.

Checklist

  • I have followed Sample Format Guide
  • pom.xml parent set to latest shared-configuration
  • Appropriate changes to README are included in PR
  • These samples need a new API enabled in testing projects to pass (let us know which ones)
  • These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
  • Tests pass: mvn clean verify required
  • Lint passes: mvn -P lint checkstyle:check required
  • Static Analysis: mvn -P lint clean compile pmd:cpd-check spotbugs:check advisory only
  • This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
  • This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
  • Please merge this PR for me once it is approved
@vipul7499 vipul7499 requested review from yoshi-approver and a team as code owners July 2, 2025 12:25
Copy link

snippet-bot bot commented Jul 2, 2025

Here is the summary of changes.

You are about to add 2 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment
@product-auto-label product-auto-label bot added samples Issues that are directly related to samples. api: secretmanager Issues related to the Secret Manager API. labels Jul 2, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary of Changes

Hello @vipul7499, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request introduces new functionality to the Google Cloud Secret Manager samples, enabling users to create secrets with associated tags. It includes both global and regional examples, along with comprehensive integration tests to ensure the new features work as expected. The changes involve updating dependencies, adding new sample files, and extending the test suite to cover the new tagging capabilities.

Highlights

  • New Feature: Secret Tagging: I've added new sample code that demonstrates how to create Google Cloud Secret Manager secrets with user-defined tags. This functionality allows for better organization and management of secrets.
  • Regional Secret Tagging Support: A dedicated sample has been included to show how to create regional Secret Manager secrets with tags, illustrating the process of interacting with regional Secret Manager endpoints.
  • Dependency Updates: The project dependencies have been updated to incorporate the latest Google Cloud BOM and explicitly include the google-cloud-resourcemanager library, which is essential for programmatically managing tags.
  • Comprehensive Integration Tests: New integration tests have been implemented to cover the new tagging functionality. These tests include the setup and teardown of tag keys and values, and verify the successful creation of secrets with these tags for both global and regional scenarios.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces new code samples for creating secrets with tags in Google Cloud Secret Manager, for both global and regional secrets. The changes include adding new dependencies, the sample code files, and corresponding integration tests. I've identified several areas for improvement, especially within the integration tests. My feedback focuses on strengthening test assertions and improving exception handling to prevent silent failures.

vipul7499 and others added 4 commits July 2, 2025 18:33
….java Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
…teRegionalSecretWithTags.java Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Copy link

@YashSaraf11 YashSaraf11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@vipul7499 vipul7499 merged commit 5d5f26f into main Jul 3, 2025
10 checks passed
@vipul7499 vipul7499 deleted the sm-tags branch July 3, 2025 10:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: secretmanager Issues related to the Secret Manager API. samples Issues that are directly related to samples.
2 participants