Skip to content

[DECO-28072] Fix Major-Only Runtime Version Selection - #1805

Merged
rauchy merged 1 commit into
mainfrom
omer-lachish_data/deco-28072-fix-major-only-runtime-version-selection
Aug 14, 2026
Merged

[DECO-28072] Fix Major-Only Runtime Version Selection#1805
rauchy merged 1 commit into
mainfrom
omer-lachish_data/deco-28072-fix-major-only-runtime-version-selection

Conversation

@rauchy

@rauchy rauchy commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes Spark runtime selection so major-only runtime keys such as 18.x sort correctly alongside legacy major-minor keys.

Why

This behavior was originally reported in terraform-provider-databricks#5866.

SelectSparkVersion normalizes runtime keys before comparing them as semantic versions. The existing normalization only recognized major.minor.x, leaving major.x keys invalid and sorting them behind older valid versions.

What changed

Interface changes

None.

Behavioral changes

Latest-version queries now correctly prefer newer major-only runtime versions when they match the requested filters.

Internal changes

The runtime-version parser accepts both numbering formats, with a regression test covering LTS selection across 17.3.x and 18.x.

@rauchy
rauchy deployed to test-trigger-is August 14, 2026 10:56 — with GitHub Actions Active
@rauchy
rauchy deployed to test-trigger-is August 14, 2026 10:56 — with GitHub Actions Active
@rauchy
rauchy deployed to test-trigger-is August 14, 2026 10:57 — with GitHub Actions Active
@rauchy
rauchy requested a review from parthban-db August 14, 2026 11:02
@rauchy
rauchy marked this pull request as ready for review August 14, 2026 11:02
@rauchy
rauchy force-pushed the omer-lachish_data/deco-28072-fix-major-only-runtime-version-selection branch from 3d7238c to ee5464b Compare August 14, 2026 11:17
@rauchy
rauchy deployed to test-trigger-is August 14, 2026 11:17 — with GitHub Actions Active
@rauchy
rauchy deployed to test-trigger-is August 14, 2026 11:17 — with GitHub Actions Active
@rauchy
rauchy deployed to test-trigger-is August 14, 2026 11:18 — with GitHub Actions Active
Signed-off-by: Omer Lachish <rauchy@users.noreply.github.com>
@rauchy
rauchy force-pushed the omer-lachish_data/deco-28072-fix-major-only-runtime-version-selection branch from ee5464b to 66cdea7 Compare August 14, 2026 11:20
@rauchy
rauchy deployed to test-trigger-is August 14, 2026 11:20 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-go

Inputs:

  • PR number: 1805
  • Commit SHA: 66cdea7e85ceab443188276779810aff2c3c9412

Checks will be approved automatically on success.

@rauchy
rauchy deployed to test-trigger-is August 14, 2026 11:21 — with GitHub Actions Active
@rauchy
rauchy deployed to test-trigger-is August 14, 2026 11:22 — with GitHub Actions Active
@rauchy
rauchy enabled auto-merge August 14, 2026 11:23
@rauchy
rauchy added this pull request to the merge queue Aug 14, 2026
Merged via the queue into main with commit 404204a Aug 14, 2026
13 checks passed
@rauchy
rauchy deleted the omer-lachish_data/deco-28072-fix-major-only-runtime-version-selection branch August 14, 2026 11:39
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.

2 participants