Skip to content

Fix validation to enforce same upgrade strategy for cp and workers fo… #23895

Fix validation to enforce same upgrade strategy for cp and workers fo…

Fix validation to enforce same upgrade strategy for cp and workers fo… #23895

Triggered via push February 10, 2024 01:03
Status Failure
Total duration 50s
Artifacts
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 11 warnings
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/check_image_existence.go#L29
cyclomatic complexity 13 of func `(CheckImageExistence).Run` is high (> 10) (gocyclo)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/download.go#L51
cyclomatic complexity 11 of func `(Download).Run` is high (> 10) (gocyclo)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/check_image_existence.go#L20
exported: exported type CheckImageExistence should have comment or be unexported (revive)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/download.go#L15
exported: exported type Reader should have comment or be unexported (revive)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/download.go#L21
exported: exported type ImageMover should have comment or be unexported (revive)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/import.go#L11
exported: exported type Import should have comment or be unexported (revive)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/check_image_existence.go#L21
var-naming: struct field ImageUri should be ImageURI (revive)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/import.go#L20
exported: exported type ChartImporter should have comment or be unexported (revive)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/download.go#L25
exported: exported type ChartDownloader should have comment or be unexported (revive)
lint: cmd/eksctl-anywhere/cmd/internal/commands/artifacts/import_tools_image.go#L12
exported: exported type ImportToolsImage should have comment or be unexported (revive)
lint
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: golangci/golangci-lint-action@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
lint: pkg/dependencies/factory.go#L507
var-naming: var localIp should be localIP (revive)
lint: pkg/dependencies/factory.go#L962
var-naming: var clusterId should be clusterID (revive)
lint: pkg/dependencies/factory.go#L62
exported: exported type Dependencies should have comment or be unexported (revive)
lint: pkg/dependencies/factory.go#L129
exported: exported method Dependencies.Close should have comment or be unexported (revive)
lint: pkg/dependencies/factory.go#L179
exported: exported function NewFactory should have comment or be unexported (revive)
lint: pkg/dependencies/factory.go#L189
exported: exported method Factory.Build should have comment or be unexported (revive)
lint: pkg/dependencies/factory.go#L205
exported: exported method Factory.WithWriterFolder should have comment or be unexported (revive)
lint: pkg/dependencies/factory.go#L217
exported: exported method Factory.UseProxyConfiguration should have comment or be unexported (revive)
lint: pkg/dependencies/factory.go#L222
exported: exported method Factory.GetProxyConfiguration should have comment or be unexported (revive)
lint: pkg/dependencies/factory.go#L226
exported: exported method Factory.WithProxyConfiguration should have comment or be unexported (revive)