Skip to content

Repair cron ownership on container restart - #41976

Merged
benbarclay merged 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/41966-cron-ownership
Jun 10, 2026
Merged

benbarclay merged 1 commit into
NousResearch:mainfrom
LeonSGP43:fix/41966-cron-ownership

Conversation

@LeonSGP43

Copy link
Copy Markdown

Summary

  • always repair $HERMES_HOME/cron ownership on container boot, matching the existing profiles repair path
  • keep cron state readable after root-context docker exec writes or scheduler writes
  • add a regression test that locks in the unconditional ownership repairs for both profiles/ and cron/

Testing

  • bash -n docker/stage2-hook.sh
  • uv run --frozen --extra dev pytest tests/test_docker_home_override_scripts.py -q
  • uv run --frozen ruff check tests/test_docker_home_override_scripts.py
  • git diff --check

Closes #41966

@alt-glitch alt-glitch added type/bug Something isn't working area/docker comp/cron Cron scheduler and job management P2 Medium — degraded but workaround exists labels Jun 8, 2026
@benbarclay
benbarclay merged commit 702f4df into NousResearch:main Jun 10, 2026
22 checks passed
davidgut1982 pushed a commit to davidgut1982/hermes-agent that referenced this pull request Jun 17, 2026
T02200059 pushed a commit to T02200059/hermes-agent that referenced this pull request Jun 18, 2026
waefrebeorn pushed a commit to waefrebeorn/slermes that referenced this pull request Jul 2, 2026
santhreal pushed a commit to santhreal/hermes-agent that referenced this pull request Jul 13, 2026
donbowman pushed a commit to donbowman/hermes-agent that referenced this pull request Jul 13, 2026
Gravezzz pushed a commit to Gravezzz/hermes-agent that referenced this pull request Jul 21, 2026
leewenjie pushed a commit to leewenjie/hermes-agent that referenced this pull request Aug 7, 2026
melon-xf added a commit to melon-xf/hermes-agent that referenced this pull request Sep 3, 2026
fsca8 pushed a commit to fsca8/hermes-agent that referenced this pull request Oct 6, 2026
@alt-glitch alt-glitch added area/install-update Installer, updater, packaging, wheels, doctor and removed area/docker labels Oct 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/install-update Installer, updater, packaging, wheels, doctor comp/cron Cron scheduler and job management P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: cron/ directory ownership not restored on container restart

3 participants