RepoMirrors/ceph
1
0
Fork 0
mirror of https://github.com/ceph/ceph synced 2025-01-31 23:44:10 +00:00
Code Issues Packages Projects Releases Wiki Activity

qa/tasks/mgr: fix test_pool.py

Browse Source 
* per command desc of "osd pool set", "val" parameter should be a "string"
  not a number. so update the test_pool.py accordingly.
* also as a follow-up of 65925511bc, we need to update the test
  accordingly to reflect the change in the error's str representation.

Fixes: http://tracker.ceph.com/issues/24077
Signed-off-by: Kefu Chai <kchai@redhat.com>
This commit is contained in:
Kefu Chai 2018-05-11 22:12:43 +08:00
parent ce7afd435e
commit 01c38041bc
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
qa/tasks/mgr/dashboard/test_pool.py
View File

@ -97,9 +97,10 @@ class PoolTest(DashboardTestCase):
data['application_metadata'].split(','))
elif k == 'pool':
self.assertEqual(pool['pool_name'], v)
elif k in ['compression_mode', 'compression_algorithm',
'compression_max_blob_size']:
elif k in ['compression_mode', 'compression_algorithm']:
self.assertEqual(pool['options'][k], data[k])
elif k == 'compression_max_blob_size':
self.assertEqual(pool['options'][k], int(data[k]))
elif k == 'compression_required_ratio':
self.assertEqual(pool['options'][k], float(data[k]))
else:
@ -136,7 +137,7 @@ class PoolTest(DashboardTestCase):
'pool_type': 'replicated',
'compression_algorithm': 'zstd',
'compression_mode': 'aggressive',
'compression_max_blob_size': 10000000,
'compression_max_blob_size': "10000000",
'compression_required_ratio': '0.8',
}]
for data in pools:
@ -150,7 +151,7 @@ class PoolTest(DashboardTestCase):
self.assertJsonBody({
'component': 'pool',
'code': "2",
'detail': "specified rule dnf doesn't exist"
'detail': "[errno -2] specified rule dnf doesn't exist"
})
@authenticate