Subprocess call to curl must DIE!

Bug #911972 reported by Gabriel Hurley on 2012-01-04
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
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

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

Changed in horizon:
status: Confirmed → In Progress

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) on 2012-01-25
Changed in horizon:
milestone: none → essex-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-04-05
Changed in horizon:
milestone: essex-3 → 2012.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers