Skip to content

Document + modifier syntax in bundle run --only help#5760

Open
afberg wants to merge 2 commits into
databricks:mainfrom
afberg:jobs-run-now-only-flag
Open

Document + modifier syntax in bundle run --only help#5760
afberg wants to merge 2 commits into
databricks:mainfrom
afberg:jobs-run-now-only-flag

Conversation

@afberg

@afberg afberg commented Jun 29, 2026

Copy link
Copy Markdown

Changes

bundle run --only takes a comma-separated list of task keys to run a subset of a job's tasks. Its help text did not mention that a task key can be prefixed or suffixed with + to also pull in upstream/downstream tasks (the value is forwarded to the run-now only API field, which interprets the modifiers). This documents that syntax in the flag's help:

  • +my_task: run my_task and everything upstream of it
  • my_task+: run my_task and everything downstream of it
  • +my_task+: both

Adresses #4244

Tests

  • New acceptance/bundle/run/jobs/partial_run case asserts --only +task_1,task_2+ is forwarded unchanged to the run-now request body ("only": ["+task_1", "task_2+"]), proving the documented + syntax is not mangled or rejected client-side.
  • Regenerated the bundle run --help golden for the updated --only description.

This pull request and its description were written by Isaac.

@afberg afberg force-pushed the jobs-run-now-only-flag branch from 4b5431e to e34491a Compare June 29, 2026 08:17
@afberg afberg marked this pull request as ready for review June 29, 2026 08:24
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Approval status: pending

/acceptance/bundle/ - needs approval

Files: acceptance/bundle/help/bundle-run/output.txt, acceptance/bundle/run/jobs/partial_run/output.txt, acceptance/bundle/run/jobs/partial_run/script
Suggested: @denik
Also eligible: @janniklasrose, @anton-107, @pietern, @andrewnester, @shreyas-goenka, @lennartkats-db

/bundle/ - needs approval

Files: bundle/run/job_options.go
Suggested: @denik
Also eligible: @janniklasrose, @anton-107, @pietern, @andrewnester, @shreyas-goenka, @lennartkats-db

General files (require maintainer)

Files: NEXT_CHANGELOG.md
Based on git history:

  • @denik -- recent work in ./, acceptance/bundle/run/jobs/partial_run/

Any maintainer (@andrewnester, @anton-107, @denik, @pietern, @shreyas-goenka, @simonfaltum, @renaudhartert-db) can approve all areas.
See OWNERS for ownership rules.

`bundle run --only` accepts task keys to run a subset of a job's tasks.
A task key may be prefixed with `+` (also run upstream tasks) or suffixed
with `+` (also run downstream tasks); document that in the flag's help text.

Co-authored-by: Isaac
@afberg afberg force-pushed the jobs-run-now-only-flag branch from e34491a to 6ba3ad7 Compare June 29, 2026 08:34
@afberg afberg changed the title Add --only flag to jobs run-now Jun 29, 2026
Add a partial_run case asserting `--only +task_1,task_2+` forwards
unchanged to the run-now `only` field, proving the CLI does not mangle or
reject the upstream/downstream modifier syntax documented on the flag.

Co-authored-by: Isaac
@github-actions

Copy link
Copy Markdown
Contributor

An authorized user can trigger integration tests manually by following the instructions below:

Trigger:
go/deco-tests-run/cli

Inputs:

  • PR number: 5760
  • Commit SHA: 3c8922a321fc6ef489a83954ee4179ec2d0ee888

Checks will be approved automatically on success.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant