networking: apply CNI args to the default networks as well #2985
Conversation
// We don't do this earlier because we also load networks in other contexts | ||
for _, n := range nets { | ||
n.runtime.Args = e.netsLoadList.SpecificArgs(n.conf.Name) | ||
} |
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.
0.02$: I would've inserted a newline here
Please add a test for this functionality for
and checks if that IP has actually been allocated. |
8df0654
to
16bb179
Compare
Tests done - ready for review ( @steveej ) |
|
||
func TestNetIPConflict(t *testing.T) { | ||
NewNetIPConflictTest().Execute(t) | ||
} |
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.
Same tests imo should be added into rkt_net_kvm_test.go
These tests are only for first mentioned by @steveej bullet, but IMO they look really good. |
16bb179
to
bc077ba
Compare
@jellonek We already tested the non-default-network case before this PR. So we should have everything covered. |
Ack, but what about |
Ack. LGTM. |
Fixes #2980