From a9efb9b33083bf8c97491c65c0d95f3500f5aeb1 Mon Sep 17 00:00:00 2001 From: Friedrich Gonzalez Date: Mon, 25 Mar 2024 14:02:09 +0100 Subject: [PATCH] Fix linting Signed-off-by: Friedrich Gonzalez --- cmd/e2ealerting/main.go | 3 ++- cmd/logtool/main.go | 12 ++++++------ pkg/alerting/runner.go | 5 +++-- pkg/analyse/ruler.go | 3 ++- pkg/bench/query_runner.go | 4 ++-- pkg/bench/write_client.go | 2 +- pkg/chunk/cassandra/storage_client.go | 2 +- pkg/chunk/migrate/writer/writer.go | 2 +- pkg/client/alerts.go | 2 +- pkg/client/client.go | 2 +- pkg/client/rules.go | 10 +++++----- pkg/commands/access_control.go | 2 +- pkg/commands/alerts.go | 12 ++++++------ pkg/commands/analyse_dashboards.go | 2 +- pkg/commands/analyse_grafana.go | 8 ++------ pkg/commands/analyse_prometheus.go | 8 ++------ pkg/commands/analyse_ruler.go | 6 +----- pkg/commands/analyse_rules_test.go | 5 +++-- pkg/commands/block_gen.go | 3 ++- pkg/commands/bucket_validation.go | 2 +- pkg/commands/chunks.go | 2 +- pkg/commands/logger.go | 2 +- pkg/commands/migrate.go | 2 +- pkg/commands/push_gateway.go | 2 +- pkg/commands/rules.go | 2 +- pkg/commands/rules_test.go | 3 ++- pkg/rules/compare_test.go | 3 ++- pkg/rules/rules.go | 2 +- 28 files changed, 54 insertions(+), 59 deletions(-) diff --git a/cmd/e2ealerting/main.go b/cmd/e2ealerting/main.go index d6804706d..32ed77204 100644 --- a/cmd/e2ealerting/main.go +++ b/cmd/e2ealerting/main.go @@ -4,9 +4,10 @@ import ( "flag" "os" - "github.com/cortexproject/cortex-tools/pkg/alerting" "github.com/go-kit/log/level" + "github.com/cortexproject/cortex-tools/pkg/alerting" + "github.com/cortexproject/cortex/pkg/util/flagext" util_log "github.com/cortexproject/cortex/pkg/util/log" "github.com/prometheus/client_golang/prometheus" diff --git a/cmd/logtool/main.go b/cmd/logtool/main.go index df0b77306..d64e77d70 100644 --- a/cmd/logtool/main.go +++ b/cmd/logtool/main.go @@ -45,7 +45,7 @@ func main() { for scanner.Scan() { show := false var qt time.Time - var dur, len time.Duration + var dur, length time.Duration var status, path, query, trace string var err error @@ -99,13 +99,13 @@ func main() { fmt.Println(err, line) continue } - len = et.Sub(st) + length = et.Sub(st) } else { // Loki queries are nanosecond, simple check to see if it's a second or nanosecond timestamp if st > 9999999999 { - len = time.Unix(0, et).Sub(time.Unix(0, st)) + length = time.Unix(0, et).Sub(time.Unix(0, st)) } else { - len = time.Unix(et, 0).Sub(time.Unix(st, 0)) + length = time.Unix(et, 0).Sub(time.Unix(st, 0)) } } @@ -132,9 +132,9 @@ func main() { ts = fmt.Sprint(qt.Local()) } if *showQuery { - fmt.Fprintf(w, "%v\t%v\t%v\t%v\t%v\t%v\t%v\n", ts, trace, len, dur, status, path, query) + fmt.Fprintf(w, "%v\t%v\t%v\t%v\t%v\t%v\t%v\n", ts, trace, length, dur, status, path, query) } else { - fmt.Fprintf(w, "%v\t%v\t%v\t%v\t%v\t%v\n", ts, trace, len, dur, status, path) + fmt.Fprintf(w, "%v\t%v\t%v\t%v\t%v\t%v\n", ts, trace, length, dur, status, path) } //If looking at stdin, flush after every line as someone would only paste one line at a time at the terminal if !isPipe { diff --git a/pkg/alerting/runner.go b/pkg/alerting/runner.go index b96a3c9d0..3586894b2 100644 --- a/pkg/alerting/runner.go +++ b/pkg/alerting/runner.go @@ -9,12 +9,13 @@ import ( "sync" "time" - "github.com/cortexproject/cortex-tools/pkg/client" - "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" "github.com/go-kit/log" "github.com/go-kit/log/level" "github.com/prometheus/client_golang/prometheus" yaml "gopkg.in/yaml.v3" + + "github.com/cortexproject/cortex-tools/pkg/client" + "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" ) // Case represents a metric that can be used for exporting, then verified against an Alertmanager webhook diff --git a/pkg/analyse/ruler.go b/pkg/analyse/ruler.go index 999391f5b..9fe3d192a 100644 --- a/pkg/analyse/ruler.go +++ b/pkg/analyse/ruler.go @@ -3,10 +3,11 @@ package analyse import ( "sort" - "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" "github.com/pkg/errors" "github.com/prometheus/prometheus/promql/parser" log "github.com/sirupsen/logrus" + + "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" ) type MetricsInRuler struct { diff --git a/pkg/bench/query_runner.go b/pkg/bench/query_runner.go index 792b817c5..362e1a81c 100644 --- a/pkg/bench/query_runner.go +++ b/pkg/bench/query_runner.go @@ -204,8 +204,8 @@ func (q *queryRunner) executeQuery(ctx context.Context, queryReq query) error { now := time.Now() var ( - queryType string = "instant" - status string = "success" + queryType = "instant" + status = "success" ) if queryReq.timeRange > 0 { queryType = "range" diff --git a/pkg/bench/write_client.go b/pkg/bench/write_client.go index eb98f0e3d..770e67ad7 100644 --- a/pkg/bench/write_client.go +++ b/pkg/bench/write_client.go @@ -39,7 +39,7 @@ type writeClient struct { } // newWriteClient creates a new client for remote write. -func newWriteClient(name string, tenantName string, conf *remote.ClientConfig, logger log.Logger, requestHistogram *prometheus.HistogramVec) (*writeClient, error) { +func newWriteClient(name string, tenantName string, conf *remote.ClientConfig, _ log.Logger, requestHistogram *prometheus.HistogramVec) (*writeClient, error) { httpClient, err := config_util.NewClientFromConfig(conf.HTTPClientConfig, "bench_write_client", config_util.WithHTTP2Disabled()) if err != nil { return nil, err diff --git a/pkg/chunk/cassandra/storage_client.go b/pkg/chunk/cassandra/storage_client.go index 64aaf828e..2b770443a 100644 --- a/pkg/chunk/cassandra/storage_client.go +++ b/pkg/chunk/cassandra/storage_client.go @@ -196,4 +196,4 @@ func (noopConvictionPolicy) AddFailure(err error, host *gocql.HostInfo) bool { } // Implementats gocql.ConvictionPolicy. -func (noopConvictionPolicy) Reset(host *gocql.HostInfo) {} +func (noopConvictionPolicy) Reset(_ *gocql.HostInfo) {} diff --git a/pkg/chunk/migrate/writer/writer.go b/pkg/chunk/migrate/writer/writer.go index bbe23b798..604f3fba3 100644 --- a/pkg/chunk/migrate/writer/writer.go +++ b/pkg/chunk/migrate/writer/writer.go @@ -91,7 +91,7 @@ func (w *Writer) Run(ctx context.Context, inChan chan chunk.Chunk) { } } -func (w *Writer) writeLoop(ctx context.Context, workerID int, inChan chan chunk.Chunk, errChan chan error) { +func (w *Writer) writeLoop(ctx context.Context, _ int, inChan chan chunk.Chunk, errChan chan error) { defer w.workerGroup.Done() for { diff --git a/pkg/client/alerts.go b/pkg/client/alerts.go index 61e4f41e0..fc1ec69a9 100644 --- a/pkg/client/alerts.go +++ b/pkg/client/alerts.go @@ -17,7 +17,7 @@ type configCompat struct { } // CreateAlertmanagerConfig creates a new alertmanager config -func (r *CortexClient) CreateAlertmanagerConfig(ctx context.Context, cfg string, templates map[string]string) error { +func (r *CortexClient) CreateAlertmanagerConfig(_ context.Context, cfg string, templates map[string]string) error { payload, err := yaml.Marshal(&configCompat{ TemplateFiles: templates, AlertmanagerConfig: cfg, diff --git a/pkg/client/client.go b/pkg/client/client.go index fb055d377..e78925a41 100644 --- a/pkg/client/client.go +++ b/pkg/client/client.go @@ -102,7 +102,7 @@ func New(cfg Config) (*CortexClient, error) { } // Query executes a PromQL query against the Cortex cluster. -func (r *CortexClient) Query(ctx context.Context, query string) (*http.Response, error) { +func (r *CortexClient) Query(_ context.Context, query string) (*http.Response, error) { query = fmt.Sprintf("query=%s&time=%d", query, time.Now().Unix()) escapedQuery := url.PathEscape(query) diff --git a/pkg/client/rules.go b/pkg/client/rules.go index 52c1dc3d2..a41775f01 100644 --- a/pkg/client/rules.go +++ b/pkg/client/rules.go @@ -14,7 +14,7 @@ import ( ) // CreateRuleGroup creates a new rule group -func (r *CortexClient) CreateRuleGroup(ctx context.Context, namespace string, rg rwrulefmt.RuleGroup) error { +func (r *CortexClient) CreateRuleGroup(_ context.Context, namespace string, rg rwrulefmt.RuleGroup) error { payload, err := yaml.Marshal(&rg) if err != nil { return err @@ -34,7 +34,7 @@ func (r *CortexClient) CreateRuleGroup(ctx context.Context, namespace string, rg } // DeleteRuleGroup deletes a rule group -func (r *CortexClient) DeleteRuleGroup(ctx context.Context, namespace, groupName string) error { +func (r *CortexClient) DeleteRuleGroup(_ context.Context, namespace, groupName string) error { escapedNamespace := url.PathEscape(namespace) escapedGroupName := url.PathEscape(groupName) path := r.apiPath + "/" + escapedNamespace + "/" + escapedGroupName @@ -50,7 +50,7 @@ func (r *CortexClient) DeleteRuleGroup(ctx context.Context, namespace, groupName } // DeleteRuleNamespace deletes a rule namespace -func (r *CortexClient) DeleteRuleNamespace(ctx context.Context, namespace string) error { +func (r *CortexClient) DeleteRuleNamespace(_ context.Context, namespace string) error { escapedNamespace := url.PathEscape(namespace) path := r.apiPath + "/" + escapedNamespace @@ -65,7 +65,7 @@ func (r *CortexClient) DeleteRuleNamespace(ctx context.Context, namespace string } // GetRuleGroup retrieves a rule group -func (r *CortexClient) GetRuleGroup(ctx context.Context, namespace, groupName string) (*rwrulefmt.RuleGroup, error) { +func (r *CortexClient) GetRuleGroup(_ context.Context, namespace, groupName string) (*rwrulefmt.RuleGroup, error) { escapedNamespace := url.PathEscape(namespace) escapedGroupName := url.PathEscape(groupName) path := r.apiPath + "/" + escapedNamespace + "/" + escapedGroupName @@ -97,7 +97,7 @@ func (r *CortexClient) GetRuleGroup(ctx context.Context, namespace, groupName st } // ListRules retrieves a rule group -func (r *CortexClient) ListRules(ctx context.Context, namespace string) (map[string][]rwrulefmt.RuleGroup, error) { +func (r *CortexClient) ListRules(_ context.Context, namespace string) (map[string][]rwrulefmt.RuleGroup, error) { path := r.apiPath if namespace != "" { path = path + "/" + namespace diff --git a/pkg/commands/access_control.go b/pkg/commands/access_control.go index 0c2c73fee..52dd71ea6 100644 --- a/pkg/commands/access_control.go +++ b/pkg/commands/access_control.go @@ -25,7 +25,7 @@ func (a *AccessControlCommand) Register(app *kingpin.Application) { generateHeaderCmd.Flag("rule", "The access control rules (Prometheus selectors). Set it multiple times to set multiple rules.").Required().StringsVar(&a.ACLs) } -func (a *AccessControlCommand) generateHeader(k *kingpin.ParseContext) error { +func (a *AccessControlCommand) generateHeader(_ *kingpin.ParseContext) error { for _, acl := range a.ACLs { _, err := parser.ParseMetricSelector(acl) if err != nil { diff --git a/pkg/commands/alerts.go b/pkg/commands/alerts.go index 61e37665c..6c69c0dbe 100644 --- a/pkg/commands/alerts.go +++ b/pkg/commands/alerts.go @@ -83,7 +83,7 @@ func (a *AlertmanagerCommand) Register(app *kingpin.Application) { loadalertCmd.Arg("template-files", "The template files to load").ExistingFilesVar(&a.TemplateFiles) } -func (a *AlertmanagerCommand) setup(k *kingpin.ParseContext) error { +func (a *AlertmanagerCommand) setup(_ *kingpin.ParseContext) error { cli, err := client.New(a.ClientConfig) if err != nil { return err @@ -93,7 +93,7 @@ func (a *AlertmanagerCommand) setup(k *kingpin.ParseContext) error { return nil } -func (a *AlertmanagerCommand) getConfig(k *kingpin.ParseContext) error { +func (a *AlertmanagerCommand) getConfig(_ *kingpin.ParseContext) error { cfg, templates, err := a.cli.GetAlertmanagerConfig(context.Background()) if err != nil { if err == client.ErrResourceNotFound { @@ -108,7 +108,7 @@ func (a *AlertmanagerCommand) getConfig(k *kingpin.ParseContext) error { return p.PrintAlertmanagerConfig(cfg, templates) } -func (a *AlertmanagerCommand) loadConfig(k *kingpin.ParseContext) error { +func (a *AlertmanagerCommand) loadConfig(_ *kingpin.ParseContext) error { content, err := ioutil.ReadFile(a.AlertmanagerConfigFile) if err != nil { return errors.Wrap(err, "unable to load config file: "+a.AlertmanagerConfigFile) @@ -132,7 +132,7 @@ func (a *AlertmanagerCommand) loadConfig(k *kingpin.ParseContext) error { return a.cli.CreateAlertmanagerConfig(context.Background(), cfg, templates) } -func (a *AlertmanagerCommand) deleteConfig(k *kingpin.ParseContext) error { +func (a *AlertmanagerCommand) deleteConfig(_ *kingpin.ParseContext) error { err := a.cli.DeleteAlermanagerConfig(context.Background()) if err != nil && err != client.ErrResourceNotFound { return err @@ -155,7 +155,7 @@ func (a *AlertCommand) Register(app *kingpin.Application) { verifyAlertsCmd.Flag("frequency", "Setting this value will turn cortextool into a long-running process, running the alerts verify check every # of minutes specified").IntVar(&a.CheckFrequency) } -func (a *AlertCommand) setup(k *kingpin.ParseContext) error { +func (a *AlertCommand) setup(_ *kingpin.ParseContext) error { cli, err := client.New(a.ClientConfig) if err != nil { return err @@ -179,7 +179,7 @@ type metric struct { Metric map[string]string `json:"metric"` } -func (a *AlertCommand) verifyConfig(k *kingpin.ParseContext) error { +func (a *AlertCommand) verifyConfig(_ *kingpin.ParseContext) error { var empty interface{} if a.IgnoreString != "" { a.IgnoreAlerts = make(map[string]interface{}) diff --git a/pkg/commands/analyse_dashboards.go b/pkg/commands/analyse_dashboards.go index 491f5176e..c99bb2e4e 100644 --- a/pkg/commands/analyse_dashboards.go +++ b/pkg/commands/analyse_dashboards.go @@ -16,7 +16,7 @@ type DashboardAnalyseCommand struct { outputFile string } -func (cmd *DashboardAnalyseCommand) run(k *kingpin.ParseContext) error { +func (cmd *DashboardAnalyseCommand) run(_ *kingpin.ParseContext) error { output := &analyse.MetricsInGrafana{} output.OverallMetrics = make(map[string]struct{}) diff --git a/pkg/commands/analyse_grafana.go b/pkg/commands/analyse_grafana.go index 90cbfd5c6..26d2b7332 100644 --- a/pkg/commands/analyse_grafana.go +++ b/pkg/commands/analyse_grafana.go @@ -23,7 +23,7 @@ type GrafanaAnalyseCommand struct { outputFile string } -func (cmd *GrafanaAnalyseCommand) run(k *kingpin.ParseContext) error { +func (cmd *GrafanaAnalyseCommand) run(_ *kingpin.ParseContext) error { output := &analyse.MetricsInGrafana{} output.OverallMetrics = make(map[string]struct{}) @@ -70,9 +70,5 @@ func writeOut(mig *analyse.MetricsInGrafana, outputFile string) error { return err } - if err := ioutil.WriteFile(outputFile, out, os.FileMode(int(0666))); err != nil { - return err - } - - return nil + return ioutil.WriteFile(outputFile, out, os.FileMode(int(0666))) } diff --git a/pkg/commands/analyse_prometheus.go b/pkg/commands/analyse_prometheus.go index b2e32d12e..b5fa42ca6 100644 --- a/pkg/commands/analyse_prometheus.go +++ b/pkg/commands/analyse_prometheus.go @@ -31,7 +31,7 @@ type PrometheusAnalyseCommand struct { outputFile string } -func (cmd *PrometheusAnalyseCommand) run(k *kingpin.ParseContext) error { +func (cmd *PrometheusAnalyseCommand) run(_ *kingpin.ParseContext) error { var ( hasGrafanaMetrics, hasRulerMetrics = false, false grafanaMetrics = analyse.MetricsInGrafana{} @@ -228,9 +228,5 @@ func (cmd *PrometheusAnalyseCommand) run(k *kingpin.ParseContext) error { return err } - if err := ioutil.WriteFile(cmd.outputFile, out, os.FileMode(int(0666))); err != nil { - return err - } - - return nil + return ioutil.WriteFile(cmd.outputFile, out, os.FileMode(int(0666))) } diff --git a/pkg/commands/analyse_ruler.go b/pkg/commands/analyse_ruler.go index 553ce86a2..2e53e3747 100644 --- a/pkg/commands/analyse_ruler.go +++ b/pkg/commands/analyse_ruler.go @@ -65,9 +65,5 @@ func writeOutRuleMetrics(mir *analyse.MetricsInRuler, outputFile string) error { return err } - if err := ioutil.WriteFile(outputFile, out, os.FileMode(int(0666))); err != nil { - return err - } - - return nil + return ioutil.WriteFile(outputFile, out, os.FileMode(int(0666))) } diff --git a/pkg/commands/analyse_rules_test.go b/pkg/commands/analyse_rules_test.go index e6bce4a01..4214cf226 100644 --- a/pkg/commands/analyse_rules_test.go +++ b/pkg/commands/analyse_rules_test.go @@ -4,10 +4,11 @@ import ( "sort" "testing" - "github.com/cortexproject/cortex-tools/pkg/analyse" - "github.com/cortexproject/cortex-tools/pkg/rules" "github.com/stretchr/testify/assert" "github.com/stretchr/testify/require" + + "github.com/cortexproject/cortex-tools/pkg/analyse" + "github.com/cortexproject/cortex-tools/pkg/rules" ) var metricsInRuleGroup = []string{ diff --git a/pkg/commands/block_gen.go b/pkg/commands/block_gen.go index c3cc2f45b..940e78158 100644 --- a/pkg/commands/block_gen.go +++ b/pkg/commands/block_gen.go @@ -7,7 +7,6 @@ import ( "sort" "time" - "github.com/cortexproject/cortex-tools/pkg/bench" "github.com/go-kit/log" "github.com/go-kit/log/level" "github.com/pkg/errors" @@ -17,6 +16,8 @@ import ( "github.com/prometheus/prometheus/tsdb" "gopkg.in/alecthomas/kingpin.v2" "gopkg.in/yaml.v3" + + "github.com/cortexproject/cortex-tools/pkg/bench" ) // BlockGenCommand is the kingpin command to generate blocks of mock data. diff --git a/pkg/commands/bucket_validation.go b/pkg/commands/bucket_validation.go index 2b08f9f25..d83508642 100644 --- a/pkg/commands/bucket_validation.go +++ b/pkg/commands/bucket_validation.go @@ -299,7 +299,7 @@ func (b *BucketValidationCommand) deleteTestObjects(ctx context.Context) error { return errors.Wrapf(err, "failed to list objects") } if foundDeletedDir { - return errors.Errorf("List returned directory which is supposed to be deleted.") + return errors.Errorf("list returned directory which is supposed to be deleted") } expectedDirCount := len(b.objectNames) - iteration if foundDirCount != expectedDirCount { diff --git a/pkg/commands/chunks.go b/pkg/commands/chunks.go index a4daa07b0..6436a3521 100644 --- a/pkg/commands/chunks.go +++ b/pkg/commands/chunks.go @@ -181,7 +181,7 @@ func setup(k *kingpin.ParseContext) error { return nil } -func (c *chunkCleanCommandOptions) run(k *kingpin.ParseContext) error { +func (c *chunkCleanCommandOptions) run(_ *kingpin.ParseContext) error { cortexCfg := &cortex.Config{} flagext.RegisterFlags(cortexCfg) err := LoadConfig(c.CortexConfigFile, true, cortexCfg) diff --git a/pkg/commands/logger.go b/pkg/commands/logger.go index ab87c6f41..4e28fd9b8 100644 --- a/pkg/commands/logger.go +++ b/pkg/commands/logger.go @@ -11,7 +11,7 @@ type LoggerConfig struct { Level string } -func (l *LoggerConfig) registerLogLevel(pc *kingpin.ParseContext) error { +func (l *LoggerConfig) registerLogLevel(_ *kingpin.ParseContext) error { var logLevel logrus.Level switch l.Level { case "debug": diff --git a/pkg/commands/migrate.go b/pkg/commands/migrate.go index 0060e50a3..b804253d9 100644 --- a/pkg/commands/migrate.go +++ b/pkg/commands/migrate.go @@ -23,7 +23,7 @@ func registerMigrateChunksCommandOptions(cmd *kingpin.CmdClause) { migrateChunksCommandOptions.Planner.Register(migrateChunksCommand) } -func (c *migrateChunksCommandOptions) run(k *kingpin.ParseContext) error { +func (c *migrateChunksCommandOptions) run(_ *kingpin.ParseContext) error { f, err := os.Open(c.ConfigFile) if err != nil { return err diff --git a/pkg/commands/push_gateway.go b/pkg/commands/push_gateway.go index 3b77738d7..70886c048 100644 --- a/pkg/commands/push_gateway.go +++ b/pkg/commands/push_gateway.go @@ -29,7 +29,7 @@ func (l *PushGatewayConfig) Register(app *kingpin.Application) { app.Flag("push-gateway.interval", "interval to forward metrics to the push gateway").Default("1m").DurationVar(&l.Interval) } -func (l *PushGatewayConfig) setup(pc *kingpin.ParseContext) error { +func (l *PushGatewayConfig) setup(_ *kingpin.ParseContext) error { if l.Endpoint == nil || l.JobName == "" { logrus.Debugln("push-gateway not configured") return nil diff --git a/pkg/commands/rules.go b/pkg/commands/rules.go index 94c1a42ce..70f367c0d 100644 --- a/pkg/commands/rules.go +++ b/pkg/commands/rules.go @@ -567,7 +567,7 @@ func (r *RuleCommand) syncRules(k *kingpin.ParseContext) error { err = r.executeChanges(context.Background(), changes) if err != nil { - return errors.Wrap(err, "sync operation unsuccessful, unable to complete executing changes.") + return errors.Wrap(err, "sync operation unsuccessful, unable to complete executing changes") } return nil diff --git a/pkg/commands/rules_test.go b/pkg/commands/rules_test.go index a9cd59be3..706912955 100644 --- a/pkg/commands/rules_test.go +++ b/pkg/commands/rules_test.go @@ -3,10 +3,11 @@ package commands import ( "testing" - "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" "github.com/prometheus/prometheus/model/rulefmt" "github.com/stretchr/testify/assert" "gopkg.in/yaml.v3" + + "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" ) func TestCheckDuplicates(t *testing.T) { diff --git a/pkg/rules/compare_test.go b/pkg/rules/compare_test.go index 390c9a8df..40671686f 100644 --- a/pkg/rules/compare_test.go +++ b/pkg/rules/compare_test.go @@ -3,9 +3,10 @@ package rules import ( "testing" - "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" "github.com/prometheus/prometheus/model/rulefmt" yaml "gopkg.in/yaml.v3" + + "github.com/cortexproject/cortex-tools/pkg/rules/rwrulefmt" ) func Test_rulesEqual(t *testing.T) { diff --git a/pkg/rules/rules.go b/pkg/rules/rules.go index d66b9ba4a..8e9bd6b37 100644 --- a/pkg/rules/rules.go +++ b/pkg/rules/rules.go @@ -25,7 +25,7 @@ type RuleNamespace struct { // LintExpressions runs the `expr` from a rule through the PromQL parser and // compares its output. If it differs from the parser, it uses the parser's instead. func (r RuleNamespace) LintExpressions() (int, int, error) { - var parseFn func(string) (fmt.Stringer, error) = func(s string) (fmt.Stringer, error) { + var parseFn = func(s string) (fmt.Stringer, error) { return parser.ParseExpr(s) }