oblachno-oss/grm
3
0
Fork 0
Code Issues 33 Pull Requests Actions Packages Projects Releases 22 Wiki Activity

GRM-170: Fix stall-detection robustness bugs in runner healthcheck #277

Merged
kireto merged 2 commits from GRM-170-stall-diag-robustness into master 2026-09-18 11:19:16 +00:00
Conversation 2 Commits 2 Files Changed 2
+87 -5
2 Commits
Author SHA1 Message Date
Emil SimeonovandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com> 8cd0d4dd57 fix: redact secret env values in stall diagnostics inspect output
CI / validate (pull_request) Successful in 1m9s
Details
CI / molecule-tests (1) (pull_request) Successful in 5m23s
Details
CI / molecule-tests (3) (pull_request) Successful in 7m5s
Details
CI / molecule-tests (2) (pull_request) Successful in 7m27s
Details
CI / molecule-tests (4) (pull_request) Successful in 7m31s
Details
CI / auto-merge (pull_request) Successful in 7m50s
Details 
docker inspect captures the container Env block, which for CI job
containers carries CI_GITEA_API_TOKEN and similar secrets — the diag
bundle must not become a secret artifact (OBL-INFRA-548 S02).

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
 2026-09-18 13:10:04 +02:00
Emil SimeonovandDevin <158243242+devin-ai-integration[bot]@users.noreply.github.com> a61075ebde fix: harden stall-detection enumeration and timestamp parsing
CI / validate (pull_request) Successful in 1m9s
Details
CI / molecule-tests (1) (pull_request) Successful in 5m49s
Details
CI / molecule-tests (3) (pull_request) Successful in 6m47s
Details
CI / molecule-tests (2) (pull_request) Successful in 7m10s
Details
CI / molecule-tests (4) (pull_request) Successful in 7m12s
Details
CI / auto-merge (pull_request) Failing after 7m50s
Details 
Post-merge review of GRM-168 found three defects: the docker ps
feeding the probe loop lacked a failure guard, so a slow daemon would
abort the healthcheck under pipefail exactly when diagnostics matter;
CreatedAt contains spaces so whitespace-splitting read only captured
the date and broke the 15-minute age gate (GNU date also rejects the
+0000 UTC suffix); and head -200 could truncate the inspect State
block the spec requires. Pipe-separated fields, guarded enumeration,
stripped UTC suffix, and full inspect output fix all three.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
 2026-09-18 13:01:38 +02:00