LAVA Dispatcher

Code review comment for lp:~le-chi-thu/lava-dispatcher/cache-tarballs-v1

  1. cache-tarballs-v1
  2. Merge into trunk
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

I agree with mwhudson on the feel that we should do less caching but at the same time this code probably saves a good deal of time and IO. In the long term I see a way to get that fixed but it's not something I want to talk about now.

I agree on method names, we should keep a close watch on that to ensure they stay readable.

+1

review: Approve

« Back to merge proposal