Azure: query builder cleanup (#25956)

public/app/plugins/datasource/grafana-azure-monitor-datasource/azure_monitor/azure_monitor_datasource.test.ts

@@ -931,11 +931,22 @@ describe('AzureMonitorDatasource', () => {
           'Transactions'
         )
         .then((results: any) => {
-          expect(results.dimensions.length).toEqual(4);
+          expect(results.dimensions).toMatchInlineSnapshot(`
-          expect(results.dimensions[0].text).toEqual('None');
+            Array [
-          expect(results.dimensions[0].value).toEqual('None');
+              Object {
-          expect(results.dimensions[1].text).toEqual('Response type');
+                "text": "Response type",
-          expect(results.dimensions[1].value).toEqual('ResponseType');
+                "value": "ResponseType",
+              },
+              Object {
+                "text": "Geo type",
+                "value": "GeoType",
+              },
+              Object {
+                "text": "API name",
+                "value": "ApiName",
+              },
+            ]
+          `);
         });
     });
 

public/app/plugins/datasource/grafana-azure-monitor-datasource/azure_monitor/azure_monitor_datasource.ts

@@ -74,13 +74,16 @@ export default class AzureMonitorDatasource extends DataSourceWithBackend<AzureM
     const aggregation = templateSrv.replace(item.aggregation, scopedVars);
     const top = templateSrv.replace(item.top || '', scopedVars);
 
-    const dimensionsFilters = item.dimensionFilters.map(f => {
+    const dimensionsFilters = item.dimensionFilters
-      return {
+      .filter(f => f.dimension && f.dimension !== 'None')
-        dimension: templateSrv.replace(f.dimension, scopedVars),
+      .map(f => {
-        operator: f.operator || 'eq',
+        const filter = templateSrv.replace(f.filter, scopedVars);
-        filter: templateSrv.replace(f.filter, scopedVars),
+        return {
-      };
+          dimension: templateSrv.replace(f.dimension, scopedVars),
-    });
+          operator: f.operator || 'eq',
+          filter: filter || '*', // send * when empty
+        };
+      });
 
     return {
       refId: target.refId,

public/app/plugins/datasource/grafana-azure-monitor-datasource/azure_monitor/response_parser.ts

@@ -92,10 +92,6 @@ export default class ResponseParser {
       return dimensions;
     }
 
-    if (!metricData.isDimensionRequired) {
-      dimensions.push({ text: 'None', value: 'None' });
-    }
-
     for (let i = 0; i < metricData.dimensions.length; i++) {
       const text = metricData.dimensions[i].localizedValue;
       const value = metricData.dimensions[i].value;

public/app/plugins/datasource/grafana-azure-monitor-datasource/partials/query.editor.html

@@ -100,7 +100,7 @@
         >
         </gf-form-dropdown>
       </div>
-      <div class="gf-form gf-form--grow aggregation-dropdown-wrapper">
+      <div class="gf-form">
         <label class="gf-form-label query-keyword width-9">Aggregation</label>
         <div class="gf-form-select-wrapper gf-form-select-wrapper--caret-indent">
           <select
@@ -111,6 +111,9 @@
           ></select>
         </div>
       </div>
+      <div class="gf-form gf-form--grow">
+        <div class="gf-form-label gf-form-label--grow"></div>
+      </div>
     </div>
     <div class="gf-form-inline">
       <div class="gf-form">

public/app/plugins/datasource/grafana-azure-monitor-datasource/query_ctrl.ts

@@ -533,7 +533,6 @@ export class AzureMonitorQueryCtrl extends QueryCtrl {
       operator: 'eq',
       filter: '',
     });
-    this.refresh();
   }
 
   azureMonitorRemoveDimensionFilter(index: number) {