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

[Dentrax]

Signed-off-by: Furkan [email protected]

What this PR does

Also adds missing errcheck linter since we already have a errcheckexclude file in the root.

Previous PR: grafana/cortex-tools#237

$ go run cmd/mimirtool/main.go remote-read stats --address http://10.11.12.13:30090 --selector '{__name__!=""}'

INFO[0000] Created remote read client using endpoint 'http://10.11.12.13:30090/prometheus/api/v1/read'
INFO[0000] Querying time from=2022-09-27T08:48:54+03:00 to=2022-09-27T09:48:54+03:00 with selector={__name__!=""}
main: error: remote server http://10.11.12.13:30090/prometheus/api/v1/read returned HTTP status 404 Not Found: 404 page not found, try --help
exit status 1

Which issue(s) this PR fixes or relates to

Fixes grafana/cortex-tools#236

Checklist

• Tests updated
• Documentation added
• CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

cc @aknuds1

[CLAassistant]

All committers have signed the CLA.

[aknuds1]

At first pass, I see two changes that shouldn't be part of the PR.

[Dentrax]

@aknuds1 Accidentally closed, can you please re-open? 🙏

[aknuds1]

The Reopen button is disabled for me, saying the branch was force-pushed or recreated.

[Dentrax]

Oh, opened a new one: #3053 (Resolved your reviews!)