Wrong variables request about grafanajsondatasource HOT 6 CLOSED

This is how original datasource worked and we did not change it. So the behaviour is correct.

I agree having a way to issue also a /query request is handy but currently I don't see a way to do so without breaking the original behaviour.

from grafanajsondatasource.

Well then I think this is still no the correct behaviour, but a bug in the original datasource, too. Since the InfluxDB datasource plugin seems to resolve the query correctly:

from grafanajsondatasource.

InfluxDB datasource has AFAIK only one entry via query. We have 5. So it cannot be compared. And people have found use cases for current behaviour (see older issues) so now it's feature anyway.

from grafanajsondatasource.

InfluxDB datasource has AFAIK only one entry via query. We have 5. So it cannot be compared. And people have found use cases for current behaviour (see older issues) so now it's feature anyway.

Not sure this is addressing the core of the question. Imho:

/search retrieves the metrics, /query the data. The query editor's metric selection is wired to findMetrics:

<div ng-if="!ctrl.target.rawQuery">
  <div class="gf-form">
    <label class="gf-form-label width-5 query-keyword">Metric</label>
    <gf-form-dropdown model="ctrl.target.target"
      allow-custom="true"
      lookup-text="true"
      get-options="ctrl.findMetrics($query)"
      on-change="ctrl.onChangeInternal()">
    </gf-form-dropdown>
  </div>
</div>

The behaviour of query variables is imho controlled by Grafana, not by this plugin, anyway variables are.

So not an issue here but with Grafana.

from grafanajsondatasource.

@andig thanks for your reply. But doesn't this plugin not controll whether the /search or /query route is triggered when creating a Query-variable?

from grafanajsondatasource.

Seems like @andig is right. I did a quick check out of curiosity https://github.com/simPod/grafana-json-datasource/search?q=findMetrics&unscoped_q=findMetrics We do not call it explicitly.

from grafanajsondatasource.