Skip to content

feat(gateway): add elevated gateway info#2202

Open
elezar wants to merge 1 commit into
mainfrom
feat/gateway-status-driver-info
Open

feat(gateway): add elevated gateway info#2202
elezar wants to merge 1 commit into
mainfrom
feat/gateway-status-driver-info

Conversation

@elezar

@elezar elezar commented Jul 9, 2026

Copy link
Copy Markdown
Member

Summary

Add an elevated gateway info surface for live runtime metadata, including the initialized compute driver name and driver-reported capability snapshot.

openshell status now remains aligned with main: it uses the existing Health RPC and HTTP fallback behavior. Runtime driver details are exposed through openshell info, backed by the admin-gated GetGatewayInfo RPC.

Related Issue

None.

Related PRs

Changes

  • Added GetGatewayInfo with gateway version, service status, and initialized compute driver info.
  • Snapshot driver identity/version during compute runtime initialization and return it from the new admin-authenticated RPC.
  • Added top-level openshell info -o table|json|yaml.
  • Restored openshell status to the behavior present on main.
  • Updated gateway docs and compute runtime architecture notes.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Attempted devenv shell -- mise --cd /home/elezar/dev/OpenShell run pre-commit through the host devenv at ~/.dotfiles/devenvs/openshell; it fails in this environment due to:

  • helm:lint: chart metadata missing dependency postgresql
  • rust:check/rust:lint: sqlx_macros dynamic library requires GLIBC_2.39
  • test:rust: openshell-supervisor-network process identity shared-socket test failed

Additional checks run:

  • nix develop -c cargo fmt --all
  • nix develop -c cargo check -p openshell-cli --tests
  • nix develop -c cargo check -p openshell-server --tests
  • nix develop -c cargo check -p openshell-core --tests
  • nix develop -c cargo test -p openshell-cli gateway_info_json -- --nocapture
  • nix develop -c cargo test -p openshell-cli info_accepts_output_json -- --nocapture
  • nix develop -c cargo test -p openshell-server --lib every_proto_rpc_has_an_annotation -- --nocapture
  • nix develop -c cargo test -p openshell-server --lib every_annotated_path_matches_a_real_rpc -- --nocapture
  • git diff --check

Manual smoke check against a branch-local Docker gateway:

target/debug/openshell info -o json
{
  "auth": null,
  "compute_drivers": [
    {
      "capabilities": {
        "driver_name": "docker",
        "driver_version": "29.2.1"
      },
      "name": "docker"
    }
  ],
  "gateway": "info-dev",
  "server": "http://127.0.0.1:19080",
  "status": "healthy",
  "version": "0.0.80-dev.7+g8fb2782a"
}

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

@elezar elezar requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners July 9, 2026 18:20
@copy-pr-bot

copy-pr-bot Bot commented Jul 9, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@elezar elezar force-pushed the feat/gateway-status-driver-info branch from 62b489b to 8fb2782 Compare July 9, 2026 18:43
@github-actions

Copy link
Copy Markdown

@elezar elezar changed the title feat(gateway): report compute driver status feat(gateway): add elevated gateway info Jul 10, 2026
@elezar elezar force-pushed the feat/gateway-status-driver-info branch from addc9c5 to decd2c0 Compare July 10, 2026 11:06
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar elezar force-pushed the feat/gateway-status-driver-info branch from decd2c0 to 6345c48 Compare July 10, 2026 11:39
TaylorMutch
TaylorMutch previously approved these changes Jul 10, 2026

@TaylorMutch TaylorMutch left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it 🚢

@TaylorMutch TaylorMutch dismissed their stale review July 10, 2026 15:39

Stale review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants