fuelmenu dhcp check should run once per interface

Bug #1365395 reported by Matthew Mosesohn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Triaged
Low
Fuel Sustaining

Bug Description

We should run dhcp check early on in the background and set dhcp probe results globally in fuelmenu. There is an event scheduler already used by urwid, documented here: http://urwid.org/manual/mainloop.html#event-loops
Getting this DHCP information early and not obtrusively will allow fuelmenu to operate much faster and greatly improve user experience.

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

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

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Matthew Mosesohn (raytrac3r)
status: New → In Progress
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/119014
Committed: https://git.openstack.org/cgit/stackforge/fuel-web/commit/?id=fc24c30cac34b9edad62f4f368d8b98f2ccb4530
Submitter: Jenkins
Branch: master

commit fc24c30cac34b9edad62f4f368d8b98f2ccb4530
Author: Matthew Mosesohn <email address hidden>
Date: Thu Sep 4 13:27:52 2014 +0400

    Add check for dhcp range overlap in fuelmenu

    Change-Id: Ie666ee7e83b99f0ba544e8e90f82000b0aa12a96
    Closes-Bug: #1365395

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

This is not a duplicate and I marked the commit fc24c30cac34b9edad62f4f368d8b98f2ccb4530 as related to this bug in error. Please disregard and this is actually still a confirmed bug unresolved.

Changed in fuel:
status: Fix Committed → Confirmed
Changed in fuel:
milestone: 6.0 → 6.1
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Nikolay said he would help work on a threading function to evaluate dhcpcheck on all interfaces simultaneously in the background.

Changed in fuel:
assignee: Matthew Mosesohn (raytrac3r) → Nikolay Markov (nmarkov)
Nikolay Markov (nmarkov)
Changed in fuel:
milestone: 6.1 → next
Dmitry Pyzhov (dpyzhov)
tags: added: module-fuelmenu
removed: fuelmenu
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: next → 7.0
Revision history for this message
Nikolay Markov (nmarkov) wrote :

I'm not sure we really need this as well as fuelmenu itself. Moving this back to team for now.

Changed in fuel:
assignee: Nikolay Markov (nmarkov) → Fuel Python Team (fuel-python)
Revision history for this message
Vladimir Sharshov (vsharshov) wrote :

Moving to 8.0 because it is feature.

tags: added: feature need-bp
Changed in fuel:
status: Confirmed → Won't Fix
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
no longer affects: fuel/8.0.x
Dmitry Pyzhov (dpyzhov)
tags: added: area-python
Revision history for this message
Alexander Kislitsky (akislitsky) wrote :

We passed SCF in 8.0. Moving the bug to 9.0.

Changed in fuel:
milestone: 8.0 → 9.0
Fedor Zhadaev (fzhadaev)
Changed in fuel:
milestone: 9.0 → 10.0
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
assignee: Fuel Python (Deprecated) (fuel-python) → Fuel Sustaining (fuel-sustaining-team)
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.