No longer set default value for AWS_VPC_K8S_CNI_CONFIGURE_RPFILTER (#737

) Co-authored-by: Paul Stack <public@paulstack.co.uk>
pulumi · Jul 27, 2022 · 8ae0e00 · 8ae0e00
1 parent dcba1d3
commit 8ae0e00
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,8 @@
 ## Unreleased
-- Due to https://github.com/pulumi/pulumi/issues/7012 including the provider in the generated SDK never really worked. This is removed now. Note - existing uses in Nodejs should not be affected.
+- BREAKING CHANGE: Due to https://github.com/pulumi/pulumi/issues/7012 including the provider in the generated SDK never really worked. This is removed now. Note - existing uses in Nodejs should not be affected.
   [#746](https://github.com/pulumi/pulumi-eks/pull/746)
+- Fix issue with duplicated AWS_VPC_K8S_CNI_CONFIGURE_RPFILTER env var on aws-node daemonset. 
+  [#737](https://github.com/pulumi/pulumi-eks/pull/737)
 
 ## 0.41.2 (Released Jul 12, 2022)
 - Allow removal of default Kubernetes addons

diff --git a/nodejs/eks/cmd/provider/cni.ts b/nodejs/eks/cmd/provider/cni.ts
@@ -142,8 +142,6 @@ function computeVpcCniYaml(cniYamlText: string, args: VpcCniInputs): string {
     }
     if (args.cniConfigureRpfilter) {
         env.push({ name: "AWS_VPC_K8S_CNI_CONFIGURE_RPFILTER", value: args.cniConfigureRpfilter ? "true" : "false" });
-    } else {
-        env.push({ name: "AWS_VPC_K8S_CNI_CONFIGURE_RPFILTER", value: "false" });
     }
     if (args.cniCustomNetworkCfg) {
         env.push({ name: "AWS_VPC_K8S_CNI_CUSTOM_NETWORK_CFG", value: args.cniCustomNetworkCfg ? "true" : "false" });