feat(netconfig): extract standalone interface bootstrap - #1
Merged
Merged
Conversation
borislitv
force-pushed
the
feat/extract-netconfig
branch
from
September 14, 2026 16:33
cb097b9 to
10fecd3
Compare
borislitv
marked this pull request as ready for review
September 17, 2026 12:12
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.
Summary
-onceand-checkmodes, scratch runtime images for Linux amd64/arm64, CI and GHCR publication workflows.go.yaml.in/yaml/v33.0.5. Other selected libraries are already at their latest stable versions.Execution contract and simplification
Current HEAD:
5767486b44a0b11aff981134e1c3aa938ff18a8c.::1on non-loopback interfaces are rejected before kernel access.internal/config, sharing the field parser while keeping strict native restrictions and Netplan compatibility explicit. Remove the intermediate native configuration model and conversion pass.The parser follow-up at
85238823ae8760298108ab18da886ece1c87f56aremoved 551 net lines relative to3e12a48. At that revision, compared with the initial extraction at10fecd3, this was 1012 fewer lines, comprising 482 fewer production Go lines, 546 fewer test lines, and 16 added documentation/build/dependency lines. The suite is 26 top-level tests / 240 leaf scenarios, versus 43 / 381 initially.Scratch runtime
FROM scratch; it contains theCGO_ENABLED=0executable and LICENSE/NOTICE only. Go 1.27.1 build/test stages are unchanged./proc/sys.-checkpassed with no network/capabilities and a read-only root filesystem.Verification
go vet, module checksum verification, gofmt and diff whitespace checks.go test -race -count=1 -p=1 ./..., with both opt-in environment variables enabled.testing/synctestverifies actual backoff timing and cancellation during the retry timer; a constant-backoff mutation fails the test.