Code review comment for lp://qastaging/~jtv/maas/custom-commands

Revision history for this message
Gavin Panella (allenap) wrote :

Nice :)

[1]

+ check_call(['bin/maasdb', 'delete-cluster', 'db'], stderr=stderr)

I think subprocess doesn't redirect stderr, so the last argument above is superfluous I think.

review: Approve

« Back to merge proposal