Code review comment for lp:~hazmat/pyjuju/var-run-mass-restart

Revision history for this message
Benjamin Saller (bcsaller) wrote :

LGTM w/trivial

https://codereview.appspot.com/8583048/diff/1/juju/agents/base.py
File juju/agents/base.py (right):

https://codereview.appspot.com/8583048/diff/1/juju/agents/base.py#newcode41
juju/agents/base.py:41: if e.errno != 17:
Could you use errno.EEXIST here rather than the magic number?

https://codereview.appspot.com/8583048/

« Back to merge proposal