Update technical-overview.md doc with current code by vadikgo · Pull Request #86 · apple/container
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 676
Merged
Update technical-overview.md doc with current code#86
jglogan merged 1 commit intoapple:mainfrom
Update technical-overview.md doc with current code#86
jglogan merged 1 commit intoapple:mainfrom
Conversation
Copy link
Contributor
vadikgo
commented
Jun 10, 2025
vadikgo
commented
com.apple.container.defaults default subnet key must be network.subnet, not default.subnet.
container/Sources/Services/ContainerNetworkService/AllocationOnlyVmnetNetwork.swift
Line 70 in ef4462e
| defaultSubnet = suite.string(forKey: "network.subnet") ?? defaultSubnet |
wlan0 approved these changes Jun 10, 2025
vadikgo
force-pushed
the
main
branch
from
b0701e0 to
383b74a
Compare
Copy link
Contributor
jglogan
commented
Jun 11, 2025
jglogan commented
Jun 11, 2025@vadikgo Merged, thank you for your contribution!
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
Labels
None yet
3 participants