Skip to content

feat(gateway): add elevated gateway info#2202

Merged
elezar merged 2 commits into
mainfrom
feat/gateway-status-driver-info
Jul 15, 2026
Merged

feat(gateway): add elevated gateway info#2202
elezar merged 2 commits 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 remains aligned with main: it uses the existing Health RPC and HTTP fallback behavior. Runtime driver details are exposed through openshell gateway info, backed by the admin-gated GetGatewayInfo RPC.

This also moves the old local registration metadata formerly shown by openshell gateway info into openshell gateway list / openshell gateway list -o json.

Related Issue

None.

Related PRs

Changes

  • Folded local gateway registration metadata into openshell gateway list, including is_remote, remote_host, and resolved_host in structured output.
  • 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 openshell gateway info -o table|json|yaml.
  • Restored openshell status to the behavior present on main.
  • Updated gateway docs and compute runtime architecture notes.

Breaking Change

openshell gateway info no longer shows local gateway registration metadata. Use openshell gateway list or openshell gateway list -o json for local registration details.

Previous behavior from the globally installed openshell 0.0.75 CLI:

/home/elezar/.nix-profile/bin/openshell gateway info
Gateway Info

  Gateway: info-dev
  Gateway endpoint: http://127.0.0.1:19080

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 --check
  • nix develop -c cargo test -p openshell-cli gateway_list
  • nix develop -c cargo test -p openshell-cli info
  • git diff --check origin/main..HEAD

Manual smoke check against a branch-local Docker gateway:

target/debug/openshell gateway 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"
}

Example local registration metadata output:

target/debug/openshell gateway list
  NAME       ENDPOINT                 TYPE   SOURCE  AUTH
* info-dev   http://127.0.0.1:19080   local  user    plaintext
  openshell  https://127.0.0.1:17670  local  user    mtls
target/debug/openshell gateway list -o json
[
  {
    "active": true,
    "auth": "plaintext",
    "endpoint": "http://127.0.0.1:19080",
    "is_remote": false,
    "name": "info-dev",
    "remote_host": null,
    "resolved_host": null,
    "source": "user",
    "type": "local"
  },
  {
    "active": false,
    "auth": "mtls",
    "endpoint": "https://127.0.0.1:17670",
    "is_remote": false,
    "name": "openshell",
    "remote_host": null,
    "resolved_host": null,
    "source": "user",
    "type": "local"
  }
]

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 2 times, most recently 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

TaylorMutch
TaylorMutch previously approved these changes Jul 10, 2026
@elezar
elezar force-pushed the feat/gateway-status-driver-info branch 2 times, most recently from 4d4f910 to 0c776f8 Compare July 15, 2026 20:29
elezar added 2 commits July 15, 2026 22:41
BREAKING CHANGE: openshell gateway info no longer shows local gateway registration metadata. Use openshell gateway list or openshell gateway list -o json for local registration details.

Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
@elezar
elezar force-pushed the feat/gateway-status-driver-info branch from 0c776f8 to 9125b06 Compare July 15, 2026 20:41
@elezar
elezar added this pull request to the merge queue Jul 15, 2026
@elezar
elezar added this pull request to the merge queue Jul 15, 2026
Merged via the queue into main with commit 21aaa89 Jul 15, 2026
32 checks passed
@elezar
elezar deleted the feat/gateway-status-driver-info branch July 15, 2026 22:37
@pimlock pimlock mentioned this pull request Jul 15, 2026
6 tasks
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.

3 participants