remove enable filter and manage defaults

Author: jpinsonneau

scripts/functions.sh

@@ -315,7 +315,6 @@ function flows_usage {
   echo "          --enable_dns:             enable DNS tracking                        (default: false)"
   echo "          --enable_rtt:             enable RTT tracking                        (default: false)"
   echo "          --enable_network_events:  enable Network events monitoring           (default: false)"
-  echo "          --enable_filter:          enable flow filter                         (default: false)"
   # common
   common_usage
   # specific filters
@@ -349,6 +348,11 @@ function updateFLPConfig {
 function edit_manifest() {
   ## replace the configuration in the manifest file
   echo "opt: $1, evalue: $2"
+
+  if [[ $1 == "filter_"* ]]; then
+    "$YQ_BIN" e --inplace ".spec.template.spec.containers[0].env[] |= select(.name==\"ENABLE_FLOW_FILTER\").value|=\"true\"" "$3"
+  fi
+
   case "$1" in
   "interfaces")
     "$YQ_BIN" e --inplace ".spec.template.spec.containers[0].env[] |= select(.name==\"INTERFACES\").value|=\"$2\"" "$3"
@@ -397,9 +401,6 @@ function edit_manifest() {
       fi
     fi
     ;;
-  "filter_enable")
-    "$YQ_BIN" e --inplace ".spec.template.spec.containers[0].env[] |= select(.name==\"ENABLE_FLOW_FILTER\").value|=\"$2\"" "$3"
-    ;;
   "filter_direction")
     "$YQ_BIN" e --inplace " .spec.template.spec.containers[0].env[] |= select(.name == \"FLOW_FILTER_RULES\").value |=(fromjson | map(.direction = \"$2\")| tostring)" "$3"
     ;;
@@ -452,6 +453,10 @@ function edit_manifest() {
     "$YQ_BIN" e --inplace " .spec.template.spec.containers[0].env[] |= select(.name == \"FLOW_FILTER_RULES\").value |=(fromjson | map(.tcp_flags = \"$2\")| tostring)" "$3"
     ;;
   "filter_pkt_drops")
+    if [[ "$2" == "true" ]]; then
+      # force enable drops before setting filter
+      edit_manifest "pktdrop_enable" "$2" "$3"
+    fi
     "$YQ_BIN" e --inplace " .spec.template.spec.containers[0].env[] |= select(.name == \"FLOW_FILTER_RULES\").value |=(fromjson | map(.drops = $2)| tostring)" "$3"
     ;;
   "filter_regexes")
@@ -495,6 +500,18 @@ function edit_manifest() {
   esac
 }
 
+
+# define key and value at script level to make them available all the time
+# these will be updated by check_args_and_apply first and overriden by defaultValue when needed
+key=""
+value=""
+
+function defaultValue() {
+  if [ "$key" == "$value" ]; then
+    value="$1"
+  fi
+}
+
 # Check if the arguments are valid
 #$1: options
 #$2: manifest
@@ -506,14 +523,15 @@ function check_args_and_apply() {
     value="${option#*=}"
     case "$key" in
     --background) # Run command in background
+      defaultValue "true"
       if [[ "$value" == "true" || "$value" == "false" ]]; then
-        echo "param: $key, param_value: $value"
         runBackground="$value"
       else
         echo "invalid value for --background"
       fi
       ;;
     --copy) # Copy or skip without prompt
+      defaultValue "true"
       if [[ "$value" == "true" || "$value" == "false" || "$value" == "prompt" ]]; then
         copy="$value"
       else
@@ -525,6 +543,7 @@ function check_args_and_apply() {
       ;;
     --enable_pktdrop) # Enable packet drop
       if [[ "$3" == "flows" ]]; then
+        defaultValue "true"
         if [[ "$value" == "true" || "$value" == "false" ]]; then
           edit_manifest "pktdrop_enable" "$value" "$2"
         else
@@ -537,6 +556,7 @@ function check_args_and_apply() {
       ;;
     --enable_dns) # Enable DNS
       if [[ "$3" == "flows" ]]; then
+        defaultValue "true"
         if [[ "$value" == "true" || "$value" == "false" ]]; then
           edit_manifest "dns_enable" "$value" "$2"
         else
@@ -549,6 +569,7 @@ function check_args_and_apply() {
       ;;
     --enable_rtt) # Enable RTT
       if [[ "$3" == "flows" ]]; then
+        defaultValue "true"
         if [[ "$value" == "true" || "$value" == "false" ]]; then
           edit_manifest "rtt_enable" "$value" "$2"
         else
@@ -561,6 +582,7 @@ function check_args_and_apply() {
       ;;
     --enable_network_events) # Enable Network events monitoring
       if [[ "$3" == "flows" ]]; then
+        defaultValue "true"
         if [[ "$value" == "true" || "$value" == "false" ]]; then
           edit_manifest "network_events_enable" "$value" "$2"
         else
@@ -571,18 +593,6 @@ function check_args_and_apply() {
         exit 1
       fi
       ;;
-    --enable_filter) # Enable flow filter
-      if [[ "$3" == "flows" ]]; then
-        if [[ "$value" == "true" || "$value" == "false" ]]; then
-          edit_manifest "filter_enable" "$value" "$2"
-        else
-          echo "invalid value for --enable_filter"
-        fi
-      else
-        echo "--enable_filter is invalid option for packets"
-        exit 1
-      fi
-      ;;
     --direction) # Configure filter direction
       if [[ "$value" == "Ingress" || "$value" == "Egress" ]]; then
         edit_manifest "filter_direction" "$value" "$2"
@@ -635,6 +645,7 @@ function check_args_and_apply() {
       fi
       ;;
     --drops) # Filter packet drops
+      defaultValue "true"
       if [[ "$value" == "true" || "$value" == "false" ]]; then
         edit_manifest "filter_pkt_drops" "$value" "$2"
       else
@@ -692,6 +703,7 @@ function check_args_and_apply() {
       fi
       ;;
     --get-subnets) # Get subnets
+      defaultValue "true"
       if [[ "$value" == "true" || "$value" == "false" ]]; then
         edit_manifest "get_subnets" "$value" "$2"
       else