Skip to content

Conversation

@RakeshGuptaDev
Copy link
Contributor

What changes were proposed in this pull request?

We should add Summary section to existing Ranger Admin metrics response. The summary section will have following

Total number of services
Total number of policies
Total number of tags
Total number of users
Total number of groups
Total number of roles
Total number of admin audits (x_trx_log rows)
Total number of login records (x_auth_sess rows)
Total number of plugin records (x_plugin_info rows)
Total number of plugin download records (x_policy_export_audit rows)
This will help us improved debugging.

How was this patch tested?

Tested and Verified: The /metrics/json API now successfully returns the summary section.

Copy link
Contributor

@kishorgollapalliwar kishorgollapalliwar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look good to me. Please make sure to have enough DEV testing before merging the change.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants