)]}'
{"neutron/tests/unit/test_extension_extradhcpopts.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"f9e452dc8596026002ede0e5b04c3ca56a0e2c13","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        super(ExtraDhcpOptDBTestCase, self).setUp(plugin\u003dplugin)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def tearDown(self):"},{"line_number":65,"context_line":"        gettextutils.install(\u0027neutron\u0027, lazy\u003dFalse)"},{"line_number":66,"context_line":"        super(ExtraDhcpOptDBTestCase, self).tearDown()"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAVX%2F%2Fpus%3D","line":65,"updated":"2014-03-01 22:58:48.000000000","message":"This does not seem to within the scope of the other work.  Why are we installing in the tearDown()?","commit_id":"e1ca8c71cd00051c3c6694c31d4115d2005c6072"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"8777f6f0deba896bd2ac1b1529ca3f5a14526f53","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        super(ExtraDhcpOptDBTestCase, self).setUp(plugin\u003dplugin)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def tearDown(self):"},{"line_number":65,"context_line":"        gettextutils.install(\u0027neutron\u0027, lazy\u003dFalse)"},{"line_number":66,"context_line":"        super(ExtraDhcpOptDBTestCase, self).tearDown()"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAVn%2F%2FGqc%3D","line":65,"in_reply_to":"AAAAVX%2F%2Fpus%3D","updated":"2014-03-09 18:04:02.000000000","message":"markmcclain,\n\nThanks for your review. During the execution of this unit test, gettextutils is installed with lazy\u003dTrue and left that way. That causes other unit tests to fail in the python26 and python27 gates. All I am doing is restoring gettextutils with lazy\u003dFalse, so subsequent unit tests don\u0027t fail in those gates.\n\nCheers","commit_id":"e1ca8c71cd00051c3c6694c31d4115d2005c6072"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"9c65e40de967a9fd34ca03bec9a8c4ed775d2941","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        super(ExtraDhcpOptDBTestCase, self).setUp(plugin\u003dplugin)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def tearDown(self):"},{"line_number":65,"context_line":"        gettextutils.install(\u0027neutron\u0027, lazy\u003dFalse)"},{"line_number":66,"context_line":"        super(ExtraDhcpOptDBTestCase, self).tearDown()"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAVn%2F%2B%2F6A%3D","line":65,"in_reply_to":"AAAAVn%2F%2FBQ8%3D","updated":"2014-03-10 19:36:55.000000000","message":"markmcclain,\n\nThanks for the comment. I\u0027ll dig deeper","commit_id":"e1ca8c71cd00051c3c6694c31d4115d2005c6072"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"4aa7cf6a94f521c123bf49e6cf127189fa2b5420","unresolved":false,"context_lines":[{"line_number":62,"context_line":"        super(ExtraDhcpOptDBTestCase, self).setUp(plugin\u003dplugin)"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"    def tearDown(self):"},{"line_number":65,"context_line":"        gettextutils.install(\u0027neutron\u0027, lazy\u003dFalse)"},{"line_number":66,"context_line":"        super(ExtraDhcpOptDBTestCase, self).tearDown()"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"AAAAVn%2F%2FBQ8%3D","line":65,"in_reply_to":"AAAAVn%2F%2FGqc%3D","updated":"2014-03-10 16:58:33.000000000","message":"Given that we do not explicitly install it in the inheritance chain for this test case, I don\u0027t we should patching this behavior here.  Instead we need to find/understand where it is getting installed with lazy\u003dTrue vs adding code that magically fixes this test but may not fix the issue later when tests are run out of order.","commit_id":"e1ca8c71cd00051c3c6694c31d4115d2005c6072"}]}
