Browse Source

Merge remote-tracking branch 'upstream/master' into postgres-docs-timefilter

Sven Klemm 7 years ago
parent
commit
eb994ad72e

+ 1 - 5
.circleci/config.yml

@@ -117,7 +117,7 @@ jobs:
       - image: circleci/python:2.7-stretch
     steps:
       - attach_workspace:
-          at: dist
+          at: .
       - run:
           name: install awscli
           command: 'sudo pip install awscli'
@@ -139,10 +139,6 @@ workflows:
           filters:
             tags:
               only: /.*/
-      - gometalinter:
-          filters:
-            tags:
-              only: /.*/
       - build:
           filters:
             tags:

+ 1 - 1
CHANGELOG.md

@@ -1,4 +1,4 @@
-# 5.1.0 (unreleased)
+# 5.1.0-beta1 (2018-04-20)
 
 * **MSSQL**: New Microsoft SQL Server data source [#10093](https://github.com/grafana/grafana/pull/10093), [#11298](https://github.com/grafana/grafana/pull/11298), thx [@linuxchips](https://github.com/linuxchips)
 * **Prometheus**: The heatmap panel now support Prometheus histograms [#10009](https://github.com/grafana/grafana/issues/10009)

+ 2 - 0
docs/sources/http_api/annotations.md

@@ -36,6 +36,8 @@ Query Parameters:
 - `alertId`: number. Optional. Find annotations for a specified alert.
 - `dashboardId`: number. Optional. Find annotations that are scoped to a specific dashboard
 - `panelId`: number. Optional. Find annotations that are scoped to a specific panel
+- `userId`: number. Optional. Find annotations created by a specific user
+- `type`: string. Optional. `alert`|`annotation` Return alerts or user created annotations
 - `tags`: string. Optional. Use this to filter global annotations. Global annotations are annotations from an annotation data source that are not connected specifically to a dashboard or panel. To do an "AND" filtering with multiple tags, specify the tags parameter multiple times e.g. `tags=tag1&tags=tag2`.
 
 **Example Response**:

+ 8 - 0
docs/sources/installation/debian.md

@@ -16,6 +16,7 @@ weight = 1
 Description | Download
 ------------ | -------------
 Stable for Debian-based Linux | [grafana_5.0.4_amd64.deb](https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana_5.0.4_amd64.deb)
+Beta for Debian-based Linux | [grafana_5.1.0-beta1_amd64.deb](https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana_5.1.0-beta1_amd64.deb)
 
 Read [Upgrading Grafana]({{< relref "installation/upgrading.md" >}}) for tips and guidance on updating an existing
 installation.
@@ -29,6 +30,13 @@ sudo apt-get install -y adduser libfontconfig
 sudo dpkg -i grafana_5.0.4_amd64.deb
 ```
 
+## Install Latest Beta
+```bash
+wget https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana_5.1.0-beta1_amd64.deb
+sudo apt-get install -y adduser libfontconfig
+sudo dpkg -i grafana_5.1.0-beta1_amd64.deb
+```
+
 ## APT Repository
 
 Add the following line to your `/etc/apt/sources.list` file.

+ 7 - 0
docs/sources/installation/rpm.md

@@ -16,6 +16,7 @@ weight = 2
 Description | Download
 ------------ | -------------
 Stable for CentOS / Fedora / OpenSuse / Redhat Linux | [5.0.4 (x86-64 rpm)](https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana-5.0.4-1.x86_64.rpm)
+Latest Beta for CentOS / Fedora / OpenSuse / Redhat Linux | [5.1.0-beta1 (x86-64 rpm)](https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana-5.1.0-beta1.x86_64.rpm)
 
 
 Read [Upgrading Grafana]({{< relref "installation/upgrading.md" >}}) for tips and guidance on updating an existing
@@ -29,6 +30,12 @@ You can install Grafana using Yum directly.
 $ sudo yum install https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana-5.0.4-1.x86_64.rpm
 ```
 
+## Install Beta
+
+```bash
+$ sudo yum install https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana-5.1.0-beta1.x86_64.rpm
+```
+
 Or install manually using `rpm`.
 
 #### On CentOS / Fedora / Redhat:

+ 1 - 0
docs/sources/installation/windows.md

@@ -13,6 +13,7 @@ weight = 3
 Description | Download
 ------------ | -------------
 Latest stable package for Windows | [grafana-5.0.4.windows-x64.zip](https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana-5.0.4.windows-x64.zip)
+Latest beta package for Windows | [grafana.5.1.0-beta1.windows-x64.zip](https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana-5.0.0-beta5.windows-x64.zip)
 
 Read [Upgrading Grafana]({{< relref "installation/upgrading.md" >}}) for tips and guidance on updating an existing
 installation.

+ 2 - 2
packaging/publish/publish_testing.sh

@@ -1,6 +1,6 @@
 #! /usr/bin/env bash
-deb_ver=5.0.0-beta5
-rpm_ver=5.0.0-beta5
+deb_ver=5.1.0-beta1
+rpm_ver=5.1.0-beta1
 
 wget https://s3-us-west-2.amazonaws.com/grafana-releases/release/grafana_${deb_ver}_amd64.deb
 

+ 9 - 16
pkg/api/annotations.go

@@ -2,7 +2,6 @@ package api
 
 import (
 	"strings"
-	"time"
 
 	"github.com/grafana/grafana/pkg/api/dtos"
 	"github.com/grafana/grafana/pkg/components/simplejson"
@@ -15,9 +14,10 @@ import (
 func GetAnnotations(c *m.ReqContext) Response {
 
 	query := &annotations.ItemQuery{
-		From:        c.QueryInt64("from") / 1000,
-		To:          c.QueryInt64("to") / 1000,
+		From:        c.QueryInt64("from"),
+		To:          c.QueryInt64("to"),
 		OrgId:       c.OrgId,
+		UserId:      c.QueryInt64("userId"),
 		AlertId:     c.QueryInt64("alertId"),
 		DashboardId: c.QueryInt64("dashboardId"),
 		PanelId:     c.QueryInt64("panelId"),
@@ -37,7 +37,7 @@ func GetAnnotations(c *m.ReqContext) Response {
 		if item.Email != "" {
 			item.AvatarUrl = dtos.GetGravatarUrl(item.Email)
 		}
-		item.Time = item.Time * 1000
+		item.Time = item.Time
 	}
 
 	return JSON(200, items)
@@ -68,16 +68,12 @@ func PostAnnotation(c *m.ReqContext, cmd dtos.PostAnnotationsCmd) Response {
 		UserId:      c.UserId,
 		DashboardId: cmd.DashboardId,
 		PanelId:     cmd.PanelId,
-		Epoch:       cmd.Time / 1000,
+		Epoch:       cmd.Time,
 		Text:        cmd.Text,
 		Data:        cmd.Data,
 		Tags:        cmd.Tags,
 	}
 
-	if item.Epoch == 0 {
-		item.Epoch = time.Now().Unix()
-	}
-
 	if err := repo.Save(&item); err != nil {
 		return Error(500, "Failed to save annotation", err)
 	}
@@ -97,7 +93,7 @@ func PostAnnotation(c *m.ReqContext, cmd dtos.PostAnnotationsCmd) Response {
 		}
 
 		item.Id = 0
-		item.Epoch = cmd.TimeEnd / 1000
+		item.Epoch = cmd.TimeEnd
 
 		if err := repo.Save(&item); err != nil {
 			return Error(500, "Failed save annotation for region end time", err)
@@ -132,9 +128,6 @@ func PostGraphiteAnnotation(c *m.ReqContext, cmd dtos.PostGraphiteAnnotationsCmd
 		return Error(500, "Failed to save Graphite annotation", err)
 	}
 
-	if cmd.When == 0 {
-		cmd.When = time.Now().Unix()
-	}
 	text := formatGraphiteAnnotation(cmd.What, cmd.Data)
 
 	// Support tags in prior to Graphite 0.10.0 format (string of tags separated by space)
@@ -163,7 +156,7 @@ func PostGraphiteAnnotation(c *m.ReqContext, cmd dtos.PostGraphiteAnnotationsCmd
 	item := annotations.Item{
 		OrgId:  c.OrgId,
 		UserId: c.UserId,
-		Epoch:  cmd.When,
+		Epoch:  cmd.When * 1000,
 		Text:   text,
 		Tags:   tagsArray,
 	}
@@ -191,7 +184,7 @@ func UpdateAnnotation(c *m.ReqContext, cmd dtos.UpdateAnnotationsCmd) Response {
 		OrgId:  c.OrgId,
 		UserId: c.UserId,
 		Id:     annotationID,
-		Epoch:  cmd.Time / 1000,
+		Epoch:  cmd.Time,
 		Text:   cmd.Text,
 		Tags:   cmd.Tags,
 	}
@@ -203,7 +196,7 @@ func UpdateAnnotation(c *m.ReqContext, cmd dtos.UpdateAnnotationsCmd) Response {
 	if cmd.IsRegion {
 		itemRight := item
 		itemRight.RegionId = item.Id
-		itemRight.Epoch = cmd.TimeEnd / 1000
+		itemRight.Epoch = cmd.TimeEnd
 
 		// We don't know id of region right event, so set it to 0 and find then using query like
 		// ... WHERE region_id = <item.RegionId> AND id != <item.RegionId> ...

+ 3 - 0
pkg/cmd/grafana-cli/services/services.go

@@ -10,6 +10,7 @@ import (
 	"net/http"
 	"net/url"
 	"path"
+	"runtime"
 	"time"
 
 	"github.com/grafana/grafana/pkg/cmd/grafana-cli/logger"
@@ -155,6 +156,8 @@ func sendRequest(repoUrl string, subPaths ...string) ([]byte, error) {
 	req, err := http.NewRequest(http.MethodGet, u.String(), nil)
 
 	req.Header.Set("grafana-version", grafanaVersion)
+	req.Header.Set("grafana-os", runtime.GOOS)
+	req.Header.Set("grafana-arch", runtime.GOARCH)
 	req.Header.Set("User-Agent", "grafana "+grafanaVersion)
 
 	if err != nil {

+ 1 - 1
pkg/services/alerting/result_handler.go

@@ -77,7 +77,7 @@ func (handler *DefaultResultHandler) Handle(evalContext *EvalContext) error {
 			Text:        "",
 			NewState:    string(evalContext.Rule.State),
 			PrevState:   string(evalContext.PrevAlertState),
-			Epoch:       time.Now().Unix(),
+			Epoch:       time.Now().UnixNano() / int64(time.Millisecond),
 			Data:        annotationData,
 		}
 

+ 5 - 0
pkg/services/annotations/annotations.go

@@ -13,6 +13,7 @@ type ItemQuery struct {
 	OrgId        int64    `json:"orgId"`
 	From         int64    `json:"from"`
 	To           int64    `json:"to"`
+	UserId       int64    `json:"userId"`
 	AlertId      int64    `json:"alertId"`
 	DashboardId  int64    `json:"dashboardId"`
 	PanelId      int64    `json:"panelId"`
@@ -63,6 +64,8 @@ type Item struct {
 	PrevState   string           `json:"prevState"`
 	NewState    string           `json:"newState"`
 	Epoch       int64            `json:"epoch"`
+	Created     int64            `json:"created"`
+	Updated     int64            `json:"updated"`
 	Tags        []string         `json:"tags"`
 	Data        *simplejson.Json `json:"data"`
 
@@ -80,6 +83,8 @@ type ItemDTO struct {
 	UserId      int64            `json:"userId"`
 	NewState    string           `json:"newState"`
 	PrevState   string           `json:"prevState"`
+	Created     int64            `json:"created"`
+	Updated     int64            `json:"updated"`
 	Time        int64            `json:"time"`
 	Text        string           `json:"text"`
 	RegionId    int64            `json:"regionId"`

+ 18 - 1
pkg/services/sqlstore/annotation.go

@@ -5,6 +5,7 @@ import (
 	"errors"
 	"fmt"
 	"strings"
+	"time"
 
 	"github.com/grafana/grafana/pkg/models"
 	"github.com/grafana/grafana/pkg/services/annotations"
@@ -17,6 +18,12 @@ func (r *SqlAnnotationRepo) Save(item *annotations.Item) error {
 	return inTransaction(func(sess *DBSession) error {
 		tags := models.ParseTagPairs(item.Tags)
 		item.Tags = models.JoinTagPairs(tags)
+		item.Created = time.Now().UnixNano() / int64(time.Millisecond)
+		item.Updated = item.Created
+		if item.Epoch == 0 {
+			item.Epoch = item.Created
+		}
+
 		if _, err := sess.Table("annotation").Insert(item); err != nil {
 			return err
 		}
@@ -79,6 +86,7 @@ func (r *SqlAnnotationRepo) Update(item *annotations.Item) error {
 			return errors.New("Annotation not found")
 		}
 
+		existing.Updated = time.Now().UnixNano() / int64(time.Millisecond)
 		existing.Epoch = item.Epoch
 		existing.Text = item.Text
 		if item.RegionId != 0 {
@@ -102,7 +110,7 @@ func (r *SqlAnnotationRepo) Update(item *annotations.Item) error {
 
 		existing.Tags = item.Tags
 
-		_, err = sess.Table("annotation").Id(existing.Id).Cols("epoch", "text", "region_id", "tags").Update(existing)
+		_, err = sess.Table("annotation").Id(existing.Id).Cols("epoch", "text", "region_id", "updated", "tags").Update(existing)
 		return err
 	})
 }
@@ -124,6 +132,8 @@ func (r *SqlAnnotationRepo) Find(query *annotations.ItemQuery) ([]*annotations.I
 			annotation.text,
 			annotation.tags,
 			annotation.data,
+			annotation.created,
+			annotation.updated,
 			usr.email,
 			usr.login,
 			alert.name as alert_name
@@ -161,6 +171,11 @@ func (r *SqlAnnotationRepo) Find(query *annotations.ItemQuery) ([]*annotations.I
 		params = append(params, query.PanelId)
 	}
 
+	if query.UserId != 0 {
+		sql.WriteString(` AND annotation.user_id = ?`)
+		params = append(params, query.UserId)
+	}
+
 	if query.From > 0 && query.To > 0 {
 		sql.WriteString(` AND annotation.epoch BETWEEN ? AND ?`)
 		params = append(params, query.From, query.To)
@@ -168,6 +183,8 @@ func (r *SqlAnnotationRepo) Find(query *annotations.ItemQuery) ([]*annotations.I
 
 	if query.Type == "alert" {
 		sql.WriteString(` AND annotation.alert_id > 0`)
+	} else if query.Type == "annotation" {
+		sql.WriteString(` AND annotation.alert_id = 0`)
 	}
 
 	if len(query.Tags) > 0 {

+ 10 - 0
pkg/services/sqlstore/annotation_test.go

@@ -79,6 +79,12 @@ func TestAnnotations(t *testing.T) {
 				Convey("Can read tags", func() {
 					So(items[0].Tags, ShouldResemble, []string{"outage", "error", "type:outage", "server:server-1"})
 				})
+
+				Convey("Has created and updated values", func() {
+					So(items[0].Created, ShouldBeGreaterThan, 0)
+					So(items[0].Updated, ShouldBeGreaterThan, 0)
+					So(items[0].Updated, ShouldEqual, items[0].Created)
+				})
 			})
 
 			Convey("Can query for annotation by id", func() {
@@ -231,6 +237,10 @@ func TestAnnotations(t *testing.T) {
 					So(items[0].Tags, ShouldResemble, []string{"newtag1", "newtag2"})
 					So(items[0].Text, ShouldEqual, "something new")
 				})
+
+				Convey("Updated time has increased", func() {
+					So(items[0].Updated, ShouldBeGreaterThan, items[0].Created)
+				})
 			})
 
 			Convey("Can delete annotation", func() {

+ 25 - 0
pkg/services/sqlstore/migrations/annotation_mig.go

@@ -90,4 +90,29 @@ func addAnnotationMig(mg *Migrator) {
 		Sqlite(updateTextFieldSql).
 		Postgres(updateTextFieldSql).
 		Mysql(updateTextFieldSql))
+
+	//
+	// Add a 'created' & 'updated' column
+	//
+	mg.AddMigration("Add created time to annotation table", NewAddColumnMigration(table, &Column{
+		Name: "created", Type: DB_BigInt, Nullable: true, Default: "0",
+	}))
+	mg.AddMigration("Add updated time to annotation table", NewAddColumnMigration(table, &Column{
+		Name: "updated", Type: DB_BigInt, Nullable: true, Default: "0",
+	}))
+	mg.AddMigration("Add index for created in annotation table", NewAddIndexMigration(table, &Index{
+		Cols: []string{"org_id", "created"}, Type: IndexType,
+	}))
+	mg.AddMigration("Add index for updated in annotation table", NewAddIndexMigration(table, &Index{
+		Cols: []string{"org_id", "updated"}, Type: IndexType,
+	}))
+
+	//
+	// Convert epoch saved as seconds to miliseconds
+	//
+	updateEpochSql := "UPDATE annotation SET epoch = (epoch*1000) where epoch < 9999999999"
+	mg.AddMigration("Convert existing annotations from seconds to milliseconds", new(RawSqlMigration).
+		Sqlite(updateEpochSql).
+		Postgres(updateEpochSql).
+		Mysql(updateEpochSql))
 }

+ 68 - 88
public/app/plugins/datasource/graphite/specs/datasource_specs.ts → public/app/plugins/datasource/graphite/specs/datasource.jest.ts

@@ -1,30 +1,19 @@
-import { describe, beforeEach, it, expect, angularMocks } from 'test/lib/common';
-import helpers from 'test/specs/helpers';
 import { GraphiteDatasource } from '../datasource';
 import moment from 'moment';
 import _ from 'lodash';
+import $q from 'q';
+import { TemplateSrvStub } from 'test/specs/helpers';
 
-describe('graphiteDatasource', function() {
-  let ctx = new helpers.ServiceTestContext();
-  let instanceSettings: any = { url: [''], name: 'graphiteProd', jsonData: {} };
-
-  beforeEach(angularMocks.module('grafana.core'));
-  beforeEach(angularMocks.module('grafana.services'));
-  beforeEach(ctx.providePhase(['backendSrv', 'templateSrv']));
-  beforeEach(
-    angularMocks.inject(function($q, $rootScope, $httpBackend, $injector) {
-      ctx.$q = $q;
-      ctx.$httpBackend = $httpBackend;
-      ctx.$rootScope = $rootScope;
-      ctx.$injector = $injector;
-      $httpBackend.when('GET', /\.html$/).respond('');
-    })
-  );
+describe('graphiteDatasource', () => {
+  let ctx: any = {
+    backendSrv: {},
+    $q: $q,
+    templateSrv: new TemplateSrvStub(),
+  };
 
   beforeEach(function() {
-    ctx.ds = ctx.$injector.instantiate(GraphiteDatasource, {
-      instanceSettings: instanceSettings,
-    });
+    ctx.instanceSettings = { url: [''], name: 'graphiteProd', jsonData: {} };
+    ctx.ds = new GraphiteDatasource(ctx.instanceSettings, ctx.$q, ctx.backendSrv, ctx.templateSrv);
   });
 
   describe('When querying graphite with one target using query editor target spec', function() {
@@ -38,7 +27,7 @@ describe('graphiteDatasource', function() {
     let results;
     let requestOptions;
 
-    beforeEach(function() {
+    beforeEach(async () => {
       ctx.backendSrv.datasourceRequest = function(options) {
         requestOptions = options;
         return ctx.$q.when({
@@ -46,40 +35,39 @@ describe('graphiteDatasource', function() {
         });
       };
 
-      ctx.ds.query(query).then(function(data) {
+      await ctx.ds.query(query).then(function(data) {
         results = data;
       });
-      ctx.$rootScope.$apply();
     });
 
     it('should generate the correct query', function() {
-      expect(requestOptions.url).to.be('/render');
+      expect(requestOptions.url).toBe('/render');
     });
 
     it('should set unique requestId', function() {
-      expect(requestOptions.requestId).to.be('graphiteProd.panelId.3');
+      expect(requestOptions.requestId).toBe('graphiteProd.panelId.3');
     });
 
     it('should query correctly', function() {
       let params = requestOptions.data.split('&');
-      expect(params).to.contain('target=prod1.count');
-      expect(params).to.contain('target=prod2.count');
-      expect(params).to.contain('from=-1h');
-      expect(params).to.contain('until=now');
+      expect(params).toContain('target=prod1.count');
+      expect(params).toContain('target=prod2.count');
+      expect(params).toContain('from=-1h');
+      expect(params).toContain('until=now');
     });
 
     it('should exclude undefined params', function() {
       let params = requestOptions.data.split('&');
-      expect(params).to.not.contain('cacheTimeout=undefined');
+      expect(params).not.toContain('cacheTimeout=undefined');
     });
 
     it('should return series list', function() {
-      expect(results.data.length).to.be(1);
-      expect(results.data[0].target).to.be('prod1.count');
+      expect(results.data.length).toBe(1);
+      expect(results.data[0].target).toBe('prod1.count');
     });
 
     it('should convert to millisecond resolution', function() {
-      expect(results.data[0].datapoints[0][0]).to.be(10);
+      expect(results.data[0].datapoints[0][0]).toBe(10);
     });
   });
 
@@ -110,22 +98,21 @@ describe('graphiteDatasource', function() {
         ],
       };
 
-      beforeEach(() => {
+      beforeEach(async () => {
         ctx.backendSrv.datasourceRequest = function(options) {
           return ctx.$q.when(response);
         };
 
-        ctx.ds.annotationQuery(options).then(function(data) {
+        await ctx.ds.annotationQuery(options).then(function(data) {
           results = data;
         });
-        ctx.$rootScope.$apply();
       });
 
       it('should parse the tags string into an array', () => {
-        expect(_.isArray(results[0].tags)).to.eql(true);
-        expect(results[0].tags.length).to.eql(2);
-        expect(results[0].tags[0]).to.eql('tag1');
-        expect(results[0].tags[1]).to.eql('tag2');
+        expect(_.isArray(results[0].tags)).toEqual(true);
+        expect(results[0].tags.length).toEqual(2);
+        expect(results[0].tags[0]).toEqual('tag1');
+        expect(results[0].tags[1]).toEqual('tag2');
       });
     });
 
@@ -149,14 +136,14 @@ describe('graphiteDatasource', function() {
         ctx.ds.annotationQuery(options).then(function(data) {
           results = data;
         });
-        ctx.$rootScope.$apply();
+        // ctx.$rootScope.$apply();
       });
 
       it('should parse the tags string into an array', () => {
-        expect(_.isArray(results[0].tags)).to.eql(true);
-        expect(results[0].tags.length).to.eql(2);
-        expect(results[0].tags[0]).to.eql('tag1');
-        expect(results[0].tags[1]).to.eql('tag2');
+        expect(_.isArray(results[0].tags)).toEqual(true);
+        expect(results[0].tags.length).toEqual(2);
+        expect(results[0].tags[0]).toEqual('tag1');
+        expect(results[0].tags[1]).toEqual('tag2');
       });
     });
   });
@@ -166,21 +153,21 @@ describe('graphiteDatasource', function() {
       let results = ctx.ds.buildGraphiteParams({
         targets: [{}],
       });
-      expect(results.length).to.be(0);
+      expect(results.length).toBe(0);
     });
 
     it('should uri escape targets', function() {
       let results = ctx.ds.buildGraphiteParams({
         targets: [{ target: 'prod1.{test,test2}' }, { target: 'prod2.count' }],
       });
-      expect(results).to.contain('target=prod1.%7Btest%2Ctest2%7D');
+      expect(results).toContain('target=prod1.%7Btest%2Ctest2%7D');
     });
 
     it('should replace target placeholder', function() {
       let results = ctx.ds.buildGraphiteParams({
         targets: [{ target: 'series1' }, { target: 'series2' }, { target: 'asPercent(#A,#B)' }],
       });
-      expect(results[2]).to.be('target=asPercent(series1%2Cseries2)');
+      expect(results[2]).toBe('target=asPercent(series1%2Cseries2)');
     });
 
     it('should replace target placeholder for hidden series', function() {
@@ -191,35 +178,35 @@ describe('graphiteDatasource', function() {
           { target: 'asPercent(#A,#B)' },
         ],
       });
-      expect(results[0]).to.be('target=' + encodeURIComponent('asPercent(series1,sumSeries(series1))'));
+      expect(results[0]).toBe('target=' + encodeURIComponent('asPercent(series1,sumSeries(series1))'));
     });
 
     it('should replace target placeholder when nesting query references', function() {
       let results = ctx.ds.buildGraphiteParams({
         targets: [{ target: 'series1' }, { target: 'sumSeries(#A)' }, { target: 'asPercent(#A,#B)' }],
       });
-      expect(results[2]).to.be('target=' + encodeURIComponent('asPercent(series1,sumSeries(series1))'));
+      expect(results[2]).toBe('target=' + encodeURIComponent('asPercent(series1,sumSeries(series1))'));
     });
 
     it('should fix wrong minute interval parameters', function() {
       let results = ctx.ds.buildGraphiteParams({
         targets: [{ target: "summarize(prod.25m.count, '25m', 'sum')" }],
       });
-      expect(results[0]).to.be('target=' + encodeURIComponent("summarize(prod.25m.count, '25min', 'sum')"));
+      expect(results[0]).toBe('target=' + encodeURIComponent("summarize(prod.25m.count, '25min', 'sum')"));
     });
 
     it('should fix wrong month interval parameters', function() {
       let results = ctx.ds.buildGraphiteParams({
         targets: [{ target: "summarize(prod.5M.count, '5M', 'sum')" }],
       });
-      expect(results[0]).to.be('target=' + encodeURIComponent("summarize(prod.5M.count, '5mon', 'sum')"));
+      expect(results[0]).toBe('target=' + encodeURIComponent("summarize(prod.5M.count, '5mon', 'sum')"));
     });
 
     it('should ignore empty targets', function() {
       let results = ctx.ds.buildGraphiteParams({
         targets: [{ target: 'series1' }, { target: '' }],
       });
-      expect(results.length).to.be(2);
+      expect(results.length).toBe(2);
     });
   });
 
@@ -231,7 +218,7 @@ describe('graphiteDatasource', function() {
       ctx.backendSrv.datasourceRequest = function(options) {
         requestOptions = options;
         return ctx.$q.when({
-          data: [{ target: 'prod1.count', datapoints: [[10, 1], [12, 1]] }],
+          data: ['backend_01', 'backend_02'],
         });
       };
     });
@@ -241,10 +228,9 @@ describe('graphiteDatasource', function() {
         results = data;
       });
 
-      ctx.$rootScope.$apply();
-      expect(requestOptions.url).to.be('/tags/autoComplete/tags');
-      expect(requestOptions.params.expr).to.eql([]);
-      expect(results).not.to.be(null);
+      expect(requestOptions.url).toBe('/tags/autoComplete/tags');
+      expect(requestOptions.params.expr).toEqual([]);
+      expect(results).not.toBe(null);
     });
 
     it('should generate tags query with a filter expression', () => {
@@ -252,21 +238,19 @@ describe('graphiteDatasource', function() {
         results = data;
       });
 
-      ctx.$rootScope.$apply();
-      expect(requestOptions.url).to.be('/tags/autoComplete/tags');
-      expect(requestOptions.params.expr).to.eql(['server=backend_01']);
-      expect(results).not.to.be(null);
+      expect(requestOptions.url).toBe('/tags/autoComplete/tags');
+      expect(requestOptions.params.expr).toEqual(['server=backend_01']);
+      expect(results).not.toBe(null);
     });
 
-    it('should generate tag query for an expression with whitespace after', () => {
+    it('should generate tags query for an expression with whitespace after', () => {
       ctx.ds.metricFindQuery('tags(server=backend_01 )').then(data => {
         results = data;
       });
 
-      ctx.$rootScope.$apply();
-      expect(requestOptions.url).to.be('/tags/autoComplete/tags');
-      expect(requestOptions.params.expr).to.eql(['server=backend_01']);
-      expect(results).not.to.be(null);
+      expect(requestOptions.url).toBe('/tags/autoComplete/tags');
+      expect(requestOptions.params.expr).toEqual(['server=backend_01']);
+      expect(results).not.toBe(null);
     });
 
     it('should generate tag values query for one tag', () => {
@@ -274,11 +258,10 @@ describe('graphiteDatasource', function() {
         results = data;
       });
 
-      ctx.$rootScope.$apply();
-      expect(requestOptions.url).to.be('/tags/autoComplete/values');
-      expect(requestOptions.params.tag).to.be('server');
-      expect(requestOptions.params.expr).to.eql([]);
-      expect(results).not.to.be(null);
+      expect(requestOptions.url).toBe('/tags/autoComplete/values');
+      expect(requestOptions.params.tag).toBe('server');
+      expect(requestOptions.params.expr).toEqual([]);
+      expect(results).not.toBe(null);
     });
 
     it('should generate tag values query for a tag and expression', () => {
@@ -286,11 +269,10 @@ describe('graphiteDatasource', function() {
         results = data;
       });
 
-      ctx.$rootScope.$apply();
-      expect(requestOptions.url).to.be('/tags/autoComplete/values');
-      expect(requestOptions.params.tag).to.be('server');
-      expect(requestOptions.params.expr).to.eql(['server=~backend*']);
-      expect(results).not.to.be(null);
+      expect(requestOptions.url).toBe('/tags/autoComplete/values');
+      expect(requestOptions.params.tag).toBe('server');
+      expect(requestOptions.params.expr).toEqual(['server=~backend*']);
+      expect(results).not.toBe(null);
     });
 
     it('should generate tag values query for a tag with whitespace after', () => {
@@ -298,11 +280,10 @@ describe('graphiteDatasource', function() {
         results = data;
       });
 
-      ctx.$rootScope.$apply();
-      expect(requestOptions.url).to.be('/tags/autoComplete/values');
-      expect(requestOptions.params.tag).to.be('server');
-      expect(requestOptions.params.expr).to.eql([]);
-      expect(results).not.to.be(null);
+      expect(requestOptions.url).toBe('/tags/autoComplete/values');
+      expect(requestOptions.params.tag).toBe('server');
+      expect(requestOptions.params.expr).toEqual([]);
+      expect(results).not.toBe(null);
     });
 
     it('should generate tag values query for a tag and expression with whitespace after', () => {
@@ -310,11 +291,10 @@ describe('graphiteDatasource', function() {
         results = data;
       });
 
-      ctx.$rootScope.$apply();
-      expect(requestOptions.url).to.be('/tags/autoComplete/values');
-      expect(requestOptions.params.tag).to.be('server');
-      expect(requestOptions.params.expr).to.eql(['server=~backend*']);
-      expect(results).not.to.be(null);
+      expect(requestOptions.url).toBe('/tags/autoComplete/values');
+      expect(requestOptions.params.tag).toBe('server');
+      expect(requestOptions.params.expr).toEqual(['server=~backend*']);
+      expect(results).not.toBe(null);
     });
   });
 });

+ 2 - 2
public/sass/components/_gf-form.scss

@@ -341,19 +341,19 @@ $input-border: 1px solid $input-border-color;
   margin-right: $gf-form-margin;
   position: relative;
   background-color: $input-bg;
-  padding-right: $input-padding-x;
   border: $input-border;
   border-radius: $input-border-radius;
 
   &::after {
     position: absolute;
     top: 35%;
-    right: $input-padding-x/2;
+    right: $input-padding-x;
     background-color: transparent;
     color: $input-color;
     font: normal normal normal $font-size-sm/1 FontAwesome;
     content: '\f0d7';
     pointer-events: none;
+    font-size: 11px;
   }
 
   .gf-form-input {