Skip to content

acc: Make run_as/job_default test run only on direct#5012

Open
andrewnester wants to merge 2 commits intomainfrom
fix/job_run_as_test
Open

acc: Make run_as/job_default test run only on direct#5012
andrewnester wants to merge 2 commits intomainfrom
fix/job_run_as_test

Conversation

@andrewnester
Copy link
Copy Markdown
Contributor

Changes

Make run_as/job_default test run only on direct

Why

This test on TF causes a permanent drift and differnce in the output. But for this test there is not real reason to run it on both direct and TF.
See the difference (source field is included) here:

https://github.com/databricks-eng/eng-dev-ecosystem/actions/runs/24552546859/job/71781373042

 --- bundle/run_as/job_default/output.txt
          +++ /tmp/TestAcceptbundlerun_asjob_defaultDATABRICKS_BUNDLE_ENGINE=te1133212625/001/output.txt
          @@ -33,7 +33,8 @@
                     "spark_version": "13.3.x-snapshot-scala2.12"
                   },
                   "notebook_task": {
          -          "notebook_path": "/Workspace/Users/[USERNAME]/.bundle/run_as_job_default/default/files/test"
          +          "notebook_path": "/Workspace/Users/[USERNAME]/.bundle/run_as_job_default/default/files/test",
          +          "source": "***"
                   },
                   "task_key": "task_one"
                 }

Tests

@andrewnester andrewnester enabled auto-merge April 17, 2026 12:49
@andrewnester andrewnester requested a review from denik April 17, 2026 12:49
@github-actions
Copy link
Copy Markdown

Waiting for approval

Could not determine reviewers from git history.
Round-robin suggestion: @lennartkats-db

Eligible reviewers: @anton-107, @denik, @janniklasrose, @lennartkats-db, @pietern, @shreyas-goenka

Suggestions based on git history. See OWNERS for ownership rules.

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.

1 participant