Merge pull request #109 from RHsyseng/ovn_conntrack_fix

Adding clarification about the Node subnet
RHsyseng · Nov 8, 2022 · 195cc0d · 195cc0d
2 parents 8c1bb1a + d3ac4b1
commit 195cc0d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/scripts/README.md b/scripts/README.md
@@ -14,7 +14,7 @@ This script gives the potential list of commands to clean up wrong conntracks
 It only supports UDP stale entries
 It only considers clusterIP services
 It only works on IPV4 single stack env
-Assumes node subnet is the default /24 cidr
+Assumes node subnet is the default /24 cidr (it also works for /23)
 Assumes Cluster CIDR is /16
 Checks for the Service CIDR to have one of the networks /8 /16 or /24
 

diff --git a/scripts/ovn_cleanConntrack.sh b/scripts/ovn_cleanConntrack.sh
@@ -8,7 +8,7 @@
 NOW=$(date +"%Y-%m-%d_%H-%M-%S")
 # Logfile to save some DEBUG output
 LOG="/tmp/ovn_cleanConntrack.sh.${NOW}.log"
-# IP of the ovn-k8s-mp0 interface for a node subnet with mask /24
+# IP of the ovn-k8s-mp0 interface for a node subnet with mask /24 or /23
 NODESUBNETIP=2
 # Debug var to write DEBUG lines into the log
 DEBUG=false
@@ -27,7 +27,7 @@ function usage() {
   echo "It only supports UDP stale entries"
   echo "It only considers clusterIP services"
   echo "It only works on IPV4 single stack env"
-  echo "Assumes node subnet is the default /24 cidr"
+  echo "Assumes node subnet is the default /24 cidr (it also works for /23)"
   echo "Assumes Cluster CIDR is /16"
   echo "Checks for the Service CIDR to have one of the networks /8 /16 or /24"
   echo -e