ovs_stats: Implement OVS statistics plugin.
authorMytnyk, VolodymyrX <volodymyrx.mytnyk@intel.com>
Tue, 3 Jan 2017 16:36:43 +0000 (16:36 +0000)
committerMytnyk, VolodymyrX <volodymyrx.mytnyk@intel.com>
Tue, 3 Jan 2017 16:36:43 +0000 (16:36 +0000)
OVS stats plugin uses OVS utils to get available
bridge/interface statistics from OVSDB.

Change-Id: Iabe71d64fef7fdbfd5a272c3599a92bce2adf055
Signed-off-by: Mytnyk, VolodymyrX <volodymyrx.mytnyk@intel.com>
Makefile.am
README
configure.ac
src/collectd.conf.in
src/collectd.conf.pod
src/ovs_stats.c [new file with mode: 0644]

index 3460cf6..4f479e8 100644 (file)
@@ -1288,6 +1288,17 @@ ovs_events_la_LDFLAGS = $(PLUGIN_LDFLAGS) $(BUILD_WITH_LIBYAJL_LDFLAGS)
 ovs_events_la_LIBADD = $(BUILD_WITH_LIBYAJL_LIBS)
 endif
 
+if BUILD_PLUGIN_OVS_STATS
+pkglib_LTLIBRARIES += ovs_stats.la
+ovs_stats_la_SOURCES = \
+       src/ovs_stats.c \
+       src/utils_ovs.c \
+       src/utils_ovs.h
+ovs_stats_la_CPPFLAGS = $(AM_CPPFLAGS) $(BUILD_WITH_LIBYAJL_CPPFLAGS)
+ovs_stats_la_LDFLAGS = $(PLUGIN_LDFLAGS) $(BUILD_WITH_LIBYAJL_LDFLAGS)
+ovs_stats_la_LIBADD = $(BUILD_WITH_LIBYAJL_LIBS)
+endif
+
 if BUILD_PLUGIN_PERL
 pkglib_LTLIBRARIES += perl.la
 perl_la_SOURCES = src/perl.c
diff --git a/README b/README
index fb3aa27..1860281 100644 (file)
--- a/README
+++ b/README
@@ -297,6 +297,13 @@ Features
       OVS documentation.
       <http://openvswitch.org/support/dist-docs/INSTALL.rst.html>
 
+    - ovs_stats
+      The plugin collects the statistics of OVS connected bridges and
+      interfaces. It requires YAJL library to be installed.
+      Detailed instructions for installing and setting up Open vSwitch, see
+      OVS documentation.
+      <http://openvswitch.org/support/dist-docs/INSTALL.rst.html>
+
     - perl
       The perl plugin implements a Perl-interpreter into collectd. You can
       write your own plugins in Perl and return arbitrary values using this
@@ -935,8 +942,8 @@ Prerequisites
     <http://www.xmms.org/>
 
   * libyajl (optional)
-    Parse JSON data. This is needed for the `ceph', `curl_json', 'ovs_events'
-    and `log_logstash' plugins.
+    Parse JSON data. This is needed for the `ceph', `curl_json', 'ovs_events',
+    'ovs_stats' and `log_logstash' plugins.
     <http://github.com/lloyd/yajl>
 
   * libvarnish (optional)
index c0fe54e..4d56015 100644 (file)
@@ -5893,6 +5893,7 @@ plugin_multimeter="no"
 plugin_nfs="no"
 plugin_numa="no"
 plugin_ovs_events="no"
+plugin_ovs_stats="no"
 plugin_perl="no"
 plugin_pinba="no"
 plugin_processes="no"
@@ -6142,6 +6143,7 @@ fi
 
 if test "x$with_libyajl" = "xyes" && test "x$with_libyajl2" = "xyes"; then
   plugin_ovs_events="yes"
+  plugin_ovs_stats="yes"
 fi
 
 if test "x$with_libperl" = "xyes" && test "x$c_cv_have_perl_ithreads" = "xyes"; then
@@ -6331,6 +6333,7 @@ AC_PLUGIN([openldap],            [$with_libldap],           [OpenLDAP statistics
 AC_PLUGIN([openvpn],             [yes],                     [OpenVPN client statistics])
 AC_PLUGIN([oracle],              [$with_oracle],            [Oracle plugin])
 AC_PLUGIN([ovs_events],          [$plugin_ovs_events],      [OVS events plugin])
+AC_PLUGIN([ovs_stats],           [$plugin_ovs_stats],       [OVS statistics plugin])
 AC_PLUGIN([perl],                [$plugin_perl],            [Embed a Perl interpreter])
 AC_PLUGIN([pf],                  [$have_net_pfvar_h],       [BSD packet filter (PF) statistics])
 # FIXME: Check for libevent, too.
@@ -6756,6 +6759,7 @@ AC_MSG_RESULT([    openldap  . . . . . . $enable_openldap])
 AC_MSG_RESULT([    openvpn . . . . . . . $enable_openvpn])
 AC_MSG_RESULT([    oracle  . . . . . . . $enable_oracle])
 AC_MSG_RESULT([    ovs_events  . . . . . $enable_ovs_events])
+AC_MSG_RESULT([    ovs_stats . . . . . . $enable_ovs_stats])
 AC_MSG_RESULT([    perl  . . . . . . . . $enable_perl])
 AC_MSG_RESULT([    pf  . . . . . . . . . $enable_pf])
 AC_MSG_RESULT([    pinba . . . . . . . . $enable_pinba])
index 1cc86af..eafe3fb 100644 (file)
 #@BUILD_PLUGIN_OPENVPN_TRUE@LoadPlugin openvpn
 #@BUILD_PLUGIN_ORACLE_TRUE@LoadPlugin oracle
 #@BUILD_PLUGIN_OVS_EVENTS_TRUE@LoadPlugin ovs_events
+#@BUILD_PLUGIN_OVS_STATS_TRUE@LoadPlugin ovs_stats
 #@BUILD_PLUGIN_PERL_TRUE@LoadPlugin perl
 #@BUILD_PLUGIN_PINBA_TRUE@LoadPlugin pinba
 #@BUILD_PLUGIN_PING_TRUE@LoadPlugin ping
 #  SendNotification false
 #</Plugin>
 
+#<Plugin ovs_stats>
+#  Port "6640"
+#  Address "127.0.0.1"
+#  Socket "/var/run/openvswitch/db.sock"
+#  Bridges "br0" "br_ext"
+#</Plugin>
+
 #<Plugin perl>
 #      IncludeDir "/my/include/path"
 #      BaseName "Collectd::Plugins"
index 51fe68d..7279f8a 100644 (file)
@@ -5524,6 +5524,54 @@ use B<Interval> option of the OVS B<LoadPlugin> block settings. For milliseconds
 simple divide the time by 1000 for example if the desired interval is 50ms, set
 interval to 0.05.
 
+=head2 Plugin C<ovs_stats>
+
+The I<ovs_stats> plugin collects a statistics of OVS connected interfaces.
+This plugin uses OVSDB management protocol (RFC7047) monitor mechanism to get
+statistics from OVSDB
+
+B<Synopsis:>
+
+ <Plugin "ovs_stats">
+   Port 6640
+   Address "127.0.0.1"
+   Socket "/var/run/openvswitch/db.sock"
+   Bridges "br0" "br_ext"
+ </Plugin>
+
+The plugin provides the following configuration options:
+
+=over 4
+
+=item B<Address> I<node>
+
+The address of the OVS DB server JSON-RPC interface used by the plugin. To
+enable the interface, OVS DB daemon should be running with C<--remote=ptcp:>
+option. See L<ovsdb-server(1)> for more details. The option may be either
+network hostname, IPv4 numbers-and-dots notation or IPv6 hexadecimal string
+format. Defaults to B<'localhost'>.
+
+=item B<Port> I<service>
+
+TCP-port to connect to. Either a service name or a port number may be given.
+Defaults to B<6640>.
+
+=item B<Socket> I<path>
+
+The UNIX domain socket path of OVS DB server JSON-RPC interface used by the
+plugin. To enable the interface, the OVS DB daemon should be running with
+C<--remote=punix:> option. See L<ovsdb-server(1)> for more details. If this
+option is set, B<Address> and B<Port> options are ignored.
+
+=item B<Bridges> [I<brname> ...]
+
+List of OVS bridge names to be monitored by this plugin. If this option is
+omitted or is empty then all OVS bridges will be monitored.
+
+Default: empty (monitor all bridges)
+
+=back
+
 =head2 Plugin C<perl>
 
 This plugin embeds a Perl-interpreter into collectd and provides an interface
diff --git a/src/ovs_stats.c b/src/ovs_stats.c
new file mode 100644 (file)
index 0000000..bbdb600
--- /dev/null
@@ -0,0 +1,982 @@
+/*
+ * collectd - src/ovs_stats.c
+ *
+ * Copyright(c) 2016 Intel Corporation. All rights reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a copy of
+ * this software and associated documentation files (the "Software"), to deal in
+ * the Software without restriction, including without limitation the rights to
+ * use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies
+ * of the Software, and to permit persons to whom the Software is furnished to do
+ * so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in all
+ * copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+ * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+ * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+ * SOFTWARE.
+ *
+ * Authors:
+ *   Taras Chornyi <tarasx.chornyi@intel.com>
+ */
+
+#include "common.h"
+
+#include "utils_ovs.h" /* OvS helpers */
+
+/* Plugin name */
+static const char plugin_name[] = "ovs_stats";
+
+typedef enum iface_counter {
+  not_supportred = -1,
+  collisions,
+  rx_bytes,
+  rx_crc_err,
+  rx_dropped,
+  rx_errors,
+  rx_frame_err,
+  rx_over_err,
+  rx_packets,
+  tx_bytes,
+  tx_dropped,
+  tx_errors,
+  tx_packets,
+  rx_1_to_64_packets,
+  rx_65_to_127_packets,
+  rx_128_to_255_packets,
+  rx_256_to_511_packets,
+  rx_512_to_1023_packets,
+  rx_1024_to_1522_packets,
+  rx_1523_to_max_packets,
+  tx_1_to_64_packets,
+  tx_65_to_127_packets,
+  tx_128_to_255_packets,
+  tx_256_to_511_packets,
+  tx_512_to_1023_packets,
+  tx_1024_to_1522_packets,
+  tx_1523_to_max_packets,
+  tx_multicast_packets,
+  rx_broadcast_packets,
+  tx_broadcast_packets,
+  rx_undersized_errors,
+  rx_oversize_errors,
+  rx_fragmented_errors,
+  rx_jabber_errors,
+  __iface_counter_max
+} iface_counter;
+
+#define IFACE_COUNTER_MAX (__iface_counter_max - 1)
+#define IFACE_COUNTER_COUNT (__iface_counter_max)
+#define PORT_NAME_SIZE_MAX 255
+#define UUID_SIZE 64
+
+typedef struct port_s {
+  char name[PORT_NAME_SIZE_MAX];      /* Port name */
+  char port_uuid[UUID_SIZE];          /* Port table _uuid */
+  char iface_uuid[UUID_SIZE];         /* Interface table uuid */
+  char ex_iface_id[UUID_SIZE];        /* External iface id */
+  char ex_vm_id[UUID_SIZE];           /* External vm id */
+  int64_t stats[IFACE_COUNTER_COUNT]; /* Port statistics */
+  struct bridge_list_s *br;           /* Pointer to bridge */
+  struct port_s *next;                /* Next port */
+} port_list_t;
+
+typedef struct bridge_list_s {
+  char *name;                 /* Bridge name */
+  struct bridge_list_s *next; /* Next bridge*/
+} bridge_list_t;
+
+static const char *const iface_counter_table[IFACE_COUNTER_COUNT] = {
+        [collisions] = "collisions",
+        [rx_bytes] = "rx_bytes",
+        [rx_crc_err] = "rx_crc_err",
+        [rx_dropped] = "rx_dropped",
+        [rx_errors] = "rx_errors",
+        [rx_frame_err] = "rx_frame_err",
+        [rx_over_err] = "rx_over_err",
+        [rx_packets] = "rx_packets",
+        [tx_bytes] = "tx_bytes",
+        [tx_dropped] = "tx_dropped",
+        [tx_errors] = "tx_errors",
+        [tx_packets] = "tx_packets",
+        [rx_1_to_64_packets] = "rx_1_to_64_packets",
+        [rx_65_to_127_packets] = "rx_65_to_127_packets",
+        [rx_128_to_255_packets] = "rx_128_to_255_packets",
+        [rx_256_to_511_packets] = "rx_256_to_511_packets",
+        [rx_512_to_1023_packets] = "rx_512_to_1023_packets",
+        [rx_1024_to_1522_packets] = "rx_1024_to_1518_packets",
+        [rx_1523_to_max_packets] = "rx_1523_to_max_packets",
+        [tx_1_to_64_packets] = "tx_1_to_64_packets",
+        [tx_65_to_127_packets] = "tx_65_to_127_packets",
+        [tx_128_to_255_packets] = "tx_128_to_255_packets",
+        [tx_256_to_511_packets] = "tx_256_to_511_packets",
+        [tx_512_to_1023_packets] = "tx_512_to_1023_packets",
+        [tx_1024_to_1522_packets] = "tx_1024_to_1518_packets",
+        [tx_1523_to_max_packets] = "tx_1523_to_max_packets",
+        [tx_multicast_packets] = "tx_multicast_packets",
+        [rx_broadcast_packets] = "rx_broadcast_packets",
+        [tx_broadcast_packets] = "tx_broadcast_packets",
+        [rx_undersized_errors] = "rx_undersized_errors",
+        [rx_oversize_errors] = "rx_oversize_errors",
+        [rx_fragmented_errors] = "rx_fragmented_errors",
+        [rx_jabber_errors] = "rx_jabber_errors",
+};
+
+/* Entry into the list of network bridges */
+static bridge_list_t *g_bridge_list_head = NULL;
+
+/* Entry into the list of monitored network bridges */
+static bridge_list_t *g_monitored_bridge_list_head = NULL;
+
+/* entry into the list of network bridges */
+static port_list_t *g_port_list_head = NULL;
+
+/* lock for statistics cache */
+static pthread_mutex_t g_stats_lock;
+
+/* OvS DB socket */
+static ovs_db_t *ovs_db = NULL;
+
+/* OVS stats configuration data */
+struct ovs_stats_config_s {
+  char ovs_db_node[OVS_DB_ADDR_NODE_SIZE];    /* OVS DB node */
+  char ovs_db_serv[OVS_DB_ADDR_SERVICE_SIZE]; /* OVS DB service */
+  char ovs_db_unix[OVS_DB_ADDR_UNIX_SIZE];    /* OVS DB unix socket path */
+};
+typedef struct ovs_stats_config_s ovs_stats_config_t;
+
+static ovs_stats_config_t ovs_stats_cfg = {
+    .ovs_db_node = "localhost", /* use default OVS DB node */
+    .ovs_db_serv = "6640",      /* use default OVS DB service */
+};
+
+static const iface_counter ovs_stats_counter_name_to_type(const char *counter) {
+  iface_counter index = not_supportred;
+
+  if (counter == NULL)
+    return not_supportred;
+
+  for (int i = 0; i < IFACE_COUNTER_COUNT; i++) {
+    if (strncmp(iface_counter_table[i], counter,
+                strlen(iface_counter_table[i])) == 0) {
+      index = i;
+      break;
+    }
+  }
+  return index;
+}
+
+static void ovs_stats_submit_one(const char *dev, const char *type,
+                                 const char *type_instance, derive_t value,
+                                 meta_data_t *meta) {
+  /* if counter is less than 0 - skip it*/
+  if (value < 0)
+    return;
+  value_list_t vl = VALUE_LIST_INIT;
+
+  vl.values = &(value_t){.derive = value};
+  vl.values_len = 1;
+  vl.meta = meta;
+
+  sstrncpy(vl.plugin, plugin_name, sizeof(vl.plugin));
+  sstrncpy(vl.plugin_instance, dev, sizeof(vl.plugin_instance));
+  sstrncpy(vl.type, type, sizeof(vl.type));
+
+  if (type_instance != NULL)
+    sstrncpy(vl.type_instance, type_instance, sizeof(vl.type_instance));
+
+  plugin_dispatch_values(&vl);
+}
+
+static void ovs_stats_submit_two(const char *dev, const char *type,
+                                 const char *type_instance, derive_t rx,
+                                 derive_t tx, meta_data_t *meta) {
+  /* if counter is less than 0 - skip it*/
+  if (rx < 0 || tx < 0)
+    return;
+  value_list_t vl = VALUE_LIST_INIT;
+  value_t values[] = {{.derive = rx}, {.derive = tx}};
+
+  vl.values = values;
+  vl.values_len = STATIC_ARRAY_SIZE(values);
+  vl.meta = meta;
+
+  sstrncpy(vl.plugin, plugin_name, sizeof(vl.plugin));
+  sstrncpy(vl.plugin_instance, dev, sizeof(vl.plugin_instance));
+  sstrncpy(vl.type, type, sizeof(vl.type));
+
+  if (type_instance != NULL)
+    sstrncpy(vl.type_instance, type_instance, sizeof(vl.type_instance));
+
+  plugin_dispatch_values(&vl);
+}
+
+static port_list_t *ovs_stats_get_port(const char *uuid) {
+  if (uuid == NULL)
+    return NULL;
+
+  for (port_list_t *port = g_port_list_head; port != NULL; port = port->next) {
+    if (strncmp(port->port_uuid, uuid, strlen(port->port_uuid)) == 0)
+      return port;
+  }
+  return NULL;
+}
+
+static port_list_t *ovs_stats_get_port_by_name(const char *name) {
+  if (name == NULL)
+    return NULL;
+
+  for (port_list_t *port = g_port_list_head; port != NULL; port = port->next)
+    if ((strncmp(port->name, name, strlen(port->name)) == 0) &&
+        strlen(name) == strlen(port->name))
+      return port;
+  return NULL;
+}
+
+/* Create or get port by port uuid */
+static port_list_t *ovs_stats_new_port(bridge_list_t *bridge,
+                                       const char *uuid) {
+  port_list_t *port = ovs_stats_get_port(uuid);
+
+  if (port == NULL) {
+    port = (port_list_t *)calloc(1, sizeof(port_list_t));
+    if (!port) {
+      ERROR("%s: Error allocating port", plugin_name);
+      return NULL;
+    }
+    memset(port->stats, -1, sizeof(int64_t[IFACE_COUNTER_COUNT]));
+    sstrncpy(port->port_uuid, uuid, sizeof(port->port_uuid));
+    pthread_mutex_lock(&g_stats_lock);
+    port->next = g_port_list_head;
+    g_port_list_head = port;
+    pthread_mutex_unlock(&g_stats_lock);
+  }
+  if (bridge != NULL) {
+    pthread_mutex_lock(&g_stats_lock);
+    port->br = bridge;
+    pthread_mutex_unlock(&g_stats_lock);
+  }
+  return port;
+}
+
+/* Get bridge by name*/
+static bridge_list_t *ovs_stats_get_bridge(bridge_list_t *head,
+                                           const char *name) {
+  if (name == NULL)
+    return NULL;
+
+  for (bridge_list_t *bridge = head; bridge != NULL; bridge = bridge->next) {
+    if ((strncmp(bridge->name, name, strlen(bridge->name)) == 0) &&
+        strlen(name) == strlen(bridge->name))
+      return bridge;
+  }
+  return NULL;
+}
+
+/* Delete bridge */
+static int ovs_stats_del_bridge(yajl_val bridge) {
+  const char *old[] = {"old", NULL};
+  const char *name[] = {"name", NULL};
+
+  yajl_val row;
+
+  if (bridge && YAJL_IS_OBJECT(bridge)) {
+    row = yajl_tree_get(bridge, old, yajl_t_object);
+    if (row && YAJL_IS_OBJECT(row)) {
+      yajl_val br_name = yajl_tree_get(row, name, yajl_t_string);
+      if (br_name && YAJL_IS_STRING(br_name)) {
+        bridge_list_t *prev_br = g_bridge_list_head;
+        for (bridge_list_t *br = g_bridge_list_head; br != NULL;
+             prev_br = br, br = br->next) {
+          if ((strncmp(br->name, br_name->u.string, strlen(br->name)) == 0) &&
+              strlen(br->name) == strlen(br_name->u.string)) {
+            if (br == g_bridge_list_head)
+              g_bridge_list_head = br->next;
+            else
+              prev_br->next = br->next;
+            sfree(br->name);
+            sfree(br);
+            break;
+          }
+        }
+      }
+    }
+  } else
+    WARNING("%s: Incorrect data for deleting bridge", plugin_name);
+  return (0);
+}
+
+/* Update Bridge. Create bridge ports*/
+static int ovs_stats_update_bridge(yajl_val bridge) {
+  const char *new[] = {"new", NULL};
+  const char *name[] = {"name", NULL};
+  const char *ports[] = {"ports", NULL};
+  bridge_list_t *br = NULL;
+
+  if (bridge && YAJL_IS_OBJECT(bridge)) {
+    yajl_val row = yajl_tree_get(bridge, new, yajl_t_object);
+    if (row && YAJL_IS_OBJECT(row)) {
+      yajl_val br_name = yajl_tree_get(row, name, yajl_t_string);
+      yajl_val br_ports = yajl_tree_get(row, ports, yajl_t_array);
+      if (br_name && YAJL_IS_STRING(br_name)) {
+        br = ovs_stats_get_bridge(g_bridge_list_head, YAJL_GET_STRING(br_name));
+        pthread_mutex_lock(&g_stats_lock);
+        if (br == NULL) {
+          br = (bridge_list_t *)calloc(1, sizeof(bridge_list_t));
+          if (!br) {
+            ERROR("%s: Error allocating memory for bridge", plugin_name);
+            return (-1);
+          }
+          char *tmp = YAJL_GET_STRING(br_name);
+
+          if (tmp != NULL)
+            br->name = strdup(tmp);
+          if (br->name == NULL) {
+            sfree(br);
+            pthread_mutex_unlock(&g_stats_lock);
+            return (-1);
+          }
+          br->next = g_bridge_list_head;
+          g_bridge_list_head = br;
+        }
+        pthread_mutex_unlock(&g_stats_lock);
+      }
+      if (br_ports && YAJL_IS_ARRAY(br_ports)) {
+        char *tmp = YAJL_GET_STRING(br_ports->u.array.values[0]);
+        if (tmp != NULL && strcmp("set", tmp) == 0) {
+          yajl_val *ports_arr =
+              YAJL_GET_ARRAY(br_ports->u.array.values[1])->values;
+          size_t ports_num = YAJL_GET_ARRAY(br_ports->u.array.values[1])->len;
+
+          for (int i = 0; i < ports_num; i++)
+            ovs_stats_new_port(
+                br, YAJL_GET_STRING(ports_arr[i]->u.array.values[1]));
+        } else
+          ovs_stats_new_port(br, YAJL_GET_STRING(br_ports->u.array.values[1]));
+      }
+    }
+  } else {
+    ERROR("Incorrect JSON Bridge data");
+    return (-1);
+  }
+  return (0);
+}
+
+/* Handle JSON with Bridge Table change event */
+static void ovs_stats_bridge_table_change_cb(yajl_val jupdates) {
+  /* Bridge Table update example JSON data
+    {
+      "Bridge": {
+        "bb1f8965-5775-46d9-b820-236ca8edbedc": {
+          "new": {
+            "name": "br0",
+            "ports": [
+              "set",
+              [
+                [
+                  "uuid",
+                  "117f1a07-7ef0-458a-865c-ec7fbb85bc01"
+                ],
+                [
+                  "uuid",
+                  "12fd8bdc-e950-4281-aaa9-46e185658f79"
+                ]
+              ]
+            ]
+          }
+        }
+      }
+    }
+   */
+  const char *path[] = {"Bridge", NULL};
+
+  yajl_val bridges = yajl_tree_get(jupdates, path, yajl_t_object);
+
+  if (bridges && YAJL_IS_OBJECT(bridges)) {
+    for (int i = 0; i < YAJL_GET_OBJECT(bridges)->len; i++) {
+      yajl_val bridge = YAJL_GET_OBJECT(bridges)->values[i];
+      ovs_stats_update_bridge(bridge);
+    }
+  }
+}
+
+/* Handle Bridge Table delete event */
+static void ovs_stats_bridge_table_delete_cb(yajl_val jupdates) {
+  const char *path[] = {"Bridge", NULL};
+  yajl_val bridges = yajl_tree_get(jupdates, path, yajl_t_object);
+  yajl_val bridge;
+  if (bridges && YAJL_IS_OBJECT(bridges)) {
+    pthread_mutex_lock(&g_stats_lock);
+    for (int i = 0; i < YAJL_GET_OBJECT(bridges)->len; i++) {
+      bridge = YAJL_GET_OBJECT(bridges)->values[i];
+      ovs_stats_del_bridge(bridge);
+    }
+    pthread_mutex_unlock(&g_stats_lock);
+  }
+  return;
+}
+
+/* Handle JSON with Bridge table initial values */
+static void ovs_stats_bridge_table_result_cb(yajl_val jresult,
+                                             yajl_val jerror) {
+  if (YAJL_IS_NULL(jerror))
+    ovs_stats_bridge_table_change_cb(jresult);
+  else
+    ERROR("%s: Error received from OvSDB. Table: Bridge", plugin_name);
+  return;
+}
+
+/* Update port name */
+static int ovs_stats_update_port(const char *uuid, yajl_val port) {
+  const char *new[] = {"new", NULL};
+  const char *name[] = {"name", NULL};
+  yajl_val row;
+  port_list_t *portentry = NULL;
+  if (port && YAJL_IS_OBJECT(port)) {
+    row = yajl_tree_get(port, new, yajl_t_object);
+    if (row && YAJL_IS_OBJECT(row)) {
+      yajl_val port_name = yajl_tree_get(row, name, yajl_t_string);
+      if (port_name && YAJL_IS_STRING(port_name)) {
+        portentry = ovs_stats_get_port(uuid);
+        if (portentry == NULL)
+          portentry = ovs_stats_new_port(NULL, uuid);
+        if (portentry) {
+          pthread_mutex_lock(&g_stats_lock);
+          sstrncpy(portentry->name, YAJL_GET_STRING(port_name),
+                   sizeof(portentry->name));
+          pthread_mutex_unlock(&g_stats_lock);
+        }
+      }
+    }
+  } else {
+    ERROR("Incorrect JSON Port data");
+    return (-1);
+  }
+  return (0);
+}
+
+/* Delete port from global port list */
+static int ovs_stats_del_port(const char *uuid) {
+  port_list_t *prev_port = g_port_list_head;
+  for (port_list_t *port = g_port_list_head; port != NULL;
+       prev_port = port, port = port->next) {
+    if (strncmp(port->port_uuid, uuid, strlen(port->port_uuid)) == 0) {
+      if (port == g_port_list_head)
+        g_port_list_head = port->next;
+      else
+        prev_port->next = port->next;
+      sfree(port);
+      break;
+    }
+  }
+  return (0);
+}
+
+/* Handle JSON with Port Table change event */
+static void ovs_stats_port_table_change_cb(yajl_val jupdates) {
+  /* Port Table update example JSON data
+    {
+      "Port": {
+        "ab107d6f-28a1-4257-b1cc-5b742821db8a": {
+          "new": {
+            "name": "br1",
+            "interfaces": [
+              "uuid",
+              "33a289a0-1d34-4e46-a3c2-3e4066fbecc6"
+            ]
+          }
+        }
+      }
+    }
+   */
+  const char *path[] = {"Port", NULL};
+  yajl_val ports = yajl_tree_get(jupdates, path, yajl_t_object);
+  yajl_val port;
+  if (ports && YAJL_IS_OBJECT(ports)) {
+    for (int i = 0; i < YAJL_GET_OBJECT(ports)->len; i++) {
+      port = YAJL_GET_OBJECT(ports)->values[i];
+      ovs_stats_update_port(YAJL_GET_OBJECT(ports)->keys[i], port);
+    }
+  }
+  return;
+}
+
+/* Handle JSON with Port table initial values */
+static void ovs_stats_port_table_result_cb(yajl_val jresult, yajl_val jerror) {
+  if (YAJL_IS_NULL(jerror))
+    ovs_stats_port_table_change_cb(jresult);
+  else
+    ERROR("%s: Error received from OvSDB. Table: Port", plugin_name);
+  return;
+}
+
+/* Handle Port Table delete event */
+static void ovs_stats_port_table_delete_cb(yajl_val jupdates) {
+  const char *path[] = {"Port", NULL};
+  yajl_val ports = yajl_tree_get(jupdates, path, yajl_t_object);
+  pthread_mutex_lock(&g_stats_lock);
+  if (ports && YAJL_IS_OBJECT(ports))
+    for (int i = 0; i < YAJL_GET_OBJECT(ports)->len; i++) {
+      ovs_stats_del_port(YAJL_GET_OBJECT(ports)->keys[i]);
+    }
+  pthread_mutex_unlock(&g_stats_lock);
+  return;
+}
+
+/* Update interface statistics */
+static int ovs_stats_update_iface_stats(port_list_t *port, yajl_val stats) {
+  yajl_val stat;
+  iface_counter counter_index = 0;
+  char *counter_name = NULL;
+  int64_t counter_value = 0;
+  if (stats && YAJL_IS_ARRAY(stats))
+    for (int i = 0; i < YAJL_GET_ARRAY(stats)->len; i++) {
+      stat = YAJL_GET_ARRAY(stats)->values[i];
+      counter_name = YAJL_GET_STRING(YAJL_GET_ARRAY(stat)->values[0]);
+      counter_index = ovs_stats_counter_name_to_type(counter_name);
+      counter_value = YAJL_GET_INTEGER(YAJL_GET_ARRAY(stat)->values[1]);
+      if (counter_index == not_supportred)
+        continue;
+      port->stats[counter_index] = counter_value;
+    }
+
+  return (0);
+}
+
+/* Update interface external_ids */
+static int ovs_stats_update_iface_ext_ids(port_list_t *port, yajl_val ext_ids) {
+  yajl_val ext_id;
+  char *key;
+  char *value;
+
+  if (ext_ids && YAJL_IS_ARRAY(ext_ids))
+    for (int i = 0; i < YAJL_GET_ARRAY(ext_ids)->len; i++) {
+      ext_id = YAJL_GET_ARRAY(ext_ids)->values[i];
+      key = YAJL_GET_STRING(YAJL_GET_ARRAY(ext_id)->values[0]);
+      value = YAJL_GET_STRING(YAJL_GET_ARRAY(ext_id)->values[1]);
+      if (key && value) {
+        if (strncmp(key, "iface-id", strlen(key)) == 0)
+          sstrncpy(port->ex_iface_id, value, sizeof(port->ex_iface_id));
+        else if (strncmp(key, "vm-uuid", strlen(key)) == 0)
+          sstrncpy(port->ex_vm_id, value, sizeof(port->ex_vm_id));
+      }
+    }
+
+  return (0);
+}
+
+/* Get interface statistic and external_ids */
+static int ovs_stats_update_iface(yajl_val iface) {
+  yajl_val row;
+  port_list_t *port = NULL;
+  if (iface && YAJL_IS_OBJECT(iface)) {
+    row = ovs_utils_get_value_by_key(iface, "new");
+    if (row && YAJL_IS_OBJECT(row)) {
+      yajl_val iface_name = ovs_utils_get_value_by_key(row, "name");
+      yajl_val iface_stats = ovs_utils_get_value_by_key(row, "statistics");
+      yajl_val iface_ext_ids = ovs_utils_get_value_by_key(row, "external_ids");
+      yajl_val iface_uuid = ovs_utils_get_value_by_key(row, "_uuid");
+      if (iface_name && YAJL_IS_STRING(iface_name)) {
+        port = ovs_stats_get_port_by_name(YAJL_GET_STRING(iface_name));
+        if (port == NULL)
+          return (0);
+      }
+      /*
+       * {
+            "statistics": [
+              "map",
+              [
+                [
+                  "collisions",
+                  0
+                ],
+                . . .
+                [
+                  "tx_packets",
+                  0
+                ]
+              ]
+            ]
+          }
+       Check that statistics is an array with 2 elements
+       */
+      if (iface_stats && YAJL_IS_ARRAY(iface_stats) &&
+          YAJL_GET_ARRAY(iface_stats)->len == 2)
+        ovs_stats_update_iface_stats(port,
+                                     YAJL_GET_ARRAY(iface_stats)->values[1]);
+      if (iface_ext_ids && YAJL_IS_ARRAY(iface_ext_ids))
+        ovs_stats_update_iface_ext_ids(
+            port, YAJL_GET_ARRAY(iface_ext_ids)->values[1]);
+      if (iface_uuid && YAJL_IS_ARRAY(iface_uuid) &&
+          YAJL_GET_ARRAY(iface_uuid)->len == 2)
+        sstrncpy(port->iface_uuid,
+                 YAJL_GET_STRING(YAJL_GET_ARRAY(iface_uuid)->values[1]),
+                 sizeof(port->iface_uuid));
+    }
+  } else {
+    ERROR("Incorrect JSON Port data");
+    return (-1);
+  }
+  return (0);
+}
+
+/* Handle JSON with Interface Table change event */
+static void ovs_stats_interface_table_change_cb(yajl_val jupdates) {
+  /* Interface Table update example JSON data
+    {
+      "Interface": {
+        "33a289a0-1d34-4e46-a3c2-3e4066fbecc6": {
+          "new": {
+            "name": "br1",
+            "statistics": [
+              "map",
+              [
+                [
+                  "collisions",
+                  0
+                ],
+                [
+                  "rx_bytes",
+                  0
+                ],
+               . . .
+                [
+                  "tx_packets",
+                  12617
+                ]
+              ]
+            ],
+            "_uuid": [
+              "uuid",
+              "33a289a0-1d34-4e46-a3c2-3e4066fbecc6"
+            ]
+            "external_ids": [
+                "map",
+                [
+                  [
+                    "attached-mac",
+                    "fa:16:3e:7c:1c:4b"
+                  ],
+                  [
+                    "iface-id",
+                    "a61b7e2b-6951-488a-b4c6-6e91343960b2"
+                  ],
+                  [
+                    "iface-status",
+                    "active"
+                  ]
+                ]
+              ]
+          }
+        }
+      }
+    }
+   */
+  const char *path[] = {"Interface", NULL};
+  yajl_val ports = yajl_tree_get(jupdates, path, yajl_t_object);
+  pthread_mutex_lock(&g_stats_lock);
+  if (ports && YAJL_IS_OBJECT(ports))
+    for (int i = 0; i < YAJL_GET_OBJECT(ports)->len; i++)
+      ovs_stats_update_iface(YAJL_GET_OBJECT(ports)->values[i]);
+  pthread_mutex_unlock(&g_stats_lock);
+  return;
+}
+
+/* Handle JSON with Interface table initial values */
+static void ovs_stats_interface_table_result_cb(yajl_val jresult,
+                                                yajl_val jerror) {
+  if (YAJL_IS_NULL(jerror))
+    ovs_stats_interface_table_change_cb(jresult);
+  else
+    ERROR("%s: Error received from OvSDB. Table: Interface", plugin_name);
+  return;
+}
+
+/* Setup OVS DB table callbacks  */
+static void ovs_stats_initialize(ovs_db_t *pdb) {
+  const char *bridge_columns[] = {"name", "ports", NULL};
+  const char *port_columns[] = {"name", "interfaces", NULL};
+  const char *interface_columns[] = {"name", "statistics", "_uuid",
+                                     "external_ids", NULL};
+
+  /* subscribe to a tables */
+  ovs_db_table_cb_register(pdb, "Bridge", bridge_columns,
+                           ovs_stats_bridge_table_change_cb,
+      ovs_stats_bridge_table_result_cb,
+                           OVS_DB_TABLE_CB_FLAG_INITIAL |
+                           OVS_DB_TABLE_CB_FLAG_INSERT |
+          OVS_DB_TABLE_CB_FLAG_MODIFY);
+
+  ovs_db_table_cb_register(pdb, "Bridge", bridge_columns,
+                           ovs_stats_bridge_table_delete_cb, NULL,
+                           OVS_DB_TABLE_CB_FLAG_DELETE);
+
+  ovs_db_table_cb_register(pdb, "Port", port_columns,
+                           ovs_stats_port_table_change_cb,
+      ovs_stats_port_table_result_cb,
+                           OVS_DB_TABLE_CB_FLAG_INITIAL |
+                           OVS_DB_TABLE_CB_FLAG_INSERT |
+          OVS_DB_TABLE_CB_FLAG_MODIFY);
+
+  ovs_db_table_cb_register(pdb, "Port", port_columns,
+                           ovs_stats_port_table_delete_cb, NULL,
+                           OVS_DB_TABLE_CB_FLAG_DELETE);
+
+  ovs_db_table_cb_register(pdb, "Interface", interface_columns,
+                           ovs_stats_interface_table_change_cb,
+      ovs_stats_interface_table_result_cb,
+                           OVS_DB_TABLE_CB_FLAG_INITIAL |
+                           OVS_DB_TABLE_CB_FLAG_INSERT |
+          OVS_DB_TABLE_CB_FLAG_MODIFY);
+}
+
+/* Check if bridge is configured to be monitored in config file */
+static int ovs_stats_is_monitored_bridge(const char *br_name) {
+  int rc = 0;
+  /* if no bridges are configured, return true */
+  if (!(rc = (g_monitored_bridge_list_head == NULL)))
+    rc = (ovs_stats_get_bridge(g_monitored_bridge_list_head, br_name) != NULL);
+  return rc;
+}
+
+/* Delete all ports from port list */
+static void ovs_stats_free_port_list(port_list_t *head) {
+  port_list_t *i, *del;
+
+  for (i = head; i != NULL;) {
+    del = i;
+    i = i->next;
+    sfree(del);
+  }
+}
+
+/* Delete all bridges from bridge list */
+static void ovs_stats_free_bridge_list(bridge_list_t *head) {
+  bridge_list_t *i, *del;
+
+  for (i = head; i != NULL;) {
+    del = i;
+    i = i->next;
+    sfree(del->name);
+    sfree(del);
+  }
+}
+
+/* Handle OVSDB lost connection callback */
+static void ovs_stats_conn_terminate() {
+
+  WARNING("Lost connection to OVSDB server");
+  pthread_mutex_lock(&g_stats_lock);
+  ovs_stats_free_bridge_list(g_bridge_list_head);
+  g_bridge_list_head = NULL;
+  ovs_stats_free_port_list(g_port_list_head);
+  g_port_list_head = NULL;
+  pthread_mutex_unlock(&g_stats_lock);
+}
+
+/* Parse plugin configuration file and store the config
+ * in allocated memory. Returns negative value in case of error.
+ */
+static int ovs_stats_plugin_config(oconfig_item_t *ci) {
+  bridge_list_t *bridge;
+  char *br_name;
+
+  for (int i = 0; i < ci->children_num; i++) {
+    oconfig_item_t *child = ci->children + i;
+    if (strcasecmp("Address", child->key) == 0) {
+      if (cf_util_get_string_buffer(child, ovs_stats_cfg.ovs_db_node,
+                                    OVS_DB_ADDR_NODE_SIZE) != 0) {
+        ERROR("%s: parse '%s' option failed", plugin_name, child->key);
+        return (-1);
+      }
+    } else if (strcasecmp("Port", child->key) == 0) {
+      if (cf_util_get_string_buffer(child, ovs_stats_cfg.ovs_db_serv,
+                                    OVS_DB_ADDR_SERVICE_SIZE) != 0) {
+        ERROR("%s: parse '%s' option failed", plugin_name, child->key);
+        return (-1);
+      }
+    } else if (strcasecmp("Socket", child->key) == 0) {
+      if (cf_util_get_string_buffer(child, ovs_stats_cfg.ovs_db_unix,
+                                    OVS_DB_ADDR_UNIX_SIZE) != 0) {
+        ERROR("%s: parse '%s' option failed", plugin_name, child->key);
+        return (-1);
+      }
+    } else if (strcasecmp("Bridges", child->key) == 0) {
+      for (int j = 0; j < child->values_num; j++) {
+        /* check value type */
+        if (child->values[j].type != OCONFIG_TYPE_STRING) {
+          ERROR("%s: Wrong bridge name [idx=%d]. "
+                "Bridge name should be string",
+                plugin_name, j);
+          goto cleanup_fail;
+        }
+        /* get value */
+        if ((br_name = strdup(child->values[j].value.string)) == NULL) {
+          ERROR("%s: strdup() copy bridge name fail", plugin_name);
+          goto cleanup_fail;
+        }
+        if ((bridge = ovs_stats_get_bridge(g_monitored_bridge_list_head,
+                                           br_name)) == NULL) {
+          if ((bridge = calloc(1, sizeof(bridge_list_t))) == NULL) {
+            ERROR("%s: Error allocating memory for bridge", plugin_name);
+            goto cleanup_fail;
+          } else {
+            pthread_mutex_lock(&g_stats_lock);
+            /* store bridge name */
+            bridge->name = br_name;
+            bridge->next = g_monitored_bridge_list_head;
+            g_monitored_bridge_list_head = bridge;
+            pthread_mutex_unlock(&g_stats_lock);
+            DEBUG("%s: found monitored interface \"%s\"", plugin_name, br_name);
+          }
+        }
+      }
+    } else {
+      WARNING("%s: option '%s' not allowed here", plugin_name, child->key);
+      goto cleanup_fail;
+    }
+  }
+  return (0);
+
+cleanup_fail:
+  ovs_stats_free_bridge_list(g_monitored_bridge_list_head);
+  return (-1);
+}
+
+/* Initialize OvS Stats plugin*/
+static int ovs_stats_plugin_init(void) {
+  ovs_db_callback_t cb = {.post_conn_init = ovs_stats_initialize,
+                          .post_conn_terminate = ovs_stats_conn_terminate};
+
+  INFO("%s: Connecting to OVS DB using address=%s, service=%s, unix=%s",
+       plugin_name, ovs_stats_cfg.ovs_db_node, ovs_stats_cfg.ovs_db_serv,
+       ovs_stats_cfg.ovs_db_unix);
+  /* connect to OvS DB */
+  if ((ovs_db = ovs_db_init (ovs_stats_cfg.ovs_db_node,
+                             ovs_stats_cfg.ovs_db_serv,
+                       ovs_stats_cfg.ovs_db_unix, &cb)) == NULL) {
+    ERROR("%s: plugin: failed to connect to OvS DB server", plugin_name);
+    return (-1);
+  }
+  int err = pthread_mutex_init(&g_stats_lock, NULL);
+  if (err < 0) {
+    ERROR("%s: plugin: failed to initialize cache lock", plugin_name);
+    ovs_db_destroy(ovs_db);
+    return (-1);
+  }
+  return (0);
+}
+
+/* OvS stats read callback. Read bridge/port information and submit it*/
+static int ovs_stats_plugin_read(__attribute__((unused)) user_data_t *ud) {
+  bridge_list_t *bridge;
+  port_list_t *port;
+  char devname[PORT_NAME_SIZE_MAX];
+
+  pthread_mutex_lock(&g_stats_lock);
+  for (bridge = g_bridge_list_head; bridge != NULL; bridge = bridge->next) {
+    if (ovs_stats_is_monitored_bridge(bridge->name)) {
+      for (port = g_port_list_head; port != NULL; port = port->next)
+        if (port->br == bridge) {
+          if (strlen(port->name) == 0)
+            /* Skip port w/o name. This is possible when read callback
+             * is called after Interface Table update callback but before
+             * Port table Update callback. Will add this port on next read */
+            continue;
+          meta_data_t *meta = meta_data_create();
+          if (meta != NULL) {
+            meta_data_add_string(meta, "uuid", port->iface_uuid);
+            if (strlen(port->ex_vm_id))
+              meta_data_add_string(meta, "vm-uuid", port->ex_vm_id);
+            if (strlen(port->ex_iface_id))
+              meta_data_add_string(meta, "iface-id", port->ex_iface_id);
+          }
+          snprintf(devname, sizeof(devname), "%s.%s", bridge->name, port->name);
+          ovs_stats_submit_one(devname, "if_collisions", NULL,
+                               port->stats[collisions], meta);
+          ovs_stats_submit_two(devname, "if_dropped", NULL,
+                               port->stats[rx_dropped], port->stats[tx_dropped],
+                               meta);
+          ovs_stats_submit_two(devname, "if_errors", NULL,
+                               port->stats[rx_errors], port->stats[tx_errors],
+                               meta);
+          ovs_stats_submit_two(devname, "if_packets", NULL,
+                               port->stats[rx_packets], port->stats[tx_packets],
+                               meta);
+          ovs_stats_submit_one(devname, "if_rx_errors", "crc",
+                               port->stats[rx_crc_err], meta);
+          ovs_stats_submit_one(devname, "if_rx_errors", "frame",
+                               port->stats[rx_frame_err], meta);
+          ovs_stats_submit_one(devname, "if_rx_errors", "over",
+                               port->stats[rx_over_err], meta);
+          ovs_stats_submit_one(devname, "if_rx_octets", NULL,
+                               port->stats[rx_bytes], meta);
+          ovs_stats_submit_one(devname, "if_tx_octets", NULL,
+                               port->stats[tx_bytes], meta);
+          ovs_stats_submit_two(devname, "if_packets", "1_to_64_packets",
+                               port->stats[rx_1_to_64_packets],
+                               port->stats[tx_1_to_64_packets], meta);
+          ovs_stats_submit_two(devname, "if_packets", "65_to_127_packets",
+                               port->stats[rx_65_to_127_packets],
+                               port->stats[tx_65_to_127_packets], meta);
+          ovs_stats_submit_two(devname, "if_packets", "128_to_255_packets",
+                               port->stats[rx_128_to_255_packets],
+                               port->stats[tx_128_to_255_packets], meta);
+          ovs_stats_submit_two(devname, "if_packets", "512_to_1023_packets",
+                               port->stats[rx_512_to_1023_packets],
+                               port->stats[tx_512_to_1023_packets], meta);
+          ovs_stats_submit_two(devname, "if_packets", "1024_to_1518_packets",
+                               port->stats[rx_1024_to_1522_packets],
+                               port->stats[tx_1024_to_1522_packets], meta);
+          ovs_stats_submit_two(devname, "if_packets", "1523_to_max_packets",
+                               port->stats[rx_1523_to_max_packets],
+                               port->stats[tx_1523_to_max_packets], meta);
+          ovs_stats_submit_two(devname, "if_packets", "broadcast_packets",
+                               port->stats[rx_broadcast_packets],
+                               port->stats[tx_broadcast_packets], meta);
+          ovs_stats_submit_one(devname, "if_multicast", "tx_multicast_packets",
+                               port->stats[tx_multicast_packets], meta);
+          ovs_stats_submit_one(devname, "if_rx_errors", "rx_undersized_errors",
+                               port->stats[rx_undersized_errors], meta);
+          ovs_stats_submit_one(devname, "if_rx_errors", "rx_oversize_errors",
+                               port->stats[rx_oversize_errors], meta);
+          ovs_stats_submit_one(devname, "if_rx_errors", "rx_fragmented_errors",
+                               port->stats[rx_fragmented_errors], meta);
+          ovs_stats_submit_one(devname, "if_rx_errors", "rx_jabber_errors",
+                               port->stats[rx_jabber_errors], meta);
+
+          meta_data_destroy(meta);
+        }
+    } else
+      continue;
+  }
+  pthread_mutex_unlock(&g_stats_lock);
+  return (0);
+}
+
+/* Shutdown OvS Stats plugin */
+static int ovs_stats_plugin_shutdown(void) {
+  pthread_mutex_lock(&g_stats_lock);
+  DEBUG("OvS Statistics plugin shutting down");
+  ovs_db_destroy(ovs_db);
+  ovs_stats_free_bridge_list(g_bridge_list_head);
+  ovs_stats_free_bridge_list(g_monitored_bridge_list_head);
+  ovs_stats_free_port_list(g_port_list_head);
+  pthread_mutex_unlock(&g_stats_lock);
+  pthread_mutex_destroy(&g_stats_lock);
+  return (0);
+}
+
+/* Register OvS Stats plugin callbacks */
+void module_register(void) {
+  plugin_register_complex_config(plugin_name, ovs_stats_plugin_config);
+  plugin_register_init(plugin_name, ovs_stats_plugin_init);
+  plugin_register_complex_read(NULL, plugin_name, ovs_stats_plugin_read, 0,
+                               NULL);
+  plugin_register_shutdown(plugin_name, ovs_stats_plugin_shutdown);
+}