-
Notifications
You must be signed in to change notification settings - Fork 34
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
fix the config file of oracledb dashboard #40
Comments
Thank you for reporting the issue, I will take a look and update it. |
We had a look at this suggestion, and we agree with your suggestion for status=ACTIVE. For the type, our thinking is that people might want to see things like jobs that run from time to time and so consume sessions too. |
Yes, it may not be necessary to limit it to users. The problem is the metric "oracledb_sessions_active" does not exist, so it shows no data on Grafana. |
Great, thank you again. I will make this change tomorrow. |
pushed the change in #42 |
closing this issue now, please feel free to reopen it if needed. |
I found a mistake in 3333_rev1.json.
The query for active sessions is written as follows,
"expr": "oracledb_sessions_active{instance=\"$host\"}"
but I think it is correctly written as follows.
"expr": "oracledb_sessions_value{instance=\"$host\", status=\"ACTIVE\", type=\"USER\"}"
This worked correctly for me.
The text was updated successfully, but these errors were encountered: