-
Notifications
You must be signed in to change notification settings - Fork 46
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Logical Network Support #160
Conversation
ca73cfa
to
fc470c1
Compare
fef4698
to
957024d
Compare
957024d
to
3c392a0
Compare
3c392a0
to
aa524ff
Compare
// AddressSpace - The AddressSpace that contains an array of IP address ranges that can be used by subnets. | ||
AddressSpace *AddressSpace `json:"addressSpace,omitempty"` | ||
// DhcpOptions - The dhcpOptions that contains an array of DNS servers available to VMs deployed in the Logical network. | ||
DhcpOptions *DhcpOptions `json:"dhcpOptions,omitempty"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do we have dhcp options here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For LNET in SDN it is per subnet so I moved the dnsservers per subnet for LNET.
wssdsubnet.Dns = &wssdcommonproto.Dns{ | ||
Servers: *subnet.DhcpOptions.DNSServers, | ||
} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why did we decide to use dhcp options for moc-sdk but just dnsservers for cloudagent?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
to keep the contract similar to virtual network.
1408d8e
to
caf8f45
Compare
caf8f45
to
4709705
Compare
4709705
to
94c0c57
Compare
94c0c57
to
ac72c73
Compare
Design Spec: Link
Sample Test Commands: Added to the spec