Auto-detect zabbix version, closes #727

src/datasource-zabbix/config.controller.js

@@ -4,12 +4,6 @@ import { migrateDSConfig } from './migrations';
 
 const SUPPORTED_SQL_DS = ['mysql', 'postgres', 'influxdb'];
 
-const zabbixVersions = [
-  { name: '2.x', value: 2 },
-  { name: '3.x', value: 3 },
-  { name: '4.x', value: 4 },
-];
-
 const defaultConfig = {
   trends: false,
   dbConnectionEnable: false,
@@ -18,7 +12,6 @@ const defaultConfig = {
   addThresholds: false,
   alertingMinSeverity: 3,
   disableReadOnlyUsersAck: false,
-  zabbixVersion: 3,
 };
 
 export class ZabbixDSConfigController {
@@ -30,8 +23,6 @@ export class ZabbixDSConfigController {
 
     this.dbConnectionDatasourceId = this.current.jsonData.dbConnectionDatasourceId;
     this.dbDataSources = this.getSupportedDBDataSources();
-    this.zabbixVersions = _.cloneDeep(zabbixVersions);
-    this.autoDetectZabbixVersion();
     if (!this.dbConnectionDatasourceId) {
       this.loadCurrentDBDatasource();
     }
@@ -60,25 +51,6 @@ export class ZabbixDSConfigController {
     });
   }
 
-  autoDetectZabbixVersion() {
-    if (!this.current.id) {
-      return;
-    }
-
-    getDataSourceSrv().loadDatasource(this.current.name)
-    .then(ds => {
-      return ds.getVersion();
-    })
-    .then(version => {
-      if (version) {
-        if (!_.find(zabbixVersions, ['value', version])) {
-          this.zabbixVersions.push({ name: version + '.x', value: version });
-        }
-        this.current.jsonData.zabbixVersion = version;
-      }
-    });
-  }
-
   onDBConnectionDatasourceChange() {
     this.current.jsonData.dbConnectionDatasourceId = this.dbConnectionDatasourceId;
   }

src/datasource-zabbix/datasource.ts

@@ -13,8 +13,6 @@ import { Zabbix } from './zabbix/zabbix';
 import { ZabbixAPIError } from './zabbix/connectors/zabbix_api/zabbixAPICore';
 import { VariableQueryTypes, ShowProblemTypes } from './types';
 
-const DEFAULT_ZABBIX_VERSION = 3;
-
 export class ZabbixDatasource {
   name: string;
   url: string;
@@ -31,7 +29,6 @@ export class ZabbixDatasource {
   addThresholds: boolean;
   alertingMinSeverity: string;
   disableReadOnlyUsersAck: boolean;
-  zabbixVersion: string;
   enableDirectDBConnection: boolean;
   dbConnectionDatasourceId: number;
   dbConnectionDatasourceName: string;
@@ -79,7 +76,6 @@ export class ZabbixDatasource {
 
     // Other options
     this.disableReadOnlyUsersAck = jsonData.disableReadOnlyUsersAck;
-    this.zabbixVersion = jsonData.zabbixVersion || DEFAULT_ZABBIX_VERSION;
 
     // Direct DB Connection options
     this.enableDirectDBConnection = jsonData.dbConnectionEnable || false;
@@ -93,7 +89,6 @@ export class ZabbixDatasource {
       password: this.password,
       basicAuth: this.basicAuth,
       withCredentials: this.withCredentials,
-      zabbixVersion: this.zabbixVersion,
       cacheTTL: this.cacheTTL,
       enableDirectDBConnection: this.enableDirectDBConnection,
       dbConnectionDatasourceId: this.dbConnectionDatasourceId,
@@ -504,20 +499,6 @@ export class ZabbixDatasource {
     });
   }
 
-  /**
-   * Get Zabbix version
-   */
-  getVersion() {
-    return this.zabbix.getVersion()
-    .then(version => {
-      const zabbixVersion = utils.parseVersion(version);
-      if (!zabbixVersion) {
-        return null;
-      }
-      return zabbixVersion.major;
-    });
-  }
-
   ////////////////
   // Templating //
   ////////////////

src/datasource-zabbix/partials/config.html

@@ -73,15 +73,6 @@
       placeholder="1h">
     </input>
   </div>
-
-  <div class="gf-form max-width-20">
-    <span class="gf-form-label width-12">Zabbix version</span>
-    <div class="gf-form-select-wrapper max-width-7">
-      <select class="gf-form-input" ng-model="ctrl.current.jsonData.zabbixVersion"
-        ng-options="s.value as s.name for s in ctrl.zabbixVersions">
-      </select>
-    </div>
-  </div>
 </div>
 
 <div class="gf-form-group">

src/datasource-zabbix/utils.ts

@@ -315,7 +315,7 @@ export function isValidVersion(version) {
   return versionPattern.exec(version);
 }
 
-export function parseVersion(version) {
+export function parseVersion(version: string) {
   const match = versionPattern.exec(version);
   if (!match) {
     return null;

src/datasource-zabbix/zabbix/connectors/zabbix_api/zabbixAPIConnector.js

@@ -1,22 +1,24 @@
 import _ from 'lodash';
+import semver from 'semver';
 import kbn from 'grafana/app/core/utils/kbn';
 import * as utils from '../../../utils';
 import { ZabbixAPICore } from './zabbixAPICore';
 import { ZBX_ACK_ACTION_NONE, ZBX_ACK_ACTION_ACK, ZBX_ACK_ACTION_ADD_MESSAGE, MIN_SLA_INTERVAL } from '../../../constants';
 import { ShowProblemTypes } from '../../../types';
 
+const DEFAULT_ZABBIX_VERSION = '3.0.0';
+
 /**
  * Zabbix API Wrapper.
  * Creates Zabbix API instance with given parameters (url, credentials and other).
  * Wraps API calls and provides high-level methods.
  */
 export class ZabbixAPIConnector {
-  constructor(api_url, username, password, version, basicAuth, withCredentials) {
+  constructor(api_url, username, password, basicAuth, withCredentials) {
     this.url              = api_url;
     this.username         = username;
     this.password         = password;
     this.auth             = '';
-    this.version          = version;
 
     this.requestOptions = {
       basicAuth: basicAuth,
@@ -31,6 +33,8 @@ export class ZabbixAPIConnector {
 
     this.getTrend = this.getTrend_ZBXNEXT1193;
     //getTrend = getTrend_30;
+
+    this.initVersion();
   }
 
   //////////////////////////
@@ -38,6 +42,10 @@ export class ZabbixAPIConnector {
   //////////////////////////
 
   request(method, params) {
+    if (!this.version) {
+      return this.initVersion().then(() => this.request(method, params));
+    }
+
     return this.zabbixAPICore.request(this.url, method, params, this.requestOptions, this.auth)
     .catch(error => {
       if (isNotAuthorized(error.data)) {
@@ -89,12 +97,31 @@ export class ZabbixAPIConnector {
     return this.zabbixAPICore.getVersion(this.url, this.requestOptions);
   }
 
+  initVersion() {
+    if (!this.getVersionPromise) {
+      this.getVersionPromise = Promise.resolve(
+        this.getVersion().then(version => {
+          if (version) {
+            console.log(`Zabbix version detected: ${version}`);
+          } else {
+            console.log(`Failed to detect Zabbix version, use default ${DEFAULT_ZABBIX_VERSION}`);
+          }
+
+          this.version = version || DEFAULT_ZABBIX_VERSION;
+          this.getVersionPromise = null;
+          return version;
+        })
+      );
+    }
+    return this.getVersionPromise;
+  }
+
   ////////////////////////////////
   // Zabbix API method wrappers //
   ////////////////////////////////
 
   acknowledgeEvent(eventid, message) {
-    const action = this.version >= 4 ? ZBX_ACK_ACTION_ACK + ZBX_ACK_ACTION_ADD_MESSAGE : ZBX_ACK_ACTION_NONE;
+    const action = semver.gte(this.version, '4.0.0') ? ZBX_ACK_ACTION_ACK + ZBX_ACK_ACTION_ADD_MESSAGE : ZBX_ACK_ACTION_NONE;
     const params = {
       eventids: eventid,
       message: message,

src/datasource-zabbix/zabbix/connectors/zabbix_api/zabbixAPICore.js

@@ -82,7 +82,10 @@ export class ZabbixAPICore {
    * Matches the version of Zabbix starting from Zabbix 2.0.4
    */
   getVersion(api_url, options) {
-    return this.request(api_url, 'apiinfo.version', [], options);
+    return this.request(api_url, 'apiinfo.version', [], options).catch(err => {
+      console.error(err);
+      return undefined;
+    });
   }
 }
 

src/datasource-zabbix/zabbix/zabbix.js

@@ -32,7 +32,6 @@ export class Zabbix {
       password,
       basicAuth,
       withCredentials,
-      zabbixVersion,
       cacheTTL,
       enableDirectDBConnection,
       dbConnectionDatasourceId,
@@ -49,7 +48,7 @@ export class Zabbix {
     };
     this.cachingProxy = new CachingProxy(cacheOptions);
 
-    this.zabbixAPI = new ZabbixAPIConnector(url, username, password, zabbixVersion, basicAuth, withCredentials);
+    this.zabbixAPI = new ZabbixAPIConnector(url, username, password, basicAuth, withCredentials);
 
     this.proxyfyRequests();
     this.cacheRequests();

src/datasource-zabbix/zabbix/zabbix.test.js

@@ -8,7 +8,6 @@ describe('Zabbix', () => {
     url: 'http://localhost',
     username: 'zabbix',
     password: 'zabbix',
-    zabbixVersion: 4,
   };
 
   beforeEach(() => {