ipvs: remove duplicated masq rules

Duplicated masq rules are created by current implementation: -A KUBE-NODE-PORT -m comment --comment "mark MASQ for externaltrafficpolicy=cluster" -j KUBE-MARK-MASQ -A KUBE-NODE-PORT -j KUBE-MARK-MASQ The last one is always there. So the one inside if statement could just be removed.

ipvs: remove duplicated masq rules
0e6f0736 · Hong Zhiguo · 24ab69d3 · 0e6f0736
Commit 0e6f0736 authored Jun 27, 2018 by Hong Zhiguo
Show whitespace changes
Inline Side-by-side

Showing with 0 additions and 7 deletions

proxier.go pkg/proxy/ipvs/proxier.go +0 -7

No files found.
--- a/pkg/proxy/ipvs/proxier.go
+++ b/pkg/proxy/ipvs/proxier.go
@@ -1283,13 +1283,6 @@ func (proxier *Proxier) writeIptablesRules() {
 			)
 			writeLine(proxier.natRules, append(args, "-j", "ACCEPT")...)
 		}
-		// mark masq for others
-		args = append(args[:0],
-			"-A", string(KubeNodePortChain),
-			"-m", "comment", "--comment",
-			fmt.Sprintf(`"mark MASQ for externaltrafficpolicy=cluster"`),
-		)
-		writeLine(proxier.natRules, append(args, "-j", string(KubeMarkMasqChain))...)
 	}
 	// mark masq for KUBE-NODE-PORT