DRAFT: OU-571: Add Perses feature flag for the monitoring-console-plugin · rhobs/observability-operator@0a435b9

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

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.

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