handle-remaining-connect-fails-cases-in-solidfire-driver

Bug #1201592 reported by John Griffith on 2013-07-15
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Critical
John Griffith
Grizzly
Critical
John Griffith

Bug Description

When fixing bug/1195910, some other failure cases weren't considered (ie not authorized, unreachable dest etc). We should fix this up to handle the various connection error states accordingly and also add some hints to the log output to help determine where to look in case of failure.

Reviewed: https://review.openstack.org/37136
Committed: http://github.com/openstack/cinder/commit/5edbc4eb9a6b0d50612fffb6261850c80515b483
Submitter: Jenkins
Branch: master

commit 5edbc4eb9a6b0d50612fffb6261850c80515b483
Author: John Griffith <email address hidden>
Date: Mon Jul 15 15:16:25 2013 -0600

    Catch additional connect fail cases.

    When fixing bug/1195910, some other failure
    cases weren't considered (ie not authorized,
    unreachable dest etc).

    We should fix this up to handle the various
    connection error states accordingly and also
    add some hints to the log output to help
    determine where to look in case of failure.

    Fixes bug: 1201592

    Change-Id: Ib099c7af705e7c49be4d0f723c8a20262c942e7f

Changed in cinder:
status: In Progress → Fix Committed
John Griffith (john-griffith) wrote :

There are two issues associated with this bug:
1. Not handling the exceptions resulting in a trace and crashing the cinder-volume service (unacceptable obvisouly)
2. The messaging doesn't provide enough info to determine what's wrong regarding the connection

Item 1 should be viewed as a critical issue, item 2 is a high priority.

Changed in cinder:
importance: Undecided → Critical
Thierry Carrez (ttx) on 2013-07-17
Changed in cinder:
status: Fix Committed → Fix Released

Reviewed: https://review.openstack.org/37321
Committed: http://github.com/openstack/cinder/commit/559b0fa33a8d4e248f50a9d1076e4ec3928f91a2
Submitter: Jenkins
Branch: stable/grizzly

commit 559b0fa33a8d4e248f50a9d1076e4ec3928f91a2
Author: John Griffith <email address hidden>
Date: Mon Jul 15 15:16:25 2013 -0600

    Catch additional connect fail cases.

    When fixing bug/1195910, some other failure
    cases weren't considered (ie not authorized,
    unreachable dest etc).

    We should fix this up to handle the various
    connection error states accordingly and also
    add some hints to the log output to help
    determine where to look in case of failure.

    Fixes bug: 1201592

    Change-Id: Ib099c7af705e7c49be4d0f723c8a20262c942e7f
    (cherry picked from commit 5edbc4eb9a6b0d50612fffb6261850c80515b483)

Alan Pevec (apevec) on 2013-08-06
tags: removed: grizzly-backport-potential
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