Code review comment for lp:~tribaal/landscape-client/ceph-usage-report

Revision history for this message
Jerry Seutter (jseutter) wrote :

+1 looks good

Just some lintian errors:
landscape/message_schemas.py:116:14: E203 whitespace before ':'
landscape/monitor/ceph.py:12:1: E302 expected 2 blank lines, found 1
landscape/monitor/tests/test_ceph.py:5:80: E501 line too long (80 characters)
landscape/monitor/tests/test_ceph.py:223:9: E301 expected 1 blank line, found 0

review: Approve

« Back to merge proposal