Revert to snapshot tests may incorrectly pass when the feature is not supported
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Triaged
|
Undecided
|
Unassigned | ||
cinder-tempest-plugin |
Triaged
|
High
|
Unassigned |
Bug Description
While testing the revert-to-snapshot support for Ceph https:/
More in details, this DNM patch was executed *before* merging the revert-to-snapshot support in Ceph: https:/
Quoting Brian Rosmaita from that review:
-------
Patch Set 1:
OK, I think I know what's going on here.
cinder_
calls revert_to_snapshot, gets a 202
waits for volume to be available, which it is since nothing is happening
waits for snapshot to be available, which it is since nothing is happening
asserts that the volume size has not changed, which it hasn't, since nothing is happening
should pass, whether the driver supports revert-to-snapshot or not
test_volume_
checks for a BadRequest being raised
gets caught at the api layer, driver is never involved
should pass whether the driver supports revert-to-snapshot or not
Changed in cinder: | |
status: | New → Confirmed |
Changed in cinder-tempest-plugin: | |
status: | New → Triaged |
Changed in cinder: | |
status: | Confirmed → Triaged |
Changed in cinder-tempest-plugin: | |
importance: | Undecided → High |