'dbi', 'oracle' and 'postgresql' plugins: Use 'Plugin' as option name
authorPavel Rochnyack <pavel2000@ngs.ru>
Mon, 19 Sep 2016 09:37:13 +0000 (15:37 +0600)
committerPavel Rochnyack <pavel2000@ngs.ru>
Mon, 10 Jul 2017 10:16:24 +0000 (17:16 +0700)
src/collectd.conf.in
src/collectd.conf.pod
src/dbi.c
src/oracle.c
src/postgresql.c

index 8ab20d1..6b677d2 100644 (file)
 #              </Result>
 #      </Query>
 #      <Database "customers_db">
-#              #PluginName "mycompany"
+#              #Plugin "mycompany"
 #              Driver "mysql"
 #              DriverOption "host" "localhost"
 #              DriverOption "username" "collectd"
 #    </Result>
 #  </Query>
 #  <Database "product_information">
-#    #PluginName "warehouse"
+#    #Plugin "warehouse"
 #    ConnectID "db01"
 #    Username "oracle"
 #    Password "secret"
 #              StoreRates true
 #      </Writer>
 #      <Database foo>
-#              #PluginName "kingdom"
+#              #Plugin "kingdom"
 #              Host "hostname"
 #              Port "5432"
 #              User "username"
index b4fe2f9..cf34e9e 100644 (file)
@@ -2017,7 +2017,7 @@ than those of other plugins. It usually looks something like this:
       </Result>
     </Query>
     <Database "product_information">
-      #PluginName "warehouse"
+      #Plugin "warehouse"
       Driver "mysql"
       Interval 120
       DriverOption "host" "localhost"
@@ -2199,9 +2199,9 @@ the daemon. Other than that, that name is not used.
 
 =over 4
 
-=item B<PluginName> I<PluginName>
+=item B<Plugin> I<Plugin>
 
-Use I<PluginName> as the plugin name when submitting query results from
+Use I<Plugin> as the plugin name when submitting query results from
 this B<Database>. Defaults to 'dbi'.
 
 =item B<Interval> I<Interval>
@@ -5595,7 +5595,7 @@ plugin's documentation above for details.
       </Result>
     </Query>
     <Database "product_information">
-      #PluginName "warehouse"
+      #Plugin "warehouse"
       ConnectID "db01"
       Username "oracle"
       Password "secret"
@@ -5618,9 +5618,9 @@ values submitted to the daemon. Other than that, that name is not used.
 
 =over 4
 
-=item B<PluginName> I<PluginName>
+=item B<Plugin> I<Plugin>
 
-Use I<PluginName> as the plugin name when submitting query results from
+Use I<Plugin> as the plugin name when submitting query results from
 this B<Database>. Defaults to 'oracle'.
 
 =item B<ConnectID> I<ID>
@@ -5976,7 +5976,7 @@ L<http://www.postgresql.org/docs/manuals/>.
     </Writer>
 
     <Database foo>
-      PluginName "kingdom"
+      Plugin "kingdom"
       Host "hostname"
       Port "5432"
       User "username"
@@ -6298,9 +6298,9 @@ activating this option. The draw-back is, that data covering the specified
 amount of time will be lost, for example, if a single statement within the
 transaction fails or if the database server crashes.
 
-=item B<PluginName> I<PluginName>
+=item B<Plugin> I<Plugin>
 
-Use I<PluginName> as the plugin name when submitting query results from
+Use I<Plugin> as the plugin name when submitting query results from
 this B<Database>. Defaults to 'postgresql'.
 
 =item B<Instance> I<name>
index ba91bff..7f3f207 100644 (file)
--- a/src/dbi.c
+++ b/src/dbi.c
@@ -302,7 +302,7 @@ static int cdbi_config_add_database(oconfig_item_t *ci) /* {{{ */
       status = cf_util_get_string(child, &db->host);
     else if (strcasecmp("Interval", child->key) == 0)
       status = cf_util_get_cdtime(child, &db->interval);
-    else if (strcasecmp("PluginName", child->key) == 0)
+    else if (strcasecmp("Plugin", child->key) == 0)
       status = cf_util_get_string(child, &db->plugin_name);
     else {
       WARNING("dbi plugin: Option `%s' not allowed here.", child->key);
index 69e519b..6d245dd 100644 (file)
@@ -214,7 +214,7 @@ static int o_config_add_database(oconfig_item_t *ci) /* {{{ */
       status = cf_util_get_string(child, &db->username);
     else if (strcasecmp("Password", child->key) == 0)
       status = cf_util_get_string(child, &db->password);
-    else if (strcasecmp("PluginName", child->key) == 0)
+    else if (strcasecmp("Plugin", child->key) == 0)
       status = cf_util_get_string(child, &db->plugin_name);
     else if (strcasecmp("Query", child->key) == 0)
       status = udb_query_pick_from_list(child, queries, queries_num,
index 70467c2..45cd001 100644 (file)
@@ -1146,7 +1146,7 @@ static int c_psql_config_database(oconfig_item_t *ci) {
       cf_util_get_string(c, &db->password);
     else if (0 == strcasecmp(c->key, "Instance"))
       cf_util_get_string(c, &db->instance);
-    else if (0 == strcasecmp (c->key, "PluginName"))
+    else if (0 == strcasecmp (c->key, "Plugin"))
       cf_util_get_string (c, &db->plugin_name);
     else if (0 == strcasecmp(c->key, "SSLMode"))
       cf_util_get_string(c, &db->sslmode);