fix(ci): repair post-merge monitor workflow #27

Merged
hongming merged 1 commits from fix/status-heartbeat-env into main 2026-05-22 18:03:23 +00:00
Owner

Fixes the post-merge uptime probe failure found while monitoring the GitHub-to-Gitea migration.

Root cause:
The workflow assigned HB="history/.monitoring-heartbeat.jsonl" as a shell-local variable, then Python read os.environ["HB"]. Because HB was not exported, every probe failed with KeyError: 'HB', and CI / all-required timed out waiting for the probe context.

Validation:

  • comprehensive-testing: parsed .gitea/workflows/uptime-probe.yml with Ruby YAML.load_file; ran git diff --check; PR checks green except SOP before ack.
  • local-postgres-e2e: not applicable; workflow-only shell/env fix with no database behavior.
  • staging-smoke: not applicable; uptime workflow is the staging/post-merge smoke surface and is rerun by Gitea Actions.
  • root-cause: missing export HB caused Python KeyError: 'HB' in Write + trim heartbeat.
  • five-axis-review: correctness, security, operability, compatibility, and rollback reviewed; one-line env export only.
  • no-backwards-compat: no compatibility shim needed; fixes intended existing workflow behavior.
  • memory-consulted: followed prior CI monitoring/merge guidance and checked current logs.
Fixes the post-merge uptime probe failure found while monitoring the GitHub-to-Gitea migration. Root cause: The workflow assigned `HB="history/.monitoring-heartbeat.jsonl"` as a shell-local variable, then Python read `os.environ["HB"]`. Because `HB` was not exported, every probe failed with `KeyError: 'HB'`, and `CI / all-required` timed out waiting for the probe context. Validation: - [x] comprehensive-testing: parsed `.gitea/workflows/uptime-probe.yml` with Ruby `YAML.load_file`; ran `git diff --check`; PR checks green except SOP before ack. - [x] local-postgres-e2e: not applicable; workflow-only shell/env fix with no database behavior. - [x] staging-smoke: not applicable; uptime workflow is the staging/post-merge smoke surface and is rerun by Gitea Actions. - [x] root-cause: missing `export HB` caused Python `KeyError: 'HB'` in `Write + trim heartbeat`. - [x] five-axis-review: correctness, security, operability, compatibility, and rollback reviewed; one-line env export only. - [x] no-backwards-compat: no compatibility shim needed; fixes intended existing workflow behavior. - [x] memory-consulted: followed prior CI monitoring/merge guidance and checked current logs.
hongming added 1 commit 2026-05-22 16:46:55 +00:00
fix(ci): repair post-merge monitor workflow
CI / all-required (pull_request) Successful in 3s
CI / noop (pull_request) Successful in 2s
CI / Workflow YAML lint (pull_request) Successful in 4s
sop-checklist-gate / gate (pull_request) Successful in 46s
sop-checklist / all-items-acked (pull_request) acked: 7/7 — body-unfilled: 7
a7d9dcc00a
Member

/sop-ack comprehensive-testing
/sop-ack local-postgres-e2e
/sop-ack staging-smoke
/sop-ack five-axis-review
/sop-ack memory-consulted

/sop-ack comprehensive-testing /sop-ack local-postgres-e2e /sop-ack staging-smoke /sop-ack five-axis-review /sop-ack memory-consulted
Member

/sop-ack root-cause
/sop-ack no-backwards-compat

/sop-ack root-cause /sop-ack no-backwards-compat
core-devops approved these changes 2026-05-22 18:03:22 +00:00
core-devops left a comment
Member

Approved post-merge monitoring workflow fix after green CI.

Approved post-merge monitoring workflow fix after green CI.
infra-sre approved these changes 2026-05-22 18:03:22 +00:00
infra-sre left a comment
Member

Approved post-merge monitoring workflow fix after green CI.

Approved post-merge monitoring workflow fix after green CI.
hongming merged commit 4a124ea78f into main 2026-05-22 18:03:23 +00:00
Sign in to join this conversation.
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/molecule-ai-status#27