Add bump-serverless-env-version skill - #6419
Merged
Merged
Conversation
Collaborator
Integration test reportCommit: 5e633fd
Top 3 slowest tests (at least 2 minutes):
|
rugpanov
marked this pull request as ready for review
August 28, 2026 09:34
rugpanov
force-pushed
the
serverless-upgrade-guidance
branch
4 times, most recently
from
September 2, 2026 12:38
9374164 to
56ed261
Compare
Automates the serverless environment version bump across bundle templates: the serverless_environment_version macro plus the hardcoded literals in default-scala and lakeflow-integrations, the coupled Python / DB Connect pins in default/, golden regeneration across both template trees, and the mandatory verify pass. Delegates policy to .agents/rules/template-versions.md. Follow-up to #6378. Co-authored-by: Isaac <no-reply@databricks.com>
- Scope: defaultServerlessVersion (libs/localenv/envkey.go) is a separate "latest LTS" Go pin, not the SSH test fixture; tell the bumper to check and bump it in sync rather than "leave it alone". - Step 2: make the grep the source of truth; the file list is orientation only. Co-authored-by: Isaac <no-reply@databricks.com>
- Remove GOTOOLCHAIN=local from the golden commands: env-specific, diverges from bump-sdk/bump-tf, and can break contributors on older local Go. - Step 6: defer PR body to pr-checklist + the PR template (Changes/Why/Tests in order + authorship disclosure) instead of a competing template that dropped ## Why; broaden pre-PR checks to fmt/checks/lint/test. - Step 2: only version-bearing grep hits get edited (macro names / template references carry no literal). - Step 5: spell out the release-notes URL slug convention (five/six). Co-authored-by: Isaac <no-reply@databricks.com>
- Scope: when bumping defaultServerlessVersion, also refresh the version shown in its help/example/error strings (cmd/environments/sync.go, libs/localenv/constraints.go), not just regenerate tests. - Step 5: don't extrapolate the changelog line from v5 — adapt the benefit to the target version's actual release notes and link its real URL. Co-authored-by: Isaac <no-reply@databricks.com>
The "-update lets both EnvMatrix variants overwrite each other's output" explanation is stale: -update now auto-enables -subset (acceptance_test.go), regenerating each output file from one covering variant. Correct the rationale (the non-update pass matters because it runs *every* variant against the goldens) and stop framing a load-induced timeout as an automatic flake. Co-authored-by: Isaac <no-reply@databricks.com>
- Intro: only the golden output under acceptance/ is regenerated (acceptance/ also holds hand-maintained inputs), not "everything under acceptance/". - Step 3: use the full path libs/template/templates/default/library/versions.tmpl. Co-authored-by: Isaac <no-reply@databricks.com>
Only the conservative_db_connect_version_spec and python_version_spec blocks carry version-specific reasoning; name them instead of implying every macro does. Co-authored-by: Isaac <no-reply@databricks.com>
Narrow the trigger, target golden regeneration, separate the localenv fallback, and cross-check prior template bumps while keeping the workflow concise. Co-authored-by: Codex <noreply@openai.com>
rugpanov
force-pushed
the
bump-serverless-env-skill
branch
from
September 8, 2026 09:52
d6add1c to
4bc5e84
Compare
lennartkats-db
approved these changes
Sep 11, 2026
lennartkats-db
left a comment
lennartkats-db
left a comment
Contributor
There was a problem hiding this comment.
Four suggested minor improvements, otherwise LGTM. Thanks!
Add compatibility and support-buffer rules, include Python pins in discovery, and identify the suggested reviewer. Co-authored-by: Codex <noreply@openai.com>
Collaborator
Integration test reportCommit: 2c890bf
152 interesting tests: 110 MISS, 17 flaky, 17 FAIL, 7 KNOWN, 1 SKIP
Top 50 slowest tests (at least 2 minutes):
|
janniklasrose
pushed a commit
that referenced
this pull request
Sep 15, 2026
## Changes Add the `bump-serverless-env-version` skill for upgrading the default serverless environment version shipped by bundle templates. The skill covers coupled Python and DB Connect compatibility, macro and hardcoded version sources, the separate `defaultServerlessVersion` fallback, acceptance regeneration, and prior-bump cross-checks. It also updates the shared template-version policy to require more than six months of remaining DBR support when selecting the conservative DB Connect floor. ## Why The version 5 bump in #6378 was performed manually. This makes future bumps repeatable and prevents newly generated projects from starting too close to a DBR end-of-support date. ## Tests - Parsed both Markdown frontmatter blocks and ran `git diff --check`. - Forward-tested a hypothetical version 6 bump with fresh agents; the skill stopped when version 6 was not published instead of guessing compatibility data. - Independently reviewed routing, scope, authorization boundaries, repository paths, commands, and policy consistency. - `./task checks` could not run because Go 1.26.8 was not installed and the environment could not reach the Go toolchain download. _This PR was written with Claude Code and Codex._ --------- Co-authored-by: Isaac <no-reply@databricks.com> Co-authored-by: Codex <noreply@openai.com>
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
Add the
bump-serverless-env-versionskill for upgrading the default serverless environment version shipped by bundle templates.The skill covers coupled Python and DB Connect compatibility, macro and hardcoded version sources, the separate
defaultServerlessVersionfallback, acceptance regeneration, and prior-bump cross-checks. It also updates the shared template-version policy to require more than six months of remaining DBR support when selecting the conservative DB Connect floor.Why
The version 5 bump in #6378 was performed manually. This makes future bumps repeatable and prevents newly generated projects from starting too close to a DBR end-of-support date.
Tests
git diff --check../task checkscould not run because Go 1.26.8 was not installed and the environment could not reach the Go toolchain download.This PR was written with Claude Code and Codex.