Code review comment for lp:~eday/nova/compute-abstraction

Revision history for this message
Michael Gundlach (gundlach) wrote :

465:s/quote/quota/ and shouldn't that be in multiline triple-quote form?

763:isn't it a pep8 violation to not have blank lines b/w class and comment? also, i'm not sure i like having an extra base class just to set 'self.db', versus making a helper function in the db module that can be called explicitly to set self.db. but that's bikeshedding so i'm ok with how it is now too.

lgtm.

review: Approve

« Back to merge proposal