Code review comment for lp://staging/~cprov/launchpad/bug-424797-buildd-manager-test-failure

Revision history for this message
Jonathan Lange (jml) wrote :

Hey Celso,

Your change to the test looks clear & correct, and I'm happy for it to land.

The fix for the "failing silently" bit is actually going to be reasonably complex, so it's fair enough to deal with it in another branch. Bug 425113 was filed to track it, and has a reasonable explanation of what's going on.

jml

review: Approve (code)

« Back to merge proposal