From 488eb3b74cd169e94de8364294586bce0925b710 Mon Sep 17 00:00:00 2001
From: Jake Correnti <jcorrenti13@gmail.com>
Date: Wed, 8 Jun 2022 10:45:37 -0400
Subject: (rootful) docker-compose now updates network MTU

Previously, the following network block did not update using
docker-compose:

```
networks:
  default:
    driver: bridge
    driver_opts:
      mtu: 9000
```

In the API, the network options were previously not being handled when the
network was being created. I translated the docker options into podman
options, and added the options to the network.

When doing `podman network inspect <network>`, the results now contain
`"mtu": "9000"`

Fixes: #14482

Signed-off-by: Jake Correnti <jcorrenti13@gmail.com>
---
 pkg/api/handlers/compat/networks.go | 46 ++++++++++++++++++++++++++++++++-----
 1 file changed, 40 insertions(+), 6 deletions(-)

(limited to 'pkg/api/handlers')

diff --git a/pkg/api/handlers/compat/networks.go b/pkg/api/handlers/compat/networks.go
index 6fdd5c6a7..9da21d15f 100644
--- a/pkg/api/handlers/compat/networks.go
+++ b/pkg/api/handlers/compat/networks.go
@@ -161,8 +161,9 @@ func ListNetworks(w http.ResponseWriter, r *http.Request) {
 
 func CreateNetwork(w http.ResponseWriter, r *http.Request) {
 	var (
-		networkCreate types.NetworkCreateRequest
-		network       nettypes.Network
+		networkCreate   types.NetworkCreateRequest
+		network         nettypes.Network
+		responseWarning string
 	)
 	runtime := r.Context().Value(api.RuntimeKey).(*libpod.Runtime)
 	if err := json.NewDecoder(r.Body).Decode(&networkCreate); err != nil {
@@ -179,8 +180,40 @@ func CreateNetwork(w http.ResponseWriter, r *http.Request) {
 	network.Internal = networkCreate.Internal
 	network.IPv6Enabled = networkCreate.EnableIPv6
 
-	// FIXME use docker options and convert them to valid libpod options
-	// network.Options = networkCreate.Options
+	network.Options = make(map[string]string)
+
+	// TODO: we should consider making this constants in c/common/libnetwork/types
+	for opt, optVal := range networkCreate.Options {
+		switch opt {
+		case "mtu":
+			fallthrough
+		case "com.docker.network.driver.mtu":
+			if network.Driver == nettypes.BridgeNetworkDriver {
+				network.Options["mtu"] = optVal
+			}
+		case "icc":
+			fallthrough
+		case "com.docker.network.bridge.enable_icc":
+			// TODO: needs to be implemented
+			if network.Driver == nettypes.BridgeNetworkDriver {
+				responseWarning = "com.docker.network.bridge.enable_icc is not currently implemented"
+			}
+		case "com.docker.network.bridge.name":
+			if network.Driver == nettypes.BridgeNetworkDriver {
+				network.NetworkInterface = optVal
+			}
+		case "mode":
+			if network.Driver == nettypes.MacVLANNetworkDriver || network.Driver == nettypes.IPVLANNetworkDriver {
+				network.Options[opt] = optVal
+			}
+		case "parent":
+			if network.Driver == nettypes.MacVLANNetworkDriver || network.Driver == nettypes.IPVLANNetworkDriver {
+				network.NetworkInterface = optVal
+			}
+		default:
+			responseWarning = "\"" + opt + ": " + optVal + "\" is not a recognized option"
+		}
+	}
 
 	// dns is only enabled for the bridge driver
 	if network.Driver == nettypes.BridgeNetworkDriver {
@@ -242,9 +275,10 @@ func CreateNetwork(w http.ResponseWriter, r *http.Request) {
 
 	body := struct {
 		ID      string `json:"Id"`
-		Warning string
+		Warning string `json:"Warning"`
 	}{
-		ID: newNetwork.ID,
+		ID:      newNetwork.ID,
+		Warning: responseWarning,
 	}
 	utils.WriteResponse(w, http.StatusCreated, body)
 }
-- 
cgit v1.2.3-54-g00ecf