Code review comment for lp:~chad.smith/landscape-client/ha-manager-skeleton

Revision history for this message
Christopher Armstrong (radix) wrote :

[8]

+ failure_string = "%s" % (failure.value)

You should probably just use failure.getErrorMessage(). (at first I was going to suggest str(failure.value) instead of "%s" %, but then I realized failure has a method for that already)

Looks good!

review: Approve

« Back to merge proposal