[Pure Storage] Replication failover issue

Bug #2028380 reported by Simon Dodsley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Committed
Medium
Simon Dodsley
Yoga
New
Medium
Unassigned

Bug Description

A patch[1] removed a specific REST version call that is required to get the correct list of snapshots to recover when doing an async replication failover.
This means that a replication failover will not correctly copy replicated volumes to the failover device.
The specific function affected is `_async_failover_host()`

Fix needs to be backport all the way to stable/yoga

[1]https://review.opendev.org/c/openstack/cinder/+/817085

Changed in cinder:
assignee: nobody → Simon Dodsley (simon-dodsley)
Changed in cinder:
status: New → In Progress
description: updated
description: updated
Changed in cinder:
importance: Undecided → Medium
tags: added: drivers pure replication storage
Changed in cinder:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 23.0.0.0rc1

This issue was fixed in the openstack/cinder 23.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/889359
Committed: https://opendev.org/openstack/cinder/commit/583eae173bfee3983477661e4d62eb4233f54eff
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit 583eae173bfee3983477661e4d62eb4233f54eff
Author: Simon Dodsley <email address hidden>
Date: Thu Jun 8 12:04:56 2023 -0400

    [Pure Storage] Fix failure in replication failover

    A previous patch incorrectly removed a call using a specific
    version of API as part of the replication failover process.

    The call being made to the backend fails as one of the
    parameters is not supported in later API version.

    This patch fixes the call to the backend to use the required
    API version.

    There are no tempest tests for replication failover, hence
    why the original patch passed tests.

    Closes-Bug: #2028380
    Change-Id: Id4f5f16a04220cc880e17ffd69a8303c93a6e8ba
    (cherry picked from commit 084d84152a4c2fcf4c94d46d4a1a97add409bd42)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 22.1.2

This issue was fixed in the openstack/cinder 22.1.2 release.

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.