Cinder

cinder is missing an equivalent to the large_ops test

Bug #1250696 reported by Joe Gordon
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Cinder
Invalid
Wishlist
Unassigned
OpenStack Core Infrastructure
Triaged
Wishlist
Unassigned
tempest
Invalid
Low
Unassigned

Bug Description

Nova uses its fake virt driver to make sure there is nothing wrong with the internal nova logic when running large operations, cinder should have a similar driver and test inside of large_ops

see tempest/scenario/test_large_ops.py

Revision history for this message
Sean Dague (sdague) wrote :

Setting to cinder Confirmed / Medium because a solid fake backend will be needed for this

Changed in tempest:
importance: Undecided → Low
status: New → Confirmed
tags: added: low-hanging-fruit test-needed
Changed in cinder:
importance: Undecided → Medium
status: New → Confirmed
John Griffith (john-griffith)
Changed in cinder:
importance: Medium → Wishlist
Revision history for this message
Joe Gordon (jogo) wrote :

Just out of curiosity whats the rational for dropping this to wishlist?

Revision history for this message
John Griffith (john-griffith) wrote :

I feel this fits more with a wishlist by definition, it's not a defect or an error condition currently, but a need to further enhance functionality/features of the fake driver which seems like a perfect fit for wishlist IMO.

I realize there are concerns with wishlist category items ever being picked up but I'll set aside some cycles to make this happen in I2 or early I3.

Revision history for this message
Jeremy Stanley (fungi) wrote :

Setting this to the same urgency in infra as for tempest, since we obviously can't add a job config until there's something for it to run.

Changed in openstack-ci:
status: New → Triaged
importance: Undecided → Low
Revision history for this message
chandan kumar (chkumar246) wrote :

Hello,

I want to work on this.
How can i get started?

Rohan (kanaderohan)
Changed in tempest:
assignee: nobody → Rohan (kanaderohan)
Revision history for this message
Joe Gordon (jogo) wrote :

Cinder would need a fake backend, similar to nova's fakevirt backend. This fake backend would just update the DB with the state, but not actually create any volumes. This can be used to test the Cinder internals.

Mike Perez (thingee)
Changed in cinder:
status: Confirmed → Triaged
vishal yadav (vishalcdac07)
Changed in openstack-ci:
assignee: nobody → vishal yadav (vishalcdac07)
assignee: vishal yadav (vishalcdac07) → nobody
Changed in cinder:
assignee: nobody → vishal yadav (vishalcdac07)
Clark Boylan (cboylan)
Changed in openstack-ci:
importance: Low → Wishlist
Rushi Agrawal (rushiagr)
Changed in cinder:
assignee: vishal yadav (vishalcdac07) → nobody
tags: added: volumes
chandan kumar (chkumar246)
Changed in tempest:
assignee: Rohan (kanaderohan) → chandan kumar (chkumar246)
Hardik Parekh (hardik-parekh047)
Changed in cinder:
assignee: nobody → hardik (hardik-parekh047)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/224413

Changed in cinder:
status: Triaged → In Progress
Revision history for this message
Hardik Parekh (hardik-parekh047) wrote :

inder.volume.driver.FakeISCSIDriver can be use for large_ops test.

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

Change abandoned by Sean McGinnis (<email address hidden>) on branch: master
Review: https://review.openstack.org/224413
Reason: This review is > 4 weeks without comment and currently blocked by a core reviewer with a -2. We are abandoning this for now. Feel free to reactivate the review by pressing the restore button and contacting the reviewer with the -2 on this review to ensure you address their concerns.

Duncan Thomas (duncan-thomas)
Changed in cinder:
assignee: hardik (hardik-parekh047) → nobody
Robin Naundorf (senk)
Changed in cinder:
status: In Progress → Confirmed
chandan kumar (chkumar246)
Changed in tempest:
assignee: chandan kumar (chkumar246) → nobody
Revision history for this message
Luz Cazares (luz-cazares) wrote :

Scenario "tempest/scenario/test_large_ops.py" was removed from tempest since it was related to performance instead of scenario. See: https://github.com/openstack/tempest/commit/1976da83a5bdb35a61d1659bb5ece1b5d248bacd
Open a new bug if functional test coverage is still needed.

Changed in tempest:
status: Confirmed → Invalid
Sean McGinnis (sean-mcginnis)
Changed in cinder:
status: Confirmed → Invalid
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.