Skip to content

chore: librarian generate pull request: 20260409T191103Z#16598

Closed
cloud-sdk-librarian-robot wants to merge 1 commit intomainfrom
librarian-20260409T191103Z
Closed

chore: librarian generate pull request: 20260409T191103Z#16598
cloud-sdk-librarian-robot wants to merge 1 commit intomainfrom
librarian-20260409T191103Z

Conversation

@cloud-sdk-librarian-robot
Copy link
Copy Markdown
Collaborator

PR created by the Librarian CLI to generate Cloud Client Libraries code from protos.

BEGIN_COMMIT

BEGIN_NESTED_COMMIT
feat: a new field base64_encoded_name is added to the Product message

PiperOrigin-RevId: 896968813
Library-IDs: google-shopping-merchant-products
Source-link: googleapis/googleapis@2aba4849
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
feat: new fields - base64_encoded_name and base64_encoded_product added to the ProductInput message

PiperOrigin-RevId: 896968813
Library-IDs: google-shopping-merchant-products
Source-link: googleapis/googleapis@2aba4849
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
docs: A comment for field name in message .google.shopping.merchant.products.v1.ProductInput is changed

PiperOrigin-RevId: 896968813
Library-IDs: google-shopping-merchant-products
Source-link: googleapis/googleapis@2aba4849
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
docs: A comment for field name in message .google.shopping.merchant.products.v1.Product is changed

PiperOrigin-RevId: 896968813
Library-IDs: google-shopping-merchant-products
Source-link: googleapis/googleapis@2aba4849
END_NESTED_COMMIT

BEGIN_NESTED_COMMIT
feat: adding DeploymentGroups, you can now manage deployment of multiple module root dependencies in a single DAG

PiperOrigin-RevId: 896570259
Library-IDs: google-cloud-config
Source-link: googleapis/googleapis@f5cb7afc
END_NESTED_COMMIT

END_COMMIT

This pull request is generated with proto changes between
googleapis/googleapis@2233f63b
(exclusive) and
googleapis/googleapis@2aba4849
(inclusive).

Librarian Version: v0.0.0-20260407211006-7cd74ea13ba9
Language Image: us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/python-librarian-generator@sha256:234b9d1f2ddb057ed7ac6a38db0bf8163d839c65c6cf88ade52530cddebce59e

Generation failed for

  • google-cloud-spanner

@cloud-sdk-librarian-robot cloud-sdk-librarian-robot requested a review from a team as a code owner April 9, 2026 19:11
@cloud-sdk-librarian-robot
Copy link
Copy Markdown
Collaborator Author

One or more libraries have failed to generate, please review PR description for a list of failed libraries.
For each failed library, open a ticket in that library’s repository and then you may resolve this comment and merge.

@cloud-sdk-librarian-robot cloud-sdk-librarian-robot requested a review from a team as a code owner April 9, 2026 19:11
@snippet-bot
Copy link
Copy Markdown

snippet-bot bot commented Apr 9, 2026

Here is the summary of changes.

You are about to add 18 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

@daniel-sanche daniel-sanche added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Apr 9, 2026
@daniel-sanche
Copy link
Copy Markdown
Contributor

daniel-sanche commented Apr 9, 2026

I'm not sure if this was created intentionally or not, so adding the do not merge tag just in case. We should make sure this doesn't interfere with the librarian migration

My understanding was that no more generations would be happening until the migration is complete

Copy link
Copy Markdown
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 support for Deployment Groups in the google-cloud-config library, enabling the management of multiple deployments as a single unit with dependency tracking. It includes new CRUD operations, long-running operations for provisioning/deprovisioning, and support for external value sources in Terraform blueprints. Additionally, the google-shopping-merchant-products library is updated with new base64-encoded name fields for products and product inputs, and a return_policy_label attribute. The review feedback suggests clarifying documentation examples for encoded resource names to avoid confusion between full resource names and product IDs.

Comment on lines +122 to +124
``accounts/123/productInputs/ZW5-VVN-c2t1LzEyMw`` for the
decoded product input name
``accounts/123/productInputs/en~US~sku/123``. This field can
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The phrase "for the decoded product input name" is confusing as it might imply the full resource name is decoded. For better clarity, it would be best to refer to the product ID directly.

Suggested change
``accounts/123/productInputs/ZW5-VVN-c2t1LzEyMw`` for the
decoded product input name
``accounts/123/productInputs/en~US~sku/123``. This field can
accounts/123/productInputs/ZW5-VVN-c2t1LzEyMw for a
product input with the ID en~US~sku/123. This field can

Comment on lines +138 to +140
``accounts/123/products/ZW5-VVN-c2t1LzEyMw`` for the decoded
product name ``accounts/123/products/en~US~sku/123``. This
field can be used directly as input to the API methods that
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The phrase "for the decoded product name" is confusing as it might imply the full resource name is decoded. For better clarity, it would be best to refer to the product ID directly.

Suggested change
``accounts/123/products/ZW5-VVN-c2t1LzEyMw`` for the decoded
product name ``accounts/123/products/en~US~sku/123``. This
field can be used directly as input to the API methods that
accounts/123/products/ZW5-VVN-c2t1LzEyMw for a product
with the ID en~US~sku/123. This field can be used
directly as input to the API methods that

Comment on lines +72 to +74
name. Example: ``accounts/123/products/ZW5-VVN-c2t1LzEyMw``
for the decoded product name
``accounts/123/products/en~US~sku/123``. This field can be
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The phrase "for the decoded product name" is confusing as it might imply the full resource name is decoded. For better clarity, it would be best to refer to the product ID directly.

Suggested change
name. Example: ``accounts/123/products/ZW5-VVN-c2t1LzEyMw``
for the decoded product name
``accounts/123/products/en~US~sku/123``. This field can be
name. Example: accounts/123/products/ZW5-VVN-c2t1LzEyMw
for a product with the ID en~US~sku/123. This field can be

@ohmayr ohmayr closed this Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants