DO NOT REVIEW/MERGE #37481

Triggered via pull request April 24, 2025 17:01

evgeniy-scherbina

synchronize #16523

dk/prebuilds

Status Failure

Total duration 10m 31s

Artifacts 2

ci.yaml

on: pull_request

Matrix: test-e2e

Matrix: test-go-pg

Matrix: test-go

test-go-tailnet-integration

Matrix: test-cli

notify-slack-on-failure

Annotations

21 errors and 3 warnings

fmt

Process completed with exit code 2.

lint: provisioner/terraform/resources.go#L907

G115: integer overflow conversion int -> int32 (gosec)

lint: enterprise/coderd/coderd.go#L1165

flag-parameter: parameter 'entitled' seems to be a control flag, avoid control coupling (revive)

lint: coderd/members_test.go#L55

empty-lines: extra empty line at the start of a block (revive)

lint: provisioner/terraform/executor.go#L383

type assertion must be checked (forcetypeassert)

lint: enterprise/coderd/prebuilds/metricscollector.go#L89

ruleguard: Using 'AsPrebuildsOrchestrator' is dangerous and should be accompanied by a comment explaining why it's ok and a nolint. (gocritic)

lint: cli/agent.go#L400

ruleguard: Error should be logged using "slog.Error" instead. (gocritic)

lint: cli/agent.go#L416

ruleguard: Message "reinitializing..." must start with lowercase, and does not end with a special characters. (gocritic)

lint: enterprise/coderd/coderd.go#L1181

ruleguard: Error should be logged using "slog.Error" instead. (gocritic)

lint: enterprise/coderd/prebuilds/claim_test.go#L236

ruleguard: Do not call functions that may call t.FailNow in a goroutine, as this can cause data races (see testing.go:834) (gocritic)

lint: enterprise/coderd/prebuilds/claim_test.go#L346

ruleguard: Do not call functions that may call t.FailNow in a goroutine, as this can cause data races (see testing.go:834) (gocritic)

test-go-pg-16

Process completed with exit code 2.

test-go (macos-latest)