Sheepdog Driver: failed to check sheepdog cluster's status

Bug #1195098 reported by Kai Zhang on 2013-06-27
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Undecided
Kai Zhang

Bug Description

Current implementation of check_for_setup_error() in sheepdog driver checks sheepdog cluster's status by finding "running" from
the splits of outputs from "collie cluster info" command.

However, as sheepdog has been upgraded, this method is not valid.

We should update it so that sheepdog driver can get the correct status of sheepdog cluster.

Kai Zhang (kyle-r) on 2013-06-27
Changed in cinder:
assignee: nobody → Kai Zhang (kyle-r)
description: updated
Kai Zhang (kyle-r) on 2013-06-27
Changed in cinder:
status: New → In Progress

Reviewed: https://review.openstack.org/34689
Committed: http://github.com/openstack/cinder/commit/039dce71d43bd69181b2502cda17f941c95f9497
Submitter: Jenkins
Branch: master

commit 039dce71d43bd69181b2502cda17f941c95f9497
Author: Kai Zhang <email address hidden>
Date: Thu Jun 27 00:31:36 2013 -0700

    Fix check_for_setup_error for sheepdog driver

    Current implementation of check_for_setup_error() of sheepdog driver
    cannot work correctly with the latest version of sheepdog cluster.
    This patch fixes it and adds tests for both old and new version.

    Fixes: bug #1195098
    Change-Id: Ia710789e4bedd7c83cd47eff6b2fc532af39d94a

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-07-17
Changed in cinder:
milestone: none → havana-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-2 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers