Subprocess call to curl must DIE!

Bug #911972 reported by Gabriel Hurley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Undecided
Gabriel Hurley

Bug Description

In the syspanel.services panel we're using a subprocess call out to curl to ping the endpoints to try and determine the up/down status of each endpoint. This is completely inappropriate and also unreliable. I recommend eliminating that call entirely.

Changed in horizon:
assignee: nobody → Gabriel Hurley (gabriel-hurley)
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/2814
Committed: http://github.com/openstack/horizon/commit/1fcb2b7ee0ad8fddafc03037e37fcd41b2a3c07a
Submitter: Jenkins
Branch: master

commit 1fcb2b7ee0ad8fddafc03037e37fcd41b2a3c07a
Author: Gabriel Hurley <email address hidden>
Date: Wed Jan 4 14:13:54 2012 -0800

    Converts syspanel.services to new tables.

    Also removes the calls to curl, fixing bug 911972.

    Change-Id: I3133b3ac4ca4617327f476f4261d1dc9e99aa13a

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → essex-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: essex-3 → 2012.1
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.