Code review comment for lp:~allenap/launchpad/add-rabbit-service-bug-806160

Revision history for this message
Robert Collins (lifeless) wrote :

On Thu, Sep 15, 2011 at 5:07 AM, Gavin Panella
<email address hidden> wrote:

> Perhaps we could do something better:
>
> - config.push() could also write out the fragments its given to
>  configs/$LPCONFIG/extra-${sequence}-${name}.conf

We might want a special verb for this to avoid it happening my mistake
on production servers. Make clean probably would want to delete them
all. The outer most test layer may want to zap them all as it comes up
as well.

Other than that this sounds good.

-Rob

« Back to merge proposal