)]}'
{"nova/cmd/manage.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"5e873c7472a1bd81fce3b610d613bd15990f382a","unresolved":false,"context_lines":[{"line_number":1008,"context_line":"                        \u0027migrated\u0027) % {\u0027total\u0027: found,"},{"line_number":1009,"context_line":"                                       \u0027meth\u0027: migration_meth.__name__,"},{"line_number":1010,"context_line":"                                       \u0027done\u0027: done})"},{"line_number":1011,"context_line":"            if max_count is not None:"},{"line_number":1012,"context_line":"                ran +\u003d done"},{"line_number":1013,"context_line":"                if ran \u003e\u003d max_count:"},{"line_number":1014,"context_line":"                    break"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_86bae221","line":1011,"updated":"2016-04-05 15:06:48.000000000","message":"Just noticing that this check is unnecessary. max_count shouldn\u0027t be None, and if it were line 998 would fail.","commit_id":"786b7d237031ca49ce3cd5fd381785c43c953f90"}],"nova/tests/unit/test_nova_manage.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"5e873c7472a1bd81fce3b610d613bd15990f382a","unresolved":false,"context_lines":[{"line_number":531,"context_line":"        with mock.patch.object(self.commands, \u0027_run_migration\u0027) as rm:"},{"line_number":532,"context_line":"            rm.return_value \u003d 5"},{"line_number":533,"context_line":"            self.assertEqual(1,"},{"line_number":534,"context_line":"                             self.commands.online_data_migrations(max_count\u003d5))"},{"line_number":535,"context_line":""},{"line_number":536,"context_line":""},{"line_number":537,"context_line":"class ApiDbCommandsTestCase(test.NoDBTestCase):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_c6ddfa15","line":534,"updated":"2016-04-05 15:06:48.000000000","message":"It would be nice to have a test to check the case where max_count is not passed in.","commit_id":"786b7d237031ca49ce3cd5fd381785c43c953f90"}]}