fix kubeproxy some tests

* NAT46: fix test cleanup, missing del keyword
* NAT66: fix kube-proxy vip, is ipv6
* add some missing kp_put_writer_lock
* wipe flowtable after each unit test
  * Add new cli api: "test kube-proxy flowtable flush" to flushes everything
  * Call this new cli function after the end of each kube-proxy unit test.
  * same as commit b3d1b203579226ca5136b9d6a2744577d07cfcc6 for the lb plugin

Change-Id: I4146f44841328ec96eb66729e3bae3d40f33e4aa
Signed-off-by: Gabriel Ganne <gabriel.ganne@enea.com>

2 files changed, 41 insertions, 2 deletions

diff --git a/src/plugins/kubeproxy/kp.c b/src/plugins/kubeproxy/kp.c
index 1a087e82073..b31b3171d90 100644
--- a/src/plugins/kubeproxy/kp.c
+++ b/src/plugins/kubeproxy/kp.c
@@ -747,8 +747,10 @@ int kp_vip_add(ip46_address_t *prefix, u8 plen, kp_vip_type_t type,
 
   if (ip46_prefix_is_ip4(prefix, plen) &&
       (type != KP_VIP_TYPE_IP4_NAT44) &&
-      (type != KP_VIP_TYPE_IP4_NAT46))
+      (type != KP_VIP_TYPE_IP4_NAT46)) {
+    kp_put_writer_lock();
     return VNET_API_ERROR_INVALID_ADDRESS_FAMILY;
+  }
 
 
   //Allocate
@@ -785,8 +787,10 @@ int kp_vip_add(ip46_address_t *prefix, u8 plen, kp_vip_type_t type,
   //Create maping from nodeport to vip_index
   key = clib_host_to_net_u16(node_port);
   entry = hash_get_mem (kpm->nodeport_by_key, &key);
-  if (entry)
+  if (entry) {
+    kp_put_writer_lock();
     return VNET_API_ERROR_VALUE_EXIST;
+  }
 
   key_copy = clib_mem_alloc (sizeof (*key_copy));
   clib_memcpy (key_copy, &key, sizeof (*key_copy));
diff --git a/src/plugins/kubeproxy/kp_cli.c b/src/plugins/kubeproxy/kp_cli.c
index 6a18834274e..43c5c51ae53 100644
--- a/src/plugins/kubeproxy/kp_cli.c
+++ b/src/plugins/kubeproxy/kp_cli.c
@@ -345,3 +345,38 @@ VLIB_CLI_COMMAND (kp_set_interface_nat4_command, static) = {
   .short_help = "kube-proxy set interface nat4 in <intfc> [del]",
 };
 
+static clib_error_t *
+kp_flowtable_flush_command_fn(vlib_main_t * vm,
+    unformat_input_t * input, vlib_cli_command_t * cmd)
+{
+  u32 thread_index;
+  vlib_thread_main_t *tm = vlib_get_thread_main();
+  kp_main_t *kpm = &kp_main;
+
+  for(thread_index = 0; thread_index < tm->n_vlib_mains; thread_index++ ) {
+    kp_hash_t *h = kpm->per_cpu[thread_index].sticky_ht;
+    if (h != NULL) {
+        kp_hash_bucket_t *b;
+        u32 i;
+        kp_hash_foreach_entry(h, b, i) {
+          vlib_refcount_add(&kpm->pod_refcount, thread_index, b->value[i], -1);
+          vlib_refcount_add(&kpm->pod_refcount, thread_index, 0, 1);
+        }
+
+        kp_hash_free(h);
+        kpm->per_cpu[thread_index].sticky_ht = NULL;
+    }
+  }
+
+  return NULL;
+}
+
+/*
+ * flush all kube-proxy flowtables
+ * This is indented for debug and unit-tests purposes only
+ */
+VLIB_CLI_COMMAND (kp_flowtable_flush_command, static) = {
+  .path = "test kube-proxy flowtable flush",
+  .short_help = "test kube-proxy flowtable flush",
+  .function = kp_flowtable_flush_command_fn,
+};