)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"d777e96476b6a98eb69356159e1ed7f297277d97","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sridhargaddam \u003csridhar.gaddam@enovance.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-24 20:51:42 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes the issue #1243878"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Including the HTTP headers in the HTTP response."},{"line_number":10,"context_line":"Changes done in neutron-ns-metadata-proxy and neutron-metadata-agent."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAU3%2F%2F1H8%3D","line":7,"updated":"2014-02-24 15:55:30.000000000","message":"please provide more meaningful commit message (https://wiki.openstack.org/wiki/GitCommitMessages)","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":2031,"name":"Nachi Ueno","email":"nati.ueno@gmail.com","username":"nati-ueno"},"change_message_id":"f253f93666d30cc7890f5b2afb8e8e83bef0ecce","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sridhargaddam \u003csridhar.gaddam@enovance.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-24 20:51:42 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes the issue #1243878"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Including the HTTP headers in the HTTP response."},{"line_number":10,"context_line":"Changes done in neutron-ns-metadata-proxy and neutron-metadata-agent."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAU3%2F%2Fup0%3D","line":7,"in_reply_to":"AAAAU3%2F%2F1H8%3D","updated":"2014-02-25 02:43:47.000000000","message":"+1","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"b3ebe64781c0f1a824039c328f5145b02513bb15","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sridhargaddam \u003csridhar.gaddam@enovance.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-24 20:51:42 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes the issue #1243878"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Including the HTTP headers in the HTTP response."},{"line_number":10,"context_line":"Changes done in neutron-ns-metadata-proxy and neutron-metadata-agent."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAU3%2F%2Fs4M%3D","line":7,"in_reply_to":"AAAAU3%2F%2Fup0%3D","updated":"2014-02-25 07:12:09.000000000","message":"Done","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"d777e96476b6a98eb69356159e1ed7f297277d97","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Including the HTTP headers in the HTTP response."},{"line_number":10,"context_line":"Changes done in neutron-ns-metadata-proxy and neutron-metadata-agent."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug #1243878"},{"line_number":13,"context_line":"Change-Id: If68f0b508fbea4ecd1eb0e58d602b5ba6ccbe263"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAU3%2F%2F1HY%3D","line":12,"updated":"2014-02-24 15:55:30.000000000","message":": is missed, should be \u0027Closes-Bug: #1243878\u0027 for the proper link","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"b3ebe64781c0f1a824039c328f5145b02513bb15","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Including the HTTP headers in the HTTP response."},{"line_number":10,"context_line":"Changes done in neutron-ns-metadata-proxy and neutron-metadata-agent."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Closes-Bug #1243878"},{"line_number":13,"context_line":"Change-Id: If68f0b508fbea4ecd1eb0e58d602b5ba6ccbe263"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAU3%2F%2Fs4A%3D","line":12,"in_reply_to":"AAAAU3%2F%2F1HY%3D","updated":"2014-02-25 07:12:09.000000000","message":"Done","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":2711,"name":"Zhang Hua","display_name":"Zhang Hua","email":"joshua.zhang@canonical.com","username":"zhhuabj"},"change_message_id":"01bac971c19e2b7bc7a599cb922db88866764960","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sridhargaddam \u003csridhar.gaddam@enovance.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-25 12:39:38 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes the issue #1243878"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Neutron namespace proxy handler and metadata agent were not setting the"},{"line_number":10,"context_line":"Content-Type in its response. Both of them were returning only the response"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAU3%2F%2Fsl4%3D","line":7,"updated":"2014-02-25 07:59:55.000000000","message":"pls response Oleg\u0027s comment for this in patch set 1 to provide more meaningful commit message here.","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"26e6beb28e6643db74ce59c2c27406904bb9af6c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sridhargaddam \u003csridhar.gaddam@enovance.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-02-25 12:39:38 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixes the issue #1243878"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Neutron namespace proxy handler and metadata agent were not setting the"},{"line_number":10,"context_line":"Content-Type in its response. Both of them were returning only the response"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAU3%2F%2Fr7I%3D","line":7,"in_reply_to":"AAAAU3%2F%2Fsl4%3D","updated":"2014-02-25 09:08:39.000000000","message":"Done","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"},{"author":{"_account_id":2711,"name":"Zhang Hua","display_name":"Zhang Hua","email":"joshua.zhang@canonical.com","username":"zhhuabj"},"change_message_id":"01bac971c19e2b7bc7a599cb922db88866764960","unresolved":false,"context_lines":[{"line_number":14,"context_line":"present in the HTTP Response. The fix now includes the Content-Type which is"},{"line_number":15,"context_line":"returned by nova-metadata-service."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug:#1243878"},{"line_number":18,"context_line":"Change-Id: If68f0b508fbea4ecd1eb0e58d602b5ba6ccbe263"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAU3%2F%2FslY%3D","line":17,"updated":"2014-02-25 07:59:55.000000000","message":"pls response Oleg\u0027s comment for this in patch set 1,  should be \u0027Closes-Bug: #1243878\u0027 for the proper link, there is a blank between : and #","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"26e6beb28e6643db74ce59c2c27406904bb9af6c","unresolved":false,"context_lines":[{"line_number":14,"context_line":"present in the HTTP Response. The fix now includes the Content-Type which is"},{"line_number":15,"context_line":"returned by nova-metadata-service."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes-Bug:#1243878"},{"line_number":18,"context_line":"Change-Id: If68f0b508fbea4ecd1eb0e58d602b5ba6ccbe263"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAU3%2F%2Fr7E%3D","line":17,"in_reply_to":"AAAAU3%2F%2FslY%3D","updated":"2014-02-25 09:08:39.000000000","message":"Done","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"}],"neutron/agent/metadata/agent.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ae0be9ac2cf2b3c67c47e43fb81c45fd7edea6b3","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import hmac"},{"line_number":21,"context_line":"import os"},{"line_number":22,"context_line":"import socket"},{"line_number":23,"context_line":"import urlparse"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import eventlet"},{"line_number":26,"context_line":"import httplib2"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAU3%2F%2Fv34%3D","line":23,"updated":"2014-02-25 00:10:21.000000000","message":"You need to rebase, the change to make this six.* just hit yesterday.","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"b3ebe64781c0f1a824039c328f5145b02513bb15","unresolved":false,"context_lines":[{"line_number":20,"context_line":"import hmac"},{"line_number":21,"context_line":"import os"},{"line_number":22,"context_line":"import socket"},{"line_number":23,"context_line":"import urlparse"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"import eventlet"},{"line_number":26,"context_line":"import httplib2"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAU3%2F%2Fs34%3D","line":23,"in_reply_to":"AAAAU3%2F%2Fv34%3D","updated":"2014-02-25 07:12:09.000000000","message":"Done","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":2711,"name":"Zhang Hua","display_name":"Zhang Hua","email":"joshua.zhang@canonical.com","username":"zhhuabj"},"change_message_id":"01bac971c19e2b7bc7a599cb922db88866764960","unresolved":false,"context_lines":[{"line_number":167,"context_line":"                                  body\u003dreq.body)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        if resp.status \u003d\u003d 200:"},{"line_number":170,"context_line":"            # Issue:1243878:: Include the proper Content-Type"},{"line_number":171,"context_line":"            # in the response headers, which is received from the server."},{"line_number":172,"context_line":"            LOG.debug(str(resp))"},{"line_number":173,"context_line":"            req.response.content_type \u003d resp[\u0027content-type\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAU3%2F%2FskM%3D","line":170,"updated":"2014-02-25 07:59:55.000000000","message":"ditto","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"26e6beb28e6643db74ce59c2c27406904bb9af6c","unresolved":false,"context_lines":[{"line_number":167,"context_line":"                                  body\u003dreq.body)"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"        if resp.status \u003d\u003d 200:"},{"line_number":170,"context_line":"            # Issue:1243878:: Include the proper Content-Type"},{"line_number":171,"context_line":"            # in the response headers, which is received from the server."},{"line_number":172,"context_line":"            LOG.debug(str(resp))"},{"line_number":173,"context_line":"            req.response.content_type \u003d resp[\u0027content-type\u0027]"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAU3%2F%2Fr6w%3D","line":170,"in_reply_to":"AAAAU3%2F%2FskM%3D","updated":"2014-02-25 09:08:39.000000000","message":"Done","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"}],"neutron/agent/metadata/namespace_proxy.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ae0be9ac2cf2b3c67c47e43fb81c45fd7edea6b3","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import httplib"},{"line_number":20,"context_line":"import socket"},{"line_number":21,"context_line":"import urlparse"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import eventlet"},{"line_number":24,"context_line":"import httplib2"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAU3%2F%2Fv3s%3D","line":21,"updated":"2014-02-25 00:10:21.000000000","message":"This needs a rebase as well.","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"b3ebe64781c0f1a824039c328f5145b02513bb15","unresolved":false,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"import httplib"},{"line_number":20,"context_line":"import socket"},{"line_number":21,"context_line":"import urlparse"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"import eventlet"},{"line_number":24,"context_line":"import httplib2"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAU3%2F%2Fs30%3D","line":21,"in_reply_to":"AAAAU3%2F%2Fv3s%3D","updated":"2014-02-25 07:12:09.000000000","message":"Done","commit_id":"09bf79f4c54a387f2bc8eea8a77b71e83386651c"},{"author":{"_account_id":2711,"name":"Zhang Hua","display_name":"Zhang Hua","email":"joshua.zhang@canonical.com","username":"zhhuabj"},"change_message_id":"01bac971c19e2b7bc7a599cb922db88866764960","unresolved":false,"context_lines":[{"line_number":105,"context_line":"            connection_type\u003dUnixDomainHTTPConnection)"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"        if resp.status \u003d\u003d 200:"},{"line_number":108,"context_line":"            # Issue:1243878:: Include the proper Content-Type"},{"line_number":109,"context_line":"            # in the response headers, which is received from the server."},{"line_number":110,"context_line":"            LOG.debug(resp)"},{"line_number":111,"context_line":"            LOG.debug(content)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAU3%2F%2FskQ%3D","line":108,"updated":"2014-02-25 07:59:55.000000000","message":"issue number is not necessary in the code which can be known by the command \u0027git blame\u0027 easily, and I think we can remove this comment because the below code is self-explanatory.","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"26e6beb28e6643db74ce59c2c27406904bb9af6c","unresolved":false,"context_lines":[{"line_number":105,"context_line":"            connection_type\u003dUnixDomainHTTPConnection)"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"        if resp.status \u003d\u003d 200:"},{"line_number":108,"context_line":"            # Issue:1243878:: Include the proper Content-Type"},{"line_number":109,"context_line":"            # in the response headers, which is received from the server."},{"line_number":110,"context_line":"            LOG.debug(resp)"},{"line_number":111,"context_line":"            LOG.debug(content)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAU3%2F%2FsJc%3D","line":108,"in_reply_to":"AAAAU3%2F%2FskQ%3D","updated":"2014-02-25 09:08:39.000000000","message":"ok Zhang Hua. Will remove it accordingly.","commit_id":"5d24d31bda588fd442d5d1793d42f3a34265e604"}],"neutron/tests/unit/test_metadata_agent.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"96a71640abc78c283c2a24cf965034957dcb17f3","unresolved":false,"context_lines":[{"line_number":206,"context_line":"        with mock.patch.object(self.handler, \u0027_sign_instance_id\u0027) as sign:"},{"line_number":207,"context_line":"            sign.return_value \u003d \u0027signed\u0027"},{"line_number":208,"context_line":"            with mock.patch(\u0027httplib2.Http\u0027) as mock_http:"},{"line_number":209,"context_line":"                resp.__getitem__ \u003d mock.Mock(return_value\u003d\"text/plain\")"},{"line_number":210,"context_line":"                mock_http.return_value.request.return_value \u003d (resp, \u0027content\u0027)"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"                retval \u003d self.handler._proxy_request(\u0027the_id\u0027, \u0027tenant_id\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2FrvU%3D","line":209,"updated":"2014-02-25 09:31:20.000000000","message":"Why not using self.resp directly?\n\nYou can just write: self.resp.__getitem__.return_value \u003d \"text/plain\". Or it doesn\u0027t work, you have to declare resp with MagicMock.","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"61cf1087c2ee36110e3db450b133a04d48492788","unresolved":false,"context_lines":[{"line_number":206,"context_line":"        with mock.patch.object(self.handler, \u0027_sign_instance_id\u0027) as sign:"},{"line_number":207,"context_line":"            sign.return_value \u003d \u0027signed\u0027"},{"line_number":208,"context_line":"            with mock.patch(\u0027httplib2.Http\u0027) as mock_http:"},{"line_number":209,"context_line":"                resp.__getitem__ \u003d mock.Mock(return_value\u003d\"text/plain\")"},{"line_number":210,"context_line":"                mock_http.return_value.request.return_value \u003d (resp, \u0027content\u0027)"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"                retval \u003d self.handler._proxy_request(\u0027the_id\u0027, \u0027tenant_id\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2FrLQ%3D","line":209,"in_reply_to":"AAAAU3%2F%2FrvU%3D","updated":"2014-02-25 10:30:55.000000000","message":"Yes Victor, it does not work as Mock is not supporting the __getitem__ attribute. I changed it to MagicMock which supports this attribute.","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"96a71640abc78c283c2a24cf965034957dcb17f3","unresolved":false,"context_lines":[{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def test_proxy_request_post(self):"},{"line_number":231,"context_line":"        self.assertEqual(self._proxy_request_test_helper(method\u003d\u0027POST\u0027),"},{"line_number":232,"context_line":"                         self.resp)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def test_proxy_request_200(self):"},{"line_number":235,"context_line":"        self.assertEqual(self._proxy_request_test_helper(200), self.resp)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2FruQ%3D","line":232,"updated":"2014-02-25 09:31:20.000000000","message":"Usually, we write assertEqual(var, expected). You exchanged the two parameters.","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"61cf1087c2ee36110e3db450b133a04d48492788","unresolved":false,"context_lines":[{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def test_proxy_request_post(self):"},{"line_number":231,"context_line":"        self.assertEqual(self._proxy_request_test_helper(method\u003d\u0027POST\u0027),"},{"line_number":232,"context_line":"                         self.resp)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def test_proxy_request_200(self):"},{"line_number":235,"context_line":"        self.assertEqual(self._proxy_request_test_helper(200), self.resp)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2FrK4%3D","line":232,"in_reply_to":"AAAAU3%2F%2FruQ%3D","updated":"2014-02-25 10:30:55.000000000","message":"Modified the code accordingly.","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"96a71640abc78c283c2a24cf965034957dcb17f3","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                         self.resp)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def test_proxy_request_200(self):"},{"line_number":235,"context_line":"        self.assertEqual(self._proxy_request_test_helper(200), self.resp)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"    def test_proxy_request_403(self):"},{"line_number":238,"context_line":"        self.assertIsInstance(self._proxy_request_test_helper(403),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2FruM%3D","line":235,"updated":"2014-02-25 09:31:20.000000000","message":"Ditto.","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"61cf1087c2ee36110e3db450b133a04d48492788","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                         self.resp)"},{"line_number":233,"context_line":""},{"line_number":234,"context_line":"    def test_proxy_request_200(self):"},{"line_number":235,"context_line":"        self.assertEqual(self._proxy_request_test_helper(200), self.resp)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"    def test_proxy_request_403(self):"},{"line_number":238,"context_line":"        self.assertIsInstance(self._proxy_request_test_helper(403),"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2FrKw%3D","line":235,"in_reply_to":"AAAAU3%2F%2FruM%3D","updated":"2014-02-25 10:30:55.000000000","message":"Done","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"}],"neutron/tests/unit/test_metadata_namespace_proxy.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"96a71640abc78c283c2a24cf965034957dcb17f3","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        resp \u003d mock.Mock(status\u003d200)"},{"line_number":99,"context_line":"        with mock.patch(\u0027httplib2.Http\u0027) as mock_http:"},{"line_number":100,"context_line":"            resp.__getitem__ \u003d mock.Mock(return_value\u003d\"text/plain\")"},{"line_number":101,"context_line":"            mock_http.return_value.request.return_value \u003d (resp, \u0027content\u0027)"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"            retval \u003d self.handler._proxy_request(\u0027192.168.1.1\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2Frtc%3D","line":100,"updated":"2014-02-25 09:31:20.000000000","message":"Ditto. Try resp.__getitem__.return_value \u003d ...","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"},{"author":{"_account_id":10257,"name":"Sridhar Gaddam","email":"sgaddam@redhat.com","username":"sridhargaddam"},"change_message_id":"61cf1087c2ee36110e3db450b133a04d48492788","unresolved":false,"context_lines":[{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        resp \u003d mock.Mock(status\u003d200)"},{"line_number":99,"context_line":"        with mock.patch(\u0027httplib2.Http\u0027) as mock_http:"},{"line_number":100,"context_line":"            resp.__getitem__ \u003d mock.Mock(return_value\u003d\"text/plain\")"},{"line_number":101,"context_line":"            mock_http.return_value.request.return_value \u003d (resp, \u0027content\u0027)"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"            retval \u003d self.handler._proxy_request(\u0027192.168.1.1\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAU3%2F%2FrKQ%3D","line":100,"in_reply_to":"AAAAU3%2F%2Frtc%3D","updated":"2014-02-25 10:30:55.000000000","message":"Changed the implementation to use MagicMock which supports this attribute.","commit_id":"dba27adf2b529a3dc254b4d2cea991be25ba2078"}]}