Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Requirements for Prometheus #136

Merged
merged 1 commit into from
Nov 6, 2023
Merged

Conversation

roehrich-hpe
Copy link
Contributor

Add a label to the ServiceMonitor resource that will be common across all DWS and NNF repos so we easily tell Prometheus which ones it should select.

Copy the namespace value into config/prometheus so that part of config/ can be deployed on its own.

Add a ClusterRoleBinding that can be used by a metrics reader process to debug the metrics endpoing of the controller. The ClusterRole is already being installed, and the ServiceAccount is already installed.

Update the deploy.sh script to determine whether or not the ServiceMonitor resource can be applied.

Add a label to the ServiceMonitor resource that will be common across all
DWS and NNF repos so we can easily tell Prometheus which ones it should select.

Copy the namespace value into config/prometheus so that part of config/ can
be deployed on its own.

Add a ClusterRoleBinding that can be used by a metrics reader process to
debug the metrics endpoint of the controller.  The ClusterRole is already
being installed, and the ServiceAccount is already installed.

Update the deploy.sh script to determine whether or not the ServiceMonitor
resource can be applied.

Signed-off-by: Dean Roehrich <[email protected]>
@roehrich-hpe roehrich-hpe force-pushed the prometheus-requirements branch from 536c6d3 to 9727c76 Compare November 6, 2023 21:41
@roehrich-hpe roehrich-hpe merged commit 3b41e1a into master Nov 6, 2023
4 checks passed
@roehrich-hpe roehrich-hpe deleted the prometheus-requirements branch November 6, 2023 22:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants