Skip to content

✨ Bump cluster-api dependency to v1.5.0-beta.1 #828

✨ Bump cluster-api dependency to v1.5.0-beta.1

✨ Bump cluster-api dependency to v1.5.0-beta.1 #828

Triggered via pull request July 4, 2023 08:07
Status Failure
Total duration 4m 52s
Artifacts

golangci-lint.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 3 warnings
lint: apis/v1beta1/vspherefailuredomain_webhook_test.go#L138
assignment mismatch: 1 variable but tt.failureDomain.ValidateCreate returns 2 values (typecheck)
lint: apis/v1beta1/vspheremachinetemplate_webhooks_test.go#L74
assignment mismatch: 1 variable but webhook.ValidateCreate returns 2 values (typecheck)
lint: apis/v1beta1/vspheremachinetemplate_webhooks_test.go#L136
assignment mismatch: 1 variable but webhook.ValidateUpdate returns 2 values (typecheck)
lint: controllers/vmware/vspherecluster_reconciler_test.go#L80
not enough arguments in call to reconciler.VSphereMachineToCluster
lint: controllers/vmware/vspherecluster_reconciler_test.go#L86
not enough arguments in call to reconciler.VSphereMachineToCluster
lint: controllers/vmware/test/controllers_test.go#L225
opts.Resync undefined (type "sigs.k8s.io/controller-runtime/pkg/cache".Options has no field or method Resync) (typecheck)
lint: test/e2e/e2e_suite_test.go#L153
cannot use LogCollector{} (value of type LogCollector) as framework.ClusterLogCollector value in argument to framework.WithMachineLogCollector: LogCollector does not implement framework.ClusterLogCollector (missing method CollectInfrastructureLogs) (typecheck)
lint: main.go#L92
SA1019: managerOpts.Namespace is deprecated: Use Cache.Namespaces instead. (staticcheck)
lint: main.go#L117
SA1019: managerOpts.Port is deprecated: Use WebhookServer instead. A WebhookServer can be created via webhook.NewServer. (staticcheck)
lint: main.go#L175
SA1019: managerOpts.Namespace is deprecated: Use Cache.Namespaces instead. (staticcheck)
lint: controllers/serviceaccount_controller.go#L622
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
lint: controllers/servicediscovery_controller.go#L449
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
lint: controllers/servicediscovery_controller.go#L458
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)