Skip to content

Shorten constraints refresh dispatch inputs and add run summary#69467

Closed
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:constraints-dispatch-ux
Closed

Shorten constraints refresh dispatch inputs and add run summary#69467
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:constraints-dispatch-ux

Conversation

@potiuk

@potiuk potiuk commented Jul 6, 2026

Copy link
Copy Markdown
Member

Follow-up to #69457 (merged).

Two small UX improvements to the manually-dispatched constraints refresh:

  • Shorter workflow_dispatch input descriptions. They rendered as long paragraphs in the "Run workflow" form, which is hard to scan. They are now concise one-liners — the full guidance still lives in the workflow header comment and dev/MANUALLY_GENERATING_IMAGE_CACHE_AND_CONSTRAINTS.md.
  • A run summary. A new job-summary step writes a small table to the run page showing how the run was triggered (push vs manual dispatch), the ref and upgrade flag it was dispatched with, and the resolved target constraints-X-Y branch — so a run's inputs are visible at a glance.

Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Opus 4.8)

Generated-by: Claude Code (Opus 4.8) following the guidelines

The workflow_dispatch input descriptions rendered as long paragraphs in
the "Run workflow" form, which is hard to scan. Shorten them to concise
one-liners (the full guidance already lives in the workflow header
comment and dev/MANUALLY_GENERATING_IMAGE_CACHE_AND_CONSTRAINTS.md).

Also add a job-summary step that shows how the run was triggered and
with which parameters (ref, upgrade flag) and the resolved target
constraints branch, so a run's inputs are visible at a glance.

@amoghrajesh amoghrajesh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

@potiuk

potiuk commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

Superseded by #69472 — same change, moved to a branch in apache/airflow so CI runs with full permissions (this touches a workflow). Closing this fork-based PR to avoid a duplicate.


Drafted-by: Claude Code (Opus 4.8); reviewed by @potiuk before posting

@potiuk potiuk closed this Jul 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants