Skip to content

feat(snap): add snap install method to install.sh when snap is available#1697

Draft
zyga wants to merge 2 commits into
NVIDIA:mainfrom
zyga:feature/install
Draft

feat(snap): add snap install method to install.sh when snap is available#1697
zyga wants to merge 2 commits into
NVIDIA:mainfrom
zyga:feature/install

Conversation

@zyga
Copy link
Copy Markdown
Contributor

@zyga zyga commented Jun 2, 2026

Summary

Update the installation script to use the snap package if snapd is available on the system and conditions are met. The installation instructions are updated to match.

Related Issue

#1674

Changes

  • The install script detects the presence of snapd and if compatible conditions (docker) are met, installs the snap from the store.

Testing

  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)
  • The install.sh script was tested in a LXD container

Checklist

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

zyga added 2 commits June 2, 2026 20:42
Add snap install support: resolve_linux_install_method auto-selects snap
when snapd is available, with OPENSHELL_INSTALL_METHOD=classic fallback.
Add preflight checks for Docker conflicts and interface connections.
Update test suite to cover has_snapd, resolve_snap_channel, and
resolve_linux_install_method functions.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
…lassic

Add full Snap installation documentation including interface connections,
gateway verification, and when to choose Snap vs classic packages.
Reword references from 'native package' to 'classic' throughout.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot Bot commented Jun 2, 2026

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.

@drew drew changed the title Feature/install feat(snap): add snap install method to install.sh when snap is available Jun 3, 2026
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.

1 participant