Code review comment for lp://staging/~doanac/ubuntu-ci-services-itself/run-worker-conversion

Revision history for this message
Vincent Ladeuil (vila) wrote :

355 + url = store.put_file(name, stream, 'text/plain')

Yeehaaa !

<vila> doanac: 7 failures in http://s-jenkins.ubuntu-ci:8080/job/uci-engine-ci/389/console for https://code.launchpad.net/~doanac/ubuntu-ci-services-itself/run-worker-conversion/+merge/210078
<vila> doanac: the fix is probably trivial, super(_worker, self).__init__()
<vila> TypeError: __init__() takes exactly 2 arguments (1 given)
<vila> doanac: additionally, since you played with it, it means it's a test-only issue. If you're peacefully sleeping, I'll try to land it with ev or psivaa help

review: Approve

« Back to merge proposal