Merge pull request #45856 from shyamjvs/save-iptables

Automatic merge from submit-queue Make fake iptables' Save operation more realistic Ref https://github.com/kubernetes/kubernetes/pull/45622#issuecomment-301624384 (2nd point) This would make fake IPtables actually return the iptable contents it stores. cc @kubernetes/sig-scalability-misc @wojtek-t

Merge pull request #45856 from shyamjvs/save-iptables
c5e9031d · Kubernetes Submit Queue · GitHub · b996c8b9 · da924fc3 · c5e9031d
Commit c5e9031d authored May 16, 2017 by Kubernetes Submit Queue Committed by GitHub May 16, 2017
Show whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

fake.go pkg/util/iptables/testing/fake.go +4 -2

No files found.
--- a/pkg/util/iptables/testing/fake.go
+++ b/pkg/util/iptables/testing/fake.go
@@ -72,8 +72,10 @@ func (*FakeIPTables) IsIpv6() bool {
 	return false
 }
-func (*FakeIPTables) Save(table iptables.Table) ([]byte, error) {
+func (f *FakeIPTables) Save(table iptables.Table) ([]byte, error) {
-	return make([]byte, 0), nil
+	lines := make([]byte, len(f.Lines))
+	copy(lines, f.Lines)
+	return lines, nil
 }
 func (*FakeIPTables) SaveAll() ([]byte, error) {