)]}'
{"bin/nova-manage":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"db8b8f7a32187ead5be5c072a9cd717fb2778124","unresolved":false,"context_lines":[{"line_number":89,"context_line":"from nova import version"},{"line_number":90,"context_line":"from nova import vsa"},{"line_number":91,"context_line":"from nova.api.ec2 import ec2utils"},{"line_number":92,"context_line":"from nova.api.ec2.admin import AdminController"},{"line_number":93,"context_line":"from nova.auth import manager"},{"line_number":94,"context_line":"from nova.cloudpipe import pipelib"},{"line_number":95,"context_line":"from nova.compute import instance_types"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAADH%2F%2F%2Fw0%3D","line":92,"updated":"2011-12-29 08:32:16.000000000","message":"Violates HACKING in 2 ways.  nova.api.ec2.admin comes before nova.api.ec2.ec2utils.  Also, should import modules, not objects.   I see that a lot of this import list violates HACKING already (not in correct order).  I\u0027d like to see the whole thing fixed, but since that\u0027s unrelated to your fix, I\u0027d be happy enough with just moving your import up 1 line and fixing it so it imports the module not the class.","commit_id":"23779e473a1aee7fc2ad98a1ae98f80d40572d50"}]}