fix(actions): make the run id for run-watch/run-get obvious from the runs listing #466
Labels
No labels
breaking
bug
documentation
enhancement
epic
good first issue
help wanted
refactoring
resolution/duplicate
resolution/invalid
resolution/wontfix
security
severity/critical
severity/high
severity/low
severity/medium
status/abandoned
status/blocked
status/needs-decision
status/needs-info
status/needs-verification
testing
upstream
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
sixfold-space/madtea#466
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
When a merge is gated on CI, I watch the run to see it pass. Going from
actions runstorun-watch/run-getis not obvious, so I wasted time digging the id out of raw JSON instead of reading it off the listing.What I saw
actions runsprints a run like this:run-watch/run-gettake arun_id, and their help says "the API id (not the web UI run number)". The listing shows#182with a#, which reads as a web UI run number, so I did not trust it. In fact182IS the API id, andrun-watch 182works. The real web UI number is different: the run'shtml_urlends in/runs/28, andindex_in_repois 28. So:#182)The one number the listing gives me is the right one, but everything about how it is presented says it is the wrong one.
Why the fallback hurts
Reading the
idfield fromactions runs raw_json=trueis the obvious plan B. Atlimit=1it returns about 180 KB, because every run carries a full nestedrepositoryobject, and it returns the whole page rather than one run.event_payloadis already stripped, so that is not the cause. Getting the id back out then needs jq or a script.Suggested fixes (any subset)
runsandrun-getoutput, label the value plainly as the id to pass. For exampleid 182, or arun-watch 182hint, instead of#182. Or show both the web number and the id.run-watch/run-getalso accept the web UI run number (index_in_repo), since that is what the URL and the web UI show.runs raw_jsonlean: honorlimit, and drop or summarize the per-runrepositoryobject, so the id is extractable without a script.Related: #438 (finish should wait on a CI-gated merge instead of failing) is what sent me to watch the run by hand in the first place.