Interesting idea... I just thought of quota as being a natural property of
the role.

Are there are quota verbs that would make sense? Maybe an automated quota
request could be a verb?




On Fri, Jan 24, 2014 at 5:35 PM, Kevin Sweeney <kevi...@apache.org> wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17332/
>
> Why is this happening on a role noun and not a quota noun.
>
> i.e.
>
> aurora quota get west/ksweeney
>
> v.s.
>
> aurora role get_quota west/ksweeney
>
> The second looks inconsistent with the noun-verb model to me and if we ever 
> decide that a quota could be set at a lower level the first is more extensible
>
> aurora quota get west/ksweeney/prod
> aurora quota get west/ksweeney/prod/appserver
>
>
> - Kevin Sweeney
>
> On January 24th, 2014, 2:07 p.m. PST, Mark Chu-Carroll wrote:
>   Review request for Aurora, Kevin Sweeney and Brian Wickman.
> By Mark Chu-Carroll.
>
> *Updated Jan. 24, 2014, 2:07 p.m.*
>  *Bugs: * aurora-107 <https://issues.apache.org/jira/browse/aurora-107>
>  *Repository: * aurora
> Description
>
> Add a "role" noun.
>
> Currently, the only operation on roles is getting the quota associated with a 
> role,
> but there are definitely others that can be added later: what packages has a 
> role created?
> How many jobs is a role running? What privileges are associated with a role? 
> How much
> system storage is being used by a role? Etc.
>
>
>   Testing
>
> [sun-wukong incubator-aurora (user)]$ ./pants 
> src/test/python/apache/aurora/client/cli:all
> Build operating on targets: 
> OrderedSet([PythonTestSuite(src/test/python/apache/aurora/client/cli/BUILD:all)])
> ==================================== test session starts 
> ====================================
> platform darwin -- Python 2.7.2 -- pytest-2.5.1
> collected 20 items
>
> src/test/python/apache/aurora/client/cli/test_create.py ....
> src/test/python/apache/aurora/client/cli/test_status.py .....
> src/test/python/apache/aurora/client/cli/test_diff.py ...
> src/test/python/apache/aurora/client/cli/test_kill.py .....
> src/test/python/apache/aurora/client/cli/test_get_quota.py ...
>
> ================================= 20 passed in 0.71 seconds 
> =================================
> src.test.python.apache.aurora.client.cli.job                                  
>   .....   SUCCESS
>
>   Diffs
>
>    - src/main/python/apache/aurora/client/cli/BUILD
>    (8828c1e68b3ae7793fb0bb081730e0ff8fff5ed1)
>    - src/main/python/apache/aurora/client/cli/__init__.py
>    (20ecbcf5d3a868f91922244162b516a66d24d32b)
>    - src/main/python/apache/aurora/client/cli/options.py
>    (1b7155409505b46451df072edd196dd7e4c88f1c)
>    - src/main/python/apache/aurora/client/cli/role.py (PRE-CREATION)
>    - src/test/python/apache/aurora/client/cli/BUILD
>    (f9ebe0cf626a040aa67654faea07b8902e558282)
>    - src/test/python/apache/aurora/client/cli/test_get_quota.py
>    (PRE-CREATION)
>    - src/test/python/apache/aurora/client/cli/util.py
>    (76f954377c154d2a7ab3292a2d772b0566ea06fa)
>
> View Diff <https://reviews.apache.org/r/17332/diff/>
>

Reply via email to