Cinder

[RBD] --pool is deprecated

Bug #1934823 reported by Sofia Enriquez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Invalid
Low
Unassigned

Bug Description

The RBD driver is using an old fashion parameter and Cinder gets the next warning message "rbd: -p [ --pool ] is deprecated, use --dest-pool".

We should update the rbd code[1].

[1] https://opendev.org/openstack/cinder/src/branch/master/cinder/volume/drivers/rbd.py#L964

See original description
Sofia Enriquez (lsofia-enriquez)
description: updated
Sofia Enriquez (lsofia-enriquez)
Changed in cinder:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/cinder/+/812216

Changed in cinder:
status: New → In Progress
Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Ceph bug for this https://tracker.ceph.com/issues/53125

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

Reviewed: https://review.opendev.org/c/openstack/cinder/+/812216
Committed: https://opendev.org/openstack/cinder/commit/b7e294599047188a04f91dda53537b262d2b3f8d
Submitter: "Zuul (22348)"
Branch: master

commit b7e294599047188a04f91dda53537b262d2b3f8d
Author: Ana Clara Cavalcante <email address hidden>
Date: Fri Oct 1 20:14:03 2021 +0000

    Update rbd warning message

    Replace --pool with --dest-pool

    Closes-bug: 1934823
    Change-Id: Ie925e1886c508d840cd68caa7ebd031f3a89822d

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/xena)

Fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/cinder/+/816342

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

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/cinder/+/816343

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/victoria)

Change abandoned by "zhen <email address hidden>" on branch: stable/victoria
Review: https://review.opendev.org/c/openstack/cinder/+/816343

Sofia Enriquez (lsofia-enriquez)
Changed in cinder:
status: Fix Released → Invalid
Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

From the pacific documentation[1] it looks like --pool is still valid. Ceph bug
https://tracker.ceph.com/issues/53125#change-208888

[1] https://docs.ceph.com/en/pacific/man/8/rbd/

Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Revert https://review.opendev.org/c/openstack/cinder/+/826623

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/xena)

Change abandoned by "Elod Illes <email address hidden>" on branch: stable/xena
Review: https://review.opendev.org/c/openstack/cinder/+/816342
Reason: stable/xena branch of openstack/cinder is about to be deleted. To be able to do that, all open patches need to be abandoned. Please cherry pick the patch to unmaintained/xena if you want to further work on this patch.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.