zaqar

Health check doesn't check storage backend health

Bug #1243268 reported by Kurt Griffiths on 2013-10-22
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
​ zaqar
Fix Released
Medium
Unassigned
zaqar 2014.1 "icehouse"
Also affects project (?) Also affects distribution/package Nominate for series

Bug Description

The health endpoint should attempt a simple operation, such as posting a message to a hidden "health check" queue, before returning OK.

See original description
Kurt Griffiths (kgriffs) wrote :

Don't forget to take into account sharding!

description: updated
Kurt Griffiths (kgriffs) on 2013-10-22
Changed in marconi:
status: New → Incomplete
status: Incomplete → Triaged
importance: Undecided → Medium
OpenStack Infra (hudson-openstack) wrote : Fix merged to marconi (master)

Reviewed: https://review.openstack.org/53457
Committed: http://github.com/openstack/marconi/commit/eb24c863976b6813ff1c0b2c53af5c192f914c29
Submitter: Jenkins
Branch: master

commit eb24c863976b6813ff1c0b2c53af5c192f914c29
Author: Zhihao Yuan <email address hidden>
Date: Wed Oct 23 17:24:46 2013 -0400

    feat(health): make health API storage-aware

    This change makes the GET request of /health to ensure that all
    connected storage shards accepts `ping` command before responding
    200; otherwise, 503.

    The HEAD request is unchanged.

    Closes-Bug: 1243268
    Change-Id: I75c3b46ccb1a5a66e6c12766d3e451fd3478822c

Changed in marconi:
status: Triaged → Fix Committed
Thierry Carrez (ttx) on 2013-12-19
Changed in marconi:
milestone: none → icehouse-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-04-17
Changed in marconi:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.
Subscribing...

Other bug subscribers