Code review comment for lp://staging/~mbp/launchpad/ec2-region

Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

There was quite a lot here, but all the changes look good, thanks for working on this!

Obviously, please don't land until you've successfully tested it all -- and the tmpfs stuff does make things a bit more complicated, so we should verify that it is actually an improvement.

Have you run the tests that the ec2 code does have?

review: Approve

« Back to merge proposal