Skip to content

DRAFT: Add Perses feature flag for the monitoring-console-plugin #2057

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

DRAFT: Add Perses feature flag for the monitoring-console-plugin #2057

Triggered via pull request January 13, 2025 21:10
Status Failure
Total duration 14m 53s
Artifacts 1

pr-checks.yaml

on: pull_request
Lint the commit messages
24s
Lint the commit messages
Lint Github Action
5s
Lint Github Action
Lint code
2m 9s
Lint code
Verify generated code
1m 3s
Verify generated code
Validate tools cache
1m 14s
Validate tools cache
Build bundle image
49s
Build bundle image
Run end-to-end tests
14m 44s
Run end-to-end tests
Fit to window
Zoom out
Zoom in

Annotations

1 error 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 ⧗ input: Adjust kubebuilder configs so that Alertmanager, Thanos Querier and PersesDashboards are not required attributes. Then run `make generate` to update CRDs (rather than manual editing). ✖ header must not be longer than 100 characters, current length is 183 [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: Run `make bundle` to update uiplugins CRD /bundle/manifests/observability.openshift.io_uiplugins.yaml ✖ header must not be longer than 100 characters, current length is 101 [header-max-length] ✖ subject may not be empty [subject-empty] ✖ type may not be empty [type-empty] ✖ found 3 problems, 0 warnings ⓘ Get help: https://github.com/conventional-changelog/commitlint/#what-is-commitlint ⧗ input: Update UIPlugin controller to use Service and UserToken Alias:"perses" instead of "perses-dashboard-proxy". ✖ header must not be longer than 100 characters, current length is 107 [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: Update UIPlugin CRDs so that PersesDashboards require a 'serviceName' rather than a URL. This because COO controls Perses Operator so COO just needs the ServiceName and Namespace to fetch the Perses Service. Requiring the exact endpoint is less flexible. ✖ header must not be longer than 100 characters, current length is 254 [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: Remove the need for "config.PersesDashboards.Url" because we only need the name, namespace, and port to connect to the Perses Service. ✖ header must not be longer than 100 characters, current length is 134 [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: Update monitoring_test.go to align with Alias name change "perses". ✖ subje
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
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
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
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
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
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
17.5 KB