diff --git a/decisions/2026-09-30-plex-quicksync.md b/decisions/2026-09-30-plex-quicksync.md index 4e9966f..35b9241 100644 --- a/decisions/2026-09-30-plex-quicksync.md +++ b/decisions/2026-09-30-plex-quicksync.md @@ -77,6 +77,31 @@ does not here). No `podSecurityContext`. No `nodeSelector`. No alert rule. No `k documented history of that going badly (a prod node hung on a warm reboot on 2026-09-03). Kept well clear of this PR. +### Rebase note (2026-10-01, Ruth Okonkwo) — resolution only, no redesign + +Rebased onto `main` @ `08deb20` (was cut from `0fbf544`), restacked on the rebased +`feat/sd-hardware-capability` (PR #27) so the stack stays coherent — this branch still carries +exactly PR #27's commit plus its own, with no duplicate of anything already in `main`. The branch +ref was force-updated with `--force-with-lease`, so PR #32 is the same PR with a new head. + +**The `catalog/render.py` conflict disappeared** once PR #27 was rebased; it was never this +branch's own conflict. **One conflict remained, in `runbooks/README.md`, purely textual.** PR #30 +(`fix/jackett-oauth2-proxy-session`, merged as `08deb20`) appended an `oauth2-proxy-session` index +entry under *Operator procedures* at the same point this branch appended its `plex-quicksync` entry. +**Both entries were kept, in full.** Ordering follows this file's existing convention, which is +grouped/narrative rather than alphabetical (the *Nodes* section runs down/io/memory/disk/cpu): +`plex-quicksync` sits next to `plex-claim` as the other Plex page, leaving `oauth2-proxy-session` +adjacent to `crafty-admin-credential`, which cross-references the same sidecar gate. The change to +`runbooks/README.md` is an 8-line pure insertion, so PR #30's entry is untouched. +`runbooks/kustomization.yaml` auto-merged and **both** `configMapGenerator` entries are present. + +Re-verified after the resolution, on the rebased stack: `--selftest` PASS (rc 0); a full re-render +reproduces every committed golden byte-identically; and the **only** `apps/` diff against the new +`main` is `apps/wave-0/plex.yaml`, confined to the expected six lines (the `dri` mount and its +`DirectoryOrCreate` hostPath volume). No other service's golden moved — PR #28 added ~176 lines of +oauth2-proxy sidecar rendering, and this confirms by measurement rather than assumption that Plex +(`auth: native`, no sidecar) is unaffected by it. + ## What I verified - **The render is exactly the intended change, and nothing else moved.** After