Code review comment for lp://qastaging/~xfactor973/charms/trusty/ceph/erasure_actions

Revision history for this message
James Page (james-page) wrote :

Hi Chris

Working through this update now; however unit test failure:

======================================================================
FAIL: test_process_requests_invalid_api_version (unit_tests.test_ceph_broker.CephBrokerTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jamespage/lib/os-charms/local/lib/python2.7/site-packages/mock/mock.py", line 1305, in patched
    return func(*args, **keywargs)
  File "/home/jamespage/src/charms/landing/ceph/unit_tests/test_ceph_broker.py", line 34, in test_process_requests_invalid_api_version
    'stderr': 'Missing or invalid api version (2)'})
AssertionError: {u'exit-code': 0} != {'exit-code': 1, 'stderr': 'Missing or invalid api version (2)'}
- {u'exit-code': 0}
+ {'exit-code': 1, 'stderr': 'Missing or invalid api version (2)'}

« Back to merge proposal