Code review comment for lp:~sandy-walsh/nova/api-parity

367/368: is this needed now that instance is returned as a dict?

Should sharedipgroups.py be renamed to shared_ip_groups.py for consistency?

I don't like the 'grab all and find w/ hash' on image ID as I mentioned in IRC, but hopefully this will get better with Glance. :)