)]}'
{"openstackclient/compute/v2/migration_list.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e59770d3890649e7bc94ababebd33cd8c5dcbe55","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"\"\"\"Migration list  action implementations\"\"\""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import re"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from novaclient import exceptions as nova_exceptions"},{"line_number":21,"context_line":"from osc_lib.command import command"}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_b2d8971e","line":18,"updated":"2020-07-21 21:35:04.000000000","message":"pep8: F401 \u0027re\u0027 imported but unused","commit_id":"1dbce2313a2305722834fd2d70e25999d4198351"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e59770d3890649e7bc94ababebd33cd8c5dcbe55","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"import re"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from novaclient import exceptions as nova_exceptions"},{"line_number":21,"context_line":"from osc_lib.command import command"},{"line_number":22,"context_line":"from osc_lib import utils"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_92d593e8","line":20,"updated":"2020-07-21 21:35:04.000000000","message":"pep8: F401 \u0027novaclient.exceptions as nova_exceptions\u0027 imported but unused","commit_id":"1dbce2313a2305722834fd2d70e25999d4198351"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e59770d3890649e7bc94ababebd33cd8c5dcbe55","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            \"--migration-type\","},{"line_number":49,"context_line":"            metavar\u003d\"\u003cmigration_type\u003e\","},{"line_number":50,"context_line":"            help\u003d_(\"Filter migrations by type. Valid values are: \""},{"line_number":51,"context_line":"                \"evacuation, live-migration, migration (cold), resize.\")"},{"line_number":52,"context_line":"        )"},{"line_number":53,"context_line":"        parser.add_argument("},{"line_number":54,"context_line":"            \"--source-compute\","}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_f2032f75","line":51,"updated":"2020-07-21 21:35:04.000000000","message":"pep8: E128 continuation line under-indented for visual indent","commit_id":"1dbce2313a2305722834fd2d70e25999d4198351"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e59770d3890649e7bc94ababebd33cd8c5dcbe55","unresolved":false,"context_lines":[{"line_number":150,"context_line":"            migration_type\u003dparsed_args.migration_type,"},{"line_number":151,"context_line":"            source_compute\u003dparsed_args.source_compute,"},{"line_number":152,"context_line":"            user_id\u003dparsed_args.user_id, project_id\u003dparsed_args.project_id"},{"line_number":153,"context_line":"            "},{"line_number":154,"context_line":"        )"},{"line_number":155,"context_line":"        table \u003d (column_headers,"},{"line_number":156,"context_line":"                 (utils.get_item_properties("}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_d208ab92","line":153,"updated":"2020-07-21 21:35:04.000000000","message":"pep8: W293 blank line contains whitespace","commit_id":"1dbce2313a2305722834fd2d70e25999d4198351"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e59770d3890649e7bc94ababebd33cd8c5dcbe55","unresolved":false,"context_lines":[{"line_number":157,"context_line":"                     s, columns,"},{"line_number":158,"context_line":"                 ) for s in data))"},{"line_number":159,"context_line":"        return table"},{"line_number":160,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"bf51134e_32fe476b","line":160,"updated":"2020-07-21 21:35:04.000000000","message":"pep8: W391 blank line at end of file","commit_id":"1dbce2313a2305722834fd2d70e25999d4198351"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"ca7782a3ac4d4051747076b298bd256d295eb03b","unresolved":false,"context_lines":[{"line_number":59,"context_line":"            \"--marker\","},{"line_number":60,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","},{"line_number":61,"context_line":"            help\u003d_(\"The last migration of the previous page;\""},{"line_number":62,"context_line":"                    \"displays list of migrations after \\\"marker\\\".\""},{"line_number":63,"context_line":"                    \"(Supported by API versions \u00272.59\u0027 - \u00272.latest\u0027)\")"},{"line_number":64,"context_line":"        )"},{"line_number":65,"context_line":"        parser.add_argument("}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_3f6b721f","line":62,"range":{"start_line":62,"start_character":20,"end_line":62,"end_character":67},"updated":"2020-07-24 12:42:03.000000000","message":"Why not using single quotes to avoid escape double quotes?","commit_id":"1a35d1e03caa0a893742be9e8b80d70decd54336"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    def get_parser(self, prog_name):"},{"line_number":30,"context_line":"        parser \u003d super(ListMigration, self).get_parser(prog_name)"},{"line_number":31,"context_line":"        parser.add_argument("},{"line_number":32,"context_line":"            \"--instance-uuid\","},{"line_number":33,"context_line":"            metavar\u003d\"\u003cinstance_uuid\u003e\","},{"line_number":34,"context_line":"            help\u003d_(\"Fetch migrations for the given instance.\")"},{"line_number":35,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_6396b7f2","line":32,"range":{"start_line":32,"start_character":15,"end_line":32,"end_character":28},"updated":"2020-08-06 03:55:48.000000000","message":"I understand that the API returns an instance UUID but should this be --server ?\n\nFor example, \"openstack port list\" has the --server argument.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":31733,"name":"jayaditya gupta","email":"jayadityagupta11@gmail.com","username":"jgupta"},"change_message_id":"79cfd6d79790bcbe0f364d245dd402ed5a7551f0","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    def get_parser(self, prog_name):"},{"line_number":30,"context_line":"        parser \u003d super(ListMigration, self).get_parser(prog_name)"},{"line_number":31,"context_line":"        parser.add_argument("},{"line_number":32,"context_line":"            \"--instance-uuid\","},{"line_number":33,"context_line":"            metavar\u003d\"\u003cinstance_uuid\u003e\","},{"line_number":34,"context_line":"            help\u003d_(\"Fetch migrations for the given instance.\")"},{"line_number":35,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_f97886c4","line":32,"range":{"start_line":32,"start_character":15,"end_line":32,"end_character":28},"in_reply_to":"9f560f44_6396b7f2","updated":"2020-08-10 09:15:14.000000000","message":"I am using same options provided by \"nova migration-list\" to keep things consistent.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"0ef09c2f48c63035593b0f698a606356f5f25c07","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    def get_parser(self, prog_name):"},{"line_number":30,"context_line":"        parser \u003d super(ListMigration, self).get_parser(prog_name)"},{"line_number":31,"context_line":"        parser.add_argument("},{"line_number":32,"context_line":"            \"--instance-uuid\","},{"line_number":33,"context_line":"            metavar\u003d\"\u003cinstance_uuid\u003e\","},{"line_number":34,"context_line":"            help\u003d_(\"Fetch migrations for the given instance.\")"},{"line_number":35,"context_line":"        )"}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_178d3431","line":32,"range":{"start_line":32,"start_character":15,"end_line":32,"end_character":28},"in_reply_to":"9f560f44_f97886c4","updated":"2020-08-11 09:09:06.000000000","message":"Within OSC, the term \"server\" is more common, so we should go for consistency with that, not with nova cli.\n\nAlso we should allow to specify the server by name like mostly everywhere else, not only by id.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":44,"context_line":"            help\u003d_(\"Fetch migrations for the given status.\")"},{"line_number":45,"context_line":"        )"},{"line_number":46,"context_line":"        parser.add_argument("},{"line_number":47,"context_line":"            \"--migration-type\","},{"line_number":48,"context_line":"            metavar\u003d\"\u003cmigration_type\u003e\","},{"line_number":49,"context_line":"            help\u003d_(\"Filter migrations by type. Valid values are: \""},{"line_number":50,"context_line":"                   \"evacuation, live-migration, migration (cold), resize.\""}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_439bb316","line":47,"range":{"start_line":47,"start_character":13,"end_line":47,"end_character":29},"updated":"2020-08-06 03:55:48.000000000","message":"For consistency with the column header, should this be --type ?","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":31733,"name":"jayaditya gupta","email":"jayadityagupta11@gmail.com","username":"jgupta"},"change_message_id":"79cfd6d79790bcbe0f364d245dd402ed5a7551f0","unresolved":false,"context_lines":[{"line_number":44,"context_line":"            help\u003d_(\"Fetch migrations for the given status.\")"},{"line_number":45,"context_line":"        )"},{"line_number":46,"context_line":"        parser.add_argument("},{"line_number":47,"context_line":"            \"--migration-type\","},{"line_number":48,"context_line":"            metavar\u003d\"\u003cmigration_type\u003e\","},{"line_number":49,"context_line":"            help\u003d_(\"Filter migrations by type. Valid values are: \""},{"line_number":50,"context_line":"                   \"evacuation, live-migration, migration (cold), resize.\""}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_f961a606","line":47,"range":{"start_line":47,"start_character":13,"end_line":47,"end_character":29},"in_reply_to":"9f560f44_439bb316","updated":"2020-08-10 09:15:14.000000000","message":"nova help migration-list gives same options. That\u0027s why using them","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":89,"context_line":"                    \"versions \u00272.66\u0027 - \u00272.latest\u0027)\")"},{"line_number":90,"context_line":"        )"},{"line_number":91,"context_line":"        parser.add_argument("},{"line_number":92,"context_line":"            \"--project-id\","},{"line_number":93,"context_line":"            metavar\u003d\"\u003cproject_id\u003e\","},{"line_number":94,"context_line":"            help\u003d_(\"Filter the migrations by the given project ID.\""},{"line_number":95,"context_line":"                    \"(Supported by API versions \u00272.80\u0027 -\""}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_a39c4f0e","line":92,"range":{"start_line":92,"start_character":13,"end_line":92,"end_character":25},"updated":"2020-08-06 03:55:48.000000000","message":"In the same spirit as --instance-uuid, like other commands this should be --project.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"0ef09c2f48c63035593b0f698a606356f5f25c07","unresolved":false,"context_lines":[{"line_number":89,"context_line":"                    \"versions \u00272.66\u0027 - \u00272.latest\u0027)\")"},{"line_number":90,"context_line":"        )"},{"line_number":91,"context_line":"        parser.add_argument("},{"line_number":92,"context_line":"            \"--project-id\","},{"line_number":93,"context_line":"            metavar\u003d\"\u003cproject_id\u003e\","},{"line_number":94,"context_line":"            help\u003d_(\"Filter the migrations by the given project ID.\""},{"line_number":95,"context_line":"                    \"(Supported by API versions \u00272.80\u0027 -\""}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_1792544c","line":92,"range":{"start_line":92,"start_character":13,"end_line":92,"end_character":25},"in_reply_to":"9f560f44_a39c4f0e","updated":"2020-08-11 09:09:06.000000000","message":"Agreed, and like with --server, allow to specify both by name and id.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":96,"context_line":"                    \"\u00272.latest\u0027)\")"},{"line_number":97,"context_line":"        )"},{"line_number":98,"context_line":"        parser.add_argument("},{"line_number":99,"context_line":"            \"--user-id\","},{"line_number":100,"context_line":"            metavar\u003d\"\u003cuser_id\u003e\","},{"line_number":101,"context_line":"            help\u003d_(\"Filter the migrations by the given user ID.\""},{"line_number":102,"context_line":"                    \"(Supported by API versions \u00272.80\u0027 - \""}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_038d3bd2","line":99,"range":{"start_line":99,"start_character":13,"end_line":99,"end_character":22},"updated":"2020-08-06 03:55:48.000000000","message":"Like other identifiers, this should be --user.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":107,"context_line":"    def take_action(self, parsed_args):"},{"line_number":108,"context_line":"        compute_client \u003d self.app.client_manager.compute"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"        column_headers \u003d ["},{"line_number":111,"context_line":"            \"Id\","},{"line_number":112,"context_line":"            \"UUID\","},{"line_number":113,"context_line":"            \"Source Node\","}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_234ddfc3","line":110,"range":{"start_line":110,"start_character":8,"end_line":110,"end_character":22},"updated":"2020-08-06 03:55:48.000000000","message":"Would it be a good idea to move some of these columns under a --long or a \"show\" command ?\n\nWhen looking at our live migrations, this number of columns makes it guaranteed that the default pretty table will wrap even on a wide screen (we have long hostnames) and it hurts readability.\nThere\u0027s always \"-f yaml\" and \"-f json\" but pretty table is the default.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        column_headers \u003d ["},{"line_number":111,"context_line":"            \"Id\","},{"line_number":112,"context_line":"            \"UUID\","},{"line_number":113,"context_line":"            \"Source Node\","},{"line_number":114,"context_line":"            \"Dest Node\","},{"line_number":115,"context_line":"            \"Source Compute\","},{"line_number":116,"context_line":"            \"Dest Compute\","},{"line_number":117,"context_line":"            \"Dest Host\","},{"line_number":118,"context_line":"            \"Status\","},{"line_number":119,"context_line":"            \"Instance UUID\","}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_e3dbe7e9","line":116,"range":{"start_line":113,"start_character":0,"end_line":116,"end_character":27},"updated":"2020-08-06 03:55:48.000000000","message":"In our case, \"Source node\" is identical to \"Source compute\" and \"Dest Node\" is identical to \"Dest Compute\". Are they ever different ? Do we need to include both ?","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":31733,"name":"jayaditya gupta","email":"jayadityagupta11@gmail.com","username":"jgupta"},"change_message_id":"79cfd6d79790bcbe0f364d245dd402ed5a7551f0","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        column_headers \u003d ["},{"line_number":111,"context_line":"            \"Id\","},{"line_number":112,"context_line":"            \"UUID\","},{"line_number":113,"context_line":"            \"Source Node\","},{"line_number":114,"context_line":"            \"Dest Node\","},{"line_number":115,"context_line":"            \"Source Compute\","},{"line_number":116,"context_line":"            \"Dest Compute\","},{"line_number":117,"context_line":"            \"Dest Host\","},{"line_number":118,"context_line":"            \"Status\","},{"line_number":119,"context_line":"            \"Instance UUID\","}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_3d2dc994","line":116,"range":{"start_line":113,"start_character":0,"end_line":116,"end_character":27},"in_reply_to":"9f560f44_e3dbe7e9","updated":"2020-08-10 09:15:14.000000000","message":"I am not sure either about the difference. Again nova migration-list commands has same columns so incorporating them.\n\nThough API does give \"source_compute\", \"source_node\" . In my small test VM I have not seen the difference but maybe there is.\n\nDocs also haven\u0027t explained this properly : https://docs.openstack.org/api-ref/compute/?expanded\u003dlist-migrations-detail","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":2033,"name":"Belmiro Moreira","email":"moreira.belmiro.email.lists@gmail.com","username":"moreira-belmiro-email-lists"},"change_message_id":"7a256bc5e6f8778b189be6e7d56454a82407b442","unresolved":false,"context_lines":[{"line_number":110,"context_line":"        column_headers \u003d ["},{"line_number":111,"context_line":"            \"Id\","},{"line_number":112,"context_line":"            \"UUID\","},{"line_number":113,"context_line":"            \"Source Node\","},{"line_number":114,"context_line":"            \"Dest Node\","},{"line_number":115,"context_line":"            \"Source Compute\","},{"line_number":116,"context_line":"            \"Dest Compute\","},{"line_number":117,"context_line":"            \"Dest Host\","},{"line_number":118,"context_line":"            \"Status\","},{"line_number":119,"context_line":"            \"Instance UUID\","}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_4aca561a","line":116,"range":{"start_line":113,"start_character":0,"end_line":116,"end_character":27},"in_reply_to":"9f560f44_e3dbe7e9","updated":"2020-08-12 10:06:05.000000000","message":"Yes, there are differences. Compute/Node can be different depending in the driver that\u0027s been used.\nBut I agree with David. Source/Dest Compute can move under a \u0027--long\u0027 flag and we keep the Node.","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":116,"context_line":"            \"Dest Compute\","},{"line_number":117,"context_line":"            \"Dest Host\","},{"line_number":118,"context_line":"            \"Status\","},{"line_number":119,"context_line":"            \"Instance UUID\","},{"line_number":120,"context_line":"            \"Old Flavor\","},{"line_number":121,"context_line":"            \"New Flavor\","},{"line_number":122,"context_line":"            \"Created At\","}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_63bfd760","line":119,"range":{"start_line":119,"start_character":13,"end_line":119,"end_character":26},"updated":"2020-08-06 03:55:48.000000000","message":"For consistency this should be \"Server\"","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":117,"context_line":"            \"Dest Host\","},{"line_number":118,"context_line":"            \"Status\","},{"line_number":119,"context_line":"            \"Instance UUID\","},{"line_number":120,"context_line":"            \"Old Flavor\","},{"line_number":121,"context_line":"            \"New Flavor\","},{"line_number":122,"context_line":"            \"Created At\","},{"line_number":123,"context_line":"            \"Updated At\","},{"line_number":124,"context_line":"            \"Type\","}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_43f61365","line":121,"range":{"start_line":120,"start_character":0,"end_line":121,"end_character":25},"updated":"2020-08-06 03:55:48.000000000","message":"These are showing up as the flavor IDs -- I guess it would be additional API calls to resolve these flavor IDs to names ?","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":122,"context_line":"            \"Created At\","},{"line_number":123,"context_line":"            \"Updated At\","},{"line_number":124,"context_line":"            \"Type\","},{"line_number":125,"context_line":"            \"Project ID\","},{"line_number":126,"context_line":"            \"User ID\""},{"line_number":127,"context_line":"        ]"},{"line_number":128,"context_line":"        columns \u003d ["}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_43c453f5","line":125,"range":{"start_line":125,"start_character":13,"end_line":125,"end_character":23},"updated":"2020-08-06 03:55:48.000000000","message":"Project","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":123,"context_line":"            \"Updated At\","},{"line_number":124,"context_line":"            \"Type\","},{"line_number":125,"context_line":"            \"Project ID\","},{"line_number":126,"context_line":"            \"User ID\""},{"line_number":127,"context_line":"        ]"},{"line_number":128,"context_line":"        columns \u003d ["},{"line_number":129,"context_line":"            \"id\","}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_a3b56f80","line":126,"range":{"start_line":126,"start_character":13,"end_line":126,"end_character":20},"updated":"2020-08-06 03:55:48.000000000","message":"User","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":9061,"name":"David Moreau Simard","email":"moi@dmsimard.com","username":"dmsimard"},"change_message_id":"2caa95a2f388137f4125b2ac23d77bf1cd65aa56","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        if parsed_args.user_id:"},{"line_number":183,"context_line":"            kwargs[\u0027user_id\u0027] \u003d parsed_args.user_id"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # prepare columns/column_headers"},{"line_number":186,"context_line":"        if compute_client.api_version \u003c api_versions.APIVersion(\u00272.23\u0027):"},{"line_number":187,"context_line":"            columns.remove(\"migration_type\")"},{"line_number":188,"context_line":"            column_headers.remove(\"Type\")"}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_23fb1f47","line":185,"updated":"2020-08-06 03:55:48.000000000","message":"Wanted to mention that these column removals based on API version work without issues on older versions of openstack \\o/","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"},{"author":{"_account_id":31733,"name":"jayaditya gupta","email":"jayadityagupta11@gmail.com","username":"jgupta"},"change_message_id":"79cfd6d79790bcbe0f364d245dd402ed5a7551f0","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        if parsed_args.user_id:"},{"line_number":183,"context_line":"            kwargs[\u0027user_id\u0027] \u003d parsed_args.user_id"},{"line_number":184,"context_line":""},{"line_number":185,"context_line":"        # prepare columns/column_headers"},{"line_number":186,"context_line":"        if compute_client.api_version \u003c api_versions.APIVersion(\u00272.23\u0027):"},{"line_number":187,"context_line":"            columns.remove(\"migration_type\")"},{"line_number":188,"context_line":"            column_headers.remove(\"Type\")"}],"source_content_type":"text/x-python","patch_set":8,"id":"9f560f44_3d52e913","line":185,"in_reply_to":"9f560f44_23fb1f47","updated":"2020-08-10 09:15:14.000000000","message":"but I was getting empty column results , that\u0027s why did it this way","commit_id":"27106cbf368a04cd9f0f72cf1ff28c838d8ea591"}],"openstackclient/compute/v2/server.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"e76c3ce16831c3d0a4dde45977ed55e8d404e26b","unresolved":false,"context_lines":[{"line_number":1788,"context_line":"            metavar\u003d\"\u003cstatus\u003e\","},{"line_number":1789,"context_line":"            help\u003d_(\"Fetch migrations for the given status.\")"},{"line_number":1790,"context_line":"        )"},{"line_number":1791,"context_line":"        parser.add_argument("},{"line_number":1792,"context_line":"            \"--migration-type\","},{"line_number":1793,"context_line":"            metavar\u003d\"\u003cmigration_type\u003e\","},{"line_number":1794,"context_line":"            help\u003d_(\"Filter migrations by type. Valid values are: \""},{"line_number":1795,"context_line":"                   \"evacuation, live-migration, migration (cold), resize.\""},{"line_number":1796,"context_line":"                   \"(Supported by API versions \u00272.23\u0027 - \u00272.latest\u0027)\")"},{"line_number":1797,"context_line":"        )"},{"line_number":1798,"context_line":"        parser.add_argument("},{"line_number":1799,"context_line":"            \"--source-compute\","},{"line_number":1800,"context_line":"            metavar\u003d\"\u003csource_compute\u003e\","},{"line_number":1801,"context_line":"            help\u003d_(\"Filter migrations by source compute host name.\")"},{"line_number":1802,"context_line":"        )"},{"line_number":1803,"context_line":"        parser.add_argument("},{"line_number":1804,"context_line":"            \"--marker\","},{"line_number":1805,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","}],"source_content_type":"text/x-python","patch_set":9,"id":"9f560f44_2d807bbf","line":1802,"range":{"start_line":1791,"start_character":0,"end_line":1802,"end_character":9},"updated":"2020-10-06 10:22:27.000000000","message":"These are missing from https://review.opendev.org/#/c/675304/. Want to do a follow-up on that to add them?","commit_id":"e4e0f01cff418f6d9d3ce801444332a5cdf6cfc3"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"b724c3784e6f81d0cc9f180dc83c11262dcf9e97","unresolved":false,"context_lines":[{"line_number":1788,"context_line":"            metavar\u003d\"\u003cstatus\u003e\","},{"line_number":1789,"context_line":"            help\u003d_(\"Fetch migrations for the given status.\")"},{"line_number":1790,"context_line":"        )"},{"line_number":1791,"context_line":"        parser.add_argument("},{"line_number":1792,"context_line":"            \"--migration-type\","},{"line_number":1793,"context_line":"            metavar\u003d\"\u003cmigration_type\u003e\","},{"line_number":1794,"context_line":"            help\u003d_(\"Filter migrations by type. Valid values are: \""},{"line_number":1795,"context_line":"                   \"evacuation, live-migration, migration (cold), resize.\""},{"line_number":1796,"context_line":"                   \"(Supported by API versions \u00272.23\u0027 - \u00272.latest\u0027)\")"},{"line_number":1797,"context_line":"        )"},{"line_number":1798,"context_line":"        parser.add_argument("},{"line_number":1799,"context_line":"            \"--source-compute\","},{"line_number":1800,"context_line":"            metavar\u003d\"\u003csource_compute\u003e\","},{"line_number":1801,"context_line":"            help\u003d_(\"Filter migrations by source compute host name.\")"},{"line_number":1802,"context_line":"        )"},{"line_number":1803,"context_line":"        parser.add_argument("},{"line_number":1804,"context_line":"            \"--marker\","},{"line_number":1805,"context_line":"            metavar\u003d\"\u003cmarker\u003e\","}],"source_content_type":"text/x-python","patch_set":9,"id":"9f560f44_c7b6ee89","line":1802,"range":{"start_line":1791,"start_character":0,"end_line":1802,"end_character":9},"in_reply_to":"9f560f44_2d807bbf","updated":"2020-10-07 07:10:49.000000000","message":"Yeah, this can be done by follow-up https://review.opendev.org/#/c/675304, jayaditya do it or I do is ok ^_^","commit_id":"e4e0f01cff418f6d9d3ce801444332a5cdf6cfc3"}],"openstackclient/tests/unit/compute/v2/test_migration_list.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":10,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import argparse"},{"line_number":14,"context_line":"import collections"},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"import getpass"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_d061d93b","line":13,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027argparse\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":11,"context_line":"#    under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import argparse"},{"line_number":14,"context_line":"import collections"},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"import getpass"},{"line_number":17,"context_line":"from unittest import mock"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_b066e550","line":14,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027collections\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"import argparse"},{"line_number":14,"context_line":"import collections"},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"import getpass"},{"line_number":17,"context_line":"from unittest import mock"},{"line_number":18,"context_line":"from unittest.mock import call"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_105cf17c","line":15,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027copy\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":13,"context_line":"import argparse"},{"line_number":14,"context_line":"import collections"},{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"import getpass"},{"line_number":17,"context_line":"from unittest import mock"},{"line_number":18,"context_line":"from unittest.mock import call"},{"line_number":19,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_f0605d35","line":16,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027getpass\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":15,"context_line":"import copy"},{"line_number":16,"context_line":"import getpass"},{"line_number":17,"context_line":"from unittest import mock"},{"line_number":18,"context_line":"from unittest.mock import call"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from novaclient import api_versions"},{"line_number":21,"context_line":"from novaclient import base"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_5055c94f","line":18,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027unittest.mock.call\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":17,"context_line":"from unittest import mock"},{"line_number":18,"context_line":"from unittest.mock import call"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from novaclient import api_versions"},{"line_number":21,"context_line":"from novaclient import base"},{"line_number":22,"context_line":"from novaclient.v2 import migrations"},{"line_number":23,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_3052d568","line":20,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027novaclient.api_versions\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":18,"context_line":"from unittest.mock import call"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from novaclient import api_versions"},{"line_number":21,"context_line":"from novaclient import base"},{"line_number":22,"context_line":"from novaclient.v2 import migrations"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from openstack import exceptions as sdk_exceptions"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_904f61bc","line":21,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027novaclient.base\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":19,"context_line":""},{"line_number":20,"context_line":"from novaclient import api_versions"},{"line_number":21,"context_line":"from novaclient import base"},{"line_number":22,"context_line":"from novaclient.v2 import migrations"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":25,"context_line":"from osc_lib import exceptions"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_704c4dc7","line":22,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027novaclient.v2.migrations\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from novaclient import base"},{"line_number":22,"context_line":"from novaclient.v2 import migrations"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":25,"context_line":"from osc_lib import exceptions"},{"line_number":26,"context_line":"from osc_lib import utils as common_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_d0c6f916","line":24,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027openstack.exceptions as sdk_exceptions\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":21,"context_line":"from novaclient import base"},{"line_number":22,"context_line":"from novaclient.v2 import migrations"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":25,"context_line":"from osc_lib import exceptions"},{"line_number":26,"context_line":"from osc_lib import utils as common_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_b0cb851c","line":24,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: I202 Additional newline in a group of imports. \u0027from openstack import exceptions\u0027 is identified as Third Party and \u0027from novaclient.v2 import migrations\u0027 is identified as Third Party.","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":22,"context_line":"from novaclient.v2 import migrations"},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":25,"context_line":"from osc_lib import exceptions"},{"line_number":26,"context_line":"from osc_lib import utils as common_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"},{"line_number":28,"context_line":"import six"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_10d191ce","line":25,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027osc_lib.exceptions\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":25,"context_line":"from osc_lib import exceptions"},{"line_number":26,"context_line":"from osc_lib import utils as common_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"},{"line_number":28,"context_line":"import six"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_f0d5fdba","line":26,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027osc_lib.utils as common_utils\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":24,"context_line":"from openstack import exceptions as sdk_exceptions"},{"line_number":25,"context_line":"from osc_lib import exceptions"},{"line_number":26,"context_line":"from osc_lib import utils as common_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"},{"line_number":28,"context_line":"import six"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"from openstackclient.compute.v2 import server"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_50dae9e8","line":27,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027oslo_utils.timeutils\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":25,"context_line":"from osc_lib import exceptions"},{"line_number":26,"context_line":"from osc_lib import utils as common_utils"},{"line_number":27,"context_line":"from oslo_utils import timeutils"},{"line_number":28,"context_line":"import six"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"from openstackclient.compute.v2 import server"},{"line_number":31,"context_line":"from openstackclient.compute.v2 import migration_list"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_30d775c2","line":28,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027six\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":27,"context_line":"from oslo_utils import timeutils"},{"line_number":28,"context_line":"import six"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"from openstackclient.compute.v2 import server"},{"line_number":31,"context_line":"from openstackclient.compute.v2 import migration_list"},{"line_number":32,"context_line":"from openstackclient.tests.unit.compute.v2 import fakes as compute_fakes"},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_90e481ae","line":30,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F401 \u0027openstackclient.compute.v2.server\u0027 imported but unused","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":28,"context_line":"import six"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"from openstackclient.compute.v2 import server"},{"line_number":31,"context_line":"from openstackclient.compute.v2 import migration_list"},{"line_number":32,"context_line":"from openstackclient.tests.unit.compute.v2 import fakes as compute_fakes"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_70e1ed9a","line":31,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: H306: imports not in alphabetical order (openstackclient.compute.v2.server, openstackclient.compute.v2.migration_list)","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":75,"context_line":"        #     \u0027user_id\u0027: None"},{"line_number":76,"context_line":"        # }"},{"line_number":77,"context_line":"        self.cmd \u003d migration_list.ListMigration(self.app, None)"},{"line_number":78,"context_line":"    "},{"line_number":79,"context_line":"    @mock.patch(\u0027novaclient.v2.migrations.list\u0027)"},{"line_number":80,"context_line":"    def test_migration_list(self, mock_ml):"},{"line_number":81,"context_line":"        arglist \u003d []"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_d0ef1985","line":78,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: W293 blank line contains whitespace","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"e90b8b23feba08ab7b54cf939d9dc32b3c404c55","unresolved":false,"context_lines":[{"line_number":82,"context_line":"        verifylist \u003d []"},{"line_number":83,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verifylist)"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"        data \u003d migration_list.ListMigration.take_action(self, parsed_args)"}],"source_content_type":"text/x-python","patch_set":6,"id":"9f560f44_b0f4255d","line":85,"updated":"2020-07-28 12:24:12.000000000","message":"pep8: F841 local variable \u0027data\u0027 is assigned to but never used","commit_id":"2ba65c54ffeae94a84f934806dec36b1ab4609a2"}]}