Open Stack

On Thu, May 29, 2014 at 5:45 AM, Day, Phil <philip.day at hp.com> wrote:

>
>
>
>
> *From:* Kieran Spear [mailto:kispear at gmail.com]
> *Sent:* 28 May 2014 06:05
>
> *To:* OpenStack Development Mailing List (not for usage questions)
> *Subject:* Re: [openstack-dev] [nova] nova default quotas
>
>
>
> Hi Joe,
>
>
>
> On 28/05/2014, at 11:21 AM, Joe Gordon <joe.gordon0 at gmail.com> wrote:
>
>
>
>
>
>
>
> On Tue, May 27, 2014 at 1:30 PM, Kieran Spear <kispear at gmail.com> wrote:
>
>
> On 28/05/2014, at 6:11 AM, Vishvananda Ishaya <vishvananda at gmail.com>
> wrote:
>
> > Phil,
> >
>
> > You are correct and this seems to be an error. I don’t think in the
> earlier ML thread[1] that anyone remembered that the quota classes were
> being used for default quotas. IMO we need to revert this removal as we
> (accidentally) removed a Havana feature with no notification to the
> community. I’ve reactivated a bug[2] and marked it critical.
>
> +1.
>
> We rely on this to set the default quotas in our cloud.
>
>
>
> Hi Kieran,
>
>
>
> Can you elaborate on this point. Do you actually use the full quota-class
> functionality that allows for quota classes, if so what provides the quota
> classes? If you only use this for setting the default quotas, why do you
> prefer the API and not setting the config file?
>
>
>
> We just need the defaults. My comment was more to indicate that yes, this
> is being used by people. I'm sure we could switch to using the config file,
> and generally I prefer to keep configuration in code, but finding out about
> this half way through a release cycle isn't ideal.
>
>
>
> I notice that only the API has been removed in Icehouse, so I'm assuming
> the impact is limited to *changing* the defaults, which we don't do often.
> I was initially worried that after upgrading to Icehouse we'd be left with
> either no quotas or whatever the config file defaults are, but it looks
> like this isn't the case.
>
>
>
> Unfortunately the API removal in Nova was followed by similar changes in
> novaclient and Horizon, so fixing Icehouse at this point is probably going
> to be difficult.
>
>
>
> *[Day, Phil]  I think we should revert the changes in all three system
> then.   We have the rules about not breaking API compatibility in place for
> a reason, if we want to be taken seriously as a stable API then we need to
> be prepared to roll back if we goof-up.*
>
>
>
> *Joe – was there a nova-specs BP for the change ?  I’m wondering how this
> one slipped through*
>

That's a good question.

the API extension quota-classes has been around for a very long time and
never actually worked [0]. This was brought up again in February 2014 and
the original author chimed in saying it doesn't work [1]. When this came up
there was no discussion around the default quota value functionality, and
it didn't come up in any of the reviews. Because this was supposed to be
just removing dead code there as no nova-specs BP for it.


[0] http://lists.openstack.org/pipermail/openstack-dev/2014-May/036053.html
[1]
http://lists.openstack.org/pipermail/openstack-dev/2014-February/027574.html



>
>
>
>
> Cheers,
>
> Kieran
>
>
>
>
>
>
> Kieran
>
>
> >
> > Vish
> >
> > [1]
> http://lists.openstack.org/pipermail/openstack-dev/2014-February/027574.html
> > [2] https://bugs.launchpad.net/nova/+bug/1299517
> >
> > On May 27, 2014, at 12:19 PM, Day, Phil <philip.day at hp.com> wrote:
> >
> >> Hi Vish,
> >>
> >> I think quota classes have been removed from Nova now.
> >>
> >> Phil
> >>
> >>
> >> Sent from Samsung Mobile
> >>
> >>
> >> -------- Original message --------
> >> From: Vishvananda Ishaya
> >> Date:27/05/2014 19:24 (GMT+00:00)
> >> To: "OpenStack Development Mailing List (not for usage questions)"
> >> Subject: Re: [openstack-dev] [nova] nova default quotas
> >>
> >> Are you aware that there is already a way to do this through the cli
> using quota-class-update?
> >>
> >> http://docs.openstack.org/user-guide-admin/content/cli_set_quotas.html
> (near the bottom)
> >>
> >> Are you suggesting that we also add the ability to use just regular
> quota-update? I’m not sure i see the need for both.
> >>
> >> Vish
> >>
> >> On May 20, 2014, at 9:52 AM, Cazzolato, Sergio J <
> sergio.j.cazzolato at intel.com> wrote:
> >>
> >>> I would to hear your thoughts about an idea to add a way to manage the
> default quota values through the API.
> >>>
> >>> The idea is to use the current quota api, but sending ''default'
> instead of the tenant_id. This change would apply to quota-show and
> quota-update methods.
> >>>
> >>> This approach will help to simplify the implementation of another
> blueprint named per-flavor-quotas
> >>>
> >>> Feedback? Suggestions?
> >>>
> >>>
> >>> Sergio Juan Cazzolato
> >>> Intel Software Argentina
> >>>
> >>> _______________________________________________
> >>> OpenStack-dev mailing list
> >>> OpenStack-dev at lists.openstack.org
> >>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >>
> >>
> >> _______________________________________________
> >> OpenStack-dev mailing list
> >> OpenStack-dev at lists.openstack.org
> >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >
> > _______________________________________________
> > OpenStack-dev mailing list
> > OpenStack-dev at lists.openstack.org
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140602/678f6c7f/attachment.html>