diff options
author | kshithijiyer <kshithij.ki@gmail.com> | 2020-03-24 10:48:11 +0530 |
---|---|---|
committer | kshithijiyer <kshithij.ki@gmail.com> | 2020-03-24 11:02:26 +0530 |
commit | 7b308bf46b2285a283497cdf0d9830903f1b7d0c (patch) | |
tree | 98d1bc5fb0c3ac090e5831a7b24fa2948bf4e253 /tests | |
parent | 2e567185989009ef0543b38282659c39ca8940f6 (diff) |
[Testfix] Fix error in logging
Problem:
Testcase test_ec_version was failing with the
below traceback:
Traceback (most recent call last):
File "/usr/lib64/python2.7/logging/__init__.py", line 851, in emit
msg = self.format(record)
File "/usr/lib64/python2.7/logging/__init__.py", line 724, in format
return fmt.format(record)
File "/usr/lib64/python2.7/logging/__init__.py", line 464, in format
record.message = record.getMessage()
File "/usr/lib64/python2.7/logging/__init__.py", line 328, in getMessage
msg = msg % self.args
TypeError: %d format: a number is required, not str
Logged from file test_ec_version_healing_whenonebrickdown.py, line 233
This was due to a missing 's' in the log message on line 233.
Solution:
Add the missing s in the log message on line 233 as
shown below:
g.log.info('Brick %s is offline successfully', brick_b2_down)
Also renaming the file for more clarity of what the
testcase does.
Change-Id: I626fbe23dfaab0dd6d77c75329664a81a120c638
Signed-off-by: kshithijiyer <kshithij.ki@gmail.com>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/functional/disperse/test_ec_version_healing_when_one_brick_down.py (renamed from tests/functional/disperse/test_ec_version_healing_whenonebrickdown.py) | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/functional/disperse/test_ec_version_healing_whenonebrickdown.py b/tests/functional/disperse/test_ec_version_healing_when_one_brick_down.py index a6e0b3203..4e0321d87 100644 --- a/tests/functional/disperse/test_ec_version_healing_whenonebrickdown.py +++ b/tests/functional/disperse/test_ec_version_healing_when_one_brick_down.py @@ -230,7 +230,7 @@ class TestEcVersionBrickdown(GlusterBaseClass): ret = bring_bricks_offline(self.volname, brick_b2_down) self.assertTrue(ret, 'Brick %s is not offline' % brick_b2_down) - g.log.info('Brick % is offline successfully', brick_b2_down) + g.log.info('Brick %s is offline successfully', brick_b2_down) # Bring brick b1 online ret = bring_bricks_online(self.mnode, self.volname, |