Skip to content

refactor(cli): remove gateway lifecycle management#1221

Open
drew wants to merge 4 commits intomainfrom
remove-gateway-management
Open

refactor(cli): remove gateway lifecycle management#1221
drew wants to merge 4 commits intomainfrom
remove-gateway-management

Conversation

@drew
Copy link
Copy Markdown
Collaborator

@drew drew commented May 6, 2026

Summary

Remove CLI-owned gateway service lifecycle commands and bootstrap fallback so package/systemd/Helm flows own gateway start/stop/destruction.

Related Issue

N/A

Changes

  • Removed openshell gateway start|stop|destroy, sandbox auto-bootstrap fallback, and the old doctor LLM prompt.
  • Added openshell gateway remove [name] for local registration cleanup.
  • Pruned openshell-bootstrap to registration/auth/PKI/build helpers and removed Docker runtime/image push bootstrap code.
  • Updated docs, architecture notes, skills, RPM troubleshooting, man pages, and the RPM spec for package-owned gateway lifecycle.

Testing

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

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew drew requested review from a team, derekwaynecarr, maxamillion and mrunalp as code owners May 6, 2026 23:19
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 6, 2026

TaylorMutch
TaylorMutch previously approved these changes May 6, 2026
@johntmyers
Copy link
Copy Markdown
Collaborator

Signed-off-by: Drew Newberry <anewberry@nvidia.com>
@drew drew added the test:e2e Requires end-to-end coverage label May 7, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

Label test:e2e applied for 2d320c4. Open the existing run and click Re-run all jobs to execute with the label set. The E2E Gate check on this PR will flip green automatically once the run finishes.

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

Labels

test:e2e Requires end-to-end coverage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants