)]}'
{"cinder/tests/unit/fake_constants.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e6876ad89a58f05c7436924ef77f266a5513cbef","unresolved":false,"context_lines":[{"line_number":24,"context_line":"BACKUP3_ID \u003d \u002730ae7641-017e-4221-a642-855687c8bd71\u0027"},{"line_number":25,"context_line":"cgsnapshot_id \u003d \u00275e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":26,"context_line":"CGSNAPSHOT_ID \u003d \u00275e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":27,"context_line":"cgsnapshot_name \u003d \u0027cgsnapshot-5e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":28,"context_line":"CGSNAPSHOT_NAME \u003d \u0027cgsnapshot-5e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":29,"context_line":"cgsnapshot2_id \u003d \u00275c36d762-d6ba-4f04-bd07-88a298cc410a\u0027"},{"line_number":30,"context_line":"CGSNAPSHOT2_ID \u003d \u00275c36d762-d6ba-4f04-bd07-88a298cc410a\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_9443c212","line":27,"updated":"2016-04-20 12:32:34.000000000","message":"?: Why do we need a lower case and upper case version of the same thing?  I think we should just have the upper case version.","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"5107b09df2b173b07397872d754b460bbc62bc6a","unresolved":false,"context_lines":[{"line_number":24,"context_line":"BACKUP3_ID \u003d \u002730ae7641-017e-4221-a642-855687c8bd71\u0027"},{"line_number":25,"context_line":"cgsnapshot_id \u003d \u00275e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":26,"context_line":"CGSNAPSHOT_ID \u003d \u00275e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":27,"context_line":"cgsnapshot_name \u003d \u0027cgsnapshot-5e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":28,"context_line":"CGSNAPSHOT_NAME \u003d \u0027cgsnapshot-5e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":29,"context_line":"cgsnapshot2_id \u003d \u00275c36d762-d6ba-4f04-bd07-88a298cc410a\u0027"},{"line_number":30,"context_line":"CGSNAPSHOT2_ID \u003d \u00275c36d762-d6ba-4f04-bd07-88a298cc410a\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_c0a60aad","line":27,"in_reply_to":"1a122d0e_9443c212","updated":"2016-04-20 15:49:25.000000000","message":"We\u0027re going to get rid of all the lower case versions at once after merging [1].  We should have used UPPER CASE constants in the first place (my bad) and are maintaining the duplicates just until we\u0027ve revisited the previous commits using the lower-case versions.\n\nIt\u0027s true that any constants being added at this point don\u0027t need lower case versions.  I just thought it easier (more mechanical) for accounting and review to have both versions of each constant for the time being and then to remove all the lower-case versions in one commit.\n\n[1] https://review.openstack.org/#/c/300189/","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"9be201007e1195e80c331c5f11ba741b65341c12","unresolved":false,"context_lines":[{"line_number":24,"context_line":"BACKUP3_ID \u003d \u002730ae7641-017e-4221-a642-855687c8bd71\u0027"},{"line_number":25,"context_line":"cgsnapshot_id \u003d \u00275e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":26,"context_line":"CGSNAPSHOT_ID \u003d \u00275e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":27,"context_line":"cgsnapshot_name \u003d \u0027cgsnapshot-5e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":28,"context_line":"CGSNAPSHOT_NAME \u003d \u0027cgsnapshot-5e34cce3-bc97-46b7-a127-5cfb95ef445d\u0027"},{"line_number":29,"context_line":"cgsnapshot2_id \u003d \u00275c36d762-d6ba-4f04-bd07-88a298cc410a\u0027"},{"line_number":30,"context_line":"CGSNAPSHOT2_ID \u003d \u00275c36d762-d6ba-4f04-bd07-88a298cc410a\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_166ca406","line":27,"in_reply_to":"1a122d0e_c0a60aad","updated":"2016-04-21 09:25:23.000000000","message":"Ok, thanks, we\u0027ll remove them later  :-)","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"}],"cinder/tests/unit/test_emc_vnx.py":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"90ffff4fda09c9104a802e03ba02d89f48b52603","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.tests.unit import fake_cgsnapshot"},{"line_number":30,"context_line":"from cinder.tests.unit import fake_consistencygroup"},{"line_number":31,"context_line":"from cinder.tests.unit.fake_constants import *  # noqa"},{"line_number":32,"context_line":"from cinder.tests.unit import fake_snapshot"},{"line_number":33,"context_line":"from cinder.tests.unit import fake_volume"},{"line_number":34,"context_line":"from cinder.tests.unit import utils"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_176f22c6","line":31,"updated":"2016-04-05 14:11:41.000000000","message":"I think it is better to import fake_constants instead of importing everything defined in fake_constants.  Then you don\u0027t need to add \"# noqa\".\n\nfrom cinder.tests.unit import fake_constants","commit_id":"a39f823db8e7f116ef0fd24fb8edc4cf868a2326"},{"author":{"_account_id":10628,"name":"Peter Wang","email":"peter.wang13@emc.com","username":"peter_wang"},"change_message_id":"647eaf0d7ff6820961ffa82069ca5aa8113ebb0a","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.tests.unit import fake_cgsnapshot"},{"line_number":30,"context_line":"from cinder.tests.unit import fake_consistencygroup"},{"line_number":31,"context_line":"from cinder.tests.unit.fake_constants import *  # noqa"},{"line_number":32,"context_line":"from cinder.tests.unit import fake_snapshot"},{"line_number":33,"context_line":"from cinder.tests.unit import fake_volume"},{"line_number":34,"context_line":"from cinder.tests.unit import utils"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_3f16d44b","line":31,"in_reply_to":"9a061dce_176f22c6","updated":"2016-04-08 03:04:15.000000000","message":"+1","commit_id":"a39f823db8e7f116ef0fd24fb8edc4cf868a2326"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"90ffff4fda09c9104a802e03ba02d89f48b52603","unresolved":false,"context_lines":[{"line_number":64,"context_line":""},{"line_number":65,"context_line":"class EMCVNXCLIDriverTestData(object):"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    base_lun_name \u003d volume_name"},{"line_number":68,"context_line":"    replication_metadata \u003d {\u0027host\u0027: \u0027host@backendsec#unit_test_pool\u0027,"},{"line_number":69,"context_line":"                            \u0027system\u0027: \u0027fake_serial\u0027}"},{"line_number":70,"context_line":"    fake_wwn \u003d \u0027fake_wwn\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a061dce_57219a4c","line":67,"updated":"2016-04-05 14:11:41.000000000","message":"If you change to import fake_constants in the beginning, here will be fake_constants.volume_name instead of volume_name.\n\nI think it is more clear to use fake_constants.volume_name instead of volume_name here.  If there happens to be another \"volume_name\" defined in another fake_xxxx file, it will cause confusion when \"volume_name\" is used directly.","commit_id":"a39f823db8e7f116ef0fd24fb8edc4cf868a2326"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"767b9176afef55c5410aeb8b3832aeab10af7121","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.tests.unit import fake_cgsnapshot"},{"line_number":30,"context_line":"from cinder.tests.unit import fake_consistencygroup"},{"line_number":31,"context_line":"from cinder.tests.unit.fake_constants import *  # noqa"},{"line_number":32,"context_line":"from cinder.tests.unit import fake_snapshot"},{"line_number":33,"context_line":"from cinder.tests.unit import fake_volume"},{"line_number":34,"context_line":"from cinder.tests.unit import utils"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_f306a71a","line":31,"updated":"2016-04-20 02:41:41.000000000","message":"from cinder.tests.unit import fake_constants\n\nThen you don\u0027t need to add \"# noqa\".","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"19ce203b1a85835f6bcec4dc540f610a5c1e2d0f","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.tests.unit import fake_cgsnapshot"},{"line_number":30,"context_line":"from cinder.tests.unit import fake_consistencygroup"},{"line_number":31,"context_line":"from cinder.tests.unit.fake_constants import *  # noqa"},{"line_number":32,"context_line":"from cinder.tests.unit import fake_snapshot"},{"line_number":33,"context_line":"from cinder.tests.unit import fake_volume"},{"line_number":34,"context_line":"from cinder.tests.unit import utils"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_bdd17525","line":31,"in_reply_to":"1a122d0e_74854658","updated":"2016-04-20 15:36:10.000000000","message":"Well, that\u0027s just the style I was following in the other patches.  Xing had a good point earlier that without the leading \u0027fake.\u0027 the fake constants could collide with variables in the tests themselves.  We\u0027ve avoided that issue now by using CAPS for the fake constants (as we should have in the first place). So now I think either using fake.CONSTANT with \u0027from cinder.tests.unit import fake constants as fake\u0027 or just CONSTANT with \u0027from cinder.tests.unit.fake_constants import *\u0027 is OK, no?","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"9be201007e1195e80c331c5f11ba741b65341c12","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.tests.unit import fake_cgsnapshot"},{"line_number":30,"context_line":"from cinder.tests.unit import fake_consistencygroup"},{"line_number":31,"context_line":"from cinder.tests.unit.fake_constants import *  # noqa"},{"line_number":32,"context_line":"from cinder.tests.unit import fake_snapshot"},{"line_number":33,"context_line":"from cinder.tests.unit import fake_volume"},{"line_number":34,"context_line":"from cinder.tests.unit import utils"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_ea7dbc6d","line":31,"in_reply_to":"1a122d0e_bdd17525","updated":"2016-04-21 09:25:23.000000000","message":"Well, using * is usually a no, no.  So except in counted cases should be avoided because you pollute current namespace, I think (could be wrong though) pyflake doesn\u0027t detect errors and explicit is always better.  ;-)","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"c5b248d73b1dfaef3de8ebcbac5964ae669c7d91","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.tests.unit import fake_cgsnapshot"},{"line_number":30,"context_line":"from cinder.tests.unit import fake_consistencygroup"},{"line_number":31,"context_line":"from cinder.tests.unit.fake_constants import *  # noqa"},{"line_number":32,"context_line":"from cinder.tests.unit import fake_snapshot"},{"line_number":33,"context_line":"from cinder.tests.unit import fake_volume"},{"line_number":34,"context_line":"from cinder.tests.unit import utils"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_65c6e84a","line":31,"in_reply_to":"1a122d0e_ea7dbc6d","updated":"2016-04-21 16:27:24.000000000","message":"I will make the change.","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e6876ad89a58f05c7436924ef77f266a5513cbef","unresolved":false,"context_lines":[{"line_number":28,"context_line":"from cinder import test"},{"line_number":29,"context_line":"from cinder.tests.unit import fake_cgsnapshot"},{"line_number":30,"context_line":"from cinder.tests.unit import fake_consistencygroup"},{"line_number":31,"context_line":"from cinder.tests.unit.fake_constants import *  # noqa"},{"line_number":32,"context_line":"from cinder.tests.unit import fake_snapshot"},{"line_number":33,"context_line":"from cinder.tests.unit import fake_volume"},{"line_number":34,"context_line":"from cinder.tests.unit import utils"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_74854658","line":31,"in_reply_to":"1a122d0e_f306a71a","updated":"2016-04-20 12:32:34.000000000","message":"That\u0027s how it is being done in other patches:\n\n from cinder.tests.unit import fake_constants as fake","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e6876ad89a58f05c7436924ef77f266a5513cbef","unresolved":false,"context_lines":[{"line_number":196,"context_line":"        \u0027display_name\u0027: \u0027thin_vol\u0027,"},{"line_number":197,"context_line":"        \u0027consistencygroup_id\u0027: None,"},{"line_number":198,"context_line":"        \u0027display_description\u0027: \u0027vol with type\u0027,"},{"line_number":199,"context_line":"        \u0027volume_type_id\u0027: \u002770000029-c2cd-4f9b-b2ec-000002219289\u0027,"},{"line_number":200,"context_line":"        \u0027provider_location\u0027: build_provider_location(1, \u0027smp\u0027, VOLUME_NAME),"},{"line_number":201,"context_line":"        \u0027volume_metadata\u0027: [{\u0027key\u0027: \u0027snapcopy\u0027, \u0027value\u0027: \u0027True\u0027}]}"},{"line_number":202,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_f4fcb6b9","line":199,"updated":"2016-04-20 12:32:34.000000000","message":"-1: Why isn\u0027t this using one of the fake constants?","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"c5b248d73b1dfaef3de8ebcbac5964ae669c7d91","unresolved":false,"context_lines":[{"line_number":196,"context_line":"        \u0027display_name\u0027: \u0027thin_vol\u0027,"},{"line_number":197,"context_line":"        \u0027consistencygroup_id\u0027: None,"},{"line_number":198,"context_line":"        \u0027display_description\u0027: \u0027vol with type\u0027,"},{"line_number":199,"context_line":"        \u0027volume_type_id\u0027: \u002770000029-c2cd-4f9b-b2ec-000002219289\u0027,"},{"line_number":200,"context_line":"        \u0027provider_location\u0027: build_provider_location(1, \u0027smp\u0027, VOLUME_NAME),"},{"line_number":201,"context_line":"        \u0027volume_metadata\u0027: [{\u0027key\u0027: \u0027snapcopy\u0027, \u0027value\u0027: \u0027True\u0027}]}"},{"line_number":202,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_2513500f","line":199,"in_reply_to":"1a122d0e_f4fcb6b9","updated":"2016-04-21 16:27:24.000000000","message":"I\u0027m changing it to do so.","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e6876ad89a58f05c7436924ef77f266a5513cbef","unresolved":false,"context_lines":[{"line_number":3338,"context_line":"        data \u003d self.testData"},{"line_number":3339,"context_line":"        test_volume \u003d data.test_volume_with_type"},{"line_number":3340,"context_line":"        lun_rename_cmd \u003d data.LUN_RENAME_CMD("},{"line_number":3341,"context_line":"            \u00271\u0027, test_volume[\u0027name\u0027])"},{"line_number":3342,"context_line":"        lun_list_cmd \u003d data.LUN_LIST_ALL_CMD(\u00271\u0027)"},{"line_number":3343,"context_line":""},{"line_number":3344,"context_line":"        commands \u003d (lun_rename_cmd, lun_list_cmd)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_94b622f7","line":3341,"updated":"2016-04-20 12:32:34.000000000","message":"?: Why has this been changed to a constant here and in the next 3 changes?","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"c5b248d73b1dfaef3de8ebcbac5964ae669c7d91","unresolved":false,"context_lines":[{"line_number":3338,"context_line":"        data \u003d self.testData"},{"line_number":3339,"context_line":"        test_volume \u003d data.test_volume_with_type"},{"line_number":3340,"context_line":"        lun_rename_cmd \u003d data.LUN_RENAME_CMD("},{"line_number":3341,"context_line":"            \u00271\u0027, test_volume[\u0027name\u0027])"},{"line_number":3342,"context_line":"        lun_list_cmd \u003d data.LUN_LIST_ALL_CMD(\u00271\u0027)"},{"line_number":3343,"context_line":""},{"line_number":3344,"context_line":"        commands \u003d (lun_rename_cmd, lun_list_cmd)"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_e56158aa","line":3341,"in_reply_to":"1a122d0e_94b622f7","updated":"2016-04-21 16:27:24.000000000","message":"Note that the argument here is a VNX lun id, not a cinder volume id.  The test used to \"work\" passing in a volume ID because the volume id was \u00271\u0027 even  though what was supposed to be passed here was the LUN ID \"1\" (or 1, it gets cast to int).","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e6876ad89a58f05c7436924ef77f266a5513cbef","unresolved":false,"context_lines":[{"line_number":4849,"context_line":"        vol1_in_new_cg \u003d self.testData.test_volume_cg.copy()"},{"line_number":4850,"context_line":"        vol1_in_new_cg.update("},{"line_number":4851,"context_line":"            {\u0027name\u0027: \u0027volume-1_in_cg\u0027,"},{"line_number":4852,"context_line":"             \u0027id\u0027: \u0027111111\u0027,"},{"line_number":4853,"context_line":"             \u0027consistencygroup_id\u0027: CONSISTENCY_GROUP_ID,"},{"line_number":4854,"context_line":"             \u0027provider_location\u0027: None})"},{"line_number":4855,"context_line":"        vol2_in_new_cg \u003d self.testData.test_volume_cg.copy()"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_14a85276","line":4852,"updated":"2016-04-20 12:32:34.000000000","message":"?: Shouldn\u0027t this also be here and in the next 3 cases a valid UUID?","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"c5b248d73b1dfaef3de8ebcbac5964ae669c7d91","unresolved":false,"context_lines":[{"line_number":4849,"context_line":"        vol1_in_new_cg \u003d self.testData.test_volume_cg.copy()"},{"line_number":4850,"context_line":"        vol1_in_new_cg.update("},{"line_number":4851,"context_line":"            {\u0027name\u0027: \u0027volume-1_in_cg\u0027,"},{"line_number":4852,"context_line":"             \u0027id\u0027: \u0027111111\u0027,"},{"line_number":4853,"context_line":"             \u0027consistencygroup_id\u0027: CONSISTENCY_GROUP_ID,"},{"line_number":4854,"context_line":"             \u0027provider_location\u0027: None})"},{"line_number":4855,"context_line":"        vol2_in_new_cg \u003d self.testData.test_volume_cg.copy()"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_4543ec6c","line":4852,"in_reply_to":"1a122d0e_14a85276","updated":"2016-04-21 16:27:24.000000000","message":"Will fix.","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"e6876ad89a58f05c7436924ef77f266a5513cbef","unresolved":false,"context_lines":[{"line_number":5392,"context_line":"        data \u003d self.testData"},{"line_number":5393,"context_line":"        test_volume \u003d data.test_volume_with_type"},{"line_number":5394,"context_line":"        lun_rename_cmd \u003d data.LUN_RENAME_CMD("},{"line_number":5395,"context_line":"            \u00271\u0027, test_volume[\u0027name\u0027])"},{"line_number":5396,"context_line":"        lun_list_cmd \u003d data.LUN_LIST_ALL_CMD(\u00271\u0027)"},{"line_number":5397,"context_line":""},{"line_number":5398,"context_line":"        commands \u003d lun_rename_cmd, lun_list_cmd"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_74e0a6cd","line":5395,"updated":"2016-04-20 12:32:34.000000000","message":"?: Why the change?","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"},{"author":{"_account_id":9003,"name":"Tom Barron","email":"tpb@dyncloud.net","username":"tbarron"},"change_message_id":"c5b248d73b1dfaef3de8ebcbac5964ae669c7d91","unresolved":false,"context_lines":[{"line_number":5392,"context_line":"        data \u003d self.testData"},{"line_number":5393,"context_line":"        test_volume \u003d data.test_volume_with_type"},{"line_number":5394,"context_line":"        lun_rename_cmd \u003d data.LUN_RENAME_CMD("},{"line_number":5395,"context_line":"            \u00271\u0027, test_volume[\u0027name\u0027])"},{"line_number":5396,"context_line":"        lun_list_cmd \u003d data.LUN_LIST_ALL_CMD(\u00271\u0027)"},{"line_number":5397,"context_line":""},{"line_number":5398,"context_line":"        commands \u003d lun_rename_cmd, lun_list_cmd"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_887b5929","line":5395,"in_reply_to":"1a122d0e_74e0a6cd","updated":"2016-04-21 16:27:24.000000000","message":"This is another case where the command argument is a LUN ID and only the fact that they were using \u00271\u0027 as a volume ID allowed the test to pass before.","commit_id":"86d4e5da713db775a66781e5c90d7e9af2c8e3bd"}]}