Code review comment for lp:~dpb/landscape-charm/vhost-config-relation

Revision history for this message
David Britton (dpb) wrote :

BTW, I committed r184 and tested it, and it hit a very weird error in the locking code. Not only did it hit an error, but the error was swallowed and the charm just got stuck by exiting 0 and blocking other relations.

I need to investigate not only why the change was bad, but also why it didn't get an error back to juju (exit 1)

« Back to merge proposal