feat(orchestrator): slice 13c(ii) — shared-gateway source-IP allocator #372
Closed
didericis-claude
wants to merge 1 commits from
orchestrator-gateway-net into orchestrator-gitgate-provision
pull from: orchestrator-gateway-net
merge into: didericis:orchestrator-gitgate-provision
didericis:main
didericis:post-build-image-smoke-test
didericis:lazy-backend-imports
didericis:orchestrator-agent-compose
didericis:orchestrator-gateway-ca
didericis:orchestrator-consolidated-launch
didericis:orchestrator-gateway-provision
didericis:orchestrator-gateway-network
didericis:orchestrator-client
didericis:orchestrator-gitgate-provision
didericis:orchestrator-registration
didericis:orchestrator-lifecycle
didericis:orchestrator-supervise-writers
didericis:orchestrator-supervise-multitenant
didericis:orchestrator-gitgate-multitenant
didericis:orchestrator-rename-gateway
didericis:orchestrator-slice8
didericis:orchestrator-slice7
didericis:orchestrator-slice6
didericis:prd-orchestrator
didericis:orchestrator-slice5
didericis:orchestrator-slice4
didericis:orchestrator-slice3
didericis:orchestrator-slice2
didericis:ci-kvm-runner
didericis:firecracker-backend
didericis:issue-330-cached-images
didericis:forge-native-integration
didericis:prd-smolmachines-linux
didericis:claude-forward-host-credentials
didericis:fold-orchestrator-subpackage
didericis:prd-egress-control-plane
didericis:manifest-break-import-cycle
didericis:dlp-supervise-quality-fixes
didericis:table-drive-dlp-tests
didericis:fix-integration-test-failures
didericis:fix/macos-container-relative-dockerfile
didericis:prd-0054-install-script
didericis:commit-bottle-state
didericis:pr-211
didericis:move-codex-auth-to-contrib
didericis:feat/pipelock-skip-scan-extensions
didericis:prd-0049-named-labelled-agents
didericis:harden-git-gate-shell-rendering
Labels
Clear labels
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Breaking change that won't be backward compatible
Something is not working
Documentation changes
Improve existing functionality
New functionality
This is security issue
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: didericis/bot-bottle#372
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "orchestrator-gateway-net"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Sub-slice of slice 13c (docker launch integration). Deterministic per-bottle address assignment on the consolidated gateway's shared docker network — the address the gateway uses as its attribution key.
What it does
next_free_ip(cidr, taken)— the lowest host address in the subnet not reserved and not already assigned:ipaddress.hosts();.1excluded here;taken= the gateway container's own address + every live bottle's (the caller reads these from the registry).NoFreeAddressErrorwhen the subnet is exhausted.Pure
ipaddresslogic — the docker-specific inputs (subnet CIDR vianetwork_inspect_cidr, the gateway container's address viadocker inspect) are gathered by the caller and passed in, so this stays unit-testable without docker.Why deterministic + pinned
The attribution invariant needs each bottle's source address to be stable and known at registration time; the launch step will
docker run --ip <allocated>so the gateway can key policy/supervise/git-gate on it. (On docker the L2 anti-spoof is weaker than Firecracker's/31+nft, which is why the identity token #366 is load-bearing here — tracked separately.)Verification
pyright 0 errors, pylint 9.83/10 (new files 10.00), unit suite green (1730 tests; the 13
test_sidecar_init/bin/sleeperrors are pre-existing NixOS-local noise).Stacked on #371.
🤖 Generated with Claude Code
Collapsed into the single stack PR #380 (per-host orchestrator, PRD 0070). Closing this slice PR; review + merge happens on #380.
Pull request closed