gabriel / musehub public
Merged
gabriel gabriel · 15 days ago · 1 agent

fix: real CloudWatch alerting on both envs, add environment/release_version to logs (#160)

fix/cloudwatch-alerts-and-log-fields dev sha256:23efc08a3fce branch
high risk
minor semver
0/2 approvals
merge type
overlay strategy
sha256:7afc59b2ee9c70d18b748d269602c588385d6e936cc22b056fd775ab9f2fd499 from
sha256:9c64dbfd65ef4e8a85f500e5909c06c2e6c65255a69e84188ee73b63f111cecd to
Proposal merged as sha256:23efc08a3fce

Symbol Delta

31 symbols
13 added 9 modified 9 deleted
deploy/cloudwatch-alerts.sh 13
+ ALERT_EMAILS
+ ENV
+ IFS
+ email
~ DB_INSTANCE breaking
~ LOG_GROUP
~ SNS_TOPIC_NAME
~ aws_cmd
× ALERT_EMAIL breaking
× ALERT_PHONE breaking
× INSTANCE_ID breaking
× THRESHOLD_DB_CONN_PCT breaking
× THRESHOLD_DISK_PCT breaking
docs/production-readiness/12-observability.md 12
+ Section 12 — Observability.Checklist assessment — updated
+ Section 12 — Observability.Remaining work — split to #181
+ Section 12 — Observability.Update (2026-09-08) — wired real alarms on top of the existing logging foundation
+ Section 12 — Observability.Update (2026-09-08) — wired real alarms on top of the existing logging foundation.Added environment and release_version to every log line
+ Section 12 — Observability.Update (2026-09-08) — wired real alarms on top of the existing logging foundation.Fixed real bugs in deploy/cloudwatch-alerts.sh, beyond just the log group name
+ Section 12 — Observability.Update (2026-09-08) — wired real alarms on top of the existing logging foundation.Ran it for real against both environments — not just fixed and left unverified
+ Section 12 — Observability.Update (2026-09-08) — wired real alarms on top of the existing logging foundation.Ran it for real against both environments — not just fixed and left unverified.code
~ Section 12 — Observability
× Section 12 — Observability.Checklist assessment
× Section 12 — Observability.The actual work items here
× Section 12 — Observability.What's actually live — verified directly against CloudWatch
× Section 12 — Observability.What's missing
musehub/config.py 1
~ Settings breaking
musehub/logging_config.py 5
+ _environment
+ _release_version
~ JsonFormatter breaking
~ JsonFormatter.format
~ configure_logging breaking

Files Changed

6 of 1,060
M deploy/cloudwatch-alerts.sh
M deploy/deploy.sh
M docs/production-readiness/12-observability.md
M musehub/config.py
M musehub/logging_config.py
M musehub/main.py

Commits

1
fix agent
fix: real CloudWatch alerting on both envs, add environment/release_version to logs...
gabriel · 15 days ago
minor sha256:7afc59b2ee9c

Divergence Analysis

fix/cloudwatch-alerts-and-log-fields → dev
0%
0% divergence
Very compatible — minimal differences

Review comments

0

No review comments yet

Be the first to leave a review comment on this merge proposal.

Muse CLI

Comment muse hub proposal comment create sha256:f84742baa9da70d084cfc30b7cbc42dd48996dafba9f2c17fce1b4567eb248ce --body "…"
Approve muse hub proposal review submit sha256:f84742baa9da70d084cfc30b7cbc42dd48996dafba9f2c17fce1b4567eb248ce --verdict approve
Request changes muse hub proposal review submit sha256:f84742baa9da70d084cfc30b7cbc42dd48996dafba9f2c17fce1b4567eb248ce --verdict request_changes --body "…"
Close muse hub proposal close sha256:f84742baa9da70d084cfc30b7cbc42dd48996dafba9f2c17fce1b4567eb248ce