DRAFT: Add Perses feature flag for the monitoring-console-plugin #2056
pr-checks.yaml
on: pull_request
Lint the commit messages
21s
Lint Github Action
5s
Lint code
46s
Verify generated code
1m 8s
Validate tools cache
1m 31s
Build bundle image
50s
Run end-to-end tests
16m 17s
Annotations
2 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
⧗ 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
|
Lint code
Process completed with exit code 2.
|
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
|
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
|
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.4 KB |
|