Repository navigation
Bump unbounded to v0.18.0 - #410
Merged
Merged
Conversation
v0.18.0 brings Azure/unbounded#962, which stops preflight warning that docker-active's and kubelet-active's service state "could not be determined" on every host where the service isn't running, and #967, which lets the agent supply the nspawn lifecycle hooks' commands instead of the helper unbounded copies onto the host. Nothing here supplies them yet. It also has NVIDIA nspawn nodes use the CDI runtime (#975). No code changes are needed.
…n it The move to the host root installs the nspawn lifecycle helper before it rewrites the active machine's hooks. With unbounded v0.18.0 a machine's config can supply the hooks' commands, and then its hooks don't run the helper. EnsureNSpawnLifecycleHelperFor installs it only for a machine whose hooks run it, as EnsureNSpawnConfig expects, so the move doesn't put a helper back on a host whose config supplies the commands. Until one does, nothing changes.
Philip Lombardi (plombardi89)
deployed
to
e2e-testing
October 23, 2026 17:10 — with
GitHub Actions
Active
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
Preflight still uses strict NVIDIA resolution, bypassing diagnostic reports and ignored-check handling when GPU readiness fails.
1 open issue
What changed in this PR
Updates the agent to unbounded v0.18.0 for preflight diagnostics, NVIDIA handling, and configurable nspawn lifecycle hooks, preparing for the containerized ACL agent work in #370.
Changes:
- Bumps unbounded and related
golang.org/xdependencies. - Refreshes dependency checksums.
- Makes host-root migration respect configured lifecycle commands when installing the helper.
| File | Description |
|---|---|
pkg/daemon/hostroot_move.go |
Uses the config-aware lifecycle helper installer and explains its behavior. |
go.sum |
Updates checksums for the dependency upgrade. |
go.mod |
Upgrades unbounded to v0.18.0 and related indirect dependencies. |
Review effort: Balanced
hbc (bcho)
approved these changes
Oct 23, 2026
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Bumps
github.com/Azure/unboundedfrom v0.16.0 to v0.18.0. Onlygolang.org/xmodules move with it, and unbounded's minimum Go is still 1.26.10.For the agent, v0.18.0 brings:
docker-activeandkubelet-activechecks no longer warn that the service's state "could not be determined" on every host where the service isn't running, Ubuntu included.It builds without code changes. One follows from #967: the move to the host root installs the helper with
EnsureNSpawnLifecycleHelperFor, rather thanEnsureNSpawnLifecycleHelper. That way the move doesn't put a helper back on a host whose config supplies the hooks' commands. Until a config does, it installs the helper as before.Testing
pkg/daemonandpkg/config, andgolangci-lintpass.