review(5): close matrix-synapse status-gap finding
Some checks failed
continuous-integration/drone/push Build is failing
Some checks failed
continuous-integration/drone/push Build is failing
This commit is contained in:
@ -267,3 +267,35 @@ the Builder's current V5 stale-test candidate plus the newly-fixed `lasuite-meet
|
||||
**Verdict:** FAIL for this live V5/V2 intersection. The PR comment surface reflects the terminal
|
||||
stale-test result, but the commit-status surface is absent, so `testme-on-pr.sh` cannot read the verdict
|
||||
back from the PR and incorrectly reports `PENDING`. Filed as `BACKLOG-5.md` item **A5-4**.
|
||||
|
||||
---
|
||||
|
||||
## Cold-verify follow-up — 2026-06-01T18:53:30Z
|
||||
|
||||
Scheduled wake noted the Builder had re-run `recipe-maintainers/matrix-synapse` PR `#1` on the current
|
||||
bridge to confirm the status surface was restored. I re-oriented from current live state and did **not**
|
||||
rely on the older A5-4 snapshot alone.
|
||||
|
||||
### A5-4 re-test: CLOSED
|
||||
- Probe target remained `recipe-maintainers/matrix-synapse` PR `#1`, head
|
||||
`21e5d84430bdc52f8fa8aa9a40fa5bda8adf06c0`.
|
||||
- Fresh poll while the rerun was active:
|
||||
`POST=0 MAX_WAIT=25 INTERVAL=5 /srv/cc-ci/.claude/skills/recipe-upgrade/testme-on-pr.sh matrix-synapse 1`
|
||||
returned:
|
||||
`VERDICT=PENDING`
|
||||
`BUILD=https://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/63`
|
||||
- At that same point, the recipe head's combined status endpoint correctly reflected the in-flight run:
|
||||
`state=pending`, `context=cc-ci/testme`, `target_url=.../63`.
|
||||
- Follow-up poll after completion:
|
||||
`POST=0 MAX_WAIT=10 INTERVAL=5 /srv/cc-ci/.claude/skills/recipe-upgrade/testme-on-pr.sh matrix-synapse 1`
|
||||
returned:
|
||||
`VERDICT=RED`
|
||||
`BUILD=https://drone.ci.commoninternet.net/recipe-maintainers/cc-ci/63`
|
||||
- The recipe head's status endpoint then reflected the terminal result:
|
||||
`state=failure`, `context=cc-ci/testme`, `target_url=.../63`.
|
||||
- The PR result comment was updated in place to the terminal result card for build `#63`
|
||||
(`issuecomment-13882`).
|
||||
|
||||
**Verdict:** A5-4 is no longer reproducible on the current live bridge flow. The stale-test/default path
|
||||
for `matrix-synapse` now exposes an in-flight status and a terminal failure status on the recipe PR head,
|
||||
and `testme-on-pr.sh` reads the verdict back correctly.
|
||||
|
||||
Reference in New Issue
Block a user