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 Document + modifier syntax in bundle run --only help 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant