-
Notifications
You must be signed in to change notification settings - Fork 272
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
Add a logic for RolloutManager to create ServiceMonitor #49
Conversation
54fb2a7
to
0822a10
Compare
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.
Thanks @Rizwana777! I have also verified that the Rollouts metrics are available from the OpenShift console UI, after a RolloutManager
CR is created.
111dece
to
4263dbd
Compare
6767d29
to
1dbd358
Compare
3d5917e
to
a2c611d
Compare
Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>
Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>
Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>
4f73244
to
31ccc21
Compare
Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>
6d1ac70
to
fdfa81c
Compare
Signed-off-by: Rizwana777 <rizwananaaz177@gmail.com>
Signed-off-by: Jonathan West <jonwest@redhat.com>
Signed-off-by: Jonathan West <jonwest@redhat.com>
Signed-off-by: Jonathan West <jonwest@redhat.com>
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.
LGTM, thanks @Rizwana777!
Issue #29
Added logic for RolloutManager to create ServiceMonitor and Added unit test and e2e test