-
Notifications
You must be signed in to change notification settings - Fork 119
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 oauth metrics into the jmx_metrics_config.yaml
#718
Conversation
Signed-off-by: Lukas Kral <lukywill16@gmail.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.
What happens when OAuth is not used?
@scholzj good question, I will try it |
@scholzj I tested this with just enabled metrics, without Oauth, and it just prints metrics, that are available. There are no Oauth metrics at all (there is no NaN or nulls etc.) |
Signed-off-by: Lukas Kral <lukywill16@gmail.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. I don't know all the OAuth metrics, so I would assume all these rules for them are necessary and we have them in the operator as well I guess?
Those metrics, which I'm adding to the Bridge, I found in Oauth repository from @mstruk , so I assumed that's all needed. |
Sorry for late chiming-in. It looks good yes. |
Signed-off-by: Lukas Kral lukywill16@gmail.com
In Oauth 0.11.0, the oauth metrics were introduced.
But, to scrape these Oauth metrics from Bridge, we need to add the correct patterns into the
jmx_metrics_config.yaml
.This PR does it.