kotovalexarian-likes-github/moby--moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
Code Releases Activity

Macvlan: Separate empty parent and internal

Browse source 
https://github.com/docker/libnetwork/pull/2419 and
https://github.com/docker/libnetwork/pull/2407
attempted to seperate out empty parent and internal for
macvlan and ipvlan networks

However it didnt pass the integration tests in moby
https://github.com/moby/moby/pull/40596 and exposed some
more plumbing that needed to be done to make sure
we separate the two things

If the -o parent is empty we create a dummylink
and if internal is set we dont add a default gateway
and make sure north-south communication cannot take place
(only east-west / container-container can)

Signed-off-by: Arko Dasgupta <arko.dasgupta@docker.com>
This commit is contained in:
Arko Dasgupta 2020-03-03 14:42:42 -08:00
parent 6a176585c6
commit 76b5905cbe
2 changed files with 11 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
libnetwork/drivers/ipvlan/ipvlan_network.go
View file

@ -57,8 +57,6 @@ func (d *driver) CreateNetwork(nid string, option map[string]interface{}, nInfo
// if parent interface not specified, create a dummy type link to use named dummy+net_id // if parent interface not specified, create a dummy type link to use named dummy+net_id
if config.Parent == "" { if config.Parent == "" {
config.Parent = getDummyName(stringid.TruncateID(config.ID)) config.Parent = getDummyName(stringid.TruncateID(config.ID))
// empty parent and --internal are handled the same. Set here to update k/v
config.Internal = true
} }
foundExisting, err := d.createNetwork(config) foundExisting, err := d.createNetwork(config)
if err != nil { if err != nil {
@ -95,19 +93,17 @@ func (d *driver) createNetwork(config *configuration) (bool, error) {
} }
} }
if !parentExists(config.Parent) { if !parentExists(config.Parent) {
// if the --internal flag is set, create a dummy link // Create a dummy link if a dummy name is set for parent
if config.Internal { if dummyName := getDummyName(stringid.TruncateID(config.ID)); dummyName == config.Parent {
err := createDummyLink(config.Parent, getDummyName(stringid.TruncateID(config.ID))) err := createDummyLink(config.Parent, dummyName)
if err != nil { if err != nil {
return false, err return false, err
} }
config.CreatedSlaveLink = true config.CreatedSlaveLink = true
// notify the user in logs they have limited communications // notify the user in logs they have limited communications
if config.Parent == getDummyName(stringid.TruncateID(config.ID)) { logrus.Debugf("Empty -o parent= flags limit communications to other containers inside of network: %s",
logrus.Debugf("Empty -o parent= and --internal flags limit communications to other containers inside of network: %s", config.Parent)
config.Parent)
}
} else { } else {
// if the subinterface parent_iface.vlan_id checks do not pass, return err. // if the subinterface parent_iface.vlan_id checks do not pass, return err.
// a valid example is 'eth0.10' for a parent iface 'eth0' with a vlan id '10' // a valid example is 'eth0.10' for a parent iface 'eth0' with a vlan id '10'

16
libnetwork/drivers/macvlan/macvlan_network.go
View file

@ -61,8 +61,6 @@ func (d *driver) CreateNetwork(nid string, option map[string]interface{}, nInfo
// if parent interface not specified, create a dummy type link to use named dummy+net_id // if parent interface not specified, create a dummy type link to use named dummy+net_id
if config.Parent == "" { if config.Parent == "" {
config.Parent = getDummyName(stringid.TruncateID(config.ID)) config.Parent = getDummyName(stringid.TruncateID(config.ID))
// empty parent and --internal are handled the same. Set here to update k/v
config.Internal = true
} }
foundExisting, err := d.createNetwork(config) foundExisting, err := d.createNetwork(config)
if err != nil { if err != nil {
@ -100,18 +98,16 @@ func (d *driver) createNetwork(config *configuration) (bool, error) {
} }
} }
if !parentExists(config.Parent) { if !parentExists(config.Parent) {
// if the --internal flag is set, create a dummy link // Create a dummy link if a dummy name is set for parent
if config.Internal { if dummyName := getDummyName(stringid.TruncateID(config.ID)); dummyName == config.Parent {
err := createDummyLink(config.Parent, getDummyName(stringid.TruncateID(config.ID))) err := createDummyLink(config.Parent, dummyName)
if err != nil { if err != nil {
return false, err return false, err
} }
config.CreatedSlaveLink = true config.CreatedSlaveLink = true
// notify the user in logs they have limited communications // notify the user in logs that they have limited communications
if config.Parent == getDummyName(stringid.TruncateID(config.ID)) { logrus.Debugf("Empty -o parent= limit communications to other containers inside of network: %s",
logrus.Debugf("Empty -o parent= and --internal flags limit communications to other containers inside of network: %s", config.Parent)
config.Parent)
}
} else { } else {
// if the subinterface parent_iface.vlan_id checks do not pass, return err. // if the subinterface parent_iface.vlan_id checks do not pass, return err.
// a valid example is 'eth0.10' for a parent iface 'eth0' with a vlan id '10' // a valid example is 'eth0.10' for a parent iface 'eth0' with a vlan id '10'