Refactoring of L3 Scheduler

Bug #1509046 reported by Armando Migliaccio
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
neutron
Expired
Wishlist
Unassigned

Bug Description

During Kilo we merged "DHCP Service LoadBalancing Scheduler" feature:

* https://review.openstack.org/#/c/111210/ (neutron-specs)
* https://review.openstack.org/#/c/137017/ (neutron)

The implementation provided as simplified framework for writing scheduler functions. It would be nice if the L3 scheduler embraced this same framework. It would neat and consistent.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

To be discussed at the drivers meeting

Changed in neutron:
status: New → Confirmed
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

A spec has been around for a while. It is relevant to this bug: https://review.openstack.org/#/c/184215/

Aman Kumar (amank)
Changed in neutron:
assignee: nobody → Aman Kumar (amank)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron-specs (master)

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/184215
Reason: This review was last updated before Mitaka started - September, 23, 2015 - and it still targets Liberty, which has released. Feel free to resubmit to Mikata, but please follow the latest feature submission guidelines as defined in: http://docs.openstack.org/developer/neutron/policies/blueprints.html

Changed in neutron:
importance: Undecided → Wishlist
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

If we have someone interested in the code clean up, I can't see why we shouldn't do it. Let's see if can agree. I can't approve my own proposal :)

Changed in neutron:
status: Confirmed → Triaged
Revision history for this message
Hong Hui Xiao (xiaohhui) wrote :

I just checked the code and found it is interesting to do the job, so I would like to do the clean up.

Changed in neutron:
assignee: Aman Kumar (amank) → Hong Hui Xiao (xiaohhui)
Aman Kumar (amank)
Changed in neutron:
assignee: Hong Hui Xiao (xiaohhui) → Aman Kumar (amank)
Akihiro Motoki (amotoki)
description: updated
Revision history for this message
Akihiro Motoki (amotoki) wrote :

At least what I can say is that it is not a good idea to have two different schemes for scheduling: DHCP and L3.
L3 scheduling needs to take into account L3-HA. If we can simplify the code base well, it is good to go with low priority.

Note that AZ feature was added in Mitaka. It may be needed to change the base scheduler if necessary.

Revision history for this message
Carl Baldwin (carl-baldwin) wrote :

I can review code for this when necessary. I think in the meeting today, we decided that this doesn't need to be tracked as an RFE. I'll let Armando confirm by setting the tags/status appropriately.

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Yes, no RFE, no spec. We look forward to the code.

tags: removed: rfe
Revision history for this message
Hong Hui Xiao (xiaohhui) wrote :

@amank, I am still available for this work item. If you don't have time for it, pls let me know.

Revision history for this message
Aman Kumar (amank) wrote :

@Hong, please Check your mail and reply over that.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
assignee: Aman Kumar (amank) → Hong Hui Xiao (xiaohhui)
status: Triaged → In Progress
Revision history for this message
Hong Hui Xiao (xiaohhui) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

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

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

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

Changed in neutron:
assignee: Hong Hui Xiao (xiaohhui) → Brian Haley (brian-haley)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/332727
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

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

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/320347
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

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

Change abandoned by Armando Migliaccio (<email address hidden>) on branch: master
Review: https://review.openstack.org/332644
Reason: This review is > 4 weeks without comment, and failed Jenkins the last time it was checked. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and leaving a 'recheck' comment to get fresh test results.

Changed in neutron:
status: In Progress → Incomplete
assignee: Brian Haley (brian-haley) → nobody
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

Changed in neutron:
status: Incomplete → Expired
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.