Skip to content

feat(suid_helper): ChrootJail.grant_api/1 wrapper for the grant-api op#46

Merged
markovejnovic merged 1 commit into
mainfrom
feat/grant-api-wrapper
Jun 30, 2026
Merged

feat(suid_helper): ChrootJail.grant_api/1 wrapper for the grant-api op#46
markovejnovic merged 1 commit into
mainfrom
feat/grant-api-wrapper

Conversation

@markovejnovic

@markovejnovic markovejnovic commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Thin wrapper over the `chroot-jail grant-api` helper subcommand (landed in
#45). Hands the firecracker API socket to the node user so the unprivileged
controller can connect(); maps the helper's "pending" result (firecracker
has not created the socket yet) to {:error, :socket_pending} so the caller
can keep polling, and "granted" to :ok.

No caller on main yet -- the FireVMM controller that polls this lands with
the boot path.
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
lib/hyper/suid_helper/chroot_jail.ex 0.00% 3 Missing ⚠️

📢 Thoughts on this report? Let us know!

@github-actions

Copy link
Copy Markdown

Test Results

337 tests  ±0   336 ✅ ±0   6s ⏱️ +2s
 60 suites ±0     1 💤 ±0 
  2 files   ±0     0 ❌ ±0 

Results for commit 4d4f6bd. ± Comparison against base commit a25b3ac.

@markovejnovic markovejnovic merged commit 7bccb8f into main Jun 30, 2026
6 checks passed
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