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
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
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)
|