)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"aee12a380915d7bbc59e2f7c3cc31deb99557852","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Follow up to I4acc15ce5f487738bb34a95c2261a5d05d827d8d"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On: I9de533ccf454b734dbae8f60bd31e34ca8f6a2b9"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Co-Authored-By: Joe Gordon \u003cjoe.gordon0@gmail.com\u003e"},{"line_number":20,"context_line":"Change-Id: I5a580fc323c3809790b4a68a9f8f8129ecdc2cf0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"5a890539_1d0df635","line":17,"updated":"2014-12-01 04:02:22.000000000","message":"s/I9de533ccf454b734dbae8f60bd31e34ca8f6a2b9/I00f1ea08a5e9dbda6e6fe765558100106026a931/ to follow Sean\u0027s patches.\n\nor we can remove this line.","commit_id":"63ac8f06d5a6176b8b618fe178804dbe2fa832fb"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"b442b0938432e9d7c9f1401ba4a8bb4749eade05","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Follow up to I4acc15ce5f487738bb34a95c2261a5d05d827d8d"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-On: I9de533ccf454b734dbae8f60bd31e34ca8f6a2b9"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Co-Authored-By: Joe Gordon \u003cjoe.gordon0@gmail.com\u003e"},{"line_number":20,"context_line":"Change-Id: I5a580fc323c3809790b4a68a9f8f8129ecdc2cf0"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"5a890539_0703dc76","line":17,"in_reply_to":"5a890539_1d0df635","updated":"2014-12-01 11:58:46.000000000","message":"Done","commit_id":"63ac8f06d5a6176b8b618fe178804dbe2fa832fb"}],"nova/api/openstack/wsgi.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"096433c8994d2841c053ca016e95325b8b0d8ca6","unresolved":false,"context_lines":[{"line_number":40,"context_line":"wsgi_opts \u003d ["},{"line_number":41,"context_line":"    cfg.BoolOpt(\u0027xml_api_v2\u0027,"},{"line_number":42,"context_line":"                default\u003dFalse,"},{"line_number":43,"context_line":"                help\u003d\u0027Disable the Nova v2 XML support\u0027),"},{"line_number":44,"context_line":"]"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_8019ed23","line":43,"range":{"start_line":43,"start_character":22,"end_line":43,"end_character":29},"updated":"2014-11-17 19:39:15.000000000","message":"from the config option name and help message its not clear what the value of True and False mean. I assume False means v2 XML is disabled, but lets be explicit about it.","commit_id":"ab98f154652ae3f455d8007f03cbc0d44099c798"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"5dba7140e5e50306938d71b3772a42d06aa774b6","unresolved":false,"context_lines":[{"line_number":40,"context_line":"wsgi_opts \u003d ["},{"line_number":41,"context_line":"    cfg.BoolOpt(\u0027xml_api_v2\u0027,"},{"line_number":42,"context_line":"                default\u003dFalse,"},{"line_number":43,"context_line":"                help\u003d\u0027Disable the Nova v2 XML support\u0027),"},{"line_number":44,"context_line":"]"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"CONF \u003d cfg.CONF"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_e0bc71bd","line":43,"in_reply_to":"5a890539_8019ed23","updated":"2014-11-17 19:45:32.000000000","message":"Picked the option name and behavior from tempest. fixed help text to make it a bit more clear.","commit_id":"ab98f154652ae3f455d8007f03cbc0d44099c798"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":20,"context_line":"from xml.dom import minidom"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"from lxml import etree"},{"line_number":23,"context_line":"from oslo.config import cfg"},{"line_number":24,"context_line":"from oslo.serialization import jsonutils"},{"line_number":25,"context_line":"from oslo.utils import strutils"},{"line_number":26,"context_line":"import six"}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_5138b9d2","line":23,"updated":"2014-11-17 21:28:16.000000000","message":"Don\u0027t need the cfg import anymore, right?","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":""},{"line_number":83,"context_line":"def get_supported_content_types():"},{"line_number":84,"context_line":"    if not xmlutil.DISABLE_XML_V2_API:"},{"line_number":85,"context_line":"        return _SUPPORTED_CONTENT_TYPES"},{"line_number":86,"context_line":"    return _SUPPORTED_CONTENT_TYPES + _SUPPORTED_XML_CONTENT_TYPES"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_7605ab12","line":84,"updated":"2014-11-17 21:28:16.000000000","message":"Shouldn\u0027t this now read:\n\n    if xmlutil.DISABLE_XML_V2_API:","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}],"nova/api/openstack/xmlutil.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"77fd08fd5142596769edc48dfd5418e03e4e0f08","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# TODO(dims): Temporary, we already deprecated the v2 XML API in"},{"line_number":30,"context_line":"# Juno, we should remove this before Kilo"},{"line_number":31,"context_line":"DISABLE_XML_V2_API \u003d True"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"XMLNS_V10 \u003d \u0027http://docs.rackspacecloud.com/servers/api/v1.0\u0027"},{"line_number":34,"context_line":"XMLNS_V11 \u003d \u0027http://docs.openstack.org/compute/api/v1.1\u0027"}],"source_content_type":"text/x-python","patch_set":22,"id":"5a890539_5d8a1e65","line":31,"updated":"2014-12-01 04:14:38.000000000","message":"Hrm I think this should probably go in openstack/wsgi.py instead. But concept is fine.","commit_id":"63ac8f06d5a6176b8b618fe178804dbe2fa832fb"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"b442b0938432e9d7c9f1401ba4a8bb4749eade05","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"# TODO(dims): Temporary, we already deprecated the v2 XML API in"},{"line_number":30,"context_line":"# Juno, we should remove this before Kilo"},{"line_number":31,"context_line":"DISABLE_XML_V2_API \u003d True"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"XMLNS_V10 \u003d \u0027http://docs.rackspacecloud.com/servers/api/v1.0\u0027"},{"line_number":34,"context_line":"XMLNS_V11 \u003d \u0027http://docs.openstack.org/compute/api/v1.1\u0027"}],"source_content_type":"text/x-python","patch_set":22,"id":"5a890539_c7d9d423","line":31,"in_reply_to":"5a890539_5d8a1e65","updated":"2014-12-01 11:58:46.000000000","message":"Done","commit_id":"63ac8f06d5a6176b8b618fe178804dbe2fa832fb"}],"nova/test.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"05e333818e58be35fdf2899c74adc1d204d8c936","unresolved":false,"context_lines":[{"line_number":245,"context_line":"            func_or_cls.setUp \u003d new_func"},{"line_number":246,"context_line":"            return func_or_cls"},{"line_number":247,"context_line":"        else:"},{"line_number":248,"context_line":"            raise TypeError(\u0027deprecated can be used only with functions or \u0027"},{"line_number":249,"context_line":"                            \u0027classes\u0027)"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":""}],"source_content_type":"text/x-python","patch_set":11,"id":"5a890539_c9e32bd7","line":248,"updated":"2014-11-17 15:23:28.000000000","message":"-\u003e \u0027skipUnless can be used...\u0027","commit_id":"e52b0ce79be6a6aa180638c834d9dc2f9ba5ea9f"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"838003006b1ffe5c5bffd2760d45717fc9ebddf9","unresolved":false,"context_lines":[{"line_number":245,"context_line":"            func_or_cls.setUp \u003d new_func"},{"line_number":246,"context_line":"            return func_or_cls"},{"line_number":247,"context_line":"        else:"},{"line_number":248,"context_line":"            raise TypeError(\u0027deprecated can be used only with functions or \u0027"},{"line_number":249,"context_line":"                            \u0027classes\u0027)"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"5a890539_259ccb5c","line":248,"updated":"2014-11-17 19:12:29.000000000","message":"nit: this isn\u0027t the @deprecated decorator…","commit_id":"6b1d16ef33df47b5f26d1da611c8fd2a197fa805"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"3180b9352c6e5323a2913559762a3e771f8cd81b","unresolved":false,"context_lines":[{"line_number":245,"context_line":"            func_or_cls.setUp \u003d new_func"},{"line_number":246,"context_line":"            return func_or_cls"},{"line_number":247,"context_line":"        else:"},{"line_number":248,"context_line":"            raise TypeError(\u0027deprecated can be used only with functions or \u0027"},{"line_number":249,"context_line":"                            \u0027classes\u0027)"},{"line_number":250,"context_line":""},{"line_number":251,"context_line":""}],"source_content_type":"text/x-python","patch_set":12,"id":"5a890539_25476b78","line":248,"in_reply_to":"5a890539_259ccb5c","updated":"2014-11-17 19:27:38.000000000","message":"Done","commit_id":"6b1d16ef33df47b5f26d1da611c8fd2a197fa805"}],"nova/tests/functional/test_api_samples.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"df4b58ae12c424f0f5eab3680c31e8c68e3c494a","unresolved":false,"context_lines":[{"line_number":4551,"context_line":"                              200)"},{"line_number":4552,"context_line":""},{"line_number":4553,"context_line":""},{"line_number":4554,"context_line":"class ServerSortKeysXmlTests(ServerSortKeysJsonTests):"},{"line_number":4555,"context_line":"    ctype \u003d \u0027xml\u0027"}],"source_content_type":"text/x-python","patch_set":26,"id":"5a890539_100f5d3a","line":4554,"updated":"2014-12-02 19:24:12.000000000","message":"Doesn\u0027t this need a @test.skipXmlTest()?","commit_id":"41b208deeb589814a59abe3fdf74ff678f9fef49"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"8d521563d5dff6be03a30a590762105c235ec720","unresolved":false,"context_lines":[{"line_number":4551,"context_line":"                              200)"},{"line_number":4552,"context_line":""},{"line_number":4553,"context_line":""},{"line_number":4554,"context_line":"class ServerSortKeysXmlTests(ServerSortKeysJsonTests):"},{"line_number":4555,"context_line":"    ctype \u003d \u0027xml\u0027"}],"source_content_type":"text/x-python","patch_set":26,"id":"5a890539_c20dd990","line":4554,"in_reply_to":"5a890539_100f5d3a","updated":"2014-12-03 18:14:24.000000000","message":"Done","commit_id":"41b208deeb589814a59abe3fdf74ff678f9fef49"}],"nova/tests/unit/api/openstack/compute/contrib/test_createserverext.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"838003006b1ffe5c5bffd2760d45717fc9ebddf9","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(response.status_int, 202)"},{"line_number":212,"context_line":"        self.assertIsNone(networks)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.skipUnless(cfg.CONF.xml_api_v2, \"Nova v2 XML support is disabled\")"},{"line_number":215,"context_line":"    def test_create_instance_with_no_networks_xml(self):"},{"line_number":216,"context_line":"        _create_inst \u003d self._create_instance_with_networks_xml"},{"line_number":217,"context_line":"        request, response, networks \u003d _create_inst(networks\u003dNone)"}],"source_content_type":"text/x-python","patch_set":12,"id":"5a890539_c5e067cf","line":214,"updated":"2014-11-17 19:12:29.000000000","message":"Wait, do we even read a config in the test suite, or are you explicitly intending to depend on the default?  If you *do* intend to depend on a configuration file here, I have to point out that said configuration file will probably not have been read at the time this test file is imported, which is the point at which the contents of the decorator will be evaluated…","commit_id":"6b1d16ef33df47b5f26d1da611c8fd2a197fa805"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"3180b9352c6e5323a2913559762a3e771f8cd81b","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(response.status_int, 202)"},{"line_number":212,"context_line":"        self.assertIsNone(networks)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.skipUnless(cfg.CONF.xml_api_v2, \"Nova v2 XML support is disabled\")"},{"line_number":215,"context_line":"    def test_create_instance_with_no_networks_xml(self):"},{"line_number":216,"context_line":"        _create_inst \u003d self._create_instance_with_networks_xml"},{"line_number":217,"context_line":"        request, response, networks \u003d _create_inst(networks\u003dNone)"}],"source_content_type":"text/x-python","patch_set":12,"id":"5a890539_e02c719b","line":214,"in_reply_to":"5a890539_c5e067cf","updated":"2014-11-17 19:27:38.000000000","message":"@Kevin, So, the neutron guys got rid of their entire xml infrastructure in one shot (https://review.openstack.org/#/c/128095/). Joe wanted a soft-landing just in case...One another option is to use an environment variable. yes, i am relying on the default value of the flag at this point.","commit_id":"6b1d16ef33df47b5f26d1da611c8fd2a197fa805"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"0e6515084ac7b3b2f36db455d0638f9de223b382","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(response.status_int, 202)"},{"line_number":212,"context_line":"        self.assertIsNone(networks)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.skipUnless(cfg.CONF.xml_api_v2, \"Nova v2 XML support is disabled\")"},{"line_number":215,"context_line":"    def test_create_instance_with_no_networks_xml(self):"},{"line_number":216,"context_line":"        _create_inst \u003d self._create_instance_with_networks_xml"},{"line_number":217,"context_line":"        request, response, networks \u003d _create_inst(networks\u003dNone)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_c029358b","line":214,"updated":"2014-11-17 19:27:56.000000000","message":"Wait, do we even read a config in the test suite, or are you explicitly intending to depend on the default? If you *do* intend to depend on a configuration file here, I have to point out that said configuration file will probably not have been read at the time this test file is imported, which is the point at which the contents of the decorator will be evaluated…","commit_id":"ab98f154652ae3f455d8007f03cbc0d44099c798"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"f81c3861a29fed759fffe698f9886d1eb85a0c75","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(response.status_int, 202)"},{"line_number":212,"context_line":"        self.assertIsNone(networks)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.skipUnless(cfg.CONF.xml_api_v2, \"Nova v2 XML support is disabled\")"},{"line_number":215,"context_line":"    def test_create_instance_with_no_networks_xml(self):"},{"line_number":216,"context_line":"        _create_inst \u003d self._create_instance_with_networks_xml"},{"line_number":217,"context_line":"        request, response, networks \u003d _create_inst(networks\u003dNone)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_c0e055cf","line":214,"in_reply_to":"5a890539_00f33d0a","updated":"2014-11-17 19:46:24.000000000","message":"y i hear ya. the main reason for the decorator was to prevent adding more lines in all the files.","commit_id":"ab98f154652ae3f455d8007f03cbc0d44099c798"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"00b6210d5d0205c4d2034744b8c23c76e9fdbc44","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(response.status_int, 202)"},{"line_number":212,"context_line":"        self.assertIsNone(networks)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.skipUnless(cfg.CONF.xml_api_v2, \"Nova v2 XML support is disabled\")"},{"line_number":215,"context_line":"    def test_create_instance_with_no_networks_xml(self):"},{"line_number":216,"context_line":"        _create_inst \u003d self._create_instance_with_networks_xml"},{"line_number":217,"context_line":"        request, response, networks \u003d _create_inst(networks\u003dNone)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_00f33d0a","line":214,"in_reply_to":"5a890539_c029358b","updated":"2014-11-17 19:33:19.000000000","message":"OK, I put in the duplicate inline before I got your followup.  As long as I know you\u0027re relying on the default value, I\u0027m fine :)  I\u0027d be tempted to suggest putting in a comment to that effect, just to make sure to catch any c\u0026p problems, but that\u0027s a lot of lines to add it to…","commit_id":"ab98f154652ae3f455d8007f03cbc0d44099c798"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"57489f7f7ac1a5c265b57e7da0a4b4474ea07516","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(response.status_int, 202)"},{"line_number":212,"context_line":"        self.assertIsNone(networks)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.skipUnless(cfg.CONF.xml_api_v2, \"Nova v2 XML support is disabled\")"},{"line_number":215,"context_line":"    def test_create_instance_with_no_networks_xml(self):"},{"line_number":216,"context_line":"        _create_inst \u003d self._create_instance_with_networks_xml"},{"line_number":217,"context_line":"        request, response, networks \u003d _create_inst(networks\u003dNone)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_e0663150","line":214,"in_reply_to":"5a890539_c0e055cf","updated":"2014-11-17 19:52:47.000000000","message":"*nod* I hear you too :)  I\u0027ve already had to stomp on a few reviews because they were evaluating config at the wrong time, and I\u0027m worried that this could exacerbate the situation.  And it\u0027s such a darn subtle bug to catch, too :(\n\nHow would you feel about a one-line addition?\n\n    # NOTE: Controlled by config default, not a config file\n\nPerhaps another way to approach it is to not use CONF at all.  What if we just stuck a global somewhere in the API, say in api/openstack/__init__.py?  It would only have to exist long enough for us to actually rip out the XML support, and wouldn\u0027t have the same c\u0026p problems for config?","commit_id":"ab98f154652ae3f455d8007f03cbc0d44099c798"},{"author":{"_account_id":5638,"name":"Davanum Srinivas","email":"davanum@gmail.com","username":"dims-v"},"change_message_id":"ff4f7c4a5dd1828c4ae390ce577d7b915bff0683","unresolved":false,"context_lines":[{"line_number":211,"context_line":"        self.assertEqual(response.status_int, 202)"},{"line_number":212,"context_line":"        self.assertIsNone(networks)"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"    @test.skipUnless(cfg.CONF.xml_api_v2, \"Nova v2 XML support is disabled\")"},{"line_number":215,"context_line":"    def test_create_instance_with_no_networks_xml(self):"},{"line_number":216,"context_line":"        _create_inst \u003d self._create_instance_with_networks_xml"},{"line_number":217,"context_line":"        request, response, networks \u003d _create_inst(networks\u003dNone)"}],"source_content_type":"text/x-python","patch_set":13,"id":"5a890539_203e79da","line":214,"in_reply_to":"5a890539_e0663150","updated":"2014-11-17 20:04:17.000000000","message":"Sounds good, let me give it a shot. thanks Kevin.","commit_id":"ab98f154652ae3f455d8007f03cbc0d44099c798"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import base64"},{"line_number":17,"context_line":"from xml.dom import minidom"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from oslo.config import cfg"},{"line_number":20,"context_line":"from oslo.serialization import jsonutils"},{"line_number":21,"context_line":"import webob"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_115ab129","line":19,"updated":"2014-11-17 21:28:16.000000000","message":"Here too, cfg no longer necessary…","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}],"nova/tests/unit/api/openstack/compute/contrib/test_extended_server_attributes.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from lxml import etree"},{"line_number":17,"context_line":"from oslo.config import cfg"},{"line_number":18,"context_line":"from oslo.serialization import jsonutils"},{"line_number":19,"context_line":"import webob"},{"line_number":20,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_11a8f1d9","line":17,"updated":"2014-11-17 21:28:16.000000000","message":"Note: This file used cfg before your patch went in; all you did was move it to the right place in the imports.","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}],"nova/tests/unit/api/openstack/compute/contrib/test_neutron_security_groups.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":17,"context_line":"from lxml import etree"},{"line_number":18,"context_line":"import mock"},{"line_number":19,"context_line":"from neutronclient.common import exceptions as n_exc"},{"line_number":20,"context_line":"from neutronclient.neutron import v2_0 as neutronv20"},{"line_number":21,"context_line":"from oslo.config import cfg"},{"line_number":22,"context_line":"from oslo.serialization import jsonutils"},{"line_number":23,"context_line":"import webob"}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_d1882973","line":20,"updated":"2014-11-17 21:28:16.000000000","message":"Note: this file also apparently already used cfg prior to your patch.","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}],"nova/tests/unit/api/openstack/compute/contrib/test_security_groups.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from lxml import etree"},{"line_number":17,"context_line":"import mock"},{"line_number":18,"context_line":"import mox"},{"line_number":19,"context_line":"from oslo.config import cfg"},{"line_number":20,"context_line":"from oslo.serialization import jsonutils"},{"line_number":21,"context_line":"import webob"}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_918a2162","line":18,"updated":"2014-11-17 21:28:16.000000000","message":"Note: this file apparently used cfg prior to your patch.","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}],"nova/tests/unit/api/openstack/compute/contrib/test_server_password.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"from lxml import etree"},{"line_number":17,"context_line":"from oslo.config import cfg"},{"line_number":18,"context_line":"from oslo.serialization import jsonutils"},{"line_number":19,"context_line":"import webob"}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_51db597b","line":16,"updated":"2014-11-17 21:28:16.000000000","message":"Note: this file apparently used cfg prior to your patch.","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}],"nova/tests/unit/api/openstack/compute/test_extensions.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":15,"context_line":"#    under the License."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"import iso8601"},{"line_number":18,"context_line":"from lxml import etree"},{"line_number":19,"context_line":"from oslo.config import cfg"},{"line_number":20,"context_line":"from oslo.serialization import jsonutils"},{"line_number":21,"context_line":"import webob"}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_f1c9a5ac","line":18,"updated":"2014-11-17 21:28:16.000000000","message":"Note: this file apparently used cfg prior to your patch.","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}],"nova/tests/unit/integrated/test_api_samples.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"ad7d8a971a331aba765e9bc6eb920418aa07b655","unresolved":false,"context_lines":[{"line_number":23,"context_line":"import uuid as uuid_lib"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"from lxml import etree"},{"line_number":26,"context_line":"import mock"},{"line_number":27,"context_line":"from oslo.config import cfg"},{"line_number":28,"context_line":"from oslo.serialization import jsonutils"},{"line_number":29,"context_line":"from oslo.utils import importutils"}],"source_content_type":"text/x-python","patch_set":15,"id":"5a890539_d13d8990","line":26,"updated":"2014-11-17 21:28:16.000000000","message":"Note: this file apparently used cfg prior to your patch.","commit_id":"a9f51afd7ffac0ef4a4680510d47d657b2c1e8ad"}]}