[PATCH conductor] fix for BZ #761509: User quota and other stats

Scott Seago sseago at redhat.com
Fri Dec 9 13:45:22 UTC 2011


On 12/09/2011 08:33 AM, Scott Seago wrote:
> The summary page at /pools was giving confusing (and incorrect) stats. Changes are as follows:
>
> Pools:
>      before: number of pools the user has view permissions on
>      after: number of pools the user has permission to launch deployments in
>
> Deployments:
>      before: number of deployments the user has view permission on
>      after: number of deployments owned (i.e. launched) by the user
>
> Instances:
>      before: number of instances the user has view permissions on
>      after: number of instances owned (i.e. launched) by the user
>
> Quota Used:
>      before: aggregated stats for environment quotas on all environments
>              that the user has view permisions on. This was the cause
>              of the BZ report, as by default, non-admin users have no
>              direct permissions on the default environment -- only on
>              the contained default pool. So for non-admin users
>              this showed 0 of 0, since it was an aggregation over
>              0 environments.
>      after: user quota stats.
>
> I've also updated the labels for the Overview to show "User Quota Usage" instead of "Quota Usage" and, for the Pool view, "Pool Quota Usage" instead of "Quota Usage" to clarify which quota measurement is being applied.
> ---
>   src/app/controllers/pools_controller.rb         |   14 ++++++--------
>   src/app/views/pools/_scoreboard_index.html.haml |   10 ++++++----
>   src/app/views/pools/_scoreboard_show.html.haml  |    4 +++-
>   src/features/pool.feature                       |    2 +-
>   src/features/step_definitions/instance_steps.rb |    6 ++++++
>   5 files changed, 22 insertions(+), 14 deletions(-)
>
>
Please push this upon review (if it's an ACK) as I'm on PTO today and 
Monday.

Scott




More information about the aeolus-devel mailing list