Merge branch 'v3.1.x'

public/app/features/dashboard/dashboardSrv.js

@@ -16,10 +16,6 @@ function (angular, $, _, moment) {
         data = {};
         data = {};
       }
       }
 
 
-      if (!data.id && data.version) {
-        data.schemaVersion = data.version;
-      }
-
       this.id = data.id || null;
       this.id = data.id || null;
       this.title = data.title || 'No Title';
       this.title = data.title || 'No Title';
       this.autoUpdate = data.autoUpdate;
       this.autoUpdate = data.autoUpdate;

public/app/features/templating/templateValuesSrv.js

@@ -11,6 +11,7 @@ function (angular, _, $, kbn) {
 
 
   module.service('templateValuesSrv', function($q, $rootScope, datasourceSrv, $location, templateSrv, timeSrv) {
   module.service('templateValuesSrv', function($q, $rootScope, datasourceSrv, $location, templateSrv, timeSrv) {
     var self = this;
     var self = this;
+    this.variableLock = {};
 
 
     function getNoneOption() { return { text: 'None', value: '', isNone: true }; }
     function getNoneOption() { return { text: 'None', value: '', isNone: true }; }
 
 
@@ -101,6 +102,8 @@ function (angular, _, $, kbn) {
         } else {
         } else {
           lock.resolve();
           lock.resolve();
         }
         }
+      }).finally(function() {
+        delete self.variableLock[variable.name];
       });
       });
     };
     };
 
 
@@ -135,7 +138,7 @@ function (angular, _, $, kbn) {
       templateSrv.setGrafanaVariable('$__auto_interval', interval);
       templateSrv.setGrafanaVariable('$__auto_interval', interval);
     };
     };
 
 
-    this.setVariableValue = function(variable, option, initPhase) {
+    this.setVariableValue = function(variable, option) {
       variable.current = angular.copy(option);
       variable.current = angular.copy(option);
 
 
       if (_.isArray(variable.current.text)) {
       if (_.isArray(variable.current.text)) {
@@ -145,13 +148,7 @@ function (angular, _, $, kbn) {
       self.selectOptionsForCurrentValue(variable);
       self.selectOptionsForCurrentValue(variable);
       templateSrv.updateTemplateData();
       templateSrv.updateTemplateData();
 
 
-      // on first load, variable loading is ordered to ensure
-      // that parents are updated before children.
-      if (initPhase) {
-        return $q.when();
-      }
-
-      return self.updateOptionsInChildVariables(variable);
+      return this.updateOptionsInChildVariables(variable);
     };
     };
 
 
     this.variableUpdated = function(variable) {
     this.variableUpdated = function(variable) {
@@ -160,6 +157,11 @@ function (angular, _, $, kbn) {
     };
     };
 
 
     this.updateOptionsInChildVariables = function(updatedVariable) {
     this.updateOptionsInChildVariables = function(updatedVariable) {
+      // if there is a variable lock ignore cascading update because we are in a boot up scenario
+      if (self.variableLock[updatedVariable.name]) {
+        return $q.when();
+      }
+
       var promises = _.map(self.variables, function(otherVariable) {
       var promises = _.map(self.variables, function(otherVariable) {
         if (otherVariable === updatedVariable) {
         if (otherVariable === updatedVariable) {
           return;
           return;