-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[BZ 1570583] Install metrics components in different namespace #8613
Conversation
Metrics components can no longer go in openshift-infra. This commit puts them in openshift-metrics.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jsanda The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@jsanda it is still valid for HOSA to still be installed in Looks like this file also needs to be updated: openshift-ansible/roles/openshift_metrics/templates/hawkular_openshift_agent_cm.j2 Lines 15 to 19 in 6790ffa
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
openshift-ansible/roles/openshift_metrics/templates/hawkular_openshift_agent_cm.j2
Lines 15 to 19 in 6790ffa
url: https://hawkular-metrics.openshift-infra.svc.cluster.local | |
credentials: | |
username: secret:openshift-infra/hawkular-metrics-account/hawkular-metrics.username | |
password: secret:openshift-infra/hawkular-metrics-account/hawkular-metrics.password | |
ca_cert_file: secret:openshift-infra/hawkular-metrics-certs/ca.crt |
Can you replace the instances of openshift-infra
there with {{ openshift_metrics_project }}
?
I will make the updates. With respect to HOSA, there is actually another ticket, BZ 1562210, to remove it. |
@@ -60,7 +60,7 @@ openshift_metrics_resolution: 30s | |||
|
|||
openshift_metrics_master_url: https://kubernetes.default.svc | |||
openshift_metrics_node_id: nodename | |||
openshift_metrics_project: openshift-infra | |||
openshift_metrics_project: openshift-metrics |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Prometheus and garbage collection are also installed under openshift-metrics project, will our change affect their functions?
Will it be ok with existing users who upgrade from older versions ? |
@ewolinetz I updated hawkular_openshift_agent_cm.j2. With respect to @juzhao's question:
Should I change the default value of openshift_metrics_project? |
@jsanda how likely is it that prometheus and hawkular would both be installed at the same time? Is there any negative repercussions from having them installed/running in the same project? With regards to @theute 's question, I think we should do something similar to what we did for logging, which was try to detect if there is a hawkular stack already installed in its current location, if so then we would maintain configuring in that project, but possibly warn users that they should migrate? Unfortunately there isn't a way to migrate persistent volumes to another project, otherwise we could just do that. |
long term the prometheus stack will be installed in openshift-monitoring not openshift-metrics as far as both hawkular and prometheus being installed during 3.10 - it is definitely possible for customers that are wanting to try out prom for cluster monitoring |
This PR is being replaced by PR 8649. |
Metrics components can no longer go in openshift-infra. This commit puts them in openshift-metrics.
@ewolinetz can you please review this?
I am operating under the assumption that additional changes will be needed, but things seem to be working after my initial smoke test.