Skip to content

Commit 2557f99

Browse files
committed
fix linting issues
Signed-off-by: Ben Ye <[email protected]>
1 parent f8b536f commit 2557f99

File tree

2 files changed

+4
-7
lines changed

2 files changed

+4
-7
lines changed

pkg/queryfrontend/metadata_codec.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -175,7 +175,7 @@ func (c metadataCodec) DecodeResponse(ctx context.Context, r *http.Response, req
175175

176176
buf, err := ioutil.ReadAll(r.Body)
177177
if err != nil {
178-
log.Error(err)
178+
log.Error(err) //nolint:errcheck
179179
return nil, httpgrpc.Errorf(http.StatusInternalServerError, "error decoding response: %v", err)
180180
}
181181

pkg/queryfrontend/roundtrip.go

+3-6
Original file line numberDiff line numberDiff line change
@@ -51,11 +51,8 @@ func NewTripperware(
5151
return nil, err
5252
}
5353

54-
metadataTripperware, err := newMetadataTripperware(config, limits, metadataCodec,
54+
metadataTripperware := newMetadataTripperware(config, limits, metadataCodec,
5555
prometheus.WrapRegistererWith(prometheus.Labels{"tripperware": "metadata"}, reg), logger)
56-
if err != nil {
57-
return nil, err
58-
}
5956

6057
return func(next http.RoundTripper) http.RoundTripper {
6158
return newRoundTripper(next, queryRangeTripperware(next), metadataTripperware(next), reg)
@@ -116,7 +113,7 @@ func getOperation(r *http.Request) string {
116113
return seriesOp
117114
default:
118115
matched, err := regexp.MatchString("/api/v1/label/.+/values$", r.URL.Path)
119-
if err == nil && matched == true {
116+
if err == nil && matched {
120117
return labelValuesOp
121118
}
122119
}
@@ -199,7 +196,7 @@ func newMetadataTripperware(
199196
codec *metadataCodec,
200197
reg prometheus.Registerer,
201198
logger log.Logger,
202-
) (frontend.Tripperware, error) {
199+
) frontend.Tripperware {
203200
metadataMiddleware := []queryrange.Middleware{queryrange.LimitsMiddleware(limits)}
204201
m := queryrange.NewInstrumentMiddlewareMetrics(reg)
205202

0 commit comments

Comments
 (0)