search

junneyang / zumastor

Automatically exported from code.google.com/p/zumastor
0 stars 1 forks source link

Parameterize cbtb tests, make them cleanup after themselves. #103

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Attaching sample cleanup for cbtb/tests/1/zero-zumastor.sh.

Basic idea:

NUMDEVS=2 # Let someone know many devices we use in the test
DEV1SIZE=4 # Size of first device in M
DEV2SIZE=8 # Size of second device in M
#DEV1NAME=/dev/null # This is what will be passed in as an env var
#DEV2NAME=/dev/null # This is what will be passed in as an env var

The cleanup at the end should help us test if we can leave a system in 
the same state as when we started after running a bunch of 
zumastor/ddsnap/etc commands. This is mainly so that we can run a bunch 
of tests on the same physical machine without having to delete volumes 
and cleanup by hand.

Original issue reported on code.google.com by williama...@gmail.com on 8 Apr 2008 at 1:25

Attachments:

GoogleCodeExporter commented 9 years ago 
adding cleanup patch for tests/1/ddsnap_msg.sh. Still needs to be parameterized.

Original comment by williama...@gmail.com on 8 Apr 2008 at 5:25

Attachments:

GoogleCodeExporter commented 9 years ago 
Cleanup patch for tests/1/snapshot-ddsnap.sh

Original comment by williama...@gmail.com on 8 Apr 2008 at 5:31

Attachments:

GoogleCodeExporter commented 9 years ago 
Cleanup patch for tests/1/snapshot-zumastor-ext2.sh
Cleanup patch for tests/1/snapshot-zumastor-ext3.sh
Cleanup patch for tests/1/snapshot-zumastor-jfs.sh
Cleanup patch for tests/1/snapshot-zumastor-reiserfs.sh

Original comment by williama...@gmail.com on 8 Apr 2008 at 5:40

Attachments:

GoogleCodeExporter commented 9 years ago 
Adding patch for tests/1/snapshot-zumastor-lvm-ext3-resize.sh
This one is questionable. (Factoring in the $DEV1NAME env variable here)

Original comment by williama...@gmail.com on 8 Apr 2008 at 6:00

Attachments:

GoogleCodeExporter commented 9 years ago 
Adding patch for tests/1/snapshot-zumastor-revert.sh

Original comment by williama...@gmail.com on 8 Apr 2008 at 6:42

Attachments:

GoogleCodeExporter commented 9 years ago 
Adding patch for tests/1/zumastor-symlink-stores.sh
Adding patch for tests/1/boot-order.sh (this test actually breaks on OS's that 
take 
advantage of upstart or the like)

Also, I'm thinking that cleanup in the test itself may not be that great. What 
if 
the test fails halfway through. The cleanup will never run?

Original comment by williama...@gmail.com on 8 Apr 2008 at 7:20

Attachments:

GoogleCodeExporter commented 9 years ago 
Drake, can you review these patches?

Original comment by williama...@gmail.com on 15 Apr 2008 at 8:11

GoogleCodeExporter commented 9 years ago 
Sounds good to me.

Original comment by drake.di...@gmail.com on 15 Apr 2008 at 9:35

GoogleCodeExporter commented 9 years ago 
Applied all the cleanup patches and patches in issue 89. Still pending the 
parameterize patch, since that will require modifying the cbtb harness. 
(Revision 
1539)

Original comment by williama...@gmail.com on 15 Apr 2008 at 10:12

GoogleCodeExporter commented 9 years ago

Original comment by williama...@gmail.com on 16 Apr 2008 at 6:35

GoogleCodeExporter commented 9 years ago 
Committed cbtb/tests/1/zero-zumastor.sh in r1574
Now we work on the CBTB harness, once that passes we can re factor the rest of 
the
tests accordingly.

Original comment by compbr...@gmail.com on 22 Apr 2008 at 5:42