)]}'
{"nova/api/openstack/contrib/createserverext.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"2e8540415b92d144891f0bd6ad83c55c7b5ee8a0","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class CreateServerController(servers.Controller):"},{"line_number":24,"context_line":"    def _build_view(self, req, instance, is_detail\u003dFalse, is_create\u003dFalse):"},{"line_number":25,"context_line":"        server \u003d servers.Controller._build_view(self,"},{"line_number":26,"context_line":"                                                req,"},{"line_number":27,"context_line":"                                                instance,"},{"line_number":28,"context_line":"                                                is_detail\u003dis_detail,"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAACX%2F%2F5nw%3D","line":25,"updated":"2011-10-31 17:33:14.000000000","message":"Why are you dropping the super() pattern here and going straight to the superclass?  The keyword arguments make sense, I just don\u0027t understand why you\u0027re dropping super()...","commit_id":"604a17746f3a4dd19cac2fdab4132ba298061c97"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"739d9d12edf91a1fb64fc9b105cc577e753d67f1","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class CreateServerController(servers.Controller):"},{"line_number":24,"context_line":"    def _build_view(self, req, instance, is_detail\u003dFalse, is_create\u003dFalse):"},{"line_number":25,"context_line":"        server \u003d servers.Controller._build_view(self,"},{"line_number":26,"context_line":"                                                req,"},{"line_number":27,"context_line":"                                                instance,"},{"line_number":28,"context_line":"                                                is_detail\u003dis_detail,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAACX%2F%2F5is%3D","line":25,"updated":"2011-10-31 20:19:44.000000000","message":"I would still like to know why you\u0027re dropping the super() pattern here in favor of a direct class reference pattern?  Is this a style that we\u0027ve chosen to go with in nova?","commit_id":"e95874e844e6bd959355bb0d14b75b1e148cb936"},{"author":{"_account_id":706,"name":"Brian Lamar","email":"brian.lamar@rackspace.com","username":"blamar"},"change_message_id":"cd6ec7eee12c92cf553ead5bd3b827d14b53136e","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"class CreateServerController(servers.Controller):"},{"line_number":24,"context_line":"    def _build_view(self, req, instance, is_detail\u003dFalse, is_create\u003dFalse):"},{"line_number":25,"context_line":"        server \u003d servers.Controller._build_view(self,"},{"line_number":26,"context_line":"                                                req,"},{"line_number":27,"context_line":"                                                instance,"},{"line_number":28,"context_line":"                                                is_detail\u003dis_detail,"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAACX%2F%2F5iE%3D","line":25,"in_reply_to":"AAAACX%2F%2F5is%3D","updated":"2011-10-31 20:57:16.000000000","message":"It\u0027s just the style I always use and I guess I didn\u0027t think twice about it. I\u0027ll revert it in favor of the super() call.","commit_id":"e95874e844e6bd959355bb0d14b75b1e148cb936"}],"nova/tests/api/openstack/test_servers.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"739d9d12edf91a1fb64fc9b105cc577e753d67f1","unresolved":false,"context_lines":[{"line_number":1447,"context_line":"        \"\"\"Test creating multiple instances but not asking for"},{"line_number":1448,"context_line":"        reservation_id"},{"line_number":1449,"context_line":"        \"\"\""},{"line_number":1450,"context_line":"        image_href \u003d \u002776fa36fc-c930-4bf3-8c8a-ea2a2420deb6\u0027"},{"line_number":1451,"context_line":"        flavor_ref \u003d \u0027http://localhost/123/flavors/3\u0027"},{"line_number":1452,"context_line":"        body \u003d {"},{"line_number":1453,"context_line":"            \u0027server\u0027: {"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAACX%2F%2F5i0%3D","line":1450,"updated":"2011-10-31 20:19:44.000000000","message":"Well, this is no longer an href...","commit_id":"e95874e844e6bd959355bb0d14b75b1e148cb936"},{"author":{"_account_id":706,"name":"Brian Lamar","email":"brian.lamar@rackspace.com","username":"blamar"},"change_message_id":"cd6ec7eee12c92cf553ead5bd3b827d14b53136e","unresolved":false,"context_lines":[{"line_number":1447,"context_line":"        \"\"\"Test creating multiple instances but not asking for"},{"line_number":1448,"context_line":"        reservation_id"},{"line_number":1449,"context_line":"        \"\"\""},{"line_number":1450,"context_line":"        image_href \u003d \u002776fa36fc-c930-4bf3-8c8a-ea2a2420deb6\u0027"},{"line_number":1451,"context_line":"        flavor_ref \u003d \u0027http://localhost/123/flavors/3\u0027"},{"line_number":1452,"context_line":"        body \u003d {"},{"line_number":1453,"context_line":"            \u0027server\u0027: {"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAACX%2F%2F5iA%3D","line":1450,"in_reply_to":"AAAACX%2F%2F5i0%3D","updated":"2011-10-31 20:57:16.000000000","message":"I don\u0027t think this changed in my branch, maybe Waldon would know what is up with this?","commit_id":"e95874e844e6bd959355bb0d14b75b1e148cb936"}]}