Code review comment for lp:~wgrant/launchpad/builderset-new-api

Revision history for this message
Celso Providelo (cprov) wrote :

Looks good!

It seems to be a good compromise before re-thinking the buildmaster component for the Cloud(tm). However, since we do not have a remove() builders should be created with caution, even by Administrators.

review: Approve

« Back to merge proposal