Project

General

Profile

Actions

Bug #16079

closed

Enable/Disable of features is allowed even the features are already enabled/disabled

Added by Tanay Ganguly almost 8 years ago. Updated over 7 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
jewel
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Description of problem:
If the feature is already enabled, still it allow user to enable it again.
Similarly goes with disable

Version-Release number of selected component (if applicable):
ceph version 10.2.1-6.el7cp

How reproducible:
Always.

Steps to Reproduce:
1.rbd info cephfs_data/img1

rbd image 'img1':
size 102400 MB in 25600 objects
order 22 (4096 kB objects)
block_name_prefix: rbd_data.1251ee238e1f29
format: 2
features: layering, deep-flatten
flags:

2. rbd feature disable cephfs_data/img1 exclusive-lock, object-map, fast-diff
3. rbd feature disable cephfs_data/img1 exclusive-lock, object-map, fast-diff
4. rbd feature disable cephfs_data/img1 exclusive-lock, object-map, fast-diff

Actual results:
User should not be allowed to disable the feature's which are not at all enabled.

Expected results:
There should be error saying that features are not enabled.

Additional info:
Same thing applies for enable.

We can enable multiple times the same features, even though its already enabled.


Related issues 1 (0 open1 closed)

Copied to rbd - Backport #17482: jewel: Enable/Disable of features is allowed even the features are already enabled/disabledResolvedLoïc DacharyActions
Actions #1

Updated by lu shi almost 8 years ago

I have alreadr midified by the pull request below
https://github.com/ceph/ceph/pull/9451

Actions #2

Updated by Jason Dillaman almost 8 years ago

  • Status changed from New to Fix Under Review
  • Backport set to jewel
Actions #3

Updated by Mykola Golub over 7 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #4

Updated by Loïc Dachary over 7 years ago

  • Copied to Backport #17482: jewel: Enable/Disable of features is allowed even the features are already enabled/disabled added
Actions #5

Updated by Loïc Dachary over 7 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF