Conversation
Regenerate cli.json, command stubs, schemas, validation, python dataclasses, and direct-engine resource types from the v0.147.0 OpenAPI spec. Manual fixups for SDK changes: - Workspace service Bundle was renamed to BundleDeployments; repoint the DAB `bundle` metadata-service wrapper and the top-level command filter at the new bundle-deployments package and drop the orphaned cmd/workspace/bundle. - Wire new SDK fields into direct-engine resources: CustomMaxRetentionHours (catalog/schema), ServerlessComputeId (pipeline), and output-only *Id fields on the postgres resource bodies. - Classify new synced-table fields (accelerated_sync, type_overrides) as recreate_on_changes; neither synced-table API supports update. - Drop the metadata.generator_version assertion in internal/cligen; the current cli_v1 producer no longer emits the cli.json metadata block. Co-authored-by: Isaac
Co-authored-by: Isaac
simonfaltum
approved these changes
Jun 17, 2026
simonfaltum
left a comment
simonfaltum
left a comment
Member
There was a problem hiding this comment.
This looks ok. Check the comments
Pick up #5588's dropShadowingPlaceholders so cli.json descriptions (e.g. gen_ai_compute_task) flow into the schema instead of being shadowed by stale PLACEHOLDER markers. Also restore 0644 mode on the tagging files that genkit marked executable. Co-authored-by: Isaac
Collaborator
Integration test reportCommit: 33a4412
621 interesting tests: 437 MISS, 152 FAIL, 26 KNOWN, 4 PANIC, 2 SKIP
Top 50 slowest tests (at least 2 minutes):
|
This was referenced Jun 18, 2026
artchen-db
pushed a commit
to artchen-db/cli
that referenced
this pull request
Jun 18, 2026
…ds (databricks#5640) The SDK bump to v0.147.0 (databricks#5636) added top-level ID fields (`project_id`, `branch_id`, `endpoint_id`, `database_id`, `role_id`, `catalog_id`) to the postgres resource types, which the real Lakebase GET APIs now return. This broke every postgres acceptance test on cloud with golden-file mismatches. Regenerated the affected goldens against cloud, and taught the testserver to emit the same top-level IDs on create so local runs match cloud. Synced tables are unaffected. Verified: postgres acceptance suite passes both on cloud (`-update`) and locally, testserver unit tests pass, lint clean. This pull request and its description were written by Isaac.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
Bumps databricks-sdk-go from v0.141.0 to v0.147.0 and regenerates
cli.json, command stubs, bundle schemas, validation code, Python dataclasses, and direct-engine resource types from the v0.147.0 OpenAPI spec.Manual fixups for SDK changes:
Bundleservice was renamed toBundleDeployments: the generated command moves tocmd/workspace/bundle-deployments, the DABbundlemetadata-service wrapper and thecmd/cmd.gotop-level filter are repointed, and the orphanedcmd/workspace/bundleis removed.CustomMaxRetentionHours(catalog/schema),ServerlessComputeId(pipeline), and output-only*Idfields on the postgres resource bodies.accelerated_sync,type_overrides) classified asrecreate_on_changes; neither synced-table API supports update.metadata.generator_versionassertion ininternal/cligen, since the current cli_v1 producer no longer emits thecli.jsonmetadata block.New top-level commands appear from the spec (
ai-search, accountdisaster-recovery).Why
Keep the CLI current with the latest Databricks Go SDK and OpenAPI spec.
Tests
All non-integration unit and acceptance tests pass.
This PR was written by Isaac.