-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17332/#review32757
-----------------------------------------------------------


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 Jan. 24, 2014, 2:07 p.m., Mark Chu-Carroll wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/17332/
> -----------------------------------------------------------
> 
> (Updated Jan. 24, 2014, 2:07 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Brian Wickman.
> 
> 
> 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.
> 
> 
> 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 
> 
> Diff: https://reviews.apache.org/r/17332/diff/
> 
> 
> 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
> 
> 
> Thanks,
> 
> Mark Chu-Carroll
> 
>

Reply via email to