@evident-ai/runner-cdk 3.4.1-dev.1856549 → 3.4.1-dev.2b2679a
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +21 -6
- package/dist/controller-lambda/handler.js +25 -14
- package/dist/evident-scale-to-zero-construct.d.ts +16 -4
- package/dist/evident-scale-to-zero-construct.js +17 -12
- package/dist/image-version-reporter-lambda/handler.js +129 -0
- package/dist/microvm/construct.d.ts +24 -0
- package/dist/microvm/construct.js +32 -1
- package/dist/microvm/controller/doorbell.d.ts +5 -0
- package/dist/microvm/controller/doorbell.js +11 -10
- package/dist/microvm/controller/handle-doorbell.js +17 -14
- package/dist/microvm/controller/microvm-client.d.ts +6 -0
- package/dist/microvm/controller/shape-catalogue.d.ts +1 -1
- package/dist/microvm/controller/shape-catalogue.js +1 -1
- package/dist/microvm/image/stage-context.d.ts +14 -5
- package/dist/microvm/image/stage-context.js +46 -16
- package/dist/microvm/image-version-reporter/construct.d.ts +35 -0
- package/dist/microvm/image-version-reporter/construct.js +91 -0
- package/dist/microvm/image-version-reporter/handler.d.ts +26 -0
- package/dist/microvm/image-version-reporter/handler.js +104 -0
- package/dist/microvm-image-context/Dockerfile +30 -48
- package/dist/microvm-image-context/hooks/common.sh +205 -718
- package/dist/microvm-image-context/hooks/resume +12 -9
- package/dist/microvm-image-context/hooks/run +18 -39
- package/dist/microvm-image-context/hooks/suspend +4 -2
- package/dist/microvm-image-context/hooks/terminate +5 -3
- package/package.json +1 -1
|
@@ -21,6 +21,12 @@ OPENCODE_PORT="${OPENCODE_PORT:-4096}"
|
|
|
21
21
|
# /proc/<pid>/environ, which runs as that same uid. /terminate removes it.
|
|
22
22
|
# shellcheck disable=SC2034 # read by the scripts that source this file
|
|
23
23
|
CONTEXT_FILE="/dev/shm/evident-run-context"
|
|
24
|
+
|
|
25
|
+
# The runtime injects MICROVM_ID only into /run, never /resume. This tmpfs file
|
|
26
|
+
# survives suspend/resume so /resume can restore the id for every fresh CLI
|
|
27
|
+
# process to self-report and acknowledge a fulfilled recycle request (#1906).
|
|
28
|
+
# shellcheck disable=SC2034 # read by the scripts that source this file
|
|
29
|
+
MICROVM_ID_FILE="/dev/shm/evident-microvm-id"
|
|
24
30
|
TUNNEL_PID_FILE="/dev/shm/evident-tunnel.pid"
|
|
25
31
|
OPENCODE_PID_FILE="/dev/shm/evident-opencode.pid"
|
|
26
32
|
LITESTREAM_PID_FILE="/dev/shm/evident-litestream.pid"
|
|
@@ -37,90 +43,38 @@ STATE_PREFIX_FILE="/dev/shm/evident-state-prefix"
|
|
|
37
43
|
# The generated litestream.yml (#812). tmpfs for the same two reasons as the
|
|
38
44
|
# files above: it must never reach the block device (Q6, `docs`), and it
|
|
39
45
|
# survives suspend/resume, which is what lets `/resume` start litestream again
|
|
40
|
-
# with no regeneration cost.
|
|
41
|
-
# (`ensure_litestream_config`, below).
|
|
46
|
+
# with no regeneration cost. The CLI regenerates it only when absent or empty.
|
|
42
47
|
LITESTREAM_CONFIG_FILE="/dev/shm/evident-litestream.yml"
|
|
43
48
|
|
|
44
|
-
# Set when
|
|
45
|
-
#
|
|
46
|
-
#
|
|
47
|
-
# so would let a partial/fresh local DB overwrite a replica this boot never
|
|
48
|
-
# proved is safe to write over. Cleared at the top of every `restore_session_db`
|
|
49
|
-
# call (i.e. every `/run`) before any other decision, so a marker left by an
|
|
50
|
-
# earlier boot cannot silently disable replication for the VM's whole life.
|
|
49
|
+
# Set by the CLI when session-DB restore or verification could not prove this
|
|
50
|
+
# boot safe to replicate. The teardown hooks read it before flushing, and
|
|
51
|
+
# `/terminate` removes it with the rest of the per-VM state.
|
|
51
52
|
SESSION_DB_NO_REPLICATE_MARKER="/dev/shm/evident-session-db-no-replicate"
|
|
52
53
|
|
|
54
|
+
# Completion evidence for the CLI-owned credential flush. It lives in tmpfs,
|
|
55
|
+
# is removed before every handshake, and is also removed by /terminate.
|
|
56
|
+
CREDENTIAL_FLUSH_MARKER_FILE="/dev/shm/evident-credential-flush"
|
|
57
|
+
|
|
53
58
|
hook_name() { printf '%s' "${0##*/}"; }
|
|
54
59
|
log() { echo "[hook:$(hook_name)] $*"; }
|
|
55
60
|
warn() { echo "[hook:$(hook_name)] $*" >&2; }
|
|
56
61
|
error() { echo "[hook:$(hook_name)] ERROR: $*" >&2; }
|
|
57
62
|
|
|
58
|
-
|
|
59
|
-
|
|
60
|
-
|
|
61
|
-
|
|
62
|
-
if [ -n "${path}" ]; then
|
|
63
|
-
printf '%s\n' "${path}"
|
|
64
|
-
return 0
|
|
65
|
-
fi
|
|
66
|
-
local home="${HOME:-}"
|
|
67
|
-
home="${home#"${home%%[![:space:]]*}"}"
|
|
68
|
-
home="${home%"${home##*[![:space:]]}"}"
|
|
69
|
-
[ -n "${home}" ] || return 1
|
|
70
|
-
printf '%s\n' "${home}/.local/state/evident/session-db-recovery.jsonl"
|
|
71
|
-
}
|
|
72
|
-
|
|
73
|
-
report_session_db_giveup() {
|
|
74
|
-
local reason="$1" exit_code="${2:-null}" outcome path at
|
|
75
|
-
case "${reason}" in
|
|
76
|
-
synchroniser_config_unresolved|synchroniser_config_unevaluable|synchroniser_config_incomplete|litestream_config_unavailable|classification_fatal) outcome="restore_misconfigured" ;;
|
|
77
|
-
restore_deadline_exceeded|restore_tool_unusable|classification_unrecognised) outcome="fresh_session_db" ;;
|
|
78
|
-
*) warn "SESSION-DB-RECOVERY-REPORT-UNKNOWN-REASON: ${reason}"; return 0 ;;
|
|
79
|
-
esac
|
|
80
|
-
if ! path="$(session_db_recovery_report_path)"; then
|
|
81
|
-
warn "SESSION-DB-RECOVERY-REPORT-PATH-UNAVAILABLE"
|
|
82
|
-
return 0
|
|
83
|
-
fi
|
|
84
|
-
at="${EPOCHREALTIME:-}"
|
|
85
|
-
[ -n "${at}" ] || warn "SESSION-DB-RECOVERY-REPORT-TIMESTAMP-UNAVAILABLE"
|
|
86
|
-
printf '{"v":1,"event":"session_db_recovery","at":"%s","stage":"restore","outcome":"%s","severity":"error","reason":"%s","litestream_exit_code":%s,"attempt":null,"replica_objects":null,"replica_bytes":null,"quarantine_destination":null,"quarantined_objects":null,"quarantine_failed_objects":null,"quarantined_bytes":null,"verified_restore_point":null,"restore_points_tried":null,"replication_suspended":true}\n' "${at}" "${outcome}" "${reason}" "${exit_code}" >>"${path}" || warn "SESSION-DB-RECOVERY-REPORT-APPEND-FAILED"
|
|
87
|
-
return 0
|
|
88
|
-
}
|
|
89
|
-
|
|
90
|
-
# Returns the CLI's own exit code. Domain outcomes (nothing persisted yet, a
|
|
91
|
-
# corrupt object) are LOGGED and exit 0, the predicates answer "no" with 10, and
|
|
63
|
+
# Returns the CLI's own exit code. `restore` logs domain outcomes and returns 0;
|
|
64
|
+
# a non-zero status means the tool itself failed. `sync-once` returns 40 for
|
|
65
|
+
# `failed`, `hashFailed`, and `localInvalid` (credentials not persisted), and 0
|
|
66
|
+
# for every other outcome. The predicates answer "no" with 10, and
|
|
92
67
|
# `session-db-classify`'s three typed answers are 30 (fatal)/31 (replica
|
|
93
|
-
# unusable)/32 (retry)
|
|
94
|
-
#
|
|
95
|
-
#
|
|
96
|
-
#
|
|
97
|
-
# that is an intentional bound firing, not a broken tool, so the caller
|
|
98
|
-
# classifies it instead (restore_credentials, #930). A trailing
|
|
99
|
-
# `--evident-deadline=N` (stripped below before forwarding to the CLI, and
|
|
100
|
-
# never produced by anything but timed_synchroniser) is how a caller opts in;
|
|
101
|
-
# every other caller — sync_credentials, restore_session_db's `env` /
|
|
102
|
-
# `litestream-config` / `session-db-classify`, and this file's own contract
|
|
103
|
-
# test — passes none, so 124/137 there still means a genuine external SIGKILL
|
|
104
|
-
# (e.g. an OOM kill) and must keep producing the ERROR below.
|
|
68
|
+
# unusable)/32 (retry), extended by `session-db-verify`'s 33 (integrity
|
|
69
|
+
# exhausted, replica separated and local disposed) / 34 (could not prove
|
|
70
|
+
# separation or disposal). Any status outside a command's contractual answers
|
|
71
|
+
# means the tool itself broke, which is the only case worth an ERROR here.
|
|
105
72
|
run_synchroniser() {
|
|
106
|
-
local -a call_args=("$@")
|
|
107
|
-
local deadline=""
|
|
108
|
-
local last=$(( ${#call_args[@]} - 1 ))
|
|
109
|
-
if [ "${last}" -ge 0 ] && [[ "${call_args[last]}" == --evident-deadline=* ]]; then
|
|
110
|
-
deadline="${call_args[last]#--evident-deadline=}"
|
|
111
|
-
call_args=("${call_args[@]:0:${last}}")
|
|
112
|
-
fi
|
|
113
|
-
|
|
114
73
|
local rc=0
|
|
115
|
-
|
|
116
|
-
[ -n "${deadline}" ] && launcher=(timeout -k "${CREDENTIAL_RESTORE_KILL_GRACE_SECONDS}" "${deadline}")
|
|
117
|
-
"${launcher[@]}" "${SYNCHRONISER}" "${call_args[@]}" || rc=$?
|
|
74
|
+
"${SYNCHRONISER}" "$@" || rc=$?
|
|
118
75
|
case "${rc}" in
|
|
119
|
-
0 | 10 | 30 | 31 | 32) ;;
|
|
120
|
-
|
|
121
|
-
[ -n "${deadline}" ] || error "synchroniser '${call_args[*]}' exited ${rc}; the '${SYNCHRONISER}' command is missing from PATH, corrupt, or it threw"
|
|
122
|
-
;;
|
|
123
|
-
*) error "synchroniser '${call_args[*]}' exited ${rc}; the '${SYNCHRONISER}' command is missing from PATH, corrupt, or it threw" ;;
|
|
76
|
+
0 | 10 | 30 | 31 | 32 | 33 | 34 | 40) ;;
|
|
77
|
+
*) error "synchroniser '$*' exited ${rc}; the '${SYNCHRONISER}' command is missing from PATH, corrupt, or it threw" ;;
|
|
124
78
|
esac
|
|
125
79
|
return "${rc}"
|
|
126
80
|
}
|
|
@@ -143,8 +97,8 @@ run_synchroniser() {
|
|
|
143
97
|
# Never fails its caller: a timing line is diagnostics, and hardening a
|
|
144
98
|
# currently-working path is worse than the gap it closes (#931). The `date`
|
|
145
99
|
# fallback covers a pre-5.0 bash; a 0 means "could not read the clock", which
|
|
146
|
-
#
|
|
147
|
-
# under `set -e` — and rather than a 0 that would read as a fast healthy call.
|
|
100
|
+
# `log_elapsed_since` turns into `elapsed_ms=unknown` rather than aborting a
|
|
101
|
+
# hook under `set -e` — and rather than a 0 that would read as a fast healthy call.
|
|
148
102
|
now_ms() {
|
|
149
103
|
local now="${EPOCHREALTIME:-}"
|
|
150
104
|
if [ -n "${now}" ]; then
|
|
@@ -155,16 +109,10 @@ now_ms() {
|
|
|
155
109
|
date +%s%3N 2>/dev/null || echo 0
|
|
156
110
|
}
|
|
157
111
|
|
|
158
|
-
# Reports what one
|
|
159
|
-
# `elapsed_ms=`, `rc=`) so "how long does this actually take in the
|
|
160
|
-
# log query rather than another spike. `at_s` is where the step
|
|
161
|
-
# hook's own SECONDS clock
|
|
162
|
-
# hook-scripts.test.ts is derived against.
|
|
163
|
-
#
|
|
164
|
-
# Callable directly, not only through timed_synchroniser below, because the
|
|
165
|
-
# steps that dominate the pre-opencode window CANNOT be wrapped: their stdout is
|
|
166
|
-
# captured (`x="$(run_synchroniser env)"`, later `eval`'d), so a `log` line
|
|
167
|
-
# emitted inside the substitution would be evaluated as configuration.
|
|
112
|
+
# Reports what one pre-warm COST, in a deliberately machine-greppable line
|
|
113
|
+
# (`op=`, `elapsed_ms=`, `rc=`) so "how long does this actually take in the
|
|
114
|
+
# fleet?" is a log query rather than another spike. `at_s` is where the step
|
|
115
|
+
# landed on the hook's own SECONDS clock.
|
|
168
116
|
#
|
|
169
117
|
# `unknown`, never a number, when either end failed to read the clock (now_ms's
|
|
170
118
|
# 0). Subtracting them would report `elapsed_ms=0` — indistinguishable from a
|
|
@@ -183,26 +131,6 @@ log_elapsed_since() {
|
|
|
183
131
|
log "SYNCHRONISER-TIMING op=${op} elapsed_ms=${elapsed_ms} rc=${rc} at_s=${SECONDS}"
|
|
184
132
|
}
|
|
185
133
|
|
|
186
|
-
# Runs ONE synchroniser call and reports what it COST, without changing what it
|
|
187
|
-
# MEANS: the call's own status is returned untouched, so every caller's
|
|
188
|
-
# `|| return 1` / `|| status=$?` keeps working exactly as before.
|
|
189
|
-
#
|
|
190
|
-
# `deadline` is an explicit parameter, forwarded to run_synchroniser as a
|
|
191
|
-
# trailing `--evident-deadline=N` (never a global or dynamically-scoped
|
|
192
|
-
# variable, #930) — every current caller (restore_credentials) always has a
|
|
193
|
-
# positive remaining-budget value by the time it calls this, having already
|
|
194
|
-
# decided to skip the call entirely otherwise.
|
|
195
|
-
timed_synchroniser() {
|
|
196
|
-
local op="$1" deadline="$2"
|
|
197
|
-
shift 2
|
|
198
|
-
|
|
199
|
-
local started_ms rc=0
|
|
200
|
-
started_ms="$(now_ms)"
|
|
201
|
-
run_synchroniser "$@" "--evident-deadline=${deadline}" || rc=$?
|
|
202
|
-
log_elapsed_since "${op}" "${started_ms}" "${rc}"
|
|
203
|
-
return "${rc}"
|
|
204
|
-
}
|
|
205
|
-
|
|
206
134
|
# Exports what `runner-synchroniser` resolves its object-store location from
|
|
207
135
|
# (runner/synchroniser/src/config.ts). It treats either being empty as
|
|
208
136
|
# "persistence disabled" and then reports every restore as a WARNING it still
|
|
@@ -224,271 +152,32 @@ load_state_config() {
|
|
|
224
152
|
|
|
225
153
|
# Mirrors runner-synchroniser's own persistenceEnabled predicate (config.ts):
|
|
226
154
|
# both non-empty, already checked above, so by this point PERSISTENCE_BUCKET
|
|
227
|
-
# is just LITESTREAM_BUCKET.
|
|
155
|
+
# is just LITESTREAM_BUCKET. The CLI's replicator and flush_session_db (#812
|
|
228
156
|
# WI-3/WI-4) read ONLY this var, never LITESTREAM_BUCKET/LITESTREAM_PREFIX
|
|
229
157
|
# directly, so every entry point agrees on one predicate regardless of which
|
|
230
|
-
# earlier step in THIS hook process set it
|
|
231
|
-
#
|
|
232
|
-
# /suspend and /terminate, none of which ever call the synchroniser before
|
|
233
|
-
# needing the answer.
|
|
158
|
+
# earlier step in THIS hook process set it; /resume, /suspend and /terminate
|
|
159
|
+
# all use this same function before any teardown decision.
|
|
234
160
|
export PERSISTENCE_BUCKET="${LITESTREAM_BUCKET}"
|
|
235
161
|
}
|
|
236
162
|
|
|
237
|
-
#
|
|
238
|
-
#
|
|
239
|
-
# depends on the model that turn's message asks for, so that question has no
|
|
240
|
-
# single boot-time answer; it is answered per message, not here. What IS still
|
|
241
|
-
# fatal is credential persistence being unavailable at all (an unset bucket/
|
|
242
|
-
# prefix, or a broken synchroniser bundle) — that would silently drop refreshed
|
|
243
|
-
# tokens on every suspend, so those two restores stay `|| return 1`.
|
|
244
|
-
# `model-auth-ready` therefore stays a boot-time diagnostic: its only output is
|
|
245
|
-
# a log line, since the runtime forwards no hook stderr to the doorbell caller.
|
|
246
|
-
|
|
247
|
-
# How long restore_credentials' shared step budget is, covering the runner-secret
|
|
248
|
-
# fetch plus its three synchroniser calls (restore claude, restore opencode,
|
|
249
|
-
# model-auth-ready) as one window rather than a timeout apiece — a fixed per-call cap at the same
|
|
250
|
-
# total would false-fire on the ordinary case of one slow call (a slow S3 GET),
|
|
251
|
-
# which is precisely the boot this exists to keep healthy. From #930's 3-boot
|
|
252
|
-
# sample: the step measured ~3-5s total (~1.2s per synchroniser call, three
|
|
253
|
-
# node cold starts of a 1.8 MB bundle) — a WEAK estimate this file's own
|
|
254
|
-
# SYNCHRONISER-TIMING lines are what will sharpen for real. Too tight and a
|
|
255
|
-
# routine slow call boots this VM with no model credentials until an operator
|
|
256
|
-
# reconnects it; too loose and a hung call burns more of the hook's own
|
|
257
|
-
# SIGTERM budget before the VM is destroyed mid-boot anyway. The env override
|
|
258
|
-
# is for tests only, so they need not burn wall clock.
|
|
259
|
-
CREDENTIAL_RESTORE_DEADLINE_SECONDS="${EVIDENT_CREDENTIAL_RESTORE_DEADLINE_SECONDS:-8}"
|
|
260
|
-
# The SIGKILL backstop `timeout -k` applies after its own SIGTERM, exactly like
|
|
261
|
-
# SESSION_DB_RESTORE_KILL_GRACE_SECONDS above (a bare `timeout` only SIGTERMs,
|
|
262
|
-
# and a call that ignored it would be unbounded again). Hard ceiling on the
|
|
263
|
-
# step: DEADLINE + this = 10s, once — not per call.
|
|
264
|
-
CREDENTIAL_RESTORE_KILL_GRACE_SECONDS=2
|
|
265
|
-
GITHUB_PROBE_DEADLINE_SECONDS="${EVIDENT_GITHUB_PROBE_DEADLINE_SECONDS:-10}"
|
|
266
|
-
|
|
267
|
-
# What is left of the shared step budget, in whole seconds, `step_started_s`
|
|
268
|
-
# seconds after it began. `SECONDS` (a bash builtin with no failure mode,
|
|
269
|
-
# unlike `now_ms`) truncates, so a step that began at true time `s0` can read
|
|
270
|
-
# less elapsed time than actually passed — the `- 1` restores the invariant
|
|
271
|
-
# that no call is granted more than DEADLINE + GRACE from step start.
|
|
272
|
-
remaining_credential_budget() {
|
|
273
|
-
local step_started_s="$1"
|
|
274
|
-
echo $(( CREDENTIAL_RESTORE_DEADLINE_SECONDS - (SECONDS - step_started_s) - 1 ))
|
|
275
|
-
}
|
|
276
|
-
|
|
277
|
-
# Runs one of the step's bounded calls: skips it with a named warn if the
|
|
278
|
-
# shared budget is already exhausted, treats a timeout (124/137) as the
|
|
279
|
-
# non-fatal warn-and-continue D1 approved, and keeps every other non-zero
|
|
280
|
-
# fatal exactly as restore_credentials always has (`|| return 1` aborts /run
|
|
281
|
-
# before opencode and the tunnel start).
|
|
282
|
-
bounded_restore_call() {
|
|
283
|
-
local op="$1" step_started_s="$2"
|
|
284
|
-
shift 2
|
|
285
|
-
|
|
286
|
-
local remaining
|
|
287
|
-
remaining="$(remaining_credential_budget "${step_started_s}")"
|
|
288
|
-
if [ "${remaining}" -lt 1 ]; then
|
|
289
|
-
warn "CREDENTIAL-RESTORE-SKIPPED: ${op} skipped; the ${CREDENTIAL_RESTORE_DEADLINE_SECONDS}s credential-restore budget is exhausted"
|
|
290
|
-
return 0
|
|
291
|
-
fi
|
|
292
|
-
|
|
293
|
-
local rc=0
|
|
294
|
-
timed_synchroniser "${op}" "${remaining}" "$@" || rc=$?
|
|
295
|
-
case "${rc}" in
|
|
296
|
-
0) ;;
|
|
297
|
-
124 | 137)
|
|
298
|
-
warn "CREDENTIAL-RESTORE-TIMEOUT: ${op} did not finish within its ${remaining}s share of the ${CREDENTIAL_RESTORE_DEADLINE_SECONDS}s credential-restore deadline; continuing without it"
|
|
299
|
-
;;
|
|
300
|
-
*) return 1 ;;
|
|
301
|
-
esac
|
|
302
|
-
return 0
|
|
303
|
-
}
|
|
304
|
-
|
|
305
|
-
fetch_runner_secret() {
|
|
306
|
-
local step_started_s="$1" remaining rc=0 payload value stderr_file started_ms populated=0 missing=0
|
|
307
|
-
if [ -z "${RUNNER_SECRET_ARN:-}" ]; then
|
|
308
|
-
log "runner secret is not configured; continuing without GitHub and MCP credentials"
|
|
309
|
-
return 0
|
|
310
|
-
fi
|
|
311
|
-
remaining="$(remaining_credential_budget "${step_started_s}")"
|
|
312
|
-
if [ "${remaining}" -lt 1 ]; then
|
|
313
|
-
warn "CREDENTIAL-RESTORE-SKIPPED: runner-secret skipped; the ${CREDENTIAL_RESTORE_DEADLINE_SECONDS}s credential-restore budget is exhausted"
|
|
314
|
-
return 0
|
|
315
|
-
fi
|
|
316
|
-
if ! stderr_file="$(mktemp /dev/shm/runner-secret-stderr.XXXXXX)"; then
|
|
317
|
-
warn "RUNNER-SECRET-STDERR-UNAVAILABLE: could not allocate diagnostic storage; continuing without runner credentials"
|
|
318
|
-
return 0
|
|
319
|
-
fi
|
|
320
|
-
started_ms="$(now_ms)"
|
|
321
|
-
payload="$(timeout -k "${CREDENTIAL_RESTORE_KILL_GRACE_SECONDS}" "${remaining}" aws secretsmanager get-secret-value --secret-id "${RUNNER_SECRET_ARN}" --query SecretString --output text 2>"${stderr_file}")" || rc=$?
|
|
322
|
-
log_elapsed_since runner-secret-fetch "${started_ms}" "${rc}"
|
|
323
|
-
if [ "${rc}" -eq 124 ] || [ "${rc}" -eq 137 ]; then
|
|
324
|
-
warn "CREDENTIAL-RESTORE-TIMEOUT: runner-secret did not finish within its ${remaining}s share of the ${CREDENTIAL_RESTORE_DEADLINE_SECONDS}s credential-restore deadline; continuing without it"
|
|
325
|
-
rm -f "${stderr_file}"
|
|
326
|
-
return 0
|
|
327
|
-
fi
|
|
328
|
-
if [ "${rc}" -ne 0 ]; then
|
|
329
|
-
warn "RUNNER-SECRET-UNREADABLE: $(<"${stderr_file}")"
|
|
330
|
-
rm -f "${stderr_file}"
|
|
331
|
-
return 0
|
|
332
|
-
fi
|
|
333
|
-
rm -f "${stderr_file}"
|
|
334
|
-
if ! jq -e 'type == "object"' >/dev/null 2>&1 <<<"${payload}"; then
|
|
335
|
-
warn "RUNNER-SECRET-UNPARSEABLE: secret value is not a JSON object"
|
|
336
|
-
return 0
|
|
337
|
-
fi
|
|
338
|
-
local -a populated_keys=()
|
|
339
|
-
for key in GH_TOKEN BRAVE_API_KEY CLOUDFLARE_API_TOKEN NEON_API_KEY; do
|
|
340
|
-
value="$(jq -r --arg k "${key}" '.[$k] // empty' <<<"${payload}")"
|
|
341
|
-
if [ -n "${value}" ]; then
|
|
342
|
-
export "${key}=${value}"
|
|
343
|
-
populated=$((populated + 1))
|
|
344
|
-
populated_keys+=("${key}")
|
|
345
|
-
else
|
|
346
|
-
missing=$((missing + 1))
|
|
347
|
-
fi
|
|
348
|
-
done
|
|
349
|
-
if [ "${populated}" -eq 0 ]; then
|
|
350
|
-
warn "RUNNER-SECRET-UNPOPULATED: populate the runner secret as documented in infrastructure/evident-microvm/README.md"
|
|
351
|
-
else
|
|
352
|
-
log "RUNNER-SECRET-OK: populated ${populated_keys[*]}; ${missing} allow-listed keys empty or absent"
|
|
353
|
-
fi
|
|
354
|
-
return 0
|
|
355
|
-
}
|
|
356
|
-
|
|
357
|
-
restore_credentials() {
|
|
358
|
-
local step_started_ms step_started_s
|
|
359
|
-
step_started_ms="$(now_ms)"
|
|
360
|
-
step_started_s="${SECONDS}"
|
|
361
|
-
|
|
362
|
-
load_state_config || return 1
|
|
363
|
-
# This shares the existing bounded window so /run's worst-case duration does not grow.
|
|
364
|
-
fetch_runner_secret "${step_started_s}"
|
|
365
|
-
bounded_restore_call restore-claude "${step_started_s}" restore claude || return 1
|
|
366
|
-
bounded_restore_call restore-opencode "${step_started_s}" restore opencode || return 1
|
|
367
|
-
|
|
368
|
-
# `restore` exits 0 whether it restored, found nothing, or discarded a corrupt
|
|
369
|
-
# object, so the synchroniser's own predicate is the only reliable answer to
|
|
370
|
-
# "can this VM authenticate a model right now". Its status is captured, not
|
|
371
|
-
# used as a condition, so it can never become this function's return status —
|
|
372
|
-
# `run` calls this bare (no `||`) under `set -euo pipefail`, and any
|
|
373
|
-
# non-zero return here would abort the hook before opencode and the tunnel
|
|
374
|
-
# ever start.
|
|
375
|
-
local auth_status=0 auth_remaining
|
|
376
|
-
auth_remaining="$(remaining_credential_budget "${step_started_s}")"
|
|
377
|
-
if [ "${auth_remaining}" -lt 1 ]; then
|
|
378
|
-
warn "CREDENTIAL-RESTORE-SKIPPED: model-auth-ready skipped; the ${CREDENTIAL_RESTORE_DEADLINE_SECONDS}s credential-restore budget is exhausted"
|
|
379
|
-
else
|
|
380
|
-
timed_synchroniser model-auth-ready "${auth_remaining}" model-auth-ready || auth_status=$?
|
|
381
|
-
case "${auth_status}" in
|
|
382
|
-
0) ;; # some model auth is configured; the two restores above already logged what they found
|
|
383
|
-
10)
|
|
384
|
-
warn "no model credentials under s3://${LITESTREAM_BUCKET}/${LITESTREAM_PREFIX}/ \
|
|
385
|
-
(neither claude/credentials.json nor opencode/auth.json yielded valid JSON) and neither \
|
|
386
|
-
ANTHROPIC_API_KEY nor OPENAI_API_KEY is set. This VM boots and connects; a turn that needs \
|
|
387
|
-
a model provider fails until one is connected. See 'Seeding a credential store' in \
|
|
388
|
-
infrastructure/evident-microvm/README.md."
|
|
389
|
-
;;
|
|
390
|
-
124 | 137)
|
|
391
|
-
warn "CREDENTIAL-RESTORE-TIMEOUT: model-auth-ready did not finish within its ${auth_remaining}s share of the ${CREDENTIAL_RESTORE_DEADLINE_SECONDS}s credential-restore deadline; continuing without it"
|
|
392
|
-
;;
|
|
393
|
-
*)
|
|
394
|
-
# run_synchroniser has already `error`ed the tool-broke line for a
|
|
395
|
-
# broken tool; any other unrecognised code lands here too. Do not
|
|
396
|
-
# claim there are no credentials — absent evidence is not contrary
|
|
397
|
-
# evidence (development-workflow.mdc).
|
|
398
|
-
warn "could not determine whether this VM has model credentials"
|
|
399
|
-
;;
|
|
400
|
-
esac
|
|
401
|
-
fi
|
|
402
|
-
# The step total, in the SAME greppable shape as the per-call lines above, so
|
|
403
|
-
# one query answers both "what does credential restore cost?" and "which of
|
|
404
|
-
# its four operations cost it". Bounded now by CREDENTIAL_RESTORE_DEADLINE_SECONDS
|
|
405
|
-
# + CREDENTIAL_RESTORE_KILL_GRACE_SECONDS as one shared window (see the
|
|
406
|
-
# comment above that constant) rather than per call. `rc=0` is deliberate
|
|
407
|
-
# here, not an absence of failure modes: a timed-out call already warned by
|
|
408
|
-
# name above and is non-fatal by design (D1); every failure that DOES abort
|
|
409
|
-
# this step has already returned 1 above.
|
|
410
|
-
log_elapsed_since credential-restore-step "${step_started_ms}" 0
|
|
411
|
-
return 0
|
|
412
|
-
}
|
|
413
|
-
|
|
414
|
-
apply_runner_opencode_config() {
|
|
415
|
-
if [ -z "${RUNNER_OPENCODE_CONFIG:-}" ]; then
|
|
416
|
-
log "runner OpenCode config is not configured; using the baked project config"
|
|
417
|
-
return 0
|
|
418
|
-
fi
|
|
419
|
-
local source="${RUNNER_OPENCODE_CONFIG}" target="opencode.json"
|
|
420
|
-
[[ "${source}" = /* ]] || source="${WORKSPACE}/${source}"
|
|
421
|
-
[ -f "${WORKSPACE}/opencode.jsonc" ] && target="opencode.jsonc"
|
|
422
|
-
if [ ! -f "${source}" ]; then
|
|
423
|
-
error "RUNNER-OPENCODE-CONFIG-MISSING: ${source} is not a file; headless turns will wedge on the first external-directory permission prompt (#563)"
|
|
424
|
-
return 0
|
|
425
|
-
fi
|
|
426
|
-
cp "${source}" "${WORKSPACE}/${target}"
|
|
427
|
-
git -C "${WORKSPACE}" update-index --skip-worktree "${target}" 2>/dev/null \
|
|
428
|
-
|| warn "could not mark ${target} skip-worktree; it may show as a local change"
|
|
429
|
-
log "Applied runner OpenCode config ${source} to ${WORKSPACE}/${target}"
|
|
430
|
-
}
|
|
431
|
-
|
|
432
|
-
configure_github_access() {
|
|
433
|
-
if [ -z "${GH_TOKEN:-}" ]; then
|
|
434
|
-
warn "GITHUB-CREDENTIALS-MISSING: GH_TOKEN is unavailable; see RUNNER-SECRET-* above"
|
|
435
|
-
return 0
|
|
436
|
-
fi
|
|
437
|
-
export GIT_CONFIG_GLOBAL=/tmp/gitconfig
|
|
438
|
-
if ! : >"${GIT_CONFIG_GLOBAL}" ||
|
|
439
|
-
! git config --global user.name "${GIT_USER_NAME:-evident-bot}" ||
|
|
440
|
-
! git config --global user.email "${GIT_USER_EMAIL:-evident-bot@users.noreply.github.com}" ||
|
|
441
|
-
! git config --global init.defaultBranch main ||
|
|
442
|
-
! printf '%s\n' '#!/usr/bin/env bash' '[ "$1" = get ] || exit 0' 'echo username=x-access-token' 'echo "password=${GH_TOKEN}"' >/tmp/git-credential-helper.sh ||
|
|
443
|
-
! chmod 0700 /tmp/git-credential-helper.sh ||
|
|
444
|
-
! git config --global credential."https://github.com".helper /tmp/git-credential-helper.sh; then
|
|
445
|
-
warn "GITHUB-SETUP-FAILED: could not configure local git credentials; continuing without GitHub access"
|
|
446
|
-
return 0
|
|
447
|
-
fi
|
|
448
|
-
(
|
|
449
|
-
local output rc=0 login repo_url repo
|
|
450
|
-
output="$(timeout -k 1 "${GITHUB_PROBE_DEADLINE_SECONDS}" gh api user --jq .login 2>&1)" || rc=$?
|
|
451
|
-
if [ "${rc}" -eq 124 ] || [ "${rc}" -eq 137 ]; then warn "GITHUB-PROBE-TIMEOUT: auth probe exceeded ${GITHUB_PROBE_DEADLINE_SECONDS}s"; return; fi
|
|
452
|
-
if [ "${rc}" -ne 0 ]; then warn "GITHUB-AUTH-REJECTED: ${output}"; return; fi
|
|
453
|
-
log "GITHUB-AUTH-OK: ${output}"
|
|
454
|
-
repo_url="$(git -C "${WORKSPACE}" remote get-url origin 2>/dev/null || true)"
|
|
455
|
-
repo="$(printf '%s' "${repo_url}" | sed -E 's#(https://github.com/|git@github.com:)##; s#\.git$##')"
|
|
456
|
-
[ -n "${repo}" ] || return
|
|
457
|
-
output="$(timeout -k 1 "${GITHUB_PROBE_DEADLINE_SECONDS}" gh api "repos/${repo}" --jq .full_name 2>&1)" || rc=$?
|
|
458
|
-
if [ "${rc}" -eq 124 ] || [ "${rc}" -eq 137 ]; then warn "GITHUB-PROBE-TIMEOUT: repository probe for ${repo} exceeded ${GITHUB_PROBE_DEADLINE_SECONDS}s"; return; fi
|
|
459
|
-
[ "${rc}" -eq 0 ] || warn "GITHUB-REPO-INACCESSIBLE: ${repo}: ${output}"
|
|
460
|
-
) &
|
|
461
|
-
}
|
|
462
|
-
|
|
463
|
-
# Best-effort by design: /suspend must still drop the tunnel and /terminate must
|
|
464
|
-
# still clean up, so a flush that cannot happen is loud but never fatal.
|
|
163
|
+
# Best-effort by design: the teardown hooks must still complete even when a
|
|
164
|
+
# credential flush cannot happen.
|
|
465
165
|
sync_credentials() {
|
|
466
166
|
load_state_config || return 0
|
|
467
167
|
run_synchroniser sync-once claude || true
|
|
468
168
|
run_synchroniser sync-once opencode || true
|
|
469
169
|
}
|
|
470
170
|
|
|
471
|
-
# ---
|
|
472
|
-
#
|
|
473
|
-
# Restores opencode.db from S3 before opencode opens it — the read half only.
|
|
474
|
-
# Writes NOTHING to S3 itself; replication back to S3 is started separately by
|
|
475
|
-
# `start_litestream` (below) from the `run`/`resume` hooks. `ensure_litestream_config`/
|
|
476
|
-
# `restore_session_db` are the MicroVM side of the same contract
|
|
477
|
-
# runner/docker-images/fargate/entrypoint.sh's inlined restore already speaks, going
|
|
478
|
-
# through the SAME runner-synchroniser CLI.
|
|
171
|
+
# --- boot pre-warms ----------------------------------------------------------
|
|
479
172
|
|
|
480
173
|
# Reads the ~30 MB litestream binary into the page cache, in the background, so
|
|
481
174
|
# the FIRST exec of it does not pay that read on the critical path.
|
|
482
175
|
#
|
|
483
|
-
#
|
|
484
|
-
#
|
|
485
|
-
#
|
|
486
|
-
#
|
|
487
|
-
#
|
|
488
|
-
# deadline is gone, the hook's own SIGTERM is not.)
|
|
489
|
-
# Boot measurements put ~6.6-7.3s between the credential restore and the
|
|
490
|
-
# litestream version line, of which only ~0.2s is accounted for by the two node
|
|
491
|
-
# calls in between — the remainder is INFERRED to be this read, never measured.
|
|
176
|
+
# The pre-warm overlaps the CLI's node startup and auth round trip, so the first
|
|
177
|
+
# litestream operation does not also pay the binary's cold page-cache read.
|
|
178
|
+
# Boot measurements put ~6.6-7.3s before the litestream version line, of which
|
|
179
|
+
# only ~0.2s is accounted for by the two node calls in between — the remainder
|
|
180
|
+
# is INFERRED to be this read, never measured.
|
|
492
181
|
# The `litestream-prewarm` and `litestream-version` timings are what settle it
|
|
493
182
|
# on the next boot.
|
|
494
183
|
#
|
|
@@ -515,272 +204,40 @@ prewarm_litestream() {
|
|
|
515
204
|
log "pre-warming ${binary} in the background"
|
|
516
205
|
}
|
|
517
206
|
|
|
518
|
-
#
|
|
519
|
-
#
|
|
520
|
-
#
|
|
521
|
-
#
|
|
522
|
-
# or empty, so `/resume` (WI-4) restarting litestream after a suspend/resume
|
|
523
|
-
# snapshot — /dev/shm intact — pays nothing to regenerate it.
|
|
524
|
-
#
|
|
525
|
-
# Returns 1 on failure — fatal to the CALLER, never to the hook: every caller
|
|
526
|
-
# (restore_session_db here, start_litestream in WI-3) must still let opencode
|
|
527
|
-
# and the tunnel start regardless.
|
|
528
|
-
ensure_litestream_config() {
|
|
529
|
-
if [ -s "${LITESTREAM_CONFIG_FILE}" ]; then
|
|
530
|
-
return 0
|
|
531
|
-
fi
|
|
532
|
-
|
|
533
|
-
# Timed OUTSIDE the substitution, not with timed_synchroniser: this call's
|
|
534
|
-
# stdout IS the config, so a timing line emitted inside `$(...)` would land in
|
|
535
|
-
# litestream.yml.
|
|
536
|
-
local rendered config_started_ms config_rc=0
|
|
537
|
-
config_started_ms="$(now_ms)"
|
|
538
|
-
rendered="$(run_synchroniser litestream-config)" || config_rc=$?
|
|
539
|
-
log_elapsed_since litestream-config "${config_started_ms}" "${config_rc}"
|
|
540
|
-
if [ "${config_rc}" -ne 0 ]; then
|
|
541
|
-
error "could not generate ${LITESTREAM_CONFIG_FILE}: runner-synchroniser litestream-config failed (see the error above)"
|
|
542
|
-
return 1
|
|
543
|
-
fi
|
|
544
|
-
printf '%s\n' "${rendered}" >"${LITESTREAM_CONFIG_FILE}" || {
|
|
545
|
-
error "could not write ${LITESTREAM_CONFIG_FILE}"
|
|
546
|
-
return 1
|
|
547
|
-
}
|
|
548
|
-
|
|
549
|
-
# Q7's diagnostic, for whoever can read this VM's CloudWatch log: which way
|
|
550
|
-
# litestream resolved its AWS region, and whether it can even see the
|
|
551
|
-
# names it would need to (never the VALUES of any OTHER AWS_* variable —
|
|
552
|
-
# AWS_SECRET_ACCESS_KEY / AWS_SESSION_TOKEN may be present here).
|
|
553
|
-
local region_empty="yes"
|
|
554
|
-
if printf '%s\n' "${rendered}" | grep -Eq '^ *region: *[^[:space:]]'; then
|
|
555
|
-
region_empty="no"
|
|
556
|
-
fi
|
|
557
|
-
# Hoisted out of the log line below so it can be timed: this is the FIRST exec
|
|
558
|
-
# of the litestream binary on the boot, and the pre-warm /run fires (see
|
|
559
|
-
# prewarm_litestream) is aimed squarely at what this number measures.
|
|
560
|
-
local version_started_ms litestream_version
|
|
561
|
-
version_started_ms="$(now_ms)"
|
|
562
|
-
litestream_version="$(litestream version 2>/dev/null || echo 'unknown')"
|
|
563
|
-
log_elapsed_since litestream-version "${version_started_ms}" 0
|
|
564
|
-
log "litestream ${litestream_version}; AWS_REGION=${AWS_REGION:-<unset>} AWS_DEFAULT_REGION=${AWS_DEFAULT_REGION:-<unset>}; rendered litestream.yml region empty: ${region_empty}"
|
|
565
|
-
}
|
|
566
|
-
|
|
567
|
-
# Writes the marker AND names the reason, in one call, so the marker can never
|
|
568
|
-
# appear silently. Every give-up path in restore_session_db calls this EXCEPT
|
|
569
|
-
# the classifier's 31 (#1106 — see that branch), and no success path ever does.
|
|
570
|
-
# It is what a later `/resume`/`start_litestream` (WI-3) reads to skip
|
|
571
|
-
# replicating this boot: a local DB that is only PARTIALLY restored must never
|
|
572
|
-
# be allowed to overwrite a replica this boot never proved safe to write over.
|
|
573
|
-
# A merely FRESH local DB is not that danger — litestream continues the txid
|
|
574
|
-
# chain — which is precisely why 31 no longer belongs here.
|
|
575
|
-
mark_no_replicate() {
|
|
576
|
-
: >"${SESSION_DB_NO_REPLICATE_MARKER}"
|
|
577
|
-
warn "SESSION-DB-NO-REPLICATE: $1"
|
|
578
|
-
}
|
|
579
|
-
|
|
580
|
-
# Removes local session-DB debris a give-up path leaves behind. Called on
|
|
581
|
-
# EVERY give-up path and NO success path: skipping it on a give-up is a real
|
|
582
|
-
# bug (a truncated file left in place makes opencode fail to open a malformed
|
|
583
|
-
# database, which fails /run outright — worse than the history loss this
|
|
584
|
-
# discards); calling it on a success path would delete a good restore. One
|
|
585
|
-
# `rm -f` per path, not a single command, so a failure removing one does not
|
|
586
|
-
# skip the other two.
|
|
587
|
-
discard_session_db_debris() {
|
|
588
|
-
rm -f "${OPENCODE_DB_PATH}" "${OPENCODE_DB_PATH}-wal" "${OPENCODE_DB_PATH}-shm"
|
|
589
|
-
}
|
|
590
|
-
|
|
591
|
-
# How long restore_session_db waits for ONE `litestream restore` before giving
|
|
592
|
-
# up. From M4 (see the plan's §2): ~50 MB/s extrapolated from the x86_64 ECS
|
|
593
|
-
# container's own live replica, so 7s covers roughly 300-350 MB — an
|
|
594
|
-
# EXTRAPOLATION, not a MicroVM measurement (different arch, network path and
|
|
595
|
-
# credential source); the boot line ensure_litestream_config logs, plus the
|
|
596
|
-
# ${SECONDS} stamps in /run, are how it gets measured here for real.
|
|
207
|
+
# Reads the aws CLI v2 install tree into the page cache, in the background, so
|
|
208
|
+
# the CLI's first runner-secret fetch does not pay first-touch I/O. The read
|
|
209
|
+
# overlaps `evident run`'s node startup and auth round trip rather than a hook
|
|
210
|
+
# deadline.
|
|
597
211
|
#
|
|
598
|
-
#
|
|
599
|
-
#
|
|
600
|
-
#
|
|
601
|
-
#
|
|
602
|
-
#
|
|
603
|
-
#
|
|
604
|
-
#
|
|
605
|
-
|
|
606
|
-
|
|
607
|
-
|
|
608
|
-
|
|
609
|
-
|
|
610
|
-
# Past it the restore is truncated (SESSION-DB-RESTORE-TRUNCATED, below) and
|
|
611
|
-
# this boot skips replication — silent, permanent history loss for that
|
|
612
|
-
# runner unless somebody reads the log. Not solved here (WI-0 Task 0.2).
|
|
613
|
-
# The env override is for tests only, so they need not burn 7s of wall clock.
|
|
614
|
-
SESSION_DB_RESTORE_DEADLINE_SECONDS="${EVIDENT_SESSION_DB_RESTORE_DEADLINE_SECONDS:-7}"
|
|
615
|
-
# The SIGKILL backstop `timeout -k` applies after its own SIGTERM: `timeout`
|
|
616
|
-
# alone only SIGTERMs, and a litestream that ignored it would be unbounded
|
|
617
|
-
# again. Hard ceiling on this step: DEADLINE + this = 9s.
|
|
618
|
-
SESSION_DB_RESTORE_KILL_GRACE_SECONDS=2
|
|
619
|
-
|
|
620
|
-
# The one bounded restore attempt, run before start_opencode (opencode opens
|
|
621
|
-
# the DB the moment it starts, so this is the only place it can happen).
|
|
622
|
-
# Every branch returns 0 — /run calls this bare under `set -euo pipefail`,
|
|
623
|
-
# exactly like restore_credentials, and nothing about the session DB may ever
|
|
624
|
-
# fail /run (Q3): a /run that exits 1 fails the whole lifecycle transition
|
|
625
|
-
# and the user gets NO runner at all, which is worse than losing history.
|
|
626
|
-
restore_session_db() {
|
|
627
|
-
# Cleanup, not a decision, and unconditionally first: /dev/shm survives
|
|
628
|
-
# suspend/resume and a /run retry, so a marker left by an earlier boot must
|
|
629
|
-
# never silently disable replication for the rest of this VM's life.
|
|
630
|
-
rm -f "${SESSION_DB_NO_REPLICATE_MARKER}"
|
|
631
|
-
local report_path
|
|
632
|
-
if report_path="$(session_db_recovery_report_path)"; then
|
|
633
|
-
# The default report directory is provisioned in the image, so this is a
|
|
634
|
-
# no-op there; a shared-path override (EVIDENT_SESSION_DB_RECOVERY_REPORT)
|
|
635
|
-
# can name a directory that isn't, and the typed synchroniser writer
|
|
636
|
-
# already creates it for that same override (session-db-recovery-report.ts).
|
|
637
|
-
# Best-effort only: report_session_db_giveup below still warns and
|
|
638
|
-
# continues if this didn't leave a writable path. `${path%/*}` leaves a
|
|
639
|
-
# bare relative filename (no `/`) unchanged rather than reducing to `.`
|
|
640
|
-
# like dirname(1) — without the explicit check, mkdir would create a
|
|
641
|
-
# directory AT the report path itself.
|
|
642
|
-
local report_dir="${report_path%/*}"
|
|
643
|
-
[ "${report_dir}" = "${report_path}" ] && report_dir="."
|
|
644
|
-
[ -d "${report_dir}" ] || mkdir -p "${report_dir}" 2>/dev/null || true
|
|
645
|
-
: >"${report_path}" || warn "SESSION-DB-RECOVERY-REPORT-TRUNCATE-FAILED"
|
|
646
|
-
else
|
|
647
|
-
warn "SESSION-DB-RECOVERY-REPORT-PATH-UNAVAILABLE"
|
|
648
|
-
fi
|
|
649
|
-
|
|
650
|
-
# Config resolved ONCE here, in the synchroniser (config.ts owns
|
|
651
|
-
# OPENCODE_DB_PATH; no second copy of that path here), exactly as
|
|
652
|
-
# entrypoint.sh's own line does. Unlike ECS's `|| die`, a failure here must
|
|
653
|
-
# not abort the hook — `x="$(cmd)"` under `set -e` WOULD abort the whole
|
|
654
|
-
# shell on a failing substitution, which is exactly why it is caught
|
|
655
|
-
# explicitly rather than left to `set -e`.
|
|
656
|
-
# Timed OUTSIDE the substitution for the same reason ensure_litestream_config
|
|
657
|
-
# is: this call's stdout is `eval`'d, so a timing line emitted inside `$(...)`
|
|
658
|
-
# would be evaluated as configuration.
|
|
659
|
-
local synchroniser_env env_started_ms env_rc=0
|
|
660
|
-
env_started_ms="$(now_ms)"
|
|
661
|
-
synchroniser_env="$(run_synchroniser env)" || env_rc=$?
|
|
662
|
-
log_elapsed_since session-db-env "${env_started_ms}" "${env_rc}"
|
|
663
|
-
if [ "${env_rc}" -ne 0 ]; then
|
|
664
|
-
mark_no_replicate "could not resolve the runner-synchroniser configuration (see the ERROR above)"
|
|
665
|
-
report_session_db_giveup synchroniser_config_unresolved
|
|
666
|
-
return 0
|
|
667
|
-
fi
|
|
668
|
-
# Guarded for the SAME reason as the substitution above, which is easy to miss:
|
|
669
|
-
# `eval` returns the status of what it ran, so a malformed line from a skewed
|
|
670
|
-
# bundle would abort /run right here under `set -e` — before start_opencode,
|
|
671
|
-
# so the user gets NO runner at all. entrypoint.sh:110 leaves the identical
|
|
672
|
-
# line bare because on ECS the blast radius is a crash-loop-and-replace; here
|
|
673
|
-
# it is the whole lifecycle transition, so Q3 makes it a give-up instead.
|
|
674
|
-
eval "${synchroniser_env}" || {
|
|
675
|
-
mark_no_replicate "the runner-synchroniser configuration could not be evaluated; the installed bundle likely does not match this hook"
|
|
676
|
-
report_session_db_giveup synchroniser_config_unevaluable
|
|
677
|
-
return 0
|
|
678
|
-
}
|
|
679
|
-
|
|
680
|
-
# #931's exact lesson, one shell over (runner/docker-images/fargate/entrypoint.sh):
|
|
681
|
-
# the `|| { ... }` above only catches a non-zero EXIT — an `env` that exits
|
|
682
|
-
# 0 with an INCOMPLETE contract (a runner-synchroniser version/build skew)
|
|
683
|
-
# would otherwise abort right here under `set -u` the moment
|
|
684
|
-
# `${OPENCODE_DB_PATH}` is dereferenced below. entrypoint.sh may `die` on
|
|
685
|
-
# this (ECS just crash-loops and replaces the task); this hook cannot —
|
|
686
|
-
# Q3 says nothing about the session DB may ever fail /run — so a missing
|
|
687
|
-
# OPENCODE_DB_PATH is a give-up, not a silent default (a guessed path would
|
|
688
|
-
# be actively wrong, not merely absent).
|
|
689
|
-
if [ -z "${OPENCODE_DB_PATH+x}" ]; then
|
|
690
|
-
mark_no_replicate "run_synchroniser env did not define OPENCODE_DB_PATH; the installed runner-synchroniser build likely does not match this hook"
|
|
691
|
-
report_session_db_giveup synchroniser_config_incomplete
|
|
692
|
-
return 0
|
|
693
|
-
fi
|
|
694
|
-
|
|
695
|
-
# #770's arm, checked FIRST of the decisions and with its OWN message —
|
|
696
|
-
# never the SESSION-DB-REPLICA-UNUSABLE one below (#764's finding: an
|
|
697
|
-
# operator grepping that marker must not hit the disabled case).
|
|
698
|
-
# PERSISTENCE_BUCKET is empty exactly when LITESTREAM_BUCKET or
|
|
699
|
-
# LITESTREAM_PREFIX is unset (config.ts's persistenceEnabled) — the
|
|
700
|
-
# IDENTICAL predicate ECS's own litestream-launch gate uses
|
|
701
|
-
# (entrypoint.sh), so the two images agree on what "disabled" means.
|
|
702
|
-
if [ -z "${PERSISTENCE_BUCKET:-}" ]; then
|
|
703
|
-
warn "SESSION-DB-PERSISTENCE-DISABLED: LITESTREAM_BUCKET/LITESTREAM_PREFIX are not both set; opencode starts with a fresh session DB and nothing is replicated."
|
|
704
|
-
return 0
|
|
705
|
-
fi
|
|
706
|
-
|
|
707
|
-
if ! ensure_litestream_config; then
|
|
708
|
-
mark_no_replicate "could not generate ${LITESTREAM_CONFIG_FILE} (see the error above)"
|
|
709
|
-
report_session_db_giveup litestream_config_unavailable
|
|
212
|
+
# A single `cat` of the `aws` entrypoint (litestream's pattern above) is NOT
|
|
213
|
+
# enough here: v2 ships as a real Python distribution (~7,500 files under the
|
|
214
|
+
# resolved binary's own directory), and a cold invocation demand-pages a
|
|
215
|
+
# scattered set of them (botocore's endpoints.json/partitions.json, service
|
|
216
|
+
# model JSON, shared libs) — one boot measured #1997's fetch at 7,243ms cold
|
|
217
|
+
# vs ~400ms warm, an ~18x gap the litestream read-ahead trick alone cannot
|
|
218
|
+
# close. `find -exec cat` walks that whole tree instead of one file.
|
|
219
|
+
prewarm_aws_cli() {
|
|
220
|
+
local binary tree
|
|
221
|
+
binary="$(command -v aws 2>/dev/null || true)"
|
|
222
|
+
if [ -z "${binary}" ]; then
|
|
223
|
+
warn "aws CLI is not on PATH; skipping the boot pre-warm"
|
|
710
224
|
return 0
|
|
711
225
|
fi
|
|
226
|
+
tree="$(dirname "$(readlink -f "${binary}")")"
|
|
712
227
|
|
|
713
|
-
|
|
714
|
-
|
|
715
|
-
|
|
716
|
-
|
|
717
|
-
|
|
718
|
-
|
|
719
|
-
|
|
720
|
-
# `timeout`'s OWN codes, handled BEFORE classifying: 124/137 are a
|
|
721
|
-
# TRUNCATED restore, not a corrupt replica — handing them to
|
|
722
|
-
# session-db-classify would mislabel a slow/large replica as one. 125-127
|
|
723
|
-
# mean `timeout` (or litestream itself) is broken, not the replica.
|
|
724
|
-
case "${restore_rc}" in
|
|
725
|
-
124 | 137)
|
|
726
|
-
discard_session_db_debris
|
|
727
|
-
mark_no_replicate "SESSION-DB-RESTORE-TRUNCATED: litestream restore did not finish within the ${SESSION_DB_RESTORE_DEADLINE_SECONDS}s deadline (+${SESSION_DB_RESTORE_KILL_GRACE_SECONDS}s kill grace), ${SECONDS}s into the hook; opencode starts with a fresh session DB and nothing is replicated this boot"
|
|
728
|
-
report_session_db_giveup restore_deadline_exceeded "${restore_rc}"
|
|
729
|
-
return 0
|
|
730
|
-
;;
|
|
731
|
-
125 | 126 | 127)
|
|
732
|
-
error "litestream restore could not even run (timeout exited ${restore_rc})"
|
|
733
|
-
discard_session_db_debris
|
|
734
|
-
mark_no_replicate "restore tool is broken (timeout exited ${restore_rc}); opencode starts with a fresh session DB and nothing is replicated this boot"
|
|
735
|
-
report_session_db_giveup restore_tool_unusable "${restore_rc}"
|
|
736
|
-
return 0
|
|
737
|
-
;;
|
|
738
|
-
esac
|
|
739
|
-
|
|
740
|
-
local classify_rc=0
|
|
741
|
-
run_synchroniser session-db-classify "${restore_rc}" 1 --on-unusable-replica=leave --fresh-db-fallback || classify_rc=$?
|
|
742
|
-
case "${classify_rc}" in
|
|
743
|
-
0) ;; # restored, or no replica yet — the CLI already logged which
|
|
744
|
-
31)
|
|
745
|
-
# The classifier already discarded the local debris (session-db.ts) — and
|
|
746
|
-
# that is exactly why this is the ONE give-up path that does NOT
|
|
747
|
-
# mark_no_replicate (#1106). Every other give-up here may be sitting on a
|
|
748
|
-
# half-restored DB, so the marker stays; a 31 is structurally guaranteed
|
|
749
|
-
# to be a FRESH one, and replicating it starts a new backup chain instead
|
|
750
|
-
# of leaving this boot with a zero-width backup window. Same reasoning,
|
|
751
|
-
# and same one-line change, as runner/docker-images/fargate/entrypoint.sh's `31)`.
|
|
752
|
-
warn "SESSION-DB-REPLICA-UNUSABLE: booting with a fresh opencode.db and replicating into the existing prefix this boot (see the WARNING above)"
|
|
753
|
-
;;
|
|
754
|
-
32)
|
|
755
|
-
# The CLI's own contract for 32 is "re-run litestream restore and ask
|
|
756
|
-
# again", but this hook has budget for exactly ONE attempt (Q4) — name
|
|
757
|
-
# the deviation so nobody reads this as a bug.
|
|
758
|
-
discard_session_db_debris
|
|
759
|
-
mark_no_replicate "session-db-classify asked for another restore attempt (32), but this hook has budget for only one; treating it as a give-up rather than retrying"
|
|
760
|
-
;;
|
|
761
|
-
30)
|
|
762
|
-
# The CLI already logged its own FATAL line above; Q3 still says boot
|
|
763
|
-
# fresh rather than fail /run.
|
|
764
|
-
discard_session_db_debris
|
|
765
|
-
mark_no_replicate "session-db-classify returned fatal (30); see the FATAL message above"
|
|
766
|
-
report_session_db_giveup classification_fatal
|
|
767
|
-
;;
|
|
768
|
-
*)
|
|
769
|
-
# run_synchroniser already logged the "tool broke" ERROR for this.
|
|
770
|
-
discard_session_db_debris
|
|
771
|
-
mark_no_replicate "session-db-classify exited ${classify_rc}, which is none of its documented answers"
|
|
772
|
-
report_session_db_giveup classification_unrecognised
|
|
773
|
-
;;
|
|
774
|
-
esac
|
|
775
|
-
|
|
776
|
-
return 0
|
|
228
|
+
(
|
|
229
|
+
local started_ms rc=0
|
|
230
|
+
started_ms="$(now_ms)"
|
|
231
|
+
find "${tree}" -type f -exec cat {} + >/dev/null 2>&1 || rc=$?
|
|
232
|
+
log_elapsed_since aws-cli-prewarm "${started_ms}" "${rc}"
|
|
233
|
+
) &
|
|
234
|
+
log "pre-warming ${tree} in the background"
|
|
777
235
|
}
|
|
778
|
-
# --- Session DB restore (end) -----------------------------------------------
|
|
779
236
|
|
|
780
237
|
# `kill -0` answers "does this pid exist", which is not the question any caller
|
|
781
238
|
# here is asking. A process that has exited but has not been reaped — a zombie —
|
|
782
239
|
# still exists, so `kill -0` reports a corpse as ALIVE. That condition is the
|
|
783
|
-
# normal case for everything these hooks start: `start_tunnel
|
|
240
|
+
# normal case for everything these hooks start: `start_tunnel` and its CLI child
|
|
784
241
|
# background a process that outlives the hook, the hook shell must return so AWS
|
|
785
242
|
# gets its 200, and PID 1 in this image is a bare node hook server with no init
|
|
786
243
|
# (Dockerfile) — so nothing ever wait()s for the orphan. It stayed a zombie for
|
|
@@ -882,24 +339,6 @@ payload_is_complete() {
|
|
|
882
339
|
' >/dev/null 2>&1
|
|
883
340
|
}
|
|
884
341
|
|
|
885
|
-
# `setsid` so opencode outlives this hook: the script must return so AWS gets its
|
|
886
|
-
# 200, while opencode keeps serving. The guard makes /run idempotent — a retry
|
|
887
|
-
# after a killed hook must not put a second instance on the same port.
|
|
888
|
-
start_opencode() {
|
|
889
|
-
if opencode_is_running; then
|
|
890
|
-
warn "opencode already running (pid $(cat "${OPENCODE_PID_FILE}")); reusing it"
|
|
891
|
-
return 0
|
|
892
|
-
fi
|
|
893
|
-
|
|
894
|
-
setsid opencode serve \
|
|
895
|
-
--hostname 127.0.0.1 \
|
|
896
|
-
--port "${OPENCODE_PORT}" \
|
|
897
|
-
--print-logs &
|
|
898
|
-
|
|
899
|
-
echo $! >"${OPENCODE_PID_FILE}"
|
|
900
|
-
log "opencode starting (pid $(cat "${OPENCODE_PID_FILE}"))"
|
|
901
|
-
}
|
|
902
|
-
|
|
903
342
|
stop_opencode() {
|
|
904
343
|
if ! opencode_is_running; then
|
|
905
344
|
# Clear the file here too, for the same reason stop_tunnel does: opencode
|
|
@@ -964,69 +403,11 @@ stop_opencode_and_wait() {
|
|
|
964
403
|
log "opencode stopped ${outcome}"
|
|
965
404
|
}
|
|
966
405
|
|
|
967
|
-
# --- litestream replicate (#812 WI-3) ---------------------------------------
|
|
968
|
-
#
|
|
969
|
-
# The write half of session-DB persistence. restore_session_db (above) is the
|
|
970
|
-
# read half only and never starts this. Q8 chose one backgrounded, unsupervised
|
|
971
|
-
# `litestream replicate` over a periodic flush or a `replicate -exec` wrapper
|
|
972
|
-
# around opencode: SIGTERM is `replicate`'s own final-sync trigger, so
|
|
973
|
-
# stop_litestream (below) doubles as the checked flush /suspend and /terminate
|
|
974
|
-
# need, and there is no supervisor in this image to hand the process to (no
|
|
975
|
-
# init, no `wait -n` shell that could stay alive after the hook returns).
|
|
976
|
-
|
|
977
406
|
# The graceful wait before the SIGKILL backstop in stop_litestream, below.
|
|
978
407
|
# litestream's own sync is normally sub-second, so 10s is generous headroom —
|
|
979
408
|
# chosen the same way TUNNEL_STOP_WAIT_SECONDS was, not a measured worst case.
|
|
980
409
|
LITESTREAM_STOP_WAIT_SECONDS="${EVIDENT_LITESTREAM_STOP_WAIT_SECONDS:-10}"
|
|
981
410
|
|
|
982
|
-
# Guards, in this exact order and no other — reordering any of them re-arms the
|
|
983
|
-
# invariant it exists to protect:
|
|
984
|
-
#
|
|
985
|
-
# 1. persistence disabled (#770) — bucket AND prefix both non-empty — is its
|
|
986
|
-
# OWN arm with its OWN message, checked FIRST, and must never log
|
|
987
|
-
# SESSION-DB-REPLICA-UNUSABLE: an operator grepping that marker must not
|
|
988
|
-
# hit the disabled case (#764).
|
|
989
|
-
# 2. the no-replicate marker (restore_session_db above) — set when this boot's
|
|
990
|
-
# local DB may be only PARTIALLY restored, so starting replicate here would
|
|
991
|
-
# let it overwrite a replica this boot never proved safe to write over.
|
|
992
|
-
# A classifier 31 no longer sets it (#1106): that DB is guaranteed fresh,
|
|
993
|
-
# and skipping replicate for it was what made the history loss unbounded.
|
|
994
|
-
# 3. no usable config — ensure_litestream_config (restore_session_db, above)
|
|
995
|
-
# already logged why it is missing; this is not a place to retry it.
|
|
996
|
-
# 4. already running — idempotence, mirroring start_tunnel's own guard.
|
|
997
|
-
#
|
|
998
|
-
# Only past all four does it actually spawn, backgrounded with `setsid` for the
|
|
999
|
-
# same reason start_opencode/start_tunnel are: the hook must return so AWS gets
|
|
1000
|
-
# its 200, while replicate keeps running. litestream's OWN stderr is left to
|
|
1001
|
-
# reach CloudWatch — never redirected — because there is no supervisor in this
|
|
1002
|
-
# image, so that is the only channel that can say why a replicator died.
|
|
1003
|
-
start_litestream() {
|
|
1004
|
-
if [ -z "${PERSISTENCE_BUCKET:-}" ]; then
|
|
1005
|
-
warn "SESSION-DB-PERSISTENCE-DISABLED: LITESTREAM_BUCKET/LITESTREAM_PREFIX are not both set; nothing is replicated."
|
|
1006
|
-
return 0
|
|
1007
|
-
fi
|
|
1008
|
-
|
|
1009
|
-
if [ -e "${SESSION_DB_NO_REPLICATE_MARKER}" ]; then
|
|
1010
|
-
log "skipping litestream replicate: this boot's session DB was not proven safe to replicate (see the SESSION-DB-* warning above)"
|
|
1011
|
-
return 0
|
|
1012
|
-
fi
|
|
1013
|
-
|
|
1014
|
-
if [ ! -s "${LITESTREAM_CONFIG_FILE}" ]; then
|
|
1015
|
-
error "no usable ${LITESTREAM_CONFIG_FILE}; not starting litestream replicate"
|
|
1016
|
-
return 0
|
|
1017
|
-
fi
|
|
1018
|
-
|
|
1019
|
-
if litestream_is_running; then
|
|
1020
|
-
warn "litestream already running (pid $(cat "${LITESTREAM_PID_FILE}")); reusing it"
|
|
1021
|
-
return 0
|
|
1022
|
-
fi
|
|
1023
|
-
|
|
1024
|
-
setsid litestream replicate -config "${LITESTREAM_CONFIG_FILE}" &
|
|
1025
|
-
|
|
1026
|
-
echo $! >"${LITESTREAM_PID_FILE}"
|
|
1027
|
-
log "litestream replicate starting (pid $(cat "${LITESTREAM_PID_FILE}"))"
|
|
1028
|
-
}
|
|
1029
|
-
|
|
1030
411
|
# The graceful stop: mirrors stop_tunnel exactly, including PROC_DEAD_REASON's
|
|
1031
412
|
# one deciding read AFTER the poll loop (never inside it, which polls 10x/s and
|
|
1032
413
|
# would bury the log) — gone / zombie / SIGKILLed are the three outcomes an
|
|
@@ -1073,7 +454,7 @@ stop_litestream() {
|
|
|
1073
454
|
# stop_tunnel's own comment argues against an argument a future caller can get
|
|
1074
455
|
# wrong. litestream_is_running is the same O(1) check stop_litestream's own
|
|
1075
456
|
# no-op branch uses, not a poll — /run's cleanup trap fires this before
|
|
1076
|
-
#
|
|
457
|
+
# the replicator start is ever reached whenever an earlier step failed, and an
|
|
1077
458
|
# operator reading that log must not be told a stop signal went to a process
|
|
1078
459
|
# that never started.
|
|
1079
460
|
kill_litestream() {
|
|
@@ -1099,19 +480,16 @@ kill_litestream() {
|
|
|
1099
480
|
# Ceiling for the one-shot `litestream replicate -once` flush below. litestream's
|
|
1100
481
|
# own sync is normally sub-second (M5, the plan's grounding); 10s is generous
|
|
1101
482
|
# headroom, chosen the same way LITESTREAM_STOP_WAIT_SECONDS was above — not a
|
|
1102
|
-
# measured worst case. The SIGKILL backstop
|
|
1103
|
-
#
|
|
1104
|
-
# identical concept (bare `timeout` only SIGTERMs, and a litestream that
|
|
1105
|
-
# ignored it would be unbounded again), and there is no reason for the restore
|
|
1106
|
-
# and flush paths to ever drift on how long a SIGTERM gets to land.
|
|
483
|
+
# measured worst case. The SIGKILL backstop gives the flush its own two-second
|
|
484
|
+
# grace after `timeout` sends SIGTERM.
|
|
1107
485
|
SESSION_DB_FLUSH_DEADLINE_SECONDS="${EVIDENT_SESSION_DB_FLUSH_DEADLINE_SECONDS:-10}"
|
|
486
|
+
SESSION_DB_FLUSH_KILL_GRACE_SECONDS=2
|
|
1108
487
|
|
|
1109
|
-
# Guards mirror
|
|
488
|
+
# Guards mirror the CLI replicator's first three exactly (disabled -> marker ->
|
|
1110
489
|
# config) — a flush must never attempt work those guards would have refused
|
|
1111
490
|
# to start in the first place. PERSISTENCE_BUCKET is the SAME variable
|
|
1112
|
-
#
|
|
1113
|
-
# THIS process set it (load_state_config
|
|
1114
|
-
# eval, on /run).
|
|
491
|
+
# the CLI reads, so both agree regardless of which earlier step in
|
|
492
|
+
# THIS process set it (load_state_config above).
|
|
1115
493
|
#
|
|
1116
494
|
# Then: stop the daemon FIRST — its SIGTERM IS litestream's own final-sync
|
|
1117
495
|
# trigger — and only once it is confirmed stopped does the synchronous -once
|
|
@@ -1151,7 +529,7 @@ flush_session_db() {
|
|
|
1151
529
|
# Only reached with the daemon confirmed stopped, or never running — never
|
|
1152
530
|
# concurrently with it (see above).
|
|
1153
531
|
local flush_rc=0
|
|
1154
|
-
timeout -k "${
|
|
532
|
+
timeout -k "${SESSION_DB_FLUSH_KILL_GRACE_SECONDS}" "${SESSION_DB_FLUSH_DEADLINE_SECONDS}" \
|
|
1155
533
|
litestream replicate -once -config "${LITESTREAM_CONFIG_FILE}" || flush_rc=$?
|
|
1156
534
|
|
|
1157
535
|
if [ "${flush_rc}" -eq 0 ]; then
|
|
@@ -1168,11 +546,14 @@ flush_session_db() {
|
|
|
1168
546
|
# suspend it (#732). Sized from the measured cost of guessing wrong rather than
|
|
1169
547
|
# the saving: suspend reaches SUSPENDED in ~7 s and a resume is RUNNING in
|
|
1170
548
|
# ~0.6 s with the tunnel back ~2 s later (README, "Measured, end to end"), so
|
|
1171
|
-
# napping a VM whose user comes straight back costs ~10 s — against
|
|
1172
|
-
#
|
|
1173
|
-
#
|
|
1174
|
-
#
|
|
1175
|
-
#
|
|
549
|
+
# napping a VM whose user comes straight back costs ~10 s — against the
|
|
550
|
+
# legacy conservative baseline input of ~$0.30/h, AWS can burst a loaded VM
|
|
551
|
+
# to a 16 GB / 8 vCPU peak (~$1.06/h at sustained full load—a worst-case
|
|
552
|
+
# ceiling, not an expectation); that is cheap enough that the balance
|
|
553
|
+
# sits far nearer the floor than the ceiling. Not AT the floor, though: the
|
|
554
|
+
# CLI's idle detector needs 2 clear poll cycles (`run.ts`'s `idlePolls >= 2`,
|
|
555
|
+
# ≥4 s of real time), so a value near that would spend more time
|
|
556
|
+
# suspending/resuming than idle.
|
|
1176
557
|
# ECS's waker uses 900 s instead only because *its* cold start is far slower
|
|
1177
558
|
# than this VM's ~2 s resume — not evidence this default should match it.
|
|
1178
559
|
#
|
|
@@ -1184,7 +565,9 @@ flush_session_db() {
|
|
|
1184
565
|
# runner/docker-images/fargate/entrypoint.sh uses for the ECS runner's own
|
|
1185
566
|
# idle-timeout flag, so an operator who knows one knows the other. A
|
|
1186
567
|
# non-numeric override must never silently DROP the flag — that degrades to
|
|
1187
|
-
# an always-on VM burning ~$
|
|
568
|
+
# an always-on VM burning ~$7.17/day at baseline, or up to ~$25.49/day at
|
|
569
|
+
# sustained full peak as load increases (a worst-case ceiling, not an
|
|
570
|
+
# expectation), exactly the bug this closes — so it
|
|
1188
571
|
# warns and falls back to the default instead.
|
|
1189
572
|
#
|
|
1190
573
|
# Deliberately NOT in the /run payload yet: doing so would touch the doorbell
|
|
@@ -1206,6 +589,26 @@ fi
|
|
|
1206
589
|
# its 200, while the tunnel keeps serving.
|
|
1207
590
|
start_tunnel() {
|
|
1208
591
|
local runner_key="$1" api_url="$2" tunnel_url="$3"
|
|
592
|
+
local restore_runner_credentials="${4:-false}" restore_history="${5:-false}"
|
|
593
|
+
local -a credential_flags=()
|
|
594
|
+
local -a session_db_flags=()
|
|
595
|
+
local -a opencode_config_flags=()
|
|
596
|
+
|
|
597
|
+
# The explicit fourth argument is set only by /run; /resume uses the default
|
|
598
|
+
# so a resumed VM never restores credentials over stores it already has.
|
|
599
|
+
if [ "${restore_runner_credentials}" = true ]; then
|
|
600
|
+
credential_flags=(--restore-runner-credentials)
|
|
601
|
+
fi
|
|
602
|
+
|
|
603
|
+
# Only a fresh /run asks the CLI to restore session history. /resume keeps
|
|
604
|
+
# the database from the snapshot and must not restore over it.
|
|
605
|
+
if [ "${restore_history}" = true ]; then
|
|
606
|
+
session_db_flags=(--restore-session-db)
|
|
607
|
+
fi
|
|
608
|
+
|
|
609
|
+
if [ -n "${RUNNER_OPENCODE_CONFIG:-}" ]; then
|
|
610
|
+
opencode_config_flags=(--opencode-config-overlay "${RUNNER_OPENCODE_CONFIG}")
|
|
611
|
+
fi
|
|
1209
612
|
|
|
1210
613
|
if tunnel_is_running; then
|
|
1211
614
|
warn "tunnel already running (pid $(cat "${TUNNEL_PID_FILE}")); not starting a second one"
|
|
@@ -1228,6 +631,14 @@ start_tunnel() {
|
|
|
1228
631
|
--port "${OPENCODE_PORT}" \
|
|
1229
632
|
--endpoint "${api_url}" \
|
|
1230
633
|
--tunnel "${tunnel_url}" \
|
|
634
|
+
--opencode-pid-file "${OPENCODE_PID_FILE}" \
|
|
635
|
+
--litestream-config "${LITESTREAM_CONFIG_FILE}" \
|
|
636
|
+
--litestream-pid-file "${LITESTREAM_PID_FILE}" \
|
|
637
|
+
--session-db-no-replicate-marker "${SESSION_DB_NO_REPLICATE_MARKER}" \
|
|
638
|
+
--credential-sync-marker "${CREDENTIAL_FLUSH_MARKER_FILE}" \
|
|
639
|
+
"${credential_flags[@]}" \
|
|
640
|
+
"${session_db_flags[@]}" \
|
|
641
|
+
"${opencode_config_flags[@]}" \
|
|
1231
642
|
--idle-timeout "${IDLE_TIMEOUT_SECONDS}" \
|
|
1232
643
|
--enable-file-sync-to "${HOME}/.claude" &
|
|
1233
644
|
|
|
@@ -1239,19 +650,17 @@ start_tunnel() {
|
|
|
1239
650
|
# whole seconds: 25 s in-flight drain (SHUTDOWN_DRAIN_TIMEOUT_MS,
|
|
1240
651
|
# apps/cli/src/commands/run.ts) + 2 s offline POST (notifyAgentDisconnected,
|
|
1241
652
|
# apps/cli/src/commands/agent-lookup.ts) + 5 s telemetry flush
|
|
1242
|
-
# (TELEMETRY_SHUTDOWN_TIMEOUT_MS, run.ts)
|
|
1243
|
-
#
|
|
1244
|
-
#
|
|
1245
|
-
#
|
|
653
|
+
# (TELEMETRY_SHUTDOWN_TIMEOUT_MS, run.ts) + 8.5 s pre-drain credential flush
|
|
654
|
+
# + 8.5 s post-drain credential flush. Each phase is bounded there, so this is
|
|
655
|
+
# a ceiling rather than a typical cost — an idle suspend finishes in a couple
|
|
656
|
+
# of seconds. This is the ONE place the hand-maintained budget is written down;
|
|
657
|
+
# the CLI points back here when its bounds change.
|
|
1246
658
|
#
|
|
1247
|
-
# DOCUMENTATION ONLY — nothing is derived from this
|
|
1248
|
-
#
|
|
1249
|
-
#
|
|
1250
|
-
# backstop chosen on its own merits and the two numbers are unrelated. Nothing
|
|
1251
|
-
# cross-checks the 32 against apps/cli either: it is a hand-maintained sum of the
|
|
1252
|
-
# three bounds cited above, so if one of them moves, update it here.
|
|
659
|
+
# DOCUMENTATION ONLY — nothing is derived from this value. It is the hand-maintained
|
|
660
|
+
# sum of the five bounded phases above (25 + 2 + 5 + 8.5 + 8.5); update it here if
|
|
661
|
+
# any of those bounds changes.
|
|
1253
662
|
# shellcheck disable=SC2034 # documentation; deliberately read by nothing
|
|
1254
|
-
CLI_SHUTDOWN_CEILING_SECONDS=
|
|
663
|
+
CLI_SHUTDOWN_CEILING_SECONDS=49
|
|
1255
664
|
|
|
1256
665
|
# How long stop_tunnel waits for that shutdown before the SIGKILL backstop.
|
|
1257
666
|
# Since #718 this binds ONLY for a CLI that is still draining: one that has
|
|
@@ -1327,6 +736,85 @@ stop_tunnel() {
|
|
|
1327
736
|
log "tunnel stopped ${outcome}"
|
|
1328
737
|
}
|
|
1329
738
|
|
|
739
|
+
# The CLI owns the interval loop and its boundary flush. Two seconds of slack
|
|
740
|
+
# over the CLI's 8s flush deadline keeps this marker handshake inside the 55s
|
|
741
|
+
# hook ceiling while leaving the fallback flush and tunnel stop budget intact.
|
|
742
|
+
#
|
|
743
|
+
# This bounds only the CLI's pre-drain flush, which always runs first and
|
|
744
|
+
# unconditionally (run.ts's cleanup(), before the channel-work drain) — not
|
|
745
|
+
# the CLI's post-drain second pass, which can take up to
|
|
746
|
+
# SHUTDOWN_DRAIN_TIMEOUT_MS longer than this wait covers. A drain that
|
|
747
|
+
# consumes the whole window loses the SECOND pass, not the credential
|
|
748
|
+
# guarantee itself: the pre-drain flush already persisted everything on disk
|
|
749
|
+
# at signal time, exactly what the old bash `sync_credentials` guaranteed in
|
|
750
|
+
# one synchronous call — so the worst case here is no worse than before this
|
|
751
|
+
# handshake existed, never a fresh data-loss window. See
|
|
752
|
+
# docs/decisions/0063-microvm-boot-orchestration-in-cli.md's two-phase-flush
|
|
753
|
+
# section for the full reasoning.
|
|
754
|
+
CREDENTIAL_FLUSH_WAIT_SECONDS="${EVIDENT_CREDENTIAL_FLUSH_WAIT_SECONDS:-10}"
|
|
755
|
+
|
|
756
|
+
# Remove the previous answer, signal the same CLI that stop_tunnel handles, and
|
|
757
|
+
# wait for either its marker or its death. A fallback sync runs only after the
|
|
758
|
+
# CLI is known to be absent, never alongside a live CLI that may still write.
|
|
759
|
+
stop_runner_and_flush_credentials() {
|
|
760
|
+
rm -f "${CREDENTIAL_FLUSH_MARKER_FILE}"
|
|
761
|
+
|
|
762
|
+
if ! tunnel_is_running; then
|
|
763
|
+
warn "CREDS-FLUSH-NO-RUNNER: no live CLI at handshake entry"
|
|
764
|
+
stop_tunnel
|
|
765
|
+
sync_credentials
|
|
766
|
+
return 0
|
|
767
|
+
fi
|
|
768
|
+
|
|
769
|
+
local pid
|
|
770
|
+
pid="$(cat "${TUNNEL_PID_FILE}")"
|
|
771
|
+
kill -TERM "${pid}" 2>/dev/null || true
|
|
772
|
+
|
|
773
|
+
local waited_ms=0 marker_found=false runner_exited=false
|
|
774
|
+
while [ "${waited_ms}" -lt $((CREDENTIAL_FLUSH_WAIT_SECONDS * 1000)) ]; do
|
|
775
|
+
if [ -s "${CREDENTIAL_FLUSH_MARKER_FILE}" ]; then
|
|
776
|
+
marker_found=true
|
|
777
|
+
break
|
|
778
|
+
fi
|
|
779
|
+
if ! process_is_alive "${pid}"; then
|
|
780
|
+
runner_exited=true
|
|
781
|
+
break
|
|
782
|
+
fi
|
|
783
|
+
sleep 0.1
|
|
784
|
+
waited_ms=$((waited_ms + 100))
|
|
785
|
+
done
|
|
786
|
+
|
|
787
|
+
# The CLI can publish the marker and exit inside one poll tick. A final
|
|
788
|
+
# marker check after a death break preserves that answer instead of falling
|
|
789
|
+
# through to the fallback.
|
|
790
|
+
if [ "${runner_exited}" = true ] && [ -s "${CREDENTIAL_FLUSH_MARKER_FILE}" ]; then
|
|
791
|
+
marker_found=true
|
|
792
|
+
runner_exited=false
|
|
793
|
+
fi
|
|
794
|
+
|
|
795
|
+
if [ "${marker_found}" = true ]; then
|
|
796
|
+
local failures
|
|
797
|
+
failures="$(grep -Ev '^(claude|opencode)=ok$' "${CREDENTIAL_FLUSH_MARKER_FILE}" || true)"
|
|
798
|
+
if [ -n "${failures}" ]; then
|
|
799
|
+
warn "CREDS-FLUSH-HAD-FAILURES: ${failures//$'\n'/ }"
|
|
800
|
+
else
|
|
801
|
+
log "CREDS-FLUSH-OK"
|
|
802
|
+
fi
|
|
803
|
+
stop_tunnel
|
|
804
|
+
return 0
|
|
805
|
+
fi
|
|
806
|
+
|
|
807
|
+
if [ "${runner_exited}" = true ]; then
|
|
808
|
+
warn "CREDS-FLUSH-RUNNER-EXITED: CLI exited without a completion marker"
|
|
809
|
+
stop_tunnel
|
|
810
|
+
sync_credentials
|
|
811
|
+
return 0
|
|
812
|
+
fi
|
|
813
|
+
|
|
814
|
+
warn "CREDS-FLUSH-TIMEOUT: live CLI did not write a completion marker within ${CREDENTIAL_FLUSH_WAIT_SECONDS}s"
|
|
815
|
+
stop_tunnel
|
|
816
|
+
}
|
|
817
|
+
|
|
1330
818
|
# --- check_runner_key (#1172) ------------------------------------------------
|
|
1331
819
|
#
|
|
1332
820
|
# Answers exactly one question before opencode/the tunnel start spending this
|
|
@@ -1360,10 +848,9 @@ check_runner_key() {
|
|
|
1360
848
|
local runner_key="$1" api_url="$2"
|
|
1361
849
|
|
|
1362
850
|
# status.ts's own exit-code contract (its header comment) means this exits
|
|
1363
|
-
# non-zero on EVERY branch except `ok
|
|
1364
|
-
#
|
|
1365
|
-
#
|
|
1366
|
-
# caller's `set -e` before the case below ever runs. Only stdout is
|
|
851
|
+
# non-zero on EVERY branch except `ok`; guard the capture so an expected
|
|
852
|
+
# non-zero does not abort this function under the caller's `set -e` before
|
|
853
|
+
# the case below ever runs. Only stdout is
|
|
1367
854
|
# captured: status.ts's own contract is one parseable JSON line and nothing
|
|
1368
855
|
# else there, and — like litestream's stderr elsewhere in this file — its
|
|
1369
856
|
# stderr is left to reach CloudWatch directly rather than being folded in,
|