Bug #1: PID key mismatch — ProtocolStart, ProtocolStop, IncidentDetector read 'git.slave.pid' but GitSlave writes 'slave.pid'. Fixed all consumers. Bug #2: release-watcher writes release.current on failed deploys, making failures permanently un-retryable. Now only updates on successful promotion. Bug #3: BlueGreen::promote() stops old containers before starting new, guaranteeing downtime. Rewritten to start new first, then stop old, then restart old on shadow ports for standby rollback. Bug #7: JsonLock/Json singleton caches never cleared in long-running release-watcher daemon. Added clearInstances() calls each poll cycle. Bug #12: GitSlave::execute() calls JsonLock::delete() on every start, wiping ALL lock file state (release.current, blue-green state, etc). Changed to only clear the stale PID key. Bug #13: release-watcher had zero logging between startup and release detection. Added comprehensive poll cycle logging and moved credential refresh before API calls. Bug #14: Shell::run() appended '2>&1' to every command, swallowing stderr and mixing it with stdout. Removed — callers that need stderr redirect explicitly. Bug #15: Shell::isLockedPIDStillRunning() read from Json (protocol.json) instead of JsonLock (protocol.lock) for PID. Fixed import and call. Also: Config::clearInstances() added for daemon cache clearing. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Uh oh!

repo:slave should kill the server if it falls out of sync #2

Description

Activity

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions