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

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

2026-11-09T17:06:27Z

cryptograss/maybelle-config#262, 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">Follow-up to #260. After today's deploy I'm running in `emergent-magent`. It has the only poller, and `justin-arthel` has none. But my commits still come out as Justin Holmes.

			Why: the container does have all four variables. `docker inspect emergent-magent-arthel` shows `GIT_AUTHOR_NAME=magent-cryptograss` and the other three, so #260's compose change rendered as intended. But `container_startup.py` starts the poller with `su - magent -c '…'`, and `su -` begins with a clean environment. The variables never reach the poller, my turns, or git.

			The fix: `poller_command` now puts `export GIT_…="…";` for each variable the container has into the poller's own shell line, after `poller.env` is sourced.
			- Not `.bashrc` or `poller.env`: both are in the home `emergent-magent` shares with `justin-arthel`. Writing the identity there would make Justin commit as magent.
			- Values are left out, not quoted: a value with any character beyond letters, digits, space and `.@+-_` is dropped. The line is wrapped in single quotes for `su`, so leaving such a value out is safer than quoting it.
			- Containers without the variables are unchanged, which covers every container but `emergent-magent`.

			Checked:
			- Three new tests: the identity is in the line, there's no `GIT_` when the container has none, and a value that would need quoting is left out. All 67 hunter tests pass.
			- I ran `env -i bash -c '<the export line> git var GIT_AUTHOR_IDENT'` in a clean shell. It prints `magent-cryptograss <magent@cryptograss.live>`.

			Also here: the log viewer's pane 0 (the follow-up promised on #261). Pane 0 used to run `docker logs -f justin-arthel`, but that container no longer runs my poller. It now tails `/tmp/mood-poller.log` in `emergent-magent-arthel`, where the poller logs its wakes, replies and costs. `docker logs` only shows container startup.

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

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

2026-11-01T17:19:01Z

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

← Older revision		Revision as of 17:19, 1 November 2026
Line 5:		Line 5:
	\|url=https://github.com/cryptograss/maybelle-config/pull/262		\|url=https://github.com/cryptograss/maybelle-config/pull/262
	\|author=magent-cryptograss		\|author=magent-cryptograss
	\|state=~~open~~		\|state=closed
	\|base=production		\|base=production
	\|branch=poller-git-identity		\|branch=poller-git-identity
Line 11:		Line 11:
	\|pushed=2026-11-01 16:32:41		\|pushed=2026-11-01 16:32:41
	\|merged=		\|merged=
	\|closed=		\|closed=2026-11-01 16:45:52
	\|worked_on_by=magent, re, justin		\|worked_on_by=magent, re, justin
	\|moods=emergence		\|moods=emergence
	\|deploy=redeploy hunter		\|deploy=redeploy hunter
	}}		}}

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

2026-11-01T16:32:47Z

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

← Older revision		Revision as of 16:32, 1 November 2026
Line 9:		Line 9:
	\|branch=poller-git-identity		\|branch=poller-git-identity
	\|opened=2026-11-01 16:28:13		\|opened=2026-11-01 16:28:13
	\|pushed=2026-11-01 16:28:32		\|pushed=2026-11-01 16:32:41
	\|merged=		\|merged=
	\|closed=		\|closed=

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

2026-11-01T16:28:42Z

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

← Older revision		Revision as of 16:28, 1 November 2026
Line 9:		Line 9:
	\|branch=poller-git-identity		\|branch=poller-git-identity
	\|opened=2026-11-01 16:28:13		\|opened=2026-11-01 16:28:13
	\|pushed=2026-11-01 16:28:09		\|pushed=2026-11-01 16:28:32
	\|merged=		\|merged=
	\|closed=		\|closed=

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

2026-11-01T16:28:17Z

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

New page

{{PR
|repo=maybelle-config
|number=262
|title=hunter: the poller gets its container's git identity through su
|url=https://github.com/cryptograss/maybelle-config/pull/262
|author=magent-cryptograss
|state=open
|base=production
|branch=poller-git-identity
|opened=2026-11-01 16:28:13
|pushed=2026-11-01 16:28:09
|merged=
|closed=
|worked_on_by=magent, re, justin
|moods=emergence
|deploy=redeploy hunter
}}