Periodic task for quota reservations expire should move out from scheduler

Bug #1420102 reported by Alex Xu
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Invalid
Wishlist
Yingxin

Bug Description

There periodic task for check expired quota reservation in scheduler https://github.com/openstack/nova/blob/master/nova/scheduler/manager.py#L80

There could be multiple nova schedulers, and quota stuff isn't related to the scheduler, so we should move the quota out of scheduler and find a new way to expire reservations.

Alex Xu (xuhj)
Changed in nova:
assignee: nobody → Alex Xu (xuhj)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/154339

Changed in nova:
status: New → In Progress
Changed in nova:
importance: Undecided → Low
Changed in nova:
importance: Low → Wishlist
tags: added: scheduler
Changed in nova:
milestone: none → kilo-3
Joe Gordon (jogo)
tags: added: quotas
Thierry Carrez (ttx)
Changed in nova:
milestone: kilo-3 → kilo-rc1
Revision history for this message
Alex Xu (xuhj) wrote :

as the discussion in the scheduler meeting http://eavesdrop.openstack.org/meetings/gantt/2015/gantt.2015-03-24-15.00.log.html, this can be defered to L.

Changed in nova:
milestone: kilo-rc1 → none
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Alex Xu (<email address hidden>) on branch: master
Review: https://review.openstack.org/154339

Yingxin (cyx1231st)
Changed in nova:
assignee: Alex Xu (xuhj) → Yingxin (cyx1231st)
Changed in nova:
assignee: Yingxin (cyx1231st) → nobody
status: In Progress → Confirmed
Yingxin (cyx1231st)
Changed in nova:
assignee: nobody → Yingxin (cyx1231st)
Changed in nova:
status: Confirmed → In Progress
Revision history for this message
Yingxin (cyx1231st) wrote :

Change to medium because this bug will block attemps to run multiple scheduler services.

description: updated
Changed in nova:
importance: Wishlist → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/290302

Revision history for this message
Yingxin (cyx1231st) wrote :

If quota expiration needs to be moved out of nova scheduler, I think the best place is nova conductor. However, there are multiple workers of conductor service, should we add periodic tasks to all of the workers? Or we should find another work-around to expire reserved quotas.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/292362

Revision history for this message
Sylvain Bauza (sylvain-bauza) wrote :

Having the DB API being called directly by multiple schedulers is not really a problem. That just means that the first scheduler will win for expiring the reservation.

Re-cateogorizing it as Wishilist again.

Changed in nova:
importance: Medium → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Alex Xu (<email address hidden>) on branch: master
Review: https://review.openstack.org/154339

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Yingxin Cheng (<email address hidden>) on branch: master
Review: https://review.openstack.org/292362
Reason: This work needs a more sophisticated solution, anyone can bring it back :)

Revision history for this message
melanie witt (melwitt) wrote :

The periodic task for expiring quota reservations was removed:

https://review.openstack.org/#/c/506822

As of Pike, the quota re-architecting work eliminated use of separate quota usages and reservations.

Changed in nova:
status: In Progress → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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