diff --git a/hooks/ceph_broker.py b/hooks/ceph_broker.py index b12491a..d01d38e 100644 --- a/hooks/ceph_broker.py +++ b/hooks/ceph_broker.py @@ -138,7 +138,7 @@ def handle_erasure_pool(request, service): log(msg, level=ERROR) return {'exit-code': 1, 'stderr': msg} - # TODO: Default to 6/3 erasure coding. I believe this requires min 9 osds + # TODO: Default to 3/2 erasure coding. I believe this requires min 5 osds if not erasure_profile_exists(service=service, name=erasure_profile): # TODO: Fail and tell them to create the profile or default msg = "erasure-profile {} does not exist. Please create it with: " \ diff --git a/unit_tests/test_ceph_broker.py b/unit_tests/test_ceph_broker.py index f8380ed..b720d94 100644 --- a/unit_tests/test_ceph_broker.py +++ b/unit_tests/test_ceph_broker.py @@ -22,9 +22,7 @@ class CephBrokerTestCase(unittest.TestCase): rc = ceph_broker.process_requests(req) self.assertEqual(json.loads(rc), { 'exit-code': 1, - 'stderr': - ('Missing or invalid api version ' - '(None)')}) + 'stderr': 'Missing or invalid api version (None)'}) @mock.patch('ceph_broker.log') def test_process_requests_invalid_api_version(self, mock_log):