)]}'
{"bin/nova-api":[{"author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"change_message_id":"05e7e710264aa73f05cf8005906545bdd5571473","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"from nova import config"},{"line_number":40,"context_line":"from nova import flags"},{"line_number":41,"context_line":"from nova.openstack.common import log as logging"},{"line_number":42,"context_line":"from nova import service"},{"line_number":43,"context_line":"from nova import utils"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAALn%2F%2F0Ps%3D","line":40,"updated":"2012-11-04 22:05:13.000000000","message":"Shouldn\u0027t this import go away now?","commit_id":"5749b8deb0f8f40fb10b41a397da82aae7e1a305"},{"author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"change_message_id":"02e438c8a2b212d8ef3d271c804f120c84f9a36d","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"from nova import config"},{"line_number":40,"context_line":"from nova import flags"},{"line_number":41,"context_line":"from nova.openstack.common import log as logging"},{"line_number":42,"context_line":"from nova import service"},{"line_number":43,"context_line":"from nova import utils"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAALn%2F%2F0Ok%3D","line":40,"in_reply_to":"AAAALn%2F%2F0PY%3D","updated":"2012-11-04 22:23:16.000000000","message":"Gah. My apologies for not fully reading the commit message.\n\nI\u0027m really not a fan of side-effects from importing modules, but I agree that it\u0027s probably easier to fix that after these changes rather than before.","commit_id":"5749b8deb0f8f40fb10b41a397da82aae7e1a305"},{"author":{"_account_id":1247,"name":"Mark McLoughlin","email":"markmc@redhat.com","username":"markmc"},"change_message_id":"09a1443e0f9733a4684d6c315e5c3b238dccc4d2","unresolved":false,"context_lines":[{"line_number":37,"context_line":""},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"from nova import config"},{"line_number":40,"context_line":"from nova import flags"},{"line_number":41,"context_line":"from nova.openstack.common import log as logging"},{"line_number":42,"context_line":"from nova import service"},{"line_number":43,"context_line":"from nova import utils"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAALn%2F%2F0PY%3D","line":40,"in_reply_to":"AAAALn%2F%2F0Ps%3D","updated":"2012-11-04 22:07:44.000000000","message":"See the commit message\n\nImporting the flags module has the side effect of registering a tonne of options - I\u0027ll follow up with further set of commits to sort all that out, but it will have to be done carefully because it\u0027s hard to tell what will break when you delete this import","commit_id":"5749b8deb0f8f40fb10b41a397da82aae7e1a305"}]}