Enable exhaustruct linter

Author: prymitive

.golangci.yml

@@ -27,9 +27,14 @@ linters:
     - exptostd
     - nilnesserr
     - usetesting
+    - exhaustruct
 
 issues:
   max-same-issues: 0
+  exclude-rules:
+  - path: ^.*_test\.go$
+    linters:
+    - exhaustruct
 
 linters-settings:
   goimports:
@@ -40,3 +45,23 @@ linters-settings:
     enable-all: true
   nakedret:
     max-func-lines: 0
+  exhaustruct:
+    exclude:
+      - net/http\..+
+      - crypto/tls\..+
+      - encoding/xml\..+
+      - .+/checks\.badMatcher
+      - .+/checks\.Problem
+      - .+/checks\.exprProblem
+      - .+/checks\..+Settings
+      - .+/config\.Match
+      - .+/discovery\.Entry
+      - .+/parser\.Rule
+      - .+/parser\.ParseError
+      - .+/parser\.VectorSelector
+      - .+/promapi\.MetricTimeRange
+      - github.com/prometheus/.+\..+
+      - github.com/urfave/cli/.+
+      - gitlab.com/gitlab-org/api/.+
+      - github.com/google/go-github/.+
+      - github.com/hashicorp/hcl/.+

cmd/pint/ci.go

