ci: re-register star history schedule - #722
Merged
Merged
Conversation
The workflow file landed on main during the 6 August Actions incident, when workflow-triggering events were dropped and could not be replayed, so its cron was never registered. Changing the minute forces a resync on the next default-branch push. Add the watch trigger so a star refreshes the chart directly rather than leaving it dependent on the cron registration alone, and cancel superseded runs since each one rebuilds from the full stargazer list. Closes #721
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The star history workflow has never run on its schedule. Two runs exist, both
manual dispatches on 6 August; filtering its runs on
event=schedulereturnszero. The chart is still the seeded one at 262 stars against 264 today.
The file landed on
mainat 18:48 UTC on 6 August, inside the Actions incidentthat ran 15:22 UTC that day to 02:04 UTC the next
(https://stspg.io/rcz3fcm83sff). That incident's closing note says
workflow-triggering events in the window were not processed and cannot be
replayed, and that the triggering action needs repeating with a new commit. A
schedule is registered off the default-branch push that introduces it, so this
one never registered. The only later push to
main, b06cd4f at 20:22 UTC, wasalso inside the window with webhooks throttled to roughly 15%, so the usual
resync did not happen either.
The workflow is
active, the file is onmain, the cron is valid, Actions areenabled, and the other seven scheduled workflows all fire, so the registration
is the only thing missing. Merging this is what re-registers it.
Closes #721
Changes
changed schedule guarantees the parsed definition differs from whatever is
cached, and the push is what triggers the resync.
watch: types: [started], so starring the repo refreshes the chartinstead of leaving it to wait out the day. The cron stays as the backstop
because
startedis the only activity typewatchhas, so unstarring neverreaches it. Any GitHub account can trigger this, which is fine here: no event
payload value reaches the job, which reads stargazers from the API, renders
with a script in this repo, and writes two SVGs to the unprotected
assetsbranch.
cancel-in-progress: true. Each run rebuilds from the full stargazerlist, so a newer run supersedes an older one and a rush of stars no longer
queues one serial run each.
Testing
actionlintis clean on the changed file and the SHA pin check passes. NoPython changed.
Worth flagging for whoever merges: the commit and force-push half of the publish
step has never executed, since both earlier runs hit the "chart unchanged" early
exit. Simulating it against a shallow clone of
assetsshows the pieces aresound, the remote-tracking ref
--force-with-leaseneeds is there and the seedcommit is a parentless orphan so amending it raises no shallow-update problem,
but the first run after merge is the real proof. The live count differs from the
published one, so a manual dispatch will exercise it: the
assetstip messageshould change from
chore: seed star history chart assetstochore: refresh star history chartand the chart should report 264.Whether the cron itself comes back only shows up over the next day or two. If no
scheduled run appears within two windows, the next step is renaming the workflow
file, then GitHub Support.
Type of Change
fix:)feat:)refactor:)docs:)ci:)chore:)Checklist
Closes #Ntype:*and onepriority:*labelphase:*label (or none when not roadmap work)feat/<slug>,fix/<slug>, etc.)mypy src/) — N/A, no Python changedruff check .) — N/A, no Python changedruff format --check .) — N/A, no Python changed