VersionControl:PRs/maybelle-config/264 - Revision history

DrivingThatTrain: cryptograss/maybelle-config#264, as GitHub has it

2026-11-09T17:06:27Z

cryptograss/maybelle-config#264, as GitHub has it

← Older revision		Revision as of 17:06, 9 November 2026
Line 16:		Line 16:
	\|deploy=redeploy hunter		\|deploy=redeploy hunter
	}}		}}
			<pre class="pr-body">Justin, #emergence, 16:42Z: "let's remove the `-arthel` suffix from everything; it hasn't been relevant in over a year."

			What changes
			- Container names: each container is now named for its inventory entry, so `emergent-magent-arthel` becomes `emergent-magent`, and the same for `emergent-re`, `emergent-blu` and `workbench`. The change is in both the compose service key and `container_name`.
			- The image: `magenta-arthel:latest` becomes `magenta:latest` in the build role, the compose template, `docker-compose-build.yml` and the playbook's task name.
			- Everything that names a container: the ssh router (`CONTAINER="${TARGET_USER}"`), `mood-previews.py` (`WORKBENCH = 'workbench'`), `agent_homes.py`'s carry-out, the startup-log task, the log viewer, and the tests' fixtures.
			- `--remove-orphans` on `docker compose up`. This is the line that matters. Without it, renaming a service leaves the old `<name>-arthel` container running next to the new one, and for me that means two pollers answering every mention. The compose project stays the same (the directory name), so the old container becomes an orphan and gets removed.

			Unchanged: historical mentions of `justin-arthel`, the container's name while it existed, in `CONTAINERS.md`, the inventory comment and the home-move task. Also unchanged: `notes/MAGENT_ENVIRONMENT.md`, which is stale in more ways than this. magenta's mentions are all in docs, plus one dead `ALLOWED_HOSTS` entry. They don't affect anything that runs, so they can follow separately.

			What the deploy does: every container is recreated under its new name. The emergents' pollers go quiet for a minute or two, and a workbench session drops. The old image stays on disk until someone prunes it.

			Checked: hunter's 64 tests pass, including the router's 6, and so does the deploy-log test. The changed YAML parses. The recovery-gate test needs a Node that loads `.ts`, which I don't have here, so CI runs it.

			Deploy: redeploy hunter
			Worked on by: magent, Justin, in #emergence</pre>

DrivingThatTrain: cryptograss/maybelle-config#264, as GitHub has it

2026-11-01T18:42:14Z

cryptograss/maybelle-config#264, as GitHub has it

← Older revision		Revision as of 18:42, 1 November 2026
Line 5:		Line 5:
	\|url=https://github.com/cryptograss/maybelle-config/pull/264		\|url=https://github.com/cryptograss/maybelle-config/pull/264
	\|author=magent-cryptograss		\|author=magent-cryptograss
	\|state=~~open~~		\|state=merged
	\|base=production		\|base=production
	\|branch=drop-arthel-suffix		\|branch=drop-arthel-suffix
	\|opened=2026-11-01 17:38:49		\|opened=2026-11-01 17:38:49
	\|pushed=2026-11-01 17:38:45		\|pushed=2026-11-01 17:38:45
	\|merged=		\|merged=2026-11-01 18:42:10
	\|closed=		\|closed=
	\|worked_on_by=magent, justin		\|worked_on_by=magent, justin

DrivingThatTrain: cryptograss/maybelle-config#264, as GitHub has it

2026-11-01T17:38:53Z

cryptograss/maybelle-config#264, as GitHub has it

New page

{{PR
|repo=maybelle-config
|number=264
|title=hunter: drop the -arthel suffix from container names and the image
|url=https://github.com/cryptograss/maybelle-config/pull/264
|author=magent-cryptograss
|state=open
|base=production
|branch=drop-arthel-suffix
|opened=2026-11-01 17:38:49
|pushed=2026-11-01 17:38:45
|merged=
|closed=
|worked_on_by=magent, justin
|moods=emergence
|deploy=redeploy hunter
}}