Use cluster name as default subnet tag for Lyft CNI

kubernetes · Feb 16, 2020 · ff68fbe · ff68fbe
1 parent 5b38b9d
commit ff68fbe
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 26 deletions.
diff --git a/docs/networking.md b/docs/networking.md
@@ -545,7 +545,7 @@ You can specify which subnets to use for allocating Pod IPs by specifying
   networking:
     lyftvpc:
       subnetTags:
-        kubernetes_kubelet: true
+        KubernetesCluster: myclustername.mydns.io
 ```
 
 In this example, new interfaces will be attached to subnets tagged with `kubernetes_kubelet = true`.

diff --git a/...odeup/resources/_lyft_vpc_cni/files/etc/cni/net.d/10-cni-ipvlan-vpc-k8s.conflist.template b/...odeup/resources/_lyft_vpc_cni/files/etc/cni/net.d/10-cni-ipvlan-vpc-k8s.conflist.template
@@ -1,26 +1,26 @@
 {
+  "cniVersion": "0.3.1",
+  "name": "cni-ipvlan-vpc-k8s",
+  "plugins": [
+  {
     "cniVersion": "0.3.1",
-    "name": "cni-ipvlan-vpc-k8s",
-    "plugins": [
-	{
-	    "cniVersion": "0.3.1",
-	    "type": "cni-ipvlan-vpc-k8s-ipam",
-	    "interfaceIndex": 1,
-		"skipDeallocation" : true,
-	    "subnetTags":  {{ SubnetTags }},
-	    "secGroupIds": {{ NodeSecurityGroups }}
-	},
-	{
-	    "cniVersion": "0.3.1",
-	    "type": "cni-ipvlan-vpc-k8s-ipvlan",
-	    "mode": "l2"
-	},
-	{
-	    "cniVersion": "0.3.1",
-	    "type": "cni-ipvlan-vpc-k8s-unnumbered-ptp",
-	    "hostInterface": "eth0",
-	    "containerInterface": "veth0",
-	    "ipMasq": true
-	}
-    ]
-}
+    "type": "cni-ipvlan-vpc-k8s-ipam",
+    "interfaceIndex": 1,
+    "skipDeallocation": true,
+    "subnetTags":  {{ SubnetTags }},
+    "secGroupIds": {{ NodeSecurityGroups }}
+  },
+  {
+    "cniVersion": "0.3.1",
+    "type": "cni-ipvlan-vpc-k8s-ipvlan",
+    "mode": "l2"
+  },
+  {
+    "cniVersion": "0.3.1",
+    "type": "cni-ipvlan-vpc-k8s-unnumbered-ptp",
+    "hostInterface": "eth0",
+    "containerInterface": "veth0",
+    "ipMasq": true
+  }
+  ]
+}
diff --git a/upup/pkg/fi/nodeup/command.go b/upup/pkg/fi/nodeup/command.go
@@ -262,7 +262,7 @@ func (c *NodeUpCommand) Run(out io.Writer) error {
 
 		loader.TemplateFunctions["SubnetTags"] = func() (string, error) {
 			tags := map[string]string{
-				"Type": "pod",
+				"KubernetesCluster": c.cluster.Name,
 			}
 			if len(c.cluster.Spec.Networking.LyftVPC.SubnetTags) > 0 {
 				tags = c.cluster.Spec.Networking.LyftVPC.SubnetTags