Code static analysis

Update sync workflow to hadle specific conflicts #113

Sign in to view logs

Triggered via push November 15, 2024 13:58

atheo89

RHOAIENG-15443

Status Failure

Total duration 1m 22s

Artifacts –

code-quality.yaml

on: push

Matrix: golangci-lint

Matrix: govulncheck

Annotations

20 errors

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/notebook_controller.go#L121

Error return value of `retry.OnError` is not checked (errcheck)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/notebook_oauth.go#L193

Error return value of `rand.Read` is not checked (errcheck)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/suite_test.go#L58

var `cfg` is unused (unused)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/suite_test.go#L67

const `timeout` is unused (unused)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/suite_test.go#L68

const `interval` is unused (unused)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/notebook_route.go#L47

SA1019: pointer.Int32Ptr is deprecated: Use ptr.To instead. (staticcheck)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/notebook_webhook.go#L188

SA1019: pointer.Int32Ptr is deprecated: Use ptr.To instead. (staticcheck)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/notebook_webhook.go#L211

SA1019: pointer.Int32Ptr is deprecated: Use ptr.To instead. (staticcheck)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/notebook_controller_test.go#L149

SA1019: pointer.BoolPtr is deprecated: Use ptr.To instead. (staticcheck)

golangci-lint (components/odh-notebook-controller): components/odh-notebook-controller/controllers/notebook_controller_test.go#L150

SA1019: pointer.BoolPtr is deprecated: Use ptr.To instead. (staticcheck)

golangci-lint (components/notebook-controller): components/notebook-controller/controllers/culling_controller_test.go#L256

Error return value is not checked (errcheck)

golangci-lint (components/notebook-controller): components/notebook-controller/api/v1beta1/notebook_webhook.go#L24

var `notebooklog` is unused (unused)

golangci-lint (components/notebook-controller): components/notebook-controller/controllers/suite_test.go#L43

var `cfg` is unused (unused)

golangci-lint (components/notebook-controller): components/notebook-controller/controllers/notebook_controller.go#L96

S1021: should merge variable declaration with assignment on next line (gosimple)

golangci-lint (components/notebook-controller): components/notebook-controller/pkg/culler/culler.go#L137

S1033: unnecessary guard around call to delete (gosimple)

golangci-lint (components/notebook-controller): components/notebook-controller/pkg/culler/culler.go#L325

S1009: should omit nil check; len() for []github.com/kubeflow/kubeflow/components/notebook-controller/pkg/culler.KernelStatus is defined as zero (gosimple)

golangci-lint (components/notebook-controller): components/notebook-controller/pkg/culler/culler.go#L362

S1009: should omit nil check; len() for []github.com/kubeflow/kubeflow/components/notebook-controller/pkg/culler.TerminalStatus is defined as zero (gosimple)

golangci-lint (components/notebook-controller): components/notebook-controller/controllers/culling_controller.go#L359

S1009: should omit nil check; len() for []github.com/kubeflow/kubeflow/components/notebook-controller/controllers.KernelStatus is defined as zero (gosimple)

golangci-lint (components/notebook-controller): components/notebook-controller/controllers/culling_controller.go#L395

S1009: should omit nil check; len() for []github.com/kubeflow/kubeflow/components/notebook-controller/controllers.TerminalStatus is defined as zero (gosimple)

golangci-lint (components/notebook-controller): components/notebook-controller/pkg/culler/culler.go#L346

S1039: unnecessary use of fmt.Sprintf (gosimple)