acc: make acceptance tests work in Databricks development environments#5662
Merged
Conversation
pietern
force-pushed
the
acc-tests-ignore-local-dev-probes
branch
from
149eac3 to
5c5df59
Compare
pietern
force-pushed
the
acc-tests-ignore-local-dev-probes
branch
from
5c5df59 to
dc7f93b
Compare
pietern
force-pushed
the
acc-tests-ignore-local-dev-probes
branch
from
dc7f93b to
8f59ba8
Compare
pietern
commented
Jun 19, 2026
| uses: astral-sh/setup-uv@08807647e7069bb48b6ef5acd8ec9567f424441b # v8.1.0 | ||
| with: | ||
| version: "0.8.9" | ||
|
|
Contributor
Author
There was a problem hiding this comment.
We now run uv find python from the Taskfile instead of assuming the right version exists.
This means we need to make sure uv is available before using the Taskfile.
pietern
requested review from
andrewnester,
denik,
shreyas-goenka and
simonfaltum
Collaborator
Integration test reportCommit: 4ece006
|
The acceptance harness blocks external network access during tests. In some Databricks-internal development environments, local tooling issues network calls that trip the sandbox and fail the suite even though CI passes. Ignore the local probes that are not test traffic and disable the background beacons so tests behave the same as on CI. Also assert the external toolchain (jq, uv, ruff) and provision python via uv up front, so a missing or stale tool fails fast with a clear message. Co-authored-by: Isaac
pietern
force-pushed
the
acc-tests-ignore-local-dev-probes
branch
from
8f59ba8 to
a9da026
Compare
denik
reviewed
Jun 21, 2026
denik
left a comment
denik
left a comment
Contributor
There was a problem hiding this comment.
Thanks, failing early and with clear message is going to save a lot of time.
nit: binary version checking & localhost probe thing feel completely unrelated and both deserve each own PRs.
Take the required version as a string argument and compare with golang.org/x/mod/semver in each tool file, with the version (and the reason for it) surfaced at the call site. Group the four checks into a single requirePrerequisites helper to keep testAccept short. On Windows, verify the python3 on PATH meets the floor before skipping provisioning. Co-authored-by: Isaac
Contributor
Author
|
@denik The two changes are related in that both are needed to get a passing test run. |
denik
approved these changes
Jun 22, 2026
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.
The acceptance harness blocks external network access during tests. In some Databricks-internal development environments, local tooling makes network calls that trip the sandbox and fail the suite (CI is unaffected), and a stale local toolchain produced confusing diffs instead of clear errors.
localhost) and disable the local background beacons, so behavior matches CI.jq,uv,ruff, andpython >= 3.11(provisioned viauv).Testing: full direct acceptance suite passes locally in a Databricks dev environment; unit tests for the tool checks and python provisioning included.
This pull request and its description were written by Isaac.