Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Bump github.com/golangci/golangci-lint from 1.55.2 to 1.56.1 in /tools #1982

Bump github.com/golangci/golangci-lint from 1.55.2 to 1.56.1 in /tools

Bump github.com/golangci/golangci-lint from 1.55.2 to 1.56.1 in /tools #1982

Triggered via pull request February 12, 2024 01:57
Status Failure
Total duration 13m 24s
Artifacts

ci.yml

on: pull_request
Matrix: Check
Matrix: API Integration Tests
Matrix: Test
Fit to window
Zoom out
Zoom in

Annotations

14 errors, 6 warnings, and 1 notice
Check (1.21.x, false): api/validation_test.go#L503
unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive)
Check (1.21.x, false): api/proxy.go#L152
unused-parameter: parameter 'req' seems to be unused, consider removing or renaming it as _ (revive)
Check (1.21.x, false): pkg/pmm/apikey.go#L47
fmt.Sprintf can be replaced with string addition (perfsprint)
Check (1.21.x, false): api/backup_storage.go#L285
fmt.Sprintf can be replaced with string addition (perfsprint)
Check (1.21.x, false): api/database_cluster_backup.go#L40
fmt.Sprintf can be replaced with string addition (perfsprint)
Check (1.21.x, false): api/monitoring_instance.go#L117
fmt.Errorf can be replaced with errors.New (perfsprint)
Check (1.21.x, false): api/monitoring_instance.go#L137
fmt.Errorf can be replaced with errors.New (perfsprint)
Check (1.21.x, false): api/monitoring_instance.go#L139
fmt.Errorf can be replaced with errors.New (perfsprint)
Check (1.21.x, false): api/validation_test.go#L503
[golangci-lint] reported by reviewdog 🐶 unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) Raw Output: api/validation_test.go:503:10: unused-parameter: parameter 'ctx' seems to be unused, consider removing or renaming it as _ (revive) func(ctx context.Context, s string) (*everestv1alpha1.BackupStorage, error) { return storage, nil }, ^
Check (1.21.x, false): api/proxy.go#L152
[golangci-lint] reported by reviewdog 🐶 unused-parameter: parameter 'req' seems to be unused, consider removing or renaming it as _ (revive) Raw Output: api/proxy.go:152:39: unused-parameter: parameter 'req' seems to be unused, consider removing or renaming it as _ (revive) return func(res http.ResponseWriter, req *http.Request, err error) { ^
[api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up: api-tests/tests/psmdb-clusters.spec.ts#L1
1) [api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up Test timeout of 30000ms exceeded.
[api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up: api-tests/tests/psmdb-clusters.spec.ts#L224
1) [api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up Error: page.waitForTimeout: Test timeout of 30000ms exceeded. 222 | }) 223 | for (let i = 0; i < 15; i++) { > 224 | await page.waitForTimeout(2000) | ^ 225 | 226 | const psmdbCluster = await request.get(`/v1/namespaces/${testsNs}/database-clusters/${clusterName}`) 227 | at /home/runner/work/percona-everest-backend/percona-everest-backend/api-tests/tests/psmdb-clusters.spec.ts:224:16
[api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up: api-tests/tests/psmdb-clusters.spec.ts#L1
1) [api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Test timeout of 30000ms exceeded.
[api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up: api-tests/tests/psmdb-clusters.spec.ts#L224
1) [api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: page.waitForTimeout: Test timeout of 30000ms exceeded. 222 | }) 223 | for (let i = 0; i < 15; i++) { > 224 | await page.waitForTimeout(2000) | ^ 225 | 226 | const psmdbCluster = await request.get(`/v1/namespaces/${testsNs}/database-clusters/${clusterName}`) 227 | at /home/runner/work/percona-everest-backend/percona-everest-backend/api-tests/tests/psmdb-clusters.spec.ts:224:16
Check (1.21.x, false)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: percona-platform/setup-go@v4, percona-platform/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Check (1.21.x, false)
Restore cache failed: Dependencies file is not found in /home/runner/work/percona-everest-backend/percona-everest-backend. Supported file pattern: go.sum
Test (1.21.x, false)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: percona-platform/setup-go@v4, percona-platform/cache@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Test (1.21.x, false)
Restore cache failed: Dependencies file is not found in /home/runner/work/percona-everest-backend/percona-everest-backend. Supported file pattern: go.sum
API Integration Tests (1.21.x, false)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: percona-platform/setup-go@v4, percona-platform/cache@v3, medyagh/setup-minikube@latest. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
API Integration Tests (1.21.x, false)
Restore cache failed: Dependencies file is not found in /home/runner/work/percona-everest-backend/percona-everest-backend. Supported file pattern: go.sum
🎭 Playwright Run Summary
1 flaky [api] › psmdb-clusters.spec.ts:190:5 › expose psmdb cluster on EKS to the public internet and scale up 51 passed (1.9m)