horizon gets the default quota for floating ips for new projects from the what the current user has

Bug #1337473 reported by Matt Fischer
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Medium
Akihiro Motoki

Bug Description

After some investigation it appears that when you login to Horizon and go to create a new project, it gets the default neutron floating IP quota value from the current value that the logged in user has. This was quite confusing.

This will likely not be fixable until this feature lands in neutron: https://bugs.launchpad.net/neutron/+bug/1204956, Since Horizon has no way to get defaults and this is probably a good reasonable solution until then.

Steps:

Login as a user with who's default tenant has a quota of 50 floating IPs.
Go to create project, note the pre-filled value is 50.
Change tenant's quota to "123"
Logout
Login again
Go to create project, note the pre-filled value is 123.

Tags: api
Revision history for this message
Matt Fischer (mfisch) wrote :

Floating ip defaults == 123 after changing them for my tenant and logging out.

Tracy Jones (tjones-i)
tags: added: api
Revision history for this message
Sean Dague (sdague) wrote :

not clear why this was a nova bug

Changed in nova:
status: New → Invalid
Revision history for this message
Akihiro Motoki (amotoki) wrote :

I removed nova from the affected project as it is only related to horizon and neutron.

no longer affects: nova
Revision history for this message
Akihiro Motoki (amotoki) wrote :

The current implementation is a workaround as neutron API did not provide a way to retrieve the quota defaults. Thus our suggestion is to keep the quota of the admin user which is used to create a project to match the default quota.

Revision history for this message
Akihiro Motoki (amotoki) wrote :

There is one good news. neutron default quota API was added in Pike cycle, so horizon can change the quota logic to leverage the new neutron API which returns the default quotas (in Queens cycle).

Considering the above, I changes the status to Triaged.

Note that this feature will not be backported to PIke or past releases (as it is a new feature) even if horizon queens supports it.

Changed in horizon:
status: New → Triaged
milestone: none → queens-2
importance: Undecided → Medium
Ying Zuo (yingzuo)
Changed in horizon:
milestone: queens-2 → queens-3
Changed in horizon:
assignee: nobody → Radomir Dopieralski (deshipu)
Revision history for this message
Akihiro Motoki (amotoki) wrote :

https://review.openstack.org/#/c/526702/ is fixing this. Sorry that I forgot to link this.

Changed in horizon:
assignee: Radomir Dopieralski (deshipu) → Akihiro Motoki (amotoki)
Changed in horizon:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/526702
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=dc7d2582db84190da0431fb7171a028b6abe2ea0
Submitter: Zuul
Branch: master

commit dc7d2582db84190da0431fb7171a028b6abe2ea0
Author: Akihiro Motoki <email address hidden>
Date: Mon Dec 4 00:05:43 2017 +0900

    quota: Use neutron default quota (read-only) API

    Neutron provides an API to retrieve default quotas since newton.
    Unfortunately there is no corresponding neutron API extension
    is not defined, so there is no way to detect it via API.
    The API exists since newton, so this commit just switches to
    the default quota API.

    Closes-Bug: #1337473
    blueprint make-quotas-great-again
    Change-Id: Ia826b67f4cec7d95335c62ef98c3040feaa06117

Changed in horizon:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/horizon 13.0.0.0b3

This issue was fixed in the openstack/horizon 13.0.0.0b3 development milestone.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.