Message ID | 20190920152804.12875-20-mreitz@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | block: Fix check_to_replace_node() | expand |
20.09.2019 18:28, Max Reitz wrote: > 041's TestRepairQuorum has its own image_len, no need to refer to > TestSingleDrive. (This patch allows uncommenting TestSingleDrive to you mean commenting > speed up 041 during test testing.) we definitely want a way to run a subset of test cases. I usually do s/def test/def ntest/, and then set needed test-case back to 'def test' > > Signed-off-by: Max Reitz <mreitz@redhat.com> > --- > tests/qemu-iotests/041 | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tests/qemu-iotests/041 b/tests/qemu-iotests/041 > index ca126de3ff..20ae9750b7 100755 > --- a/tests/qemu-iotests/041 > +++ b/tests/qemu-iotests/041 > @@ -880,7 +880,7 @@ class TestRepairQuorum(iotests.QMPTestCase): > # Add each individual quorum images > for i in self.IMAGES: > qemu_img('create', '-f', iotests.imgfmt, i, > - str(TestSingleDrive.image_len)) > + str(self.image_len)) yes, seems TestSingleDrive.image_len is a copy-pasting mistake here.. > # Assign a node name to each quorum image in order to manipulate > # them > opts = "node-name=img%i" % self.IMAGES.index(i) > Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
On 26.09.19 16:13, Vladimir Sementsov-Ogievskiy wrote: > 20.09.2019 18:28, Max Reitz wrote: >> 041's TestRepairQuorum has its own image_len, no need to refer to >> TestSingleDrive. (This patch allows uncommenting TestSingleDrive to > > you mean commenting Ah, yes, I mean commenting out. :-) >> speed up 041 during test testing.) > > we definitely want a way to run a subset of test cases. > > I usually do s/def test/def ntest/, and then set needed test-case back to > 'def test' Thanks for that tip. Max >> >> Signed-off-by: Max Reitz <mreitz@redhat.com> >> --- >> tests/qemu-iotests/041 | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/tests/qemu-iotests/041 b/tests/qemu-iotests/041 >> index ca126de3ff..20ae9750b7 100755 >> --- a/tests/qemu-iotests/041 >> +++ b/tests/qemu-iotests/041 >> @@ -880,7 +880,7 @@ class TestRepairQuorum(iotests.QMPTestCase): >> # Add each individual quorum images >> for i in self.IMAGES: >> qemu_img('create', '-f', iotests.imgfmt, i, >> - str(TestSingleDrive.image_len)) >> + str(self.image_len)) > > yes, seems TestSingleDrive.image_len is a copy-pasting mistake here.. > >> # Assign a node name to each quorum image in order to manipulate >> # them >> opts = "node-name=img%i" % self.IMAGES.index(i) >> > > > Reviewed-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com> >
diff --git a/tests/qemu-iotests/041 b/tests/qemu-iotests/041 index ca126de3ff..20ae9750b7 100755 --- a/tests/qemu-iotests/041 +++ b/tests/qemu-iotests/041 @@ -880,7 +880,7 @@ class TestRepairQuorum(iotests.QMPTestCase): # Add each individual quorum images for i in self.IMAGES: qemu_img('create', '-f', iotests.imgfmt, i, - str(TestSingleDrive.image_len)) + str(self.image_len)) # Assign a node name to each quorum image in order to manipulate # them opts = "node-name=img%i" % self.IMAGES.index(i)
041's TestRepairQuorum has its own image_len, no need to refer to TestSingleDrive. (This patch allows uncommenting TestSingleDrive to speed up 041 during test testing.) Signed-off-by: Max Reitz <mreitz@redhat.com> --- tests/qemu-iotests/041 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)