search

yahoo / mysql_perf_analyzer

MySQL performance monitoring and analysis.
Apache License 2.0
1.44k stars 214 forks source link

Allow specific group for new user #15

Closed devalias closed 9 years ago

devalias commented 9 years ago

Hi, please allow the ability to assign a label to a new user, I don't want a user to be able to see other labels.

xrao commented 9 years ago

Can you explain in detail?

devalias commented 9 years ago

I would like to create a new user that can only view certain servers under a group, for instance allow only user "A" to only see group "A" I probably should have said group instead of label. :) Admin user can see all groups.

xrao commented 9 years ago

I am considering to add a restricted user type for this purpose. Will update you when I am done.

xrao commented 9 years ago

A user type "Restricted User" has been added to "User" screen. After a restricted user is created, you can use "DB Info" page, "Access Control" tab to assign/revoke user access.

devalias commented 9 years ago

Thanks, I will try this feature out soon.Sent with SolMail App--------- Original Message --------- From: xrao To: yahoo/mysql_perf_analyzer Date: Thu Jul 09 17:27:39 EDT 2015 Subject: Re: [mysql_perf_analyzer] Allow specific group for new user (#15)

A user type "Restricted User" has been added to "User" screen. After a restricted user is created, you can use "DB Info" page, "Access Control" tab to assign/revoke user access.

—Reply to this email directly or view it on GitHub.

devalias commented 9 years ago

hi, I created a new restricted user, dashboard looks fine but when I try to view realtime this pops up: "No valid credential provided for DB ..."

xrao commented 9 years ago

Unfortunately, this is by design. We designed this tool so that every dev/dba/support can self register and use it for their own purpose, under the assumption every user will register their own database user/password for concerned databases. So, for now, you have to login as the restricted user, use DB Info page, credential tab, to add DB credentials to access the underlying DB. However, the access to the metrics data such as charts/alerts is not limited by this restriction. For the future, we can consider to copy DB credential to the restricted user when assign databases.

devalias commented 9 years ago

ok, thanks for the explanation. yes, this feature will be nice since other users that are restricted would like to be able to view their servers realtime.

xrao commented 9 years ago

Updated. Please test.

devalias commented 9 years ago

Hi Xrao, updated and this works great. Thanks!