)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"d1073d9b0c8f146268fd1f84100193d2009e00a1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Provides live migration abort feature in openstackclient"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Closes Bug : #2007489"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I0ff520ccfdf2de52c427affad7bef4554c86a06f"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"df33271e_7ff58b56","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":21},"updated":"2020-04-02 13:11:57.000000000","message":"python-openstackclient uses storyboard, so the format to get this to link properly would be:\n\nStory: 2007489\nTask: 39210","commit_id":"624d5deec530dbbbd0d1807c6cfa2f9e78865eb1"}],"openstackclient/compute/v2/server.py":[{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"fe990b05c5124f0a6ed2275690244105134ba820","unresolved":false,"context_lines":[{"line_number":1663,"context_line":"            help\u003d_(\u0027Perform a block live migration\u0027),"},{"line_number":1664,"context_line":"        )"},{"line_number":1665,"context_line":"        migration_group.add_argument("},{"line_number":1666,"context_line":"            \u0027--live-migration-abort\u0027,"},{"line_number":1667,"context_line":"            dest\u003d\u0027abort_migration\u0027,"},{"line_number":1668,"context_line":"            help\u003d_(\u0027Abort an on-going live migration. \u0027"},{"line_number":1669,"context_line":"                   \u0027Usage --live-migration-abort \u003cmigration_ID\u003e.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_9f595490","line":1666,"range":{"start_line":1666,"start_character":15,"end_line":1666,"end_character":35},"updated":"2020-07-17 11:29:58.000000000","message":"I wonder if it would be better to be a more active term:\n\n--abort-live-migration","commit_id":"fd6db2cf8a6fa3a2a15da599392e468d02b52b42"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"b15485b8488986c9cdb41a1ba1401bfbefad9506","unresolved":false,"context_lines":[{"line_number":1663,"context_line":"            help\u003d_(\u0027Perform a block live migration\u0027),"},{"line_number":1664,"context_line":"        )"},{"line_number":1665,"context_line":"        migration_group.add_argument("},{"line_number":1666,"context_line":"            \u0027--live-migration-abort\u0027,"},{"line_number":1667,"context_line":"            dest\u003d\u0027abort_migration\u0027,"},{"line_number":1668,"context_line":"            help\u003d_(\u0027Abort an on-going live migration. \u0027"},{"line_number":1669,"context_line":"                   \u0027Usage --live-migration-abort \u003cmigration_ID\u003e.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_b77b86f2","line":1666,"range":{"start_line":1666,"start_character":15,"end_line":1666,"end_character":35},"in_reply_to":"bf51134e_7b39dea1","updated":"2020-07-20 12:03:18.000000000","message":"IMO if these are related to nova (I\u0027m not sure but I think yes) then it could be worth to stay consistent with the nova terminology, and in a second time deprecate the name in the both place to stay sync and reflect the \u0027more active term\u0027, but personally I\u0027m ok with the both approach (move now ore move later).\n\nlet\u0027s Sean give its feedback.","commit_id":"fd6db2cf8a6fa3a2a15da599392e468d02b52b42"},{"author":{"_account_id":31733,"name":"jayaditya gupta","email":"jayadityagupta11@gmail.com","username":"jgupta"},"change_message_id":"4d04b642da82a368d555e5c9e64c8a80c5257ff4","unresolved":false,"context_lines":[{"line_number":1663,"context_line":"            help\u003d_(\u0027Perform a block live migration\u0027),"},{"line_number":1664,"context_line":"        )"},{"line_number":1665,"context_line":"        migration_group.add_argument("},{"line_number":1666,"context_line":"            \u0027--live-migration-abort\u0027,"},{"line_number":1667,"context_line":"            dest\u003d\u0027abort_migration\u0027,"},{"line_number":1668,"context_line":"            help\u003d_(\u0027Abort an on-going live migration. \u0027"},{"line_number":1669,"context_line":"                   \u0027Usage --live-migration-abort \u003cmigration_ID\u003e.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_7b39dea1","line":1666,"range":{"start_line":1666,"start_character":15,"end_line":1666,"end_character":35},"in_reply_to":"bf51134e_9f595490","updated":"2020-07-20 07:58:50.000000000","message":"I only took this word mainly because it was similar in nova. But yeah \"--abort-live-migration\" is better to read IMO. \n\nShould I change it and submit again ?","commit_id":"fd6db2cf8a6fa3a2a15da599392e468d02b52b42"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c698894b24acac6f971573f84323a6a8c065a9a2","unresolved":false,"context_lines":[{"line_number":1808,"context_line":"            help\u003d_(\"Migration id\"),"},{"line_number":1809,"context_line":"        )"},{"line_number":1810,"context_line":"        return parser"},{"line_number":1811,"context_line":"    "},{"line_number":1812,"context_line":"    def take_action(self, parsed_args):"},{"line_number":1813,"context_line":"        compute_client \u003d self.app.client_manager.compute"},{"line_number":1814,"context_line":"        server \u003d utils.find_resource("}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_5624ae0e","line":1811,"updated":"2020-09-15 15:38:41.000000000","message":"pep8: W293 blank line contains whitespace","commit_id":"67ed7eb31b7225cafa40151080c2a08b3e25a5c5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c698894b24acac6f971573f84323a6a8c065a9a2","unresolved":false,"context_lines":[{"line_number":1812,"context_line":"    def take_action(self, parsed_args):"},{"line_number":1813,"context_line":"        compute_client \u003d self.app.client_manager.compute"},{"line_number":1814,"context_line":"        server \u003d utils.find_resource("},{"line_number":1815,"context_line":"                    compute_client.servers,"},{"line_number":1816,"context_line":"                    parsed_args.server,"},{"line_number":1817,"context_line":"                    )"},{"line_number":1818,"context_line":"        migration \u003d parsed_args.migration_id"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_36213afe","line":1815,"updated":"2020-09-15 15:38:41.000000000","message":"pep8: E126 continuation line over-indented for hanging indent","commit_id":"67ed7eb31b7225cafa40151080c2a08b3e25a5c5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c698894b24acac6f971573f84323a6a8c065a9a2","unresolved":false,"context_lines":[{"line_number":1814,"context_line":"        server \u003d utils.find_resource("},{"line_number":1815,"context_line":"                    compute_client.servers,"},{"line_number":1816,"context_line":"                    parsed_args.server,"},{"line_number":1817,"context_line":"                    )"},{"line_number":1818,"context_line":"        migration \u003d parsed_args.migration_id"},{"line_number":1819,"context_line":"    # There is no body content for the response of a successful operation."},{"line_number":1820,"context_line":"        compute_client.server_migrations.live_migration_abort(server,migration)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_961e46b9","line":1817,"updated":"2020-09-15 15:38:41.000000000","message":"pep8: E123 closing bracket does not match indentation of opening bracket\u0027s line","commit_id":"67ed7eb31b7225cafa40151080c2a08b3e25a5c5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c698894b24acac6f971573f84323a6a8c065a9a2","unresolved":false,"context_lines":[{"line_number":1817,"context_line":"                    )"},{"line_number":1818,"context_line":"        migration \u003d parsed_args.migration_id"},{"line_number":1819,"context_line":"    # There is no body content for the response of a successful operation."},{"line_number":1820,"context_line":"        compute_client.server_migrations.live_migration_abort(server,migration)"},{"line_number":1821,"context_line":""},{"line_number":1822,"context_line":""},{"line_number":1823,"context_line":"class RebootServer(command.Command):"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_761bb2c9","line":1820,"updated":"2020-09-15 15:38:41.000000000","message":"pep8: E231 missing whitespace after \u0027,\u0027","commit_id":"67ed7eb31b7225cafa40151080c2a08b3e25a5c5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"089745327b4171d8d57755ffbe4b1ba08d456ae4","unresolved":false,"context_lines":[{"line_number":1793,"context_line":"class AbortMigration(command.Command):"},{"line_number":1794,"context_line":"    _description \u003d _(\"\"\"Abort on-going live migration"},{"line_number":1795,"context_line":""},{"line_number":1796,"context_line":"Usage : openstack server migration abort \u003cserver-id\u003e \u003cmigration-id\u003e\"\"\")"},{"line_number":1797,"context_line":""},{"line_number":1798,"context_line":"    def get_parser(self, prog_name):"},{"line_number":1799,"context_line":"        parser \u003d super(AbortMigration, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_30c9f0ab","line":1796,"range":{"start_line":1796,"start_character":0,"end_line":1796,"end_character":67},"updated":"2020-10-06 10:49:30.000000000","message":"You don\u0027t need this. Can you drop it? It\u0027s handled automatically by argparse\n\nAlso, this requires API microversion \u003e\u003d 2.24. Can you add a note here like so:\n\n  This command requires ``--os-compute-api-version`` 2.24 or greater.\n\nAlong with a check below?","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"928177c8b949a75ff8530dfdf52065d62bbe2987","unresolved":false,"context_lines":[{"line_number":1793,"context_line":"class AbortMigration(command.Command):"},{"line_number":1794,"context_line":"    _description \u003d _(\"\"\"Abort on-going live migration"},{"line_number":1795,"context_line":""},{"line_number":1796,"context_line":"Usage : openstack server migration abort \u003cserver-id\u003e \u003cmigration-id\u003e\"\"\")"},{"line_number":1797,"context_line":""},{"line_number":1798,"context_line":"    def get_parser(self, prog_name):"},{"line_number":1799,"context_line":"        parser \u003d super(AbortMigration, self).get_parser(prog_name)"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_a0a6e952","line":1796,"range":{"start_line":1796,"start_character":0,"end_line":1796,"end_character":67},"in_reply_to":"9f560f44_30c9f0ab","updated":"2020-10-13 10:35:49.000000000","message":"Done","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"089745327b4171d8d57755ffbe4b1ba08d456ae4","unresolved":false,"context_lines":[{"line_number":1796,"context_line":"Usage : openstack server migration abort \u003cserver-id\u003e \u003cmigration-id\u003e\"\"\")"},{"line_number":1797,"context_line":""},{"line_number":1798,"context_line":"    def get_parser(self, prog_name):"},{"line_number":1799,"context_line":"        parser \u003d super(AbortMigration, self).get_parser(prog_name)"},{"line_number":1800,"context_line":"        parser.add_argument("},{"line_number":1801,"context_line":"            \u0027server\u0027,"},{"line_number":1802,"context_line":"            metavar\u003d\u0027\u003cserver\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_f033f8c2","line":1799,"range":{"start_line":1799,"start_character":23,"end_line":1799,"end_character":43},"updated":"2020-10-06 10:49:30.000000000","message":"nit: not needed in Python 3","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"928177c8b949a75ff8530dfdf52065d62bbe2987","unresolved":false,"context_lines":[{"line_number":1796,"context_line":"Usage : openstack server migration abort \u003cserver-id\u003e \u003cmigration-id\u003e\"\"\")"},{"line_number":1797,"context_line":""},{"line_number":1798,"context_line":"    def get_parser(self, prog_name):"},{"line_number":1799,"context_line":"        parser \u003d super(AbortMigration, self).get_parser(prog_name)"},{"line_number":1800,"context_line":"        parser.add_argument("},{"line_number":1801,"context_line":"            \u0027server\u0027,"},{"line_number":1802,"context_line":"            metavar\u003d\u0027\u003cserver\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_e04aa180","line":1799,"range":{"start_line":1799,"start_character":23,"end_line":1799,"end_character":43},"in_reply_to":"9f560f44_f033f8c2","updated":"2020-10-13 10:35:49.000000000","message":"Done","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"089745327b4171d8d57755ffbe4b1ba08d456ae4","unresolved":false,"context_lines":[{"line_number":1804,"context_line":"        )"},{"line_number":1805,"context_line":"        parser.add_argument("},{"line_number":1806,"context_line":"            \u0027migration_id\u0027,"},{"line_number":1807,"context_line":"            metavar\u003d\u0027\u003cmigration_id\u003e\u0027,"},{"line_number":1808,"context_line":"            help\u003d_(\"Migration id\"),"},{"line_number":1809,"context_line":"        )"},{"line_number":1810,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_303e108b","line":1807,"range":{"start_line":1807,"start_character":31,"end_line":1807,"end_character":34},"updated":"2020-10-06 10:49:30.000000000","message":"drop","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"928177c8b949a75ff8530dfdf52065d62bbe2987","unresolved":false,"context_lines":[{"line_number":1804,"context_line":"        )"},{"line_number":1805,"context_line":"        parser.add_argument("},{"line_number":1806,"context_line":"            \u0027migration_id\u0027,"},{"line_number":1807,"context_line":"            metavar\u003d\u0027\u003cmigration_id\u003e\u0027,"},{"line_number":1808,"context_line":"            help\u003d_(\"Migration id\"),"},{"line_number":1809,"context_line":"        )"},{"line_number":1810,"context_line":"        return parser"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_80a1253d","line":1807,"range":{"start_line":1807,"start_character":31,"end_line":1807,"end_character":34},"in_reply_to":"9f560f44_303e108b","updated":"2020-10-13 10:35:49.000000000","message":"Done","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"089745327b4171d8d57755ffbe4b1ba08d456ae4","unresolved":false,"context_lines":[{"line_number":1805,"context_line":"        parser.add_argument("},{"line_number":1806,"context_line":"            \u0027migration_id\u0027,"},{"line_number":1807,"context_line":"            metavar\u003d\u0027\u003cmigration_id\u003e\u0027,"},{"line_number":1808,"context_line":"            help\u003d_(\"Migration id\"),"},{"line_number":1809,"context_line":"        )"},{"line_number":1810,"context_line":"        return parser"},{"line_number":1811,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_d036b4b3","line":1808,"range":{"start_line":1808,"start_character":30,"end_line":1808,"end_character":32},"updated":"2020-10-06 10:49:30.000000000","message":"ID","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"928177c8b949a75ff8530dfdf52065d62bbe2987","unresolved":false,"context_lines":[{"line_number":1805,"context_line":"        parser.add_argument("},{"line_number":1806,"context_line":"            \u0027migration_id\u0027,"},{"line_number":1807,"context_line":"            metavar\u003d\u0027\u003cmigration_id\u003e\u0027,"},{"line_number":1808,"context_line":"            help\u003d_(\"Migration id\"),"},{"line_number":1809,"context_line":"        )"},{"line_number":1810,"context_line":"        return parser"},{"line_number":1811,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_e0610113","line":1808,"range":{"start_line":1808,"start_character":30,"end_line":1808,"end_character":32},"in_reply_to":"9f560f44_d036b4b3","updated":"2020-10-13 10:35:49.000000000","message":"Done","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"089745327b4171d8d57755ffbe4b1ba08d456ae4","unresolved":false,"context_lines":[{"line_number":1816,"context_line":"            parsed_args.server,"},{"line_number":1817,"context_line":"        )"},{"line_number":1818,"context_line":"        migration \u003d parsed_args.migration_id"},{"line_number":1819,"context_line":"    # There is no body content for the response of a successful operation."},{"line_number":1820,"context_line":"        compute_client.server_migrations.live_migration_abort("},{"line_number":1821,"context_line":"            server, migration)"},{"line_number":1822,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_10414c0b","line":1819,"range":{"start_line":1819,"start_character":3,"end_line":1819,"end_character":4},"updated":"2020-10-06 10:49:30.000000000","message":"nit: missing indent\n\nAlso, there\u0027s no body content but is there a status code you can check?","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"928177c8b949a75ff8530dfdf52065d62bbe2987","unresolved":false,"context_lines":[{"line_number":1816,"context_line":"            parsed_args.server,"},{"line_number":1817,"context_line":"        )"},{"line_number":1818,"context_line":"        migration \u003d parsed_args.migration_id"},{"line_number":1819,"context_line":"    # There is no body content for the response of a successful operation."},{"line_number":1820,"context_line":"        compute_client.server_migrations.live_migration_abort("},{"line_number":1821,"context_line":"            server, migration)"},{"line_number":1822,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_807645d4","line":1819,"range":{"start_line":1819,"start_character":3,"end_line":1819,"end_character":4},"in_reply_to":"9f560f44_10414c0b","updated":"2020-10-13 10:35:49.000000000","message":"Done","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"}],"openstackclient/tests/unit/compute/v2/test_server.py":[{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c698894b24acac6f971573f84323a6a8c065a9a2","unresolved":false,"context_lines":[{"line_number":3531,"context_line":"        self.app.client_manager.compute.api_version \u003d api_versions.APIVersion("},{"line_number":3532,"context_line":"            \u00272.79\u0027)"},{"line_number":3533,"context_line":""},{"line_number":3534,"context_line":"    @mock.patch(\u0027novaclient.v2.server_migrations.live_migration_abort\u0027)    "},{"line_number":3535,"context_line":"    def test_abort_migration(self, mock_abort):"},{"line_number":3536,"context_line":"        arglist\u003d [(\u0027server\u0027, \u00271234\u0027),(\u0027migration_id\u0027, \u00272\u0027)]"},{"line_number":3537,"context_line":"        verify_args \u003d []"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_d6755e02","line":3534,"updated":"2020-09-15 15:38:41.000000000","message":"pep8: W291 trailing whitespace","commit_id":"67ed7eb31b7225cafa40151080c2a08b3e25a5c5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c698894b24acac6f971573f84323a6a8c065a9a2","unresolved":false,"context_lines":[{"line_number":3533,"context_line":""},{"line_number":3534,"context_line":"    @mock.patch(\u0027novaclient.v2.server_migrations.live_migration_abort\u0027)    "},{"line_number":3535,"context_line":"    def test_abort_migration(self, mock_abort):"},{"line_number":3536,"context_line":"        arglist\u003d [(\u0027server\u0027, \u00271234\u0027),(\u0027migration_id\u0027, \u00272\u0027)]"},{"line_number":3537,"context_line":"        verify_args \u003d []"},{"line_number":3538,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verify_args)"},{"line_number":3539,"context_line":"        result \u003d self.cmd.take_action(parsed_args)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_b67a6aed","line":3536,"updated":"2020-09-15 15:38:41.000000000","message":"pep8: E225 missing whitespace around operator","commit_id":"67ed7eb31b7225cafa40151080c2a08b3e25a5c5"},{"author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"tag":"autogenerated:zuul:check","change_message_id":"c698894b24acac6f971573f84323a6a8c065a9a2","unresolved":false,"context_lines":[{"line_number":3533,"context_line":""},{"line_number":3534,"context_line":"    @mock.patch(\u0027novaclient.v2.server_migrations.live_migration_abort\u0027)    "},{"line_number":3535,"context_line":"    def test_abort_migration(self, mock_abort):"},{"line_number":3536,"context_line":"        arglist\u003d [(\u0027server\u0027, \u00271234\u0027),(\u0027migration_id\u0027, \u00272\u0027)]"},{"line_number":3537,"context_line":"        verify_args \u003d []"},{"line_number":3538,"context_line":"        parsed_args \u003d self.check_parser(self.cmd, arglist, verify_args)"},{"line_number":3539,"context_line":"        result \u003d self.cmd.take_action(parsed_args)"}],"source_content_type":"text/x-python","patch_set":4,"id":"9f560f44_168076bf","line":3536,"updated":"2020-09-15 15:38:41.000000000","message":"pep8: E231 missing whitespace after \u0027,\u0027","commit_id":"67ed7eb31b7225cafa40151080c2a08b3e25a5c5"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"089745327b4171d8d57755ffbe4b1ba08d456ae4","unresolved":false,"context_lines":[{"line_number":3538,"context_line":"        result \u003d self.servers_mock.server_migrations.live_migration_abort("},{"line_number":3539,"context_line":"            server_id, abort_migration"},{"line_number":3540,"context_line":"        )"},{"line_number":3541,"context_line":"        self.assertIsNotNone(result.id)"},{"line_number":3542,"context_line":""},{"line_number":3543,"context_line":""},{"line_number":3544,"context_line":"class TestServerPause(TestServer):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_d05fd4e1","line":3541,"updated":"2020-10-06 10:49:30.000000000","message":"Again, you\u0027re checking the fake client, not the command. Look at the example below for how to test the command itself","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"928177c8b949a75ff8530dfdf52065d62bbe2987","unresolved":false,"context_lines":[{"line_number":3538,"context_line":"        result \u003d self.servers_mock.server_migrations.live_migration_abort("},{"line_number":3539,"context_line":"            server_id, abort_migration"},{"line_number":3540,"context_line":"        )"},{"line_number":3541,"context_line":"        self.assertIsNotNone(result.id)"},{"line_number":3542,"context_line":""},{"line_number":3543,"context_line":""},{"line_number":3544,"context_line":"class TestServerPause(TestServer):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9f560f44_c3d4c721","line":3541,"in_reply_to":"9f560f44_d05fd4e1","updated":"2020-10-13 10:35:49.000000000","message":"Done","commit_id":"2a14c02c646d57679791ea280d20bfb6a839e9c7"}]}