test(conformance): verify Snap installation status in Ubuntu VM - #3182
test(conformance): verify Snap installation status in Ubuntu VM#3182elezar wants to merge 5 commits into
Conversation
2f44036 to
da4604b
Compare
|
🌿 Preview your docs: https://nvidia-preview-pr-3182.docs.buildwithfern.com/openshell |
b52d6c4 to
44549bd
Compare
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
Signed-off-by: Evan Lezar <elezar@nvidia.com>
2332636 to
c3e9dfe
Compare
| # Do not start before the Docker interface is available. The | ||
| # connect-plug-docker hook enables and starts the gateway after Docker | ||
| # access is granted, including for store-managed auto-connections. | ||
| install-mode: disable |
There was a problem hiding this comment.
@olivercalder I was iterating on this, and it was impossible to get the package installed and connected without this change. Since the gateway service is installed in a started state, it crash loops because it can't discover a valid compute driver. At the point where the sudo snap connect openshell:docker docker:docker-daemon tries to restart the gateway in the connect-plug-docker hook, the restart is rejected because (as Codex proposes) some restart limit has been reached.
The argument of switching to install-mode: disable is then that the service will be started (and enabled) at the point where the docker plug is connected -- either automatically for installations from the Snap store directly, or manually in the case of testing.
I'm not quite certain why the existing canary infrastructure allows things to pass, but it could be that the VM-based tooling that we have now adds some other delay which allows the gateway to restart more often.
Summary
Add a VM-based Ubuntu Snap installation check to the conformance workflow. It mirrors the Snap release canary through gateway registration and
openshell status, without creating a sandbox.Related Issue
No issue required: CI packaging coverage and workflow reuse.
Changes
openshell status.Testing
mise run pre-commitpassesChecklist