)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"993c9549c4294e654b50de937ce93b542dc7a355","unresolved":false,"context_lines":[{"line_number":6063,"context_line":"        num_vm_instances \u003d self.driver.get_num_instances()"},{"line_number":6064,"context_line":"        num_db_instances \u003d len(db_instances)"},{"line_number":6065,"context_line":""},{"line_number":6066,"context_line":"        if num_vm_instances !\u003d num_db_instances:"},{"line_number":6067,"context_line":"            LOG.warning(_LW(\"While synchronizing instance power states, found \""},{"line_number":6068,"context_line":"                            \"%(num_db_instances)s instances in the database \""},{"line_number":6069,"context_line":"                            \"and %(num_vm_instances)s instances on the \""}],"source_content_type":"text/x-python","patch_set":3,"id":"1aa78d24_ce9b7a71","line":6066,"updated":"2016-07-08 14:52:07.000000000","message":"So I think this is still and important check in its own right.\n\nThis is no longer valid if you exclude all the things that are not running a task.\n\nI think the cloud steady state will be no instances running a task, so I don\u0027t think we need to worry about the extra DB load of getting instances that are running a task. It should be tiny.","commit_id":"910177f641f007d918513742d5313cba8f8b250f"}]}