Code review comment for lp:~vishvananda/nova/orm_deux

Revision history for this message
Eric Day (eday) wrote :

That sounds fine on the functional vs oo design. I would still prefer the OO design with the context member init and change during usage since this removes passing it through all the other functions. Either way I'm fine, and certainly don't want this to hold up the branch anymore given it's size and scope.

I propose we get this merged soon so we don't have as many diverging branches.

review: Approve

« Back to merge proposal