From d119a56673a95b8af60bae1e7b323fe7a27f5209 Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Wed, 28 Apr 2021 15:58:45 -0400
Subject: [PATCH 01/10] Added tslib to resolve some linting issues

---
 package.json | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/package.json b/package.json
index 63a8db1..346c116 100644
--- a/package.json
+++ b/package.json
@@ -11,7 +11,8 @@
   "author": "Circonus",
   "license": "BSD-3-clause",
   "dependencies": {
-    "memoizee": "^0.4.14"
+    "memoizee": "^0.4.14",
+    "tslib": "^2.2.0"
   },
   "devDependencies": {
     "@grafana/data": "^7.2.0",

From 3424439f490294811fe1e237f51ac5df0d88049f Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Wed, 28 Apr 2021 15:59:40 -0400
Subject: [PATCH 02/10] Adding .prettierrc.js because the grafana toolkit
 implemented it

---
 .prettierrc.js | 3 +++
 1 file changed, 3 insertions(+)
 create mode 100644 .prettierrc.js

diff --git a/.prettierrc.js b/.prettierrc.js
new file mode 100644
index 0000000..c14684b
--- /dev/null
+++ b/.prettierrc.js
@@ -0,0 +1,3 @@
+module.exports = {
+  ...require('@grafana/toolkit/src/config/prettier.plugin.config.json'),
+};

From 14de46b5baf28e898d19216c7415ca0ad56e0e97 Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Wed, 28 Apr 2021 16:01:18 -0400
Subject: [PATCH 03/10] updated gitignore with some IDE linting and testing
 artifact ignores

---
 .gitignore | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/.gitignore b/.gitignore
index 6b97c6e..600a192 100644
--- a/.gitignore
+++ b/.gitignore
@@ -3,3 +3,6 @@ npm-debug.log
 .DS_Store
 *.swp
 package-lock.json
+yarn.lock
+coverage
+jest*
\ No newline at end of file

From 41747f37ed64011e426b71903812fde68b3902fc Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Wed, 28 Apr 2021 16:01:47 -0400
Subject: [PATCH 04/10] Adding .eslintrc.json, grafana toolkit now uses eslint
 instead of tslint

---
 .eslintrc.json | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 .eslintrc.json

diff --git a/.eslintrc.json b/.eslintrc.json
new file mode 100644
index 0000000..31cb22e
--- /dev/null
+++ b/.eslintrc.json
@@ -0,0 +1,12 @@
+{
+  "root": true,
+  "parser": "@typescript-eslint/parser",
+  "plugins": ["@typescript-eslint"],
+  "extends": ["@grafana/eslint-config"],
+  "parserOptions": {
+      "tsconfigRootDir": "./"
+  },
+  "rules": {
+    "strict": "off"
+  }
+}
\ No newline at end of file

From b67d364827ef7bcf999320e57aa287cd2571bb68 Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Wed, 28 Apr 2021 16:14:27 -0400
Subject: [PATCH 05/10] resolving cherry-pick conflicts

---
 src/datasource.ts   | 35 ++++++++++-------------------------
 src/irondb_query.ts | 20 ++++++++++++++++----
 src/query_ctrl.ts   | 13 ++++++++++---
 3 files changed, 36 insertions(+), 32 deletions(-)

diff --git a/src/datasource.ts b/src/datasource.ts
index 9b0b168..b64a7e1 100644
--- a/src/datasource.ts
+++ b/src/datasource.ts
@@ -7,26 +7,23 @@ import memoize from 'memoizee';
 import { Memoized } from 'memoizee';
 import {
   metaInterpolateLabel,
-  decodeNameAndTags,
   isStatsdCounter,
   taglessName,
   taglessNameAndTags,
   decodeTag,
+  encodeTag,
   splitTags,
   mergeTags,
   TagSet,
   decodeTagsInLabel,
+  SegmentType,
 } from './irondb_query';
 
 import {
   ArrayVector,
   DataFrame,
-  Field,
   FieldType,
-  formatLabels,
-  Labels,
   MutableField,
-  ScopedVars,
   TIME_SERIES_TIME_FIELD_NAME,
   TIME_SERIES_VALUE_FIELD_NAME,
   DataSourceApi,
@@ -39,15 +36,11 @@ import {
   AnnotationQueryRequest,
   AnnotationEvent,
   LoadingState,
-  TimeSeries,
-  isTableData,
 } from '@grafana/data';
 import * as Mustache from 'mustache';
 
 const log = Log('IrondbDatasource');
 
-import { map } from 'rxjs/operators';
-
 const DURATION_UNITS_DEFAULT = 's';
 const DURATION_UNITS = {
   ms: 1,
@@ -477,20 +470,6 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
     if (typeof value === 'string') {
       return value;
     }
-
-    if (value.length === 1) {
-      return value[0];
-    }
-
-    let q = '';
-    for (let i = 0; i < value.length; i++) {
-      if (i > 0) {
-        q = q + ',';
-      }
-      q = q + value[i];
-      i = i + 1;
-    }
-    return q;
   }
 
   // This is used by the Dashboard Admin Variable Setup function
@@ -1328,7 +1307,7 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
   }
 
   buildFetchParamsAsync(cleanOptions, target, start, end) {
-    const rawQuery = this.templateSrv.replace(target['query'], cleanOptions['scopedVars']);
+    const rawQuery = this.templateSrv.replace(target['query'], cleanOptions['scopedVars'], this.interpolateExpr);
 
     return this.metricTagsQuery(rawQuery, false, start, end)
       .then((result) => {
@@ -1637,7 +1616,13 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
             vstr = v.toString();
             const tsstr = ts.toString();
             if (_.isUndefined(lookaside[vstr])) {
-              lookaside[vstr] = { target: vstr, title: vstr, tags: labels, datapoints: [], _ts: {} };
+              lookaside[vstr] = {
+                target: vstr,
+                title: vstr,
+                tags: labels,
+                datapoints: [],
+                _ts: {},
+              };
               dataFrames.push(lookaside[vstr]);
             }
             if (_.isUndefined(lookaside[vstr]._ts[tsstr])) {
diff --git a/src/irondb_query.ts b/src/irondb_query.ts
index cf9ea26..432c6ef 100644
--- a/src/irondb_query.ts
+++ b/src/irondb_query.ts
@@ -328,7 +328,10 @@ export default class IrondbQuery {
     metricName = metricName.slice(11, -1) || '*';
     const tags = metricName.split(',');
     metricName = tags.shift();
-    this.segments.push({ type: SegmentType.MetricName, value: decodeTag(metricName) });
+    this.segments.push({
+      type: SegmentType.MetricName,
+      value: decodeTag(metricName),
+    });
 
     let first = true;
     for (let tag of tags) {
@@ -349,17 +352,26 @@ export default class IrondbQuery {
         tagIndex = 3;
       }
       if (tagOp) {
-        this.segments.push({ type: SegmentType.TagOp, value: tagCat.slice(0, tagIndex) });
+        this.segments.push({
+          type: SegmentType.TagOp,
+          value: tagCat.slice(0, tagIndex),
+        });
         tagCat = tagCat.slice(tagIndex);
       }
-      this.segments.push({ type: SegmentType.TagCat, value: decodeTag(tagCat) });
+      this.segments.push({
+        type: SegmentType.TagCat,
+        value: decodeTag(tagCat),
+      });
       this.segments.push({ type: SegmentType.TagPair });
       let end = 0;
       while (tagVal.endsWith(')')) {
         tagVal = tagVal.slice(0, -1);
         end++;
       }
-      this.segments.push({ type: SegmentType.TagVal, value: decodeTag(tagVal) });
+      this.segments.push({
+        type: SegmentType.TagVal,
+        value: decodeTag(tagVal),
+      });
       for (let i = 0; i < end; i++) {
         this.segments.push({ type: SegmentType.TagPlus });
         this.segments.push({ type: SegmentType.TagEnd });
diff --git a/src/query_ctrl.ts b/src/query_ctrl.ts
index 7db6efc..f535874 100644
--- a/src/query_ctrl.ts
+++ b/src/query_ctrl.ts
@@ -1,7 +1,7 @@
 import _ from 'lodash';
 import Log from './log';
 import IrondbQuery from './irondb_query';
-/* eslint-disable-next-line no-duplicate-imports */
+// eslint-disable-next-line no-duplicate-imports
 import { SegmentType, taglessName, decodeTag, encodeTag } from './irondb_query';
 import { QueryCtrl } from 'grafana/app/plugins/sdk';
 import appEvents from 'grafana/app/core/app_events';
@@ -50,7 +50,10 @@ export class IrondbQueryCtrl extends QueryCtrl {
     { value: 'derive', text: 'rate of change (derive)' },
     { value: 'derive_stddev', text: 'rate of change σ (derive_stddev)' },
     { value: 'counter', text: 'rate of positive change (counter)' },
-    { value: 'counter_stddev', text: 'rate of positive change σ (counter_stddev)' },
+    {
+      value: 'counter_stddev',
+      text: 'rate of positive change σ (counter_stddev)',
+    },
   ];
   rollupTypeOptions = [
     { value: 'automatic', text: 'automatic' },
@@ -520,7 +523,11 @@ export class IrondbQueryCtrl extends QueryCtrl {
           segmentIndex + 1,
           0,
           this.mapSegment({ type: SegmentType.TagOp, value: segment.value }),
-          this.mapSegment({ type: SegmentType.TagCat, value: 'select tag', fake: true }),
+          this.mapSegment({
+            type: SegmentType.TagCat,
+            value: 'select tag',
+            fake: true,
+          }),
           this.mapSegment({ type: SegmentType.TagPair }),
           this.newSelectTagValSegment(),
           this.buildSelectTagPlusSegment(),

From 3e2830227a825bc9dfdd1e143b8fb05fab77a03a Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Wed, 28 Apr 2021 16:09:40 -0400
Subject: [PATCH 06/10] resolving more cherry-pick conflicts

---
 src/datasource.ts | 26 ++++++++++++++++++++------
 1 file changed, 20 insertions(+), 6 deletions(-)

diff --git a/src/datasource.ts b/src/datasource.ts
index b64a7e1..9036285 100644
--- a/src/datasource.ts
+++ b/src/datasource.ts
@@ -304,7 +304,7 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
       options.method = 'GET';
       options.url = options.url + '/alert';
       const alertId = this.templateSrv.replace(query.annotation['alertId']);
-      const alertQuery = this.templateSrv.replace(query.annotation['annotationQueryText']);
+      const alertQuery = this.templateSrv.replace(query.annotation['annotationQueryText'], null, this.interpolateExpr);
       if (alertId !== '') {
         options.url = options.url + '/' + alertId;
       } else {
@@ -462,6 +462,13 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
   }
 
   interpolateExpr(value: string | string[] = [], variable: any) {
+    if (typeof value === 'string') {
+      value = encodeTag(SegmentType.TagVal, value, false);
+    } else {
+      for (let i in value) {
+        value[i] = encodeTag(SegmentType.TagVal, value[i], false);
+      }
+    }
     // if no multi or include all do not regexEscape
     if (!variable.multi && !variable.includeAll) {
       return value;
@@ -469,6 +476,8 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
 
     if (typeof value === 'string') {
       return value;
+    } else {
+      return value.join(',');
     }
   }
 
@@ -507,7 +516,7 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
       if (tagCat !== '') {
         return this.metricFindTagValsQuery(metricQuery, tagCat, from, to).then((results) => {
           return _.map(results.data, (result) => {
-            return { value: result };
+            return { value: decodeTag(result) };
           });
         });
       } else {
@@ -776,7 +785,8 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
         end += interval;
         const caqlQuery = this.templateSrv.replace(
           irondbOptions['caql']['names'][i].leaf_name,
-          irondbOptions['scopedVars']
+          irondbOptions['scopedVars'],
+          this.interpolateExpr
         );
         options.url = options.url + '/caql_v1?format=DF4&start=' + start.toFixed(3);
         options.url = options.url + '&end=' + end.toFixed(3);
@@ -808,7 +818,11 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
         }
         options.method = 'GET';
         options.url = options.url + '/alert';
-        const alertQuery = this.templateSrv.replace(irondbOptions['alert']['names'][i], irondbOptions['scopedVars']);
+        const alertQuery = this.templateSrv.replace(
+          irondbOptions['alert']['names'][i],
+          irondbOptions['scopedVars'],
+          this.interpolateExpr
+        );
         if (alertQuery.startsWith('alert_id:')) {
           options.url = options.url + '/' + alertQuery.split(':')[1];
         } else {
@@ -1326,13 +1340,13 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
   }
 
   buildAlertQueryAsync(cleanOptions, target, start, end) {
-    let rawQuery = this.templateSrv.replace(target['query'], cleanOptions['scopedVars']);
+    let rawQuery = this.templateSrv.replace(target['query'], cleanOptions['scopedVars'], this.interpolateExpr);
     if (target['alert_id'] !== '') {
       rawQuery = 'alert_id:' + this.templateSrv.replace(target['alert_id'], cleanOptions['scopedVars']);
     }
     cleanOptions['alert']['names'].push(rawQuery);
     cleanOptions['alert']['local_filters'].push(
-      this.templateSrv.replace(target['local_filter'], cleanOptions['scopedVars'])
+      this.templateSrv.replace(target['local_filter'], cleanOptions['scopedVars'], this.interpolateExpr)
     );
     cleanOptions['alert']['local_filter_matches'].push(target['local_filter_match']);
     cleanOptions['alert']['counts_only'] = target.querytype === 'alert_counts';

From a5316c12148f3814f1d31f9759a6511030291b29 Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Thu, 29 Apr 2021 12:44:58 -0400
Subject: [PATCH 07/10] Adding Riley's changes to both tag related lookup
 functions

---
 src/datasource.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/datasource.ts b/src/datasource.ts
index 9036285..7505eba 100644
--- a/src/datasource.ts
+++ b/src/datasource.ts
@@ -534,7 +534,7 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
     return this.irondbType === 'standalone' ? '/' + this.accountId : '';
   }
 
-  metricTagsQuery(query: string, allowEmptyWildcard = false, from?: number = null, to?: number = null) {
+  metricTagsQuery(query: string, allowEmptyWildcard = false, from: number = null, to: number = null) {
     if (query === '' || query === undefined || (!allowEmptyWildcard && query === 'and(__name:*)')) {
       return Promise.resolve({ data: [] });
     }

From cc922820b818e3c8cdea12411ef0de6aba019b57 Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Tue, 11 May 2021 14:50:51 -0400
Subject: [PATCH 08/10] removing some debug code

---
 src/datasource.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/datasource.ts b/src/datasource.ts
index 7505eba..8106d15 100644
--- a/src/datasource.ts
+++ b/src/datasource.ts
@@ -389,7 +389,7 @@ export default class IrondbDatasource extends DataSourceApi<IrondbQueryInterface
               let notes =
                 rule !== undefined && rule !== null && rule['notes'] !== null && rule['notes'] !== ''
                   ? rule['notes']
-                  : 'Oh no!';
+                  : '';
 
               notes = Mustache.render(notes, tags);
 

From 4b50ad73dce2b0789e68f05889b5e8a56f5f1a15 Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Tue, 11 May 2021 14:58:15 -0400
Subject: [PATCH 09/10] updated changelog

---
 CHANGELOG.md | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index bb3d65b..08295ce 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,4 +1,9 @@
 
+## 0.9.5
+
+* Fix base64 encoding and decoding of tag values in variables
+* Made returned tag values respect activity windows
+
 ## 0.9.4
 
  * Fix labels for untagged metrics

From dc7bbb7d0dd50a05f141544acf4ea5d87dd1559c Mon Sep 17 00:00:00 2001
From: rizkybiz <justin.defrank@circonus.com>
Date: Tue, 11 May 2021 15:51:17 -0400
Subject: [PATCH 10/10] adding \n to end of .gitignore at githubs request

---
 .gitignore | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitignore b/.gitignore
index 600a192..2733e78 100644
--- a/.gitignore
+++ b/.gitignore
@@ -5,4 +5,4 @@ npm-debug.log
 package-lock.json
 yarn.lock
 coverage
-jest*
\ No newline at end of file
+jest*