RFC: census-watch workflow with issues: write #246
Unanswered
bong-water-water-bong
asked this question in
rfcs
Replies: 2 comments
|
Decision: accepted, with tighter permissions than the sketch. Implemented in #263.
Why an issue rather than the alternatives:
The development box's systemd timer stays as a second path. |
0 replies
|
Landed. The workflow is in #263, and #265 tunes the alert rule.
The first scheduled run is tomorrow at 04:30 UTC. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Problem
The daily new-model watcher (
tools/census_watch.py, merged with #242) alerts when a newly published Hugging Face model uses an architecture class thatregistry/architectures.jsondoes not map. Its only alert path today is a systemd timer on the development box, which marks its unit failed. Nobody sees that unless they look at the box. The old watcher was dead for a week (203/EXEC) and nothing turned red.Design
Add
.github/workflows/census-watch.yml:workflow_dispatch. Nopull_requesttrigger.ubuntu-latest, stdlib Python only, actions pinned by SHA.scripts/census-watch.sh --limit 120.census-watch. It does nothing while such an issue is still open.contents: read,issues: write. Theissues: writescope is the reason for this RFC.The full file is in #242's first commit (b5a9d01). The follow-up PR would restore it unchanged.
Alternatives
issues: read): a failed scheduled run notifies only whoever last edited the workflow.Testing
census_watch(offline) covers the watcher's contract.workflow_dispatchrun on the branch checks the issue path. Mapping a fake class, or running with an empty registry, forces the alert.NOTICE
Nothing new:
actions/checkoutandactions/setup-pythonare already used byci.yml.All reactions