Conversation
|
🌿 Preview your docs: https://nvidia-preview-pr-3244.docs.buildwithfern.com/openshell |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
Label |
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
The startup repair that creates version-one policy history for legacy sandboxes propagated validation failures, so a single stored policy that no longer passes current validation rules prevented the gateway from starting. Skip such sandboxes with a warning and a completion summary so they keep the pre-repair behavior where only their own configuration reads report the failure. Store errors remain fatal. Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Fleet-wide configuration changes spawned one snapshot build per connected sandbox and component with no concurrency limit, so a global setting or provider change issued every store query and credential-driver call at once. Gate builds behind a semaphore sized from the database pool and start the build deadline only once a permit is held. Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Sandboxes keep their supervisor binary until they are recreated, so a gateway upgrade meets supervisors that predate the handshake and report revision zero. Rejecting them severs every running sandbox with no automatic recovery. Accept revision zero for one release, log a warning per session, and count them in openshell_supervisor_protocol_legacy_sessions_total. The supervisor mirrors the allowance for gateways that predate the handshake. Add a shared ConnectSupervisor test harness and handler-level tests for legacy acceptance and unknown-revision rejection. Move the skill troubleshooting paragraph out of the numbered deployment list so the list renders. Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
e7729ec to
70773d3
Compare
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
pimlock
left a comment
There was a problem hiding this comment.
gator-agent
PR Review Status
I reconciled the current head against the full Gator feedback ledger and reviewed the author-only delta since e7729ecc. The earlier fanout-bounding obligation remains resolved, and this follow-up found no blocking findings. Required checks, including the dispatched E2E suite, are green.
Action required: a maintainer must review and approve the PR.
Blocking findings:
- No blocking findings remain
Carried findings:
- None
Gator metadata
- Validation: Project-valid Stage 1 of #1731; the current implementation remains within the reviewed gateway/supervisor configuration-delivery scope.
- Docs: Architecture, gateway reference, and troubleshooting guidance are updated for the current behavior.
- Checks: OpenShell Branch Checks, Helm Lint, Trivy Changes, and E2E are green for the current head.
- E2E:
test:e2eis applied; Branch E2E Checks run34994317434completed successfully. - Head SHA:
8bf93eadd823357a2adaed5621b63dca63e6dc91 - Base SHA:
c195e23267dc14becf817693252e0749a2021878 - Merge base SHA:
c195e23267dc14becf817693252e0749a2021878 - Patch ID:
eb11bcc278607383beef1a0a1417534b6fa6645e - Gator payload:
8 - Review mode:
follow_up - Previous reviewed SHA:
e7729ecca7d22751fa759e737bce2b53e33aedb8 - Review budget exhausted: no
- Maintainer decision required: no
- Next state:
gator:approval-needed
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
|
All contributors have signed the DCO ✍️ ✅ |
7f704dd to
43d7b28
Compare
Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
Summary
Add Stage 1 of gateway-pushed configuration over
ConnectSupervisor. The gateway sends complete sandbox configuration and provider-environment snapshots; supervisors receive them while polling remains authoritative.Configuration delivery flow
flowchart TD A[Configuration mutation] --> B[Commit authoritative state] B --> C[Notify scheduler of affected scope and component] C --> D[Coalesce notifications and build latest full snapshot] D --> E[Push snapshot over ConnectSupervisor] E --> F[Supervisor receives but ignores snapshot in Stage 1] B --> G[Existing polling path] G --> H[Supervisor applies configuration]Related Issue
Part of #1731. Replaces #2967. Stage 2 will apply snapshots and acknowledge revisions; Stage 3 will add durable completion semantics and remove polling.
Changes
Testing
Postgres-specific concurrency coverage requires
OPENSHELL_TEST_POSTGRES_URLand was not run.Checklist