> On Jan. 24, 2014, 5:35 p.m., Kevin Sweeney wrote: > > 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
I've been thinking about this, debating it in my own head, and I just can't agree. "Quota" isn't a standalone entity; A quota is an attribute of a role. It doesn't make sense to me to have it be a standalone noun. In terms of syntax, in noun/verb commands, the first parameter is almost always a value of the noun type. (For example, inn "job create", the first parameter is a job identifier.) For quota, there is no such thing as a quota identifier or a quota object. You specify the role who's quota you're interested in. Even in the extended scenarios you're talking about, I don't think it makes sense. The quota for "west/ksweeney/prod" is ksweeney's quota in the prod environment. - Mark ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17332/#review32757 ----------------------------------------------------------- On Jan. 24, 2014, 5: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, 5: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 > >
