Skip to content

feat: regenerated google-cloud-capacityplanner#8642

Open
jskeet wants to merge 1 commit into
googleapis:mainfrom
jskeet:local-owlbot-capacityplanning
Open

feat: regenerated google-cloud-capacityplanner#8642
jskeet wants to merge 1 commit into
googleapis:mainfrom
jskeet:local-owlbot-capacityplanning

Conversation

@jskeet

@jskeet jskeet commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@jskeet jskeet requested a review from a team as a code owner June 15, 2026 11:18
@jskeet

jskeet commented Jun 15, 2026

Copy link
Copy Markdown
Contributor Author

This has been split from #8641 due to test failures.

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

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.

Code Review

This pull request removes the CapacityPlanningService from the @google-cloud/capacityplanner package, leaving only the UsageService. It also updates the UsageService proto definitions to make several fields required (such as cloud_resource_type, reservation_type, and reservation_data_level) and updates the generated samples and client code accordingly. Feedback was provided regarding the package version downgrade to 0.1.0 (which would fail npm publishing since 0.1.3 is already released) and the use of a pre-release version of google-gax in the dependencies.

"url": "https://github.com/googleapis/google-cloud-node.git",
"directory": "packages/google-cloud-capacityplanner"
},
"version": "0.1.0",

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.

high

Downgrading the package version from 0.1.3 to 0.1.0 will cause publishing to fail on the npm registry, as 0.1.3 has already been released. Please update the generator configuration or upstream templates to preserve the correct version sequence, as manual edits to this file are not allowed.

References
  1. Do not manually edit generated client configuration JSON files. Any changes or updates must be made upstream in the generator.

},
"dependencies": {
"google-gax": "^5.0.0"
"google-gax": "^5.1.1-rc.1"

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

Using a pre-release version (^5.1.1-rc.1) in the dependencies of a production package is discouraged as it can introduce instability for downstream consumers. Please use a stable version of google-gax (e.g., ^5.1.0) in the generator configuration or upstream templates, as manual edits to this file are not allowed.

References
  1. Do not manually edit generated client configuration JSON files. Any changes or updates must be made upstream in the generator.

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