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

operators [O] prometheus (0.47.0) #1054

Closed
wants to merge 1 commit into from
Closed

operators [O] prometheus (0.47.0) #1054

wants to merge 1 commit into from

Conversation

zfrhv
Copy link
Contributor

@zfrhv zfrhv commented Apr 13, 2022

thanks to #1045 for mentioning
the operator tries to monitor all namespaces even in single namespace installation.

when the installation will be namespace scoped, the operator should work as previously.
when the operator is installed cluster scoped, he will manage all the serviceMonitors, prometheusRules... in all namespaces, and the prometheus, alertmanager instances only in his namespace.

i didnt checked the cluster scoped option yet, but the namespace scoped work as always worked before.

(i hope the signoff will work now)

@openshift-ci openshift-ci bot requested review from J0zi and mvalarh April 13, 2022 12:44
@github-actions github-actions bot changed the title fixing operator targets operators [O] prometheus (0.47.0) Apr 13, 2022
@zfrhv zfrhv closed this Apr 13, 2022
@framework-automation
Copy link
Collaborator

/merge possible

1 similar comment
@framework-automation
Copy link
Collaborator

/merge possible

@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@github-actions
Copy link
Contributor

DCO is missing. Please signoff your commits !!!

@github-actions
Copy link
Contributor

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@github-actions
Copy link
Contributor

DCO is missing. Please signoff your commits !!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

4 participants