Skip to content

DRAFT: OU-571: Add Perses feature flag for the monitoring-console-plugin #2052

DRAFT: OU-571: Add Perses feature flag for the monitoring-console-plugin

DRAFT: OU-571: Add Perses feature flag for the monitoring-console-plugin #2052

Triggered via pull request January 9, 2025 23:48
Status Failure
Total duration 14m 37s
Artifacts 1

pr-checks.yaml

on: pull_request
Lint the commit messages
22s
Lint the commit messages
Lint Github Action
8s
Lint Github Action
Lint code
46s
Lint code
Verify generated code
1m 5s
Verify generated code
Validate tools cache
1m 23s
Validate tools cache
Build bundle image
1m 18s
Build bundle image
Run end-to-end tests
14m 25s
Run end-to-end tests
Fit to window
Zoom out
Zoom in

Annotations

6 errors and 11 warnings
Lint the commit messages
You have commit messages with errors ⧗ input: Remove Requirements for UIPlugin > monitoring > thanosQuerier and alertManager. This is because we want to be able to have the options of creating a Perses proxy only and not include thanosQuerier and the alertManager. ✖ header must not be longer than 100 characters, current length is 218 [header-max-length] ✖ subject may not be empty [subject-empty] ✖ subject may not end with full stop [subject-full-stop] ✖ type may not be empty [type-empty] ✖ found 4 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: Add perses-dashboards to compatbility matrix and test ✖ subject may not be empty [subject-empty] ✖ type may not be empty [type-empty] ✖ found 2 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: Add Perses Dashboards Proxy QUESTION : where is the perses dashboard proxy URL coming from, we may need to adjust the code because right now it assumes it is in plugin.Spec.Montioring ✖ body's lines must not be longer than 100 characters [body-max-line-length] ✖ subject may not be empty [subject-empty] ✖ type may not be empty [type-empty] ⚠ body must have leading blank line [body-leading-blank] ✖ found 3 problems, 1 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint
Lint code: pkg/controllers/uiplugin/monitoring_test.go#L4
"encoding/json" imported and not used
Lint code: pkg/controllers/uiplugin/monitoring_test.go#L5
"fmt" imported and not used
Lint code: pkg/controllers/uiplugin/monitoring_test.go#L6
"log" imported and not used
Lint code
Process completed with exit code 2.
Verify generated code
Process completed with exit code 1.
Lint Github Action
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Lint the commit messages
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Lint code
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Lint code
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
Verify generated code
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Verify generated code
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
Build bundle image
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build bundle image
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
Validate tools cache
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Validate tools cache
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
Run end-to-end tests
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2

Artifacts

Produced during runtime
Name Size
cluster-state
16.8 KB