@@ -289,7 +289,7 @@ func detectCI(cfg *config.CI) *config.CI {
 
 	if cfg == nil {
 		isNil = true
-		cfg = &config.CI{}
+		cfg = &config.CI{} // nolint:exhaustruct
 	}
 
 	if bb := os.Getenv("GITHUB_BASE_REF"); bb != "" {
@@ -309,7 +309,7 @@ func detectRepository(cfg *config.Repository) *config.Repository {
 
 	if cfg == nil {
 		isNil = true
-		cfg = &config.Repository{}
+		cfg = &config.Repository{} // nolint:exhaustruct
 	}
 
 	if os.Getenv("GITHUB_ACTION") != "" {
@@ -342,7 +342,7 @@ func detectGithubActions(gh *config.GitHub) *config.GitHub {
 
 	if gh == nil {
 		isNil = true
-		gh = &config.GitHub{Timeout: time.Minute.String()}
+		gh = &config.GitHub{Timeout: time.Minute.String()} // nolint:exhaustruct
 	}
 
 	if repo := os.Getenv("GITHUB_REPOSITORY"); repo != "" {

cmd/pint/lint.go

@@ -192,6 +192,7 @@ func verifyOwners(entries []discovery.Entry, allowedOwners []*regexp.Regexp) (re
 				Path:          entry.Path,
 				ModifiedLines: entry.ModifiedLines,
 				Rule:          entry.Rule,
+				Owner:         "",
 				Problem: checks.Problem{
 					Lines:    entry.Rule.Lines,
 					Reporter: discovery.RuleOwnerComment,
@@ -211,6 +212,7 @@ func verifyOwners(entries []discovery.Entry, allowedOwners []*regexp.Regexp) (re
 			Path:          entry.Path,
 			ModifiedLines: entry.ModifiedLines,
 			Rule:          entry.Rule,
+			Owner:         "",
 			Problem: checks.Problem{
 				Lines:    entry.Rule.Lines,
 				Reporter: discovery.RuleOwnerComment,

cmd/pint/scan.go

@@ -102,7 +102,7 @@ func checkRules(ctx context.Context, workers int, isOffline bool, gen *config.Pr
 				checkList := cfg.GetChecksForEntry(ctx, gen, entry)
 				for _, check := range checkList {
 					checkIterationChecks.Inc()
-					if check.Meta().IsOnline {
+					if check.Meta().Online {
 						onlineChecksCount.Inc()
 					} else {
 						offlineChecksCount.Inc()

cmd/pint/tests/0083_github_action.txt

@@ -61,4 +61,3 @@ groups:
 
 -- src/.pint.hcl --
 repository {}
-

cmd/pint/tests/0122_lint_owner_allowed.txt

@@ -5,7 +5,7 @@ cmp stderr stderr.txt
 -- stderr.txt --
 level=INFO msg="Loading configuration file" path=.pint.hcl
 level=INFO msg="Finding all rules to check" paths=["rules"]
-level=INFO msg="Checking Prometheus rules" entries=5 workers=10 online=true
+level=INFO msg="Checking Prometheus rules" entries=6 workers=10 online=true
 rules/1.yml:4-5 Bug: `rule/owner` comments are required in all files, please add a `# pint file/owner $owner` somewhere in this file and/or `# pint rule/owner $owner` on top of each rule. (rule/owner)
  4 |   - alert: No Owner
  5 |     expr: up > 0
@@ -14,14 +14,18 @@ rules/1.yml:7-8 Bug: This rule is set as owned by `bob` but `bob` doesn't match
  7 |   - alert: Invalid
  8 |     expr: up == 0
 
+rules/1.yml:13-14 Bug: This rule is set as owned by `zed` but `zed` doesn't match any of the allowed owner values. (rule/owner)
+ 13 |   - alert: Owner Zed
+ 14 |     expr: up < 0
+
 rules/2.yml:4-5 Bug: `rule/owner` comments are required in all files, please add a `# pint file/owner $owner` somewhere in this file and/or `# pint rule/owner $owner` on top of each rule. (rule/owner)
  4 |   - alert: No Owner
  5 |     expr: up > 0
 
 rules/3.yml:1 Bug: This file is set as owned by `ax` but `ax` doesn't match any of the allowed owner values. (rule/owner)
  1 | # pint file/owner ax
 
-level=INFO msg="Problems found" Bug=4
+level=INFO msg="Problems found" Bug=5
 level=ERROR msg="Fatal error" err="found 1 problem(s) with severity Bug or higher"
 -- rules/1.yml --
 groups:
@@ -35,6 +39,9 @@ groups:
   # pint rule/owner alice
   - alert: Owner Alice
     expr: up > 0
+  # pint rule/owner zed
+  - alert: Owner Zed
+    expr: up < 0
 
 -- rules/2.yml --
 groups:

cmd/pint/watch.go

@@ -283,7 +283,7 @@ type problemCollector struct {
 }
 
 func newProblemCollector(cfg config.Config, f pathFinderFunc, minSeverity checks.Severity, maxProblems int) *problemCollector {
-	return &problemCollector{
+	return &problemCollector{ // nolint:exhaustruct
 		finder:     f,
 		cfg:        cfg,
 		fileOwners: map[string]string{},

go.mod

@@ -6,7 +6,7 @@ require (
 	github.com/cespare/xxhash/v2 v2.3.0
 	github.com/gkampitakis/go-snaps v0.5.8
 	github.com/google/go-cmp v0.6.0
-	github.com/google/go-github/v63 v63.0.0
+	github.com/google/go-github/v68 v68.0.0
 	github.com/hashicorp/hcl/v2 v2.23.0
 	github.com/klauspost/compress v1.17.11
 	github.com/neilotoole/slogt v1.1.0

go.sum

@@ -69,8 +69,8 @@ github.com/golang/snappy v0.0.4/go.mod h1:/XxbfmMg8lxefKM7IXC3fBNl/7bRcc72aCRzEW
 github.com/google/go-cmp v0.5.2/go.mod h1:v8dTdLbMG2kIc/vJvl+f65V22dbkXbowE6jgT/gNBxE=
 github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI=
 github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY=
-github.com/google/go-github/v63 v63.0.0 h1:13xwK/wk9alSokujB9lJkuzdmQuVn2QCPeck76wR3nE=
-github.com/google/go-github/v63 v63.0.0/go.mod h1:IqbcrgUmIcEaioWrGYei/09o+ge5vhffGOcxrO0AfmA=
+github.com/google/go-github/v68 v68.0.0 h1:ZW57zeNZiXTdQ16qrDiZ0k6XucrxZ2CGmoTvcCyQG6s=
+github.com/google/go-github/v68 v68.0.0/go.mod h1:K9HAUBovM2sLwM408A18h+wd9vqdLOEqTUCbnRIcx68=
 github.com/google/go-querystring v1.1.0 h1:AnCroh3fv4ZBgVIf1Iwtovgjaw/GiKJo8M8yD/fhyJ8=
 github.com/google/go-querystring v1.1.0/go.mod h1:Kcdr2DB4koayq7X8pmAG4sNG59So17icRSOU623lUBU=
 github.com/google/gofuzz v1.0.0/go.mod h1:dBl0BpW6vV/+mYPU4Po3pmUjxk6FQPldtuIdl/M65Eg=

internal/checks/alerts_absent.go

@@ -37,7 +37,8 @@ func (c AlertsAbsentCheck) Meta() CheckMeta {
 			discovery.Modified,
 			discovery.Moved,
 		},
-		IsOnline: true,
+		Online:        true,
+		AlwaysEnabled: false,
 	}
 }