Code review comment for lp:~justin-fathomdb/nova/ec2-filters

Revision history for this message
Devin Carlen (devcamcar) wrote :

Hi Justin,

Nice work here but I think it's a bit overkill for the bug. Excerpt from bug 732924:

"This makes it difficult to figure out which project a particular volume is associated with. Instances, on the other hand specifically return a project as the owner (in a separate attribute at that):

[ownerId] => testproject

Similarly, addresses also report the project as an owner:

[instanceId] => i-00000002 (testproject)"

The bug isn't that we need any advanced filtering. This only affects EC2 API and how we have to work around a few of its sharp edges.

And it looks like Ryan Lane already made this fix:

So what is the need for this merge? It doesn't seem to have anything to do with the bug you linked.

review: Needs Information

« Back to merge proposal