Skip to content

Conversation

@anton-107
Copy link
Contributor

@anton-107 anton-107 commented Oct 15, 2025

Why

Prerequisite for TF provider upgrade

Tests

Updated existing tests

@eng-dev-ecosystem-bot
Copy link
Collaborator

eng-dev-ecosystem-bot commented Oct 15, 2025

Run: 18532314149

Env 🔄​flaky ✅​pass 🙈​skip
✅​ aws linux 322 545
✅​ aws windows 323 544
✅​ aws-ucws linux 438 441
✅​ aws-ucws windows 439 440
✅​ azure linux 322 544
🔄​ azure windows 2 321 543
✅​ azure-ucws linux 438 440
🔄​ azure-ucws windows 2 437 439
🔄​ gcp linux 1 320 546
✅​ gcp windows 322 545
Test Name azure windows azure-ucws windows gcp linux
TestAccept 🔄​f 🔄​f ✅​p
TestAccept/bundle/deployment/bind/job/generate-and-bind ✅​p 🔄​f ✅​p
TestAccept/bundle/deployment/bind/model-serving-endpoint 🔄​f ✅​p ✅​p
TestAuthDescribeSuccess ✅​p ✅​p 🔄​f

@anton-107 anton-107 marked this pull request as ready for review October 15, 2025 14:46
@@ -1,13 +1,5 @@

>>> [CLI] bundle validate -o json
Warning: required field "catalog_name" is not set
Copy link
Contributor

Choose a reason for hiding this comment

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

it seems odd that this warning is lost?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

in accordance to this change - CatalogName got an omitempty attribute: databricks/databricks-sdk-go@42ad9a8#r167972266

@anton-107 anton-107 added this pull request to the merge queue Oct 15, 2025
Merged via the queue into main with commit 89b2786 Oct 15, 2025
19 checks passed
@anton-107 anton-107 deleted the anton-107/go-sdk-0.85.0 branch October 15, 2025 15:24
deco-sdk-tagging bot added a commit that referenced this pull request Oct 16, 2025
## Release v0.273.0

### Notable Changes

* (via Terraform v1.92.0) DABs will no longer try to update pipeline permissions upon pipeline deletion. This fixes PERMISSION\_ERROR upon 'bundle destroy'
  for pipelines that have run\_as setting enabled (described in https://community.databricks.com/t95/data-engineering/dab-dlt-destroy-fails-due-to-ownership-permissions-mismatch/td-p/132101)
  The downside is that if 'permissions:' block is removed from the resource, DABs will not try anymore to restore permissions to just the owner of the pipeline.

### CLI

* Add the `--configure-serverless` flag to `databricks auth login` to configure Databricks Connect to use serverless.

### Dependency updates
* Upgrade Go SDK to 0.82.0 ([#3769](#3769))
* Upgrade TF provider to 1.92.0 ([#3772](#3772))

### Bundles
* Updated the internal lakeflow-pipelines template to use an "src" layout ([#3671](#3671)).
* Fix for pip flags with equal sign being incorrectly treated as local file names ([#3766](#3766))
github-merge-queue bot pushed a commit that referenced this pull request Nov 4, 2025
## Changes

See https://github.com/databricks/databricks-sdk-go/releases/tag/v0.89.0

This change removes previously added flags to the `jobs create` and
`pipelines create` commands. These flags were not part of these commands
before and were added by accident (in #3769). Because the payload for
these commands is complex, they take a `--json` flag instead.
pietern added a commit that referenced this pull request Dec 5, 2025
## Changes

Filter out enum values with DEVELOPMENT or PRIVATE_PREVIEW launch stages
when extracting annotations from the OpenAPI spec. Only values with GA,
PUBLIC_PREVIEW, or PUBLIC_BETA stages are included in the generated schema.

## Why

Different OpenAPI spec versions have inconsistent visibility filtering,
causing enum values like LIQUID to appear and disappear across commits.
This made the annotations file non-deterministic depending on which spec
was used.

Evidence of flip-flopping (newest to oldest):
- a6fbd57 Add support for alerts to DABs (#4004) - LIQUID re-added
- ecc9d54 Regenerate autogenerated files (#3981) - LIQUID removed
- 89b2786 Upgrade go sdk to 0.85.0 (#3769) - LIQUID first added
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.

4 participants