fix(remote-read): return err instead of nil

[Dentrax]

Actual Behavior

$ cortextool remote-read stats --address http://10.10.10.10:30090 --selector '{__name__=~"istio.*"}' --remote-read-path /api/v1/read --log.level debug --read-timeout 60s

INFO[0000] log level set to debug
INFO[0000] Created remote read client using endpoint 'http://10.10.10.10:30090/api/v1/read'
INFO[0000] Querying time from=2022-01-18T16:01:52+03:00 to=2022-01-18T17:01:52+03:00 with selector={__name__=~"istio.*"}

$ echo $?
0

Expected Behavior

INFO[0000] log level set to debug                       
INFO[0000] Created remote read client using endpoint 'http://10.10.10.10:30090/api/v1/read' 
INFO[0000] Querying time from=2022-01-18T16:39:20+03:00 to=2022-01-18T17:39:20+03:00 with selector={__name__=~"istio.*"} 
___2remote_read_stats: error: remote server http://10.10.10.10:30090/api/v1/read returned HTTP status 400 Bad Request: exceeded sample limit (50000000), try --help

Process finished with the exit code 1

@eminaktas @yasintahaerol

Signed-off-by: Furkan furkan.turkal@trendyol.com
Co-authored-by: Emin emin.aktas@trendyol.com
Co-authored-by: Yasin yasintaha.erol@trendyol.com

[Dentrax]

Kind ping here 🤞 @gotjosh @gouthamve

[CLAassistant]

All committers have signed the CLA.

[Dentrax]

Kindly ping again 🤞 @gotjosh @gouthamve

[aknuds1]

cortextool is deprecated in favour of mimirtool. Is this bug still present in mimirtool?

[Dentrax]

cortextool is deprecated in favour of mimirtool. Is this bug still present in mimirtool?

Oh, didn't notice deprecation warning! Can you please make this repository achieved?

[Dentrax]

MOVED: grafana/mimir#3051