)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"change_message_id":"e18ba472822163819179a52348fba2c0b23f077b","unresolved":false,"context_lines":[{"line_number":611,"context_line":"                              task_state\u003dNone)"},{"line_number":612,"context_line":""},{"line_number":613,"context_line":"    @exception.wrap_exception(notifier\u003dnotifier, publisher_id\u003dpublisher_id())"},{"line_number":614,"context_line":"    @checks_instance_lock_uuid"},{"line_number":615,"context_line":"    def power_off_instance(self, context, instance_uuid):"},{"line_number":616,"context_line":"        \"\"\"Power off an instance on this host.\"\"\""},{"line_number":617,"context_line":"        instance \u003d self.db.instance_get_by_uuid(context, instance_uuid)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAACn%2F%2F8ac%3D","line":614,"updated":"2011-11-27 16:44:50.000000000","message":"I feel like we should be passing in the whole instance here rather than just an identifier to it. Also I think we can afford to do that since we easily control the only caller (compute/api.py)","commit_id":"2987e20c0f519bb1dcbb172758d90a3322393c40"}],"nova/tests/test_compute.py":[{"author":{"_account_id":616,"name":"Mark Washenberger","email":"mark.washenberger@markwash.net","username":"markwash"},"change_message_id":"af1f840508105498a18c6e76ecd8c779cd6a5d71","unresolved":false,"context_lines":[{"line_number":2220,"context_line":"        instance \u003d self.compute_api.get(self.context, instance_id)"},{"line_number":2221,"context_line":"        self.compute_api.get_diagnostics(self.context, instance)"},{"line_number":2222,"context_line":"        self.compute_api.delete(self.context, instance)"},{"line_number":2223,"context_line":""},{"line_number":2224,"context_line":"    def test_get_actions(self):"},{"line_number":2225,"context_line":""},{"line_number":2226,"context_line":"        called \u003d {\u0027get_actions\u0027: False}"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAACn%2F%2F8W0%3D","line":2223,"updated":"2011-11-28 15:02:57.000000000","message":"It looks like this test wasn\u0027t really testing anything before, so good catch. But I wonder if maybe you should just have the stub return something and assert that that something comes out from the manager call?","commit_id":"2987e20c0f519bb1dcbb172758d90a3322393c40"}]}