Re: Review Request 17332: Add a noun supporting operations on roles

[Bill Farner <wf...@apache.org>]

+dev, bcc reviews

-=Bill


On Tue, Feb 4, 2014 at 1:25 PM, Toby Weingartner <
tweingartner@twopensource.com> wrote:

> As a possible extension, I could see something like:
>
>   aurora quota get smf1/*  => return quota information for all users
>
> -Toby.
>
>
> On Tue, Feb 4, 2014 at 1:24 PM, Toby Weingartner <
> tweingartner@twopensource.com> wrote:
>
> > What else is hung on "aurora user"?  I think I'm in Kevin's camp... but
> > maybe I'm missing something about "aurora user"?
> >
> > -Toby.
> >
> >
> > On Tue, Feb 4, 2014 at 1:15 PM, Mark Chu-Carroll <mchucarroll@apache.org
> >wrote:
> >
> >> Toby:
> >>
> >> I guess I haven't been clear, because that is exactly Kevin's argument:
> >> that "quota" should be the noun.
> >>
> >> Quick contrast:
> >>
> >>    - Get quota for role: "aurora user get_quota west/mark" (me), "aurora
> >>    quota get west/mark" (Kevin)
> >>    - Get quota for role/env: "aurora user get_quota west/mark/test"
> (me),
> >>    "aurora quota get west/mark/test" (Kevin)
> >>    - Get quota for job: "aurora user get_quota west/mark/test/job" (me),
> >>    "aurora quota get west/mark/test/job" (Kevin)
> >>
> >> Looks like as far as the people who care go, I'm probably outnumbered.
> >> Anyone else want to chime in before I go and change this?
> >>
> >>      -Mark
> >>
> >>
> >>
> >> On Tue, Feb 4, 2014 at 4:05 PM, Toby Weingartner <
> >> tweingartner@twopensource.com> wrote:
> >>
> >> > Why can we not have:
> >> >
> >> >   aurora get_quota smf1/role
> >> >   aurora get_quota smf1/role/env
> >> >   aurora get_quota smf1/role/env/job
> >> >
> >> > To be able to set quota at each level (possibly future enhancement?).
> >>  I'm
> >> > unsure if this has been discussed, but possibly instead:
> >> >
> >> >   aurora quota get smf1/role
> >> >   aurora quota set smf1/role ...
> >> >
> >> > -Toby.
> >> >
> >> >
> >> >
> >> > On Tue, Feb 4, 2014 at 10:26 AM, Mark Chu-Carroll <
> >> mchucarroll@apache.org
> >> > >wrote:
> >> >
> >> > > I disagree.
> >> > >
> >> > > To me, the fundamental issue is just what a quota is. Is a quota a
> >> > > primitive basic object which has the role/cluster/environment as an
> >> > > attribute? Or is the role the basic object, and the quota is an
> >> attribute
> >> > > of the role? I think that the quota as attribute of a user makes
> much
> >> > more
> >> > > sense.
> >> > >
> >> > > Even in the case of things like the quota for a specific
> environment.
> >> I
> >> > > think that the question "What is roleA's quota on cluster west in
> the
> >> > prod
> >> > > environment?" is more natural than "What is the quota in the test
> >> > > environment of cluster west for roleA?" And so asking "aurora role
> >> > > get_quota smf1/mchucarroll/test" makes sense, because it's asking
> >> about
> >> > the
> >> > > quota  for a user in a particular place.
> >> > >
> >> > > I'm willing to give in if the consensus is against me, but I really
> >> think
> >> > > that the role-based quota command makes much more sense.
> >> > >
> >> > > Can others please chip in with their opinions?
> >> > >
> >> > >       -Mark
> >> > >
> >> > >
> >> > > On Fri, Jan 31, 2014 at 7:49 PM, Kevin Sweeney <ke...@apache.org>
> >> > wrote:
> >> > >
> >> > > >    This is an automatically generated e-mail. To reply, visit:
> >> > > > https://reviews.apache.org/r/17332/
> >> > > >
> >> > > > On January 24th, 2014, 2:35 p.m. PST, *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
> >> > > >
> >> > > >  On January 31st, 2014, 7:37 a.m. PST, *Mark Chu-Carroll* wrote:
> >> > > >
> >> > > > 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.
> >> > > >
> >> > > >  So it seems better to add:
> >> > > >
> >> > > > aurora role get_quota smf1/ksweeney
> >> > > > aurora environment get_quota smf1/ksweeney/prod
> >> > > > aurora job get_quota smf1/ksweeney/prod/app
> >> > > >
> >> > > > I'm not quite convinced, but I recognize I could be wrong. Anybody
> >> else
> >> > > want to chime in here?
> >> > > >
> >> > > >
> >> > > > - Kevin
> >> > > >
> >> > > > 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/>
> >> > > >
> >> > >
> >> >
> >>
> >
> >
>