)]}'
{"nova/compute/api.py":[{"author":{"_account_id":1030,"name":"Chris Behrens","email":"cbehrens@codestud.com","username":"cbehrens"},"change_message_id":"7f5697170441c9df69b1585bdd34230e48e0ae9d","unresolved":false,"context_lines":[{"line_number":969,"context_line":"        # (for notifications, etc).  Only store what we can."},{"line_number":970,"context_line":"        if \u0027system_metadata\u0027 in instance:"},{"line_number":971,"context_line":"            # Make sure we have the dict form that we need for instance_update."},{"line_number":972,"context_line":"            instance[\u0027system_metadata\u0027] \u003d utils.instance_sys_meta(instance)"},{"line_number":973,"context_line":"        else:"},{"line_number":974,"context_line":"            instance.setdefault(\u0027system_metadata\u0027, {})"},{"line_number":975,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Feeg%3D","line":972,"updated":"2013-07-01 05:20:20.000000000","message":"You could actually just leave the setdefault line above.. and add this line (972) with no \u0027if\u0027 or anything.  Ie:\n\ninstance.setdefault(..)\ninstance[\u0027system_metadata\u0027]\u003dutils.instance_sys_meta(instance)","commit_id":"41b7ae90b6702ad705aad96dbedd86ebb2917949"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"93719b58caeb27eeec14efd0ed50e9172c3c04b7","unresolved":false,"context_lines":[{"line_number":969,"context_line":"        # (for notifications, etc).  Only store what we can."},{"line_number":970,"context_line":"        if \u0027system_metadata\u0027 in instance:"},{"line_number":971,"context_line":"            # Make sure we have the dict form that we need for instance_update."},{"line_number":972,"context_line":"            instance[\u0027system_metadata\u0027] \u003d utils.instance_sys_meta(instance)"},{"line_number":973,"context_line":"        else:"},{"line_number":974,"context_line":"            instance.setdefault(\u0027system_metadata\u0027, {})"},{"line_number":975,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPn%2F%2Fdg8%3D","line":972,"in_reply_to":"AAAAPn%2F%2Feeg%3D","updated":"2013-07-01 13:54:32.000000000","message":"Done","commit_id":"41b7ae90b6702ad705aad96dbedd86ebb2917949"}],"nova/tests/compute/test_compute.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"21bae112e01d7bc1d9a82337dce7bbdd4c83f0c1","unresolved":false,"context_lines":[{"line_number":5727,"context_line":"                                                    base_options,"},{"line_number":5728,"context_line":"                                                    self.fake_image,"},{"line_number":5729,"context_line":"                                                    security_groups\u003dNone)"},{"line_number":5730,"context_line":"        self.assertEquals(base_options[\u0027image_ref\u0027],"},{"line_number":5731,"context_line":"                          instance[\u0027system_metadata\u0027][\u0027image_base_image_ref\u0027])"},{"line_number":5732,"context_line":"        self.assertEquals(vm_states.BUILDING, instance[\u0027vm_state\u0027])"},{"line_number":5733,"context_line":"        self.assertEquals(task_states.SCHEDULING, instance[\u0027task_state\u0027])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAP3%2F%2Fmsk%3D","line":5730,"updated":"2013-07-15 22:52:11.000000000","message":"assertEqual() is the canonical spelling; assertEquals() is an alias, and I believe a deprecated one.","commit_id":"fc56a3a7608fe2674f2c354e401e87910b220e04"}]}