Bug 2042384 - [rbd-mirror] : [negative scenario] peer remove returns 0 even for non existing peer uuid
Summary: [rbd-mirror] : [negative scenario] peer remove returns 0 even for non existin...
Keywords:
Status: NEW
Alias: None
Product: Red Hat Ceph Storage
Classification: Red Hat Storage
Component: RBD-Mirror
Version: 4.3
Hardware: Unspecified
OS: Unspecified
unspecified
low
Target Milestone: ---
: 8.0
Assignee: Ilya Dryomov
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-01-19 11:57 UTC by Vasishta
Modified: 2023-07-31 21:50 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Ceph Project Bug Tracker 58733 0 None None None 2023-02-22 14:23:03 UTC
Red Hat Issue Tracker RHCEPH-2998 0 None None None 2022-01-19 12:07:46 UTC

Description Vasishta 2022-01-19 11:57:07 UTC
Description of problem: [negative scenario]
rbd mirror pool peer remove returns 0 even for non existing peer's uuid

Version-Release number of selected component (if applicable):
14.2.22-49.el7cp

How reproducible:
Tried twice, faced twice

Steps to Reproduce: [negative scenario]
1. Configure rbd mirroring
2. try to remove non-existing peer's uuid and observe that command returns 0

Actual results:
Command returns zero even for non-existing peer is tried to be removed

# sudo rbd mirror pool peer remove test a-b-c-d
rbd: invalid uuid 'a-b-c-d'
# sudo rbd mirror pool peer remove test aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa
# echo $?
0
[root@DELL-R640-091 ~]# sudo rbd mirror pool info test
Mode: image
Site Name: site_a

Peer Sites: none

Expected results:
Command should error out saying there are no peers added with <uuid>


Note You need to log in before you can comment on or make changes to this bug.