docs: clarify mock-epp/llmd-ext-proc image scope in operations.md - #51
Open
jordigilh wants to merge 1 commit into
Open
docs: clarify mock-epp/llmd-ext-proc image scope in operations.md#51jordigilh wants to merge 1 commit into
jordigilh wants to merge 1 commit into
Conversation
The "Required local images" table read as if all provider/consumer gateways need locally-built `praxis-ai:llmd-ext-proc` and `praxis-ai-mock-epp` images. In practice only the generic `xtask env` harness (validate-all, verify-swim-mesh-three-node, verify-failover-under-lost-peer) uses those defaults; the named demos (grid-glb-demo, grid-combined-site, grid-llmd-pool-metrics) override them with published ghcr.io/praxis-proxy/grid-ai-rollup images via GRID_XTASK_GATEWAY_IMAGE/GRID_XTASK_MOCK_EPP_IMAGE and never touch an AI repository checkout. Also note that Containerfile.composed and a mock llm-d EPP server don't exist in the AI repository yet, so build-gateway-images --ai-repo can't produce either image today. That gap only blocks the generic-harness path (verify-failover-under-lost-peer); it doesn't affect the three named demos above. Cross-links ai#716 (tracking the gap) and ai#334 (the in-flight, other-team-owned ext_proc work it's soft-blocked on). Signed-off-by: Jordi Gil <jgil@redhat.com>
Member
|
@jordigilh thanks! 🥇 Just needs a rebase. |
nerdalert
approved these changes
Aug 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
While root-causing
ai#716(the SWIM-partition failover fixture's missing images), I found that this repo's own docs overstate the requirement and would send the next contributor down the same investigation.What's wrong
The "Required local images" table in
docs/architecture/operations.mdreads as if all provider/consumer gateways need locally-builtpraxis-ai:llmd-ext-procandpraxis-ai-mock-eppimages. In practice:xtask envharness (validate-all,verify-swim-mesh-three-node,verify-failover-under-lost-peer) uses those defaults.grid-glb-demo,grid-combined-site,grid-llmd-pool-metrics) override them viaGRID_XTASK_GATEWAY_IMAGE/GRID_XTASK_MOCK_EPP_IMAGE(seextask/src/env/image_overrides.rs) with publishedghcr.io/praxis-proxy/grid-ai-rollupimages and never touch an AI repository checkout.Containerfile.composednor a mock llm-d EPP server exists in the AI repository today, sobuild-gateway-images --ai-repocan't currently produce either image — this only blocks the generic-harness path (verify-failover-under-lost-peer), not the three named demos.Change
Added a clarifying note after the table, cross-linking
ai#716(tracking the gap) andai#334(the in-flight, other-team-ownedext_procwork it's soft-blocked on, deliberately paused per an Aug 10 AI Gateway release-timeline decision rather than abandoned).Test plan
Docs-only change, no code touched.