Code review comment for lp:~gnuoy/charms/trusty/rabbitmq-server/vhost-fix

Revision history for this message
Billy Olsen (billy-olsen) wrote :

Liam,

This looks mostly fine, but was superceded by JuanJo's MP - lp:~jjo/charms/trusty/rabbitmq-server/use-rabbitmqctl-q-for-list-cmds, which also handled the user listing info. So marking this as disapprove and you can use his.

Billy

« Back to merge proposal