jewel: common: Enable/Disable of features is allowed even the features are already enabled/disabled #11460

Merged
merged 1 commit into from Oct 20, 2016

Projects

None yet

3 participants

@s09816 @ldachary s09816 librbd: enable/disable of features is not allowed when already enable…
…d/disabled

Fixes: http://tracker.ceph.com/issues/16079

Signed-off-by: Lu Shi <shi.lu@h3c.com>
(cherry picked from commit a8a6333)
391936a
@dachary dachary self-assigned this Oct 13, 2016
@dachary dachary added this to the jewel milestone Oct 13, 2016
@dachary
Member
dachary commented Oct 13, 2016

jenkins test this please (unittest_throttle failed and does not seem to be related to this pr)

@dachary
Member
dachary commented Oct 13, 2016

jenkins test this please (src/test/cli/osdmaptool/test-map-pgs.t: failed)

@dachary
Member
dachary commented Oct 20, 2016

@dillaman does this backport look good to merge ? It passed the rbd suite http://tracker.ceph.com/issues/17487#note-17 (except for two dead nbd jobs)

@dillaman dillaman was assigned by dachary Oct 20, 2016
@dillaman
Contributor

👍

@dachary dachary merged commit ac2ce6d into ceph:jewel Oct 20, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
@theanalyst theanalyst changed the title from jewel: Enable/Disable of features is allowed even the features are already enabled/disabled to "jewel: common: Enable/Disable of features is allowed even the features are already enabled/disabled" Nov 17, 2016
@theanalyst theanalyst changed the title from "jewel: common: Enable/Disable of features is allowed even the features are already enabled/disabled" to jewel: common: Enable/Disable of features is allowed even the features are already enabled/disabled Nov 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment