Skip to content
CI

fix: stuck hook issue when a Job resource has a ttlSecondsAfterFinished field set #1714

Sign in to view logs

fix: stuck hook issue when a Job resource has a ttlSecondsAfterFinished field set

fix: stuck hook issue when a Job resource has a ttlSecondsAfterFinished field set #1714

Triggered via pull request February 7, 2025 21:47
@agaudreaultagaudreault
synchronize #646
dejanzele:fix/job-ttl-stuck-hook
Status Failure
Total duration 55s
Artifacts –

ci.yaml

on: pull_request
test
48s
test
Fit to window
Zoom out
Zoom in

Annotations

10 errors
test: pkg/sync/sync_context_test.go#L1324
increment-decrement: should replace updatedCount += 1 with updatedCount++ (revive)
test: pkg/sync/sync_context_test.go#L1381
increment-decrement: should replace updatedCount += 1 with updatedCount++ (revive)
test: pkg/sync/sync_context_test.go#L1438
increment-decrement: should replace updatedCount += 1 with updatedCount++ (revive)
test: pkg/sync/sync_context_test.go#L32
duplicated-imports: Package "github.com/argoproj/gitops-engine/pkg/sync/common" already imported (revive)
test: pkg/sync/sync_context_test.go#L1322
unused-parameter: parameter 'action' seems to be unused, consider removing or renaming it as _ (revive)
test: pkg/sync/sync_context_test.go#L1379
unused-parameter: parameter 'action' seems to be unused, consider removing or renaming it as _ (revive)
test: pkg/sync/sync_context_test.go#L1436
unused-parameter: parameter 'action' seems to be unused, consider removing or renaming it as _ (revive)
test: pkg/health/health.go#L4
File is not properly formatted (goimports)
test: pkg/sync/sync_context.go#L301
fmt.Sprintf can be replaced with string concatenation (perfsprint)
test
issues found