diff options
| author | Paul Holzinger <paul.holzinger@web.de> | 2020-12-16 20:28:07 +0100 |
|---|---|---|
| committer | Paul Holzinger <paul.holzinger@web.de> | 2020-12-16 20:59:45 +0100 |
| commit | edf0e918e32f4f109329881310b798f3b2c7defb (patch) | |
| tree | 56006ac73fd1462c5c7141f1b67567cb3cb19d83 /test/e2e/network_create_test.go | |
| parent | bacb2fc526dba7214e3c8420eccc5cf22bbb1a29 (diff) | |
| download | podman-edf0e918e32f4f109329881310b798f3b2c7defb.tar.gz podman-edf0e918e32f4f109329881310b798f3b2c7defb.tar.bz2 podman-edf0e918e32f4f109329881310b798f3b2c7defb.zip | |
Always add the default gateway to the cni config file
`podman network create` should always add a gateway to the cni config.
If no gateway is given use the first ip in the subnet. CNI does not require
the gateway field but we need it because of network inspect.
This worked with previous version but was dropped in Commit(e7a72d72fd59).
Fixes #8748
Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
Diffstat (limited to 'test/e2e/network_create_test.go')
| -rw-r--r-- | test/e2e/network_create_test.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/test/e2e/network_create_test.go b/test/e2e/network_create_test.go index 21b3074fc..7e9a18ab2 100644 --- a/test/e2e/network_create_test.go +++ b/test/e2e/network_create_test.go @@ -106,6 +106,7 @@ var _ = Describe("Podman network create", func() { Expect(bridgePlugin.IPAM.Routes[0].Dest).To(Equal("0.0.0.0/0")) Expect(bridgePlugin.IsGW).To(BeTrue()) Expect(bridgePlugin.IPMasq).To(BeTrue()) + Expect(bridgePlugin.IPAM.Ranges[0][0].Gateway).ToNot(BeEmpty()) Expect(portMapPlugin.Capabilities["portMappings"]).To(BeTrue()) }) @@ -153,6 +154,8 @@ var _ = Describe("Podman network create", func() { // JSON the bridge info bridgePlugin, err := genericPluginsToBridge(result["plugins"], "bridge") Expect(err).To(BeNil()) + // check that gateway is added to config + Expect(bridgePlugin.IPAM.Ranges[0][0].Gateway).To(Equal("10.11.12.1")) // Once a container executes a new network, the nic will be created. We should clean those up // best we can |