go-graphite / carbonapi

Implementation of graphite API (graphite-web) in golang
Other
308 stars 140 forks source link

Doesn't support | (pipe Character) in Metric Path as this is considered as Pipe Functions. #725

Open ravivish83 opened 2 years ago

ravivish83 commented 2 years ago

Describe the bug CarbonAPI doesn't support | (pipe Character) in Metric Path as this is considered as Pipe Functions. For: ap.|1234|ap.dev.hostname.memory.percent.Used

CarbonAPI Version What's the CarbonAPI version you are running? Does this issue reproducible on current master? https://github.com/go-graphite/carbonapi

Logs If applicable, add logs (please use log level debug) that shows the whole duration of the request. Request can be identified by UUID if needed.

Error : Pipe to not a function\n Parsed so far : ap.|1234|\nCould not parse :

CarbonAPI Configuration: Please include carbonapi config file that you are using (please replace all information you don't want to share, like domain names or IP addresses). https://github.com/go-graphite/carbonapi/blob/main/cmd/carbonapi/carbonapi.example.victoriametrics.yaml

Simplified query (if applicable) Please provide a query that triggered the issue, ideally narrowed down to smallest possible set of functions.

Backend response (if possible) If that's possible - please share some sample set of backend responses. Most important thing here:

  1. Are all backend responses the same? If not, what's the difference?
  2. Do they contain special values, like Inf/NaN values?
  3. Do all of them have same step?