)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"1a86f9f3e2b7d1147e16e23b04a4ed5a9d98d71d","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Transform instance.rebuild notification"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Handling rebuild.error:"},{"line_number":10,"context_line":"https://review.openstack.org/#/c/396621/"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Co-Authored-By: Béla Vancsics \u003cvancsics@inf.u-szeged.hu\u003e"},{"line_number":13,"context_line":"Co-Authored-By: Gábor Antal \u003cantal@inf.u-szeged.hu\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":20,"id":"9a629dbe_547f2bdf","line":10,"range":{"start_line":9,"start_character":0,"end_line":10,"end_character":40},"updated":"2016-11-11 13:23:01.000000000","message":"As a future suggestion, you don\u0027t have to refer this way to a commit that sits top of you commit as gerrit shows the dependency anyhow.","commit_id":"91cf2267fdc12e4eb1fa38252201e9f99dc12a26"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"17981385d038fd7136461e3259b31622262912e4","unresolved":false,"context_lines":[{"line_number":12,"context_line":"Co-Authored-By: Béla Vancsics \u003cvancsics@inf.u-szeged.hu\u003e"},{"line_number":13,"context_line":"Co-Authored-By: Gábor Antal \u003cantal@inf.u-szeged.hu\u003e"},{"line_number":14,"context_line":"Change-Id: If624c2691830584851e9c97693cd7b7e73e73bb2"},{"line_number":15,"context_line":"Implements: bp versioned-notification-transformation-ocata"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":27,"id":"fa31d9ce_84dfcc38","line":15,"updated":"2017-02-15 11:43:00.000000000","message":"if you have to respin the patch then please change this to versioned-notification-transformation-pike","commit_id":"276ffbda1ba88343b32501088ccc939a28b18766"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ccb8f9c0c86832780c02f0f206754874824150d9","unresolved":false,"context_lines":[{"line_number":12,"context_line":"Co-Authored-By: Béla Vancsics \u003cvancsics@inf.u-szeged.hu\u003e"},{"line_number":13,"context_line":"Co-Authored-By: Gábor Antal \u003cantal@inf.u-szeged.hu\u003e"},{"line_number":14,"context_line":"Change-Id: If624c2691830584851e9c97693cd7b7e73e73bb2"},{"line_number":15,"context_line":"Implements: bp versioned-notification-transformation-ocata"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":27,"id":"9a30ddce_7ec122e4","line":15,"in_reply_to":"fa31d9ce_84dfcc38","updated":"2017-03-07 20:52:40.000000000","message":"Let\u0027s fix this now please.","commit_id":"276ffbda1ba88343b32501088ccc939a28b18766"}],"doc/notification_samples/instance-rebuild-end.json":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"b9cee52ab28bdd85c2098c89ad1fa221d5081a93","unresolved":false,"context_lines":[{"line_number":19,"context_line":"            \"reservation_id\": \"r-wczvhcla\","},{"line_number":20,"context_line":"            \"flavor\": {"},{"line_number":21,"context_line":"                \"nova_object.namespace\": \"nova\","},{"line_number":22,"context_line":"                \"nova_object.version\": \"1.0\","},{"line_number":23,"context_line":"                \"nova_object.data\": {"},{"line_number":24,"context_line":"                    \"ephemeral_gb\": 0,"},{"line_number":25,"context_line":"                    \"flavorid\": \"a22d5517-147c-4147-a0d1-e698df5cd4e3\","}],"source_content_type":"application/json","patch_set":30,"id":"9a30ddce_ce42bcad","line":22,"updated":"2017-03-12 15:20:52.000000000","message":"This might be old.","commit_id":"55f25449554ba95f886440494fcc44613a9e7bba"}],"doc/notification_samples/instance-rebuild-start.json":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9d95388aa936a3c1e037a361dd75069174eb2459","unresolved":false,"context_lines":[{"line_number":58,"context_line":"            \"tenant_id\": \"6f70656e737461636b20342065766572\","},{"line_number":59,"context_line":"            \"availability_zone\": null,"},{"line_number":60,"context_line":"            \"host\": \"compute\","},{"line_number":61,"context_line":"            \"image_uuid\": \"a2459075-d96c-40d5-893e-577ff92e721c\","},{"line_number":62,"context_line":"            \"state\": \"active\","},{"line_number":63,"context_line":"            \"fault\": null"},{"line_number":64,"context_line":"        }"}],"source_content_type":"application/json","patch_set":27,"id":"9a30ddce_e1c49793","line":61,"updated":"2017-03-07 21:02:55.000000000","message":"And this is the new image used in the test, which is in here rather than the original image because the compute API stores that in the instances table before casting to the compute:\n\nhttps://github.com/openstack/nova/blob/15.0.0/nova/compute/api.py#L2982","commit_id":"276ffbda1ba88343b32501088ccc939a28b18766"}],"nova/compute/manager.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"560eadba7426191fa9d6a79b26656cb04f3a8350","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"fa6399be_f4a783ba","updated":"2016-10-14 10:45:11.000000000","message":"Could you include handling the rebuild.error notification into this or a subsequent patch?","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"406137e0e58264ab35fa3cc8d19a5a669f5c4f71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"da6895a0_35f3550c","in_reply_to":"fa6399be_f4a783ba","updated":"2016-10-21 15:51:01.000000000","message":"Hi! Yes, i create a subsequent patch for this","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"560eadba7426191fa9d6a79b26656cb04f3a8350","unresolved":false,"context_lines":[{"line_number":2793,"context_line":""},{"line_number":2794,"context_line":"        # This message should contain the new image_ref"},{"line_number":2795,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2796,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2797,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2798,"context_line":"        compute_utils.notify_about_instance_action(context, instance,"},{"line_number":2799,"context_line":"            self.host, action\u003dfields.NotificationAction.REBUILD,"},{"line_number":2800,"context_line":"            phase\u003dfields.NotificationPhase.START)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_94c2c732","line":2797,"range":{"start_line":2796,"start_character":0,"end_line":2797,"end_character":67},"updated":"2016-10-14 10:45:11.000000000","message":"I think we have to create an extended instance payload that can accommodate the extra info that is sent in the legacy notification. (here it is the image_name)","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"406137e0e58264ab35fa3cc8d19a5a669f5c4f71","unresolved":false,"context_lines":[{"line_number":2793,"context_line":""},{"line_number":2794,"context_line":"        # This message should contain the new image_ref"},{"line_number":2795,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2796,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2797,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2798,"context_line":"        compute_utils.notify_about_instance_action(context, instance,"},{"line_number":2799,"context_line":"            self.host, action\u003dfields.NotificationAction.REBUILD,"},{"line_number":2800,"context_line":"            phase\u003dfields.NotificationPhase.START)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da6895a0_af5d49eb","line":2797,"range":{"start_line":2796,"start_character":0,"end_line":2797,"end_character":67},"in_reply_to":"fa6399be_022779ca","updated":"2016-10-21 15:51:01.000000000","message":"i rebased the patch on your (Gábor), though the \u0027instance\u0027 should contain the image_uuid, which we can tell the image_name. If i know badly, then sorry","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"58725e186d6e3b69e1aa411f5d7d2ffd97e25a90","unresolved":false,"context_lines":[{"line_number":2793,"context_line":""},{"line_number":2794,"context_line":"        # This message should contain the new image_ref"},{"line_number":2795,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2796,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2797,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2798,"context_line":"        compute_utils.notify_about_instance_action(context, instance,"},{"line_number":2799,"context_line":"            self.host, action\u003dfields.NotificationAction.REBUILD,"},{"line_number":2800,"context_line":"            phase\u003dfields.NotificationPhase.START)"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_022779ca","line":2797,"range":{"start_line":2796,"start_character":0,"end_line":2797,"end_character":67},"in_reply_to":"fa6399be_94c2c732","updated":"2016-10-20 17:29:50.000000000","message":"Hi Alex,\n\nThe mentioned extended payload is implemented here: https://review.openstack.org/#/c/384621\n\nSo if you want to add this extra usage information, rebase your patch on mine, and you will have this extended payload.","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"560eadba7426191fa9d6a79b26656cb04f3a8350","unresolved":false,"context_lines":[{"line_number":2862,"context_line":"            instance.save()"},{"line_number":2863,"context_line":"            self.stop_instance(context, instance, False)"},{"line_number":2864,"context_line":"        self._update_scheduler_instance_info(context, instance)"},{"line_number":2865,"context_line":"        self._notify_about_instance_usage("},{"line_number":2866,"context_line":"                context, instance, \"rebuild.end\","},{"line_number":2867,"context_line":"                network_info\u003dnetwork_info,"},{"line_number":2868,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2869,"context_line":"        compute_utils.notify_about_instance_action(context, instance,"},{"line_number":2870,"context_line":"            self.host, action\u003dfields.NotificationAction.REBUILD,"},{"line_number":2871,"context_line":"            phase\u003dfields.NotificationPhase.END)"},{"line_number":2872,"context_line":""},{"line_number":2873,"context_line":"    def _handle_bad_volumes_detached(self, context, instance, bad_devices,"},{"line_number":2874,"context_line":"                                     block_device_info):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_f40a2389","line":2871,"range":{"start_line":2865,"start_character":1,"end_line":2871,"end_character":47},"updated":"2016-10-14 10:45:11.000000000","message":"Could you please check what kind of network info is sent in the legacy notification to see if the same information is already in the InstancePayload or not?","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"58725e186d6e3b69e1aa411f5d7d2ffd97e25a90","unresolved":false,"context_lines":[{"line_number":2862,"context_line":"            instance.save()"},{"line_number":2863,"context_line":"            self.stop_instance(context, instance, False)"},{"line_number":2864,"context_line":"        self._update_scheduler_instance_info(context, instance)"},{"line_number":2865,"context_line":"        self._notify_about_instance_usage("},{"line_number":2866,"context_line":"                context, instance, \"rebuild.end\","},{"line_number":2867,"context_line":"                network_info\u003dnetwork_info,"},{"line_number":2868,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2869,"context_line":"        compute_utils.notify_about_instance_action(context, instance,"},{"line_number":2870,"context_line":"            self.host, action\u003dfields.NotificationAction.REBUILD,"},{"line_number":2871,"context_line":"            phase\u003dfields.NotificationPhase.END)"},{"line_number":2872,"context_line":""},{"line_number":2873,"context_line":"    def _handle_bad_volumes_detached(self, context, instance, bad_devices,"},{"line_number":2874,"context_line":"                                     block_device_info):"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_d5b29de4","line":2871,"range":{"start_line":2865,"start_character":1,"end_line":2871,"end_character":47},"in_reply_to":"fa6399be_f40a2389","updated":"2016-10-20 17:29:50.000000000","message":"Hi,\n\nI just checked it, I was curious. I put the result to [1]. \nWhat do you think about it? We can check anytime with \n\n    compute_utils.get_nw_info_for_instance(instance)\n\nmethod call. so If we want to keep this information, then we\nshould add another extended payload (or edit the currently introduced extended payload).\n\n[1]: http://paste.openstack.org/show/586615/","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"2b1c1b91778a89650f2216d66ba63a6b166df2af","unresolved":false,"context_lines":[{"line_number":2798,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2799,"context_line":""},{"line_number":2800,"context_line":"        # This message should contain the new image_ref"},{"line_number":2801,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2802,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2803,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2804,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":11,"id":"ba5da102_50fa7b43","line":2801,"updated":"2016-10-28 09:44:10.000000000","message":"In the instance.create notification this image_name is the name of the image that is already referred by the image_uuid param. Is it true here as well? I mean this is the rebuild case where the image can be changed. If the image is changed does the image_uuid and this image_name kept in sync?","commit_id":"b6a2856e3220824dcbd98a957c65f04670db785c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a17fb777a4ad7cf40bd1061cfaa2173fbb45051f","unresolved":false,"context_lines":[{"line_number":2805,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2806,"context_line":""},{"line_number":2807,"context_line":"        # This message should contain the new image_ref"},{"line_number":2808,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2809,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2810,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2811,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":14,"id":"ba5da102_313b6d9e","line":2808,"updated":"2016-11-04 16:34:30.000000000","message":"See my question in PS 11","commit_id":"80650c1508f3733af3d7b3efca44e722f03f90e7"},{"author":{"_account_id":18602,"name":"Gábor Antal","email":"antal@inf.u-szeged.hu","username":"antal"},"change_message_id":"f67da01d0340ff20917a1c7155fd3268c21ca552","unresolved":false,"context_lines":[{"line_number":2805,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2806,"context_line":""},{"line_number":2807,"context_line":"        # This message should contain the new image_ref"},{"line_number":2808,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2809,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2810,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2811,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":14,"id":"ba5da102_e5020f0b","line":2808,"in_reply_to":"ba5da102_313b6d9e","updated":"2016-11-04 18:46:03.000000000","message":"Hi! \n\nThis is exactly the same name as we saw in instance.create \naction, so the following:\n\n    extra_usage_info \u003d {\u0027image_name\u0027: u\u0027fakeimage123456\u0027}\n\nSo I think, we can ignore this extra information (as we already have the image id).","commit_id":"80650c1508f3733af3d7b3efca44e722f03f90e7"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e4b7850b19fd65feab5b6a262cf758f61a9fa8f3","unresolved":false,"context_lines":[{"line_number":2805,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2806,"context_line":""},{"line_number":2807,"context_line":"        # This message should contain the new image_ref"},{"line_number":2808,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2809,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2810,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2811,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":14,"id":"9a629dbe_f17bc879","line":2808,"in_reply_to":"ba5da102_e5020f0b","updated":"2016-11-07 10:57:09.000000000","message":"OK. I see. I played with the test case a bit and it seems that the instance.image_ref (that is mapped to image_uuid in the notification) changes together with the image_name generated here. So I agree that we don\u0027t need to add image_name to the notification.","commit_id":"80650c1508f3733af3d7b3efca44e722f03f90e7"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"753fbb242d7698eace27c677a698e939adff47fe","unresolved":false,"context_lines":[{"line_number":2610,"context_line":"            self.driver.destroy(context, instance,"},{"line_number":2611,"context_line":"                                network_info\u003dnetwork_info,"},{"line_number":2612,"context_line":"                                block_device_info\u003dblock_device_info)"},{"line_number":2613,"context_line":""},{"line_number":2614,"context_line":"        instance.task_state \u003d task_states.REBUILD_BLOCK_DEVICE_MAPPING"},{"line_number":2615,"context_line":"        instance.save(expected_task_state\u003d[task_states.REBUILDING])"},{"line_number":2616,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_0bde1f6d","side":"PARENT","line":2613,"updated":"2016-11-08 08:20:23.000000000","message":"nit: seems like an unnecessary change","commit_id":"5a81b00e6b2adba2a380b90e402ff391d64ea6a5"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"018a195e773218d2ed05a5882a8e894895a94149","unresolved":false,"context_lines":[{"line_number":2610,"context_line":"            self.driver.destroy(context, instance,"},{"line_number":2611,"context_line":"                                network_info\u003dnetwork_info,"},{"line_number":2612,"context_line":"                                block_device_info\u003dblock_device_info)"},{"line_number":2613,"context_line":""},{"line_number":2614,"context_line":"        instance.task_state \u003d task_states.REBUILD_BLOCK_DEVICE_MAPPING"},{"line_number":2615,"context_line":"        instance.save(expected_task_state\u003d[task_states.REBUILDING])"},{"line_number":2616,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_e7d71cbd","side":"PARENT","line":2613,"in_reply_to":"9a629dbe_0bde1f6d","updated":"2016-11-09 12:22:15.000000000","message":"Done","commit_id":"5a81b00e6b2adba2a380b90e402ff391d64ea6a5"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"be0f3a6cf95147f8e7f5cac55b6a5e06b5a0fd62","unresolved":false,"context_lines":[{"line_number":2610,"context_line":"            self.driver.destroy(context, instance,"},{"line_number":2611,"context_line":"                                network_info\u003dnetwork_info,"},{"line_number":2612,"context_line":"                                block_device_info\u003dblock_device_info)"},{"line_number":2613,"context_line":""},{"line_number":2614,"context_line":"        instance.task_state \u003d task_states.REBUILD_BLOCK_DEVICE_MAPPING"},{"line_number":2615,"context_line":"        instance.save(expected_task_state\u003d[task_states.REBUILDING])"},{"line_number":2616,"context_line":""}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_ae9ffc01","side":"PARENT","line":2613,"in_reply_to":"9a629dbe_0bde1f6d","updated":"2016-11-08 10:22:28.000000000","message":"True, i make experiments. Unfortunately the test (test_rebuild_instance_notification) does not work and i trying realignment notification to new places. This blank line adventitious removed! :) At next patch undo :)","commit_id":"5a81b00e6b2adba2a380b90e402ff391d64ea6a5"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"753fbb242d7698eace27c677a698e939adff47fe","unresolved":false,"context_lines":[{"line_number":2666,"context_line":"                       specified by the user, this will be None"},{"line_number":2667,"context_line":"        \"\"\""},{"line_number":2668,"context_line":"        context \u003d context.elevated()"},{"line_number":2669,"context_line":""},{"line_number":2670,"context_line":"        LOG.info(_LI(\"Rebuilding instance\"), instance\u003dinstance)"},{"line_number":2671,"context_line":"        if scheduled_node is not None:"},{"line_number":2672,"context_line":"            rt \u003d self._get_resource_tracker(scheduled_node)"}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_4be81711","side":"PARENT","line":2669,"updated":"2016-11-08 08:20:23.000000000","message":"ditto","commit_id":"5a81b00e6b2adba2a380b90e402ff391d64ea6a5"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"018a195e773218d2ed05a5882a8e894895a94149","unresolved":false,"context_lines":[{"line_number":2666,"context_line":"                       specified by the user, this will be None"},{"line_number":2667,"context_line":"        \"\"\""},{"line_number":2668,"context_line":"        context \u003d context.elevated()"},{"line_number":2669,"context_line":""},{"line_number":2670,"context_line":"        LOG.info(_LI(\"Rebuilding instance\"), instance\u003dinstance)"},{"line_number":2671,"context_line":"        if scheduled_node is not None:"},{"line_number":2672,"context_line":"            rt \u003d self._get_resource_tracker(scheduled_node)"}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_27e2f49f","side":"PARENT","line":2669,"in_reply_to":"9a629dbe_4be81711","updated":"2016-11-09 12:22:15.000000000","message":"Done","commit_id":"5a81b00e6b2adba2a380b90e402ff391d64ea6a5"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"be0f3a6cf95147f8e7f5cac55b6a5e06b5a0fd62","unresolved":false,"context_lines":[{"line_number":2666,"context_line":"                       specified by the user, this will be None"},{"line_number":2667,"context_line":"        \"\"\""},{"line_number":2668,"context_line":"        context \u003d context.elevated()"},{"line_number":2669,"context_line":""},{"line_number":2670,"context_line":"        LOG.info(_LI(\"Rebuilding instance\"), instance\u003dinstance)"},{"line_number":2671,"context_line":"        if scheduled_node is not None:"},{"line_number":2672,"context_line":"            rt \u003d self._get_resource_tracker(scheduled_node)"}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_ee89f4cb","side":"PARENT","line":2669,"in_reply_to":"9a629dbe_4be81711","updated":"2016-11-08 10:22:28.000000000","message":"Thanks, at next patch undo.","commit_id":"5a81b00e6b2adba2a380b90e402ff391d64ea6a5"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ccb8f9c0c86832780c02f0f206754874824150d9","unresolved":false,"context_lines":[{"line_number":2850,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2851,"context_line":""},{"line_number":2852,"context_line":"        # This message should contain the new image_ref"},{"line_number":2853,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2854,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2855,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2856,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":27,"id":"9a30ddce_fe86f2b7","line":2853,"range":{"start_line":2853,"start_character":8,"end_line":2853,"end_character":75},"updated":"2017-03-07 20:52:40.000000000","message":"Why isn\u0027t this information included in the versioned notification for rebuild?","commit_id":"276ffbda1ba88343b32501088ccc939a28b18766"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"6dacfda6e72d037006dc081a826c8c6ad6983053","unresolved":false,"context_lines":[{"line_number":2850,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2851,"context_line":""},{"line_number":2852,"context_line":"        # This message should contain the new image_ref"},{"line_number":2853,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2854,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2855,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2856,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":27,"id":"9a30ddce_895e6274","line":2853,"range":{"start_line":2853,"start_character":8,"end_line":2853,"end_character":75},"in_reply_to":"9a30ddce_e1af37e1","updated":"2017-03-09 09:06:12.000000000","message":"I had to read the patch history. In PS14 I feel we figured out that the image_name refers to the same image as referred by the image_uuid which is already part of the payload. Therefore I feel we agreed not to include it.\n\nHowever I agree with Matt here to mention this fact at least in the commit message","commit_id":"276ffbda1ba88343b32501088ccc939a28b18766"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9d95388aa936a3c1e037a361dd75069174eb2459","unresolved":false,"context_lines":[{"line_number":2850,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2851,"context_line":""},{"line_number":2852,"context_line":"        # This message should contain the new image_ref"},{"line_number":2853,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2854,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2855,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2856,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":27,"id":"9a30ddce_e1af37e1","line":2853,"range":{"start_line":2853,"start_character":8,"end_line":2853,"end_character":75},"in_reply_to":"9a30ddce_fe86f2b7","updated":"2017-03-07 21:02:55.000000000","message":"I guess if we have the image uuid we don\u0027t really need to pass the image name so maybe that\u0027s why you didn\u0027t include it.","commit_id":"276ffbda1ba88343b32501088ccc939a28b18766"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"7b2527aeca47c5e378341b5fe697f70d347cea15","unresolved":false,"context_lines":[{"line_number":2849,"context_line":"                extra_usage_info\u003dextra_usage_info)"},{"line_number":2850,"context_line":""},{"line_number":2851,"context_line":"        # This message should contain the new image_ref"},{"line_number":2852,"context_line":"        extra_usage_info \u003d {\u0027image_name\u0027: self._get_image_name(image_meta)}"},{"line_number":2853,"context_line":"        self._notify_about_instance_usage(context, instance,"},{"line_number":2854,"context_line":"                \"rebuild.start\", extra_usage_info\u003dextra_usage_info)"},{"line_number":2855,"context_line":"        compute_utils.notify_about_instance_action("}],"source_content_type":"text/x-python","patch_set":29,"id":"9a30ddce_eec86057","line":2852,"range":{"start_line":2852,"start_character":8,"end_line":2852,"end_character":75},"updated":"2017-03-12 15:05:13.000000000","message":"Per previous comments, can we have a note in the code about why we don\u0027t include this for versioned notifications?","commit_id":"8959d4e6d772d105c43f3921f136c199fccdf56c"}],"nova/objects/fields.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"560eadba7426191fa9d6a79b26656cb04f3a8350","unresolved":false,"context_lines":[{"line_number":520,"context_line":""},{"line_number":521,"context_line":"    ALL \u003d (UPDATE, EXCEPTION, DELETE, PAUSE, UNPAUSE, RESIZE, VOLUME_SWAP,"},{"line_number":522,"context_line":"           SUSPEND, POWER_ON, REBOOT, SHUTDOWN, SNAPSHOT, ADD_FIXED_IP,"},{"line_number":523,"context_line":"           POWER_OFF, SHELVE, RESUME, RESTORE, REBUILD)"},{"line_number":524,"context_line":""},{"line_number":525,"context_line":""},{"line_number":526,"context_line":"# TODO(rlrossit): These should be changed over to be a StateMachine enum from"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_b4ed0bb9","line":523,"updated":"2016-10-14 10:45:11.000000000","message":"This causes that the EventType version needs to be bumped.","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"406137e0e58264ab35fa3cc8d19a5a669f5c4f71","unresolved":false,"context_lines":[{"line_number":520,"context_line":""},{"line_number":521,"context_line":"    ALL \u003d (UPDATE, EXCEPTION, DELETE, PAUSE, UNPAUSE, RESIZE, VOLUME_SWAP,"},{"line_number":522,"context_line":"           SUSPEND, POWER_ON, REBOOT, SHUTDOWN, SNAPSHOT, ADD_FIXED_IP,"},{"line_number":523,"context_line":"           POWER_OFF, SHELVE, RESUME, RESTORE, REBUILD)"},{"line_number":524,"context_line":""},{"line_number":525,"context_line":""},{"line_number":526,"context_line":"# TODO(rlrossit): These should be changed over to be a StateMachine enum from"}],"source_content_type":"text/x-python","patch_set":1,"id":"da6895a0_ef6581e1","line":523,"in_reply_to":"fa6399be_273460dc","updated":"2016-10-21 15:51:01.000000000","message":"Done, your patch be merged, so i rebased the patch top of https://review.openstack.org/#/c/384621","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"b35c4afc3f1bef2311b4f85a4d75d7214da723ce","unresolved":false,"context_lines":[{"line_number":520,"context_line":""},{"line_number":521,"context_line":"    ALL \u003d (UPDATE, EXCEPTION, DELETE, PAUSE, UNPAUSE, RESIZE, VOLUME_SWAP,"},{"line_number":522,"context_line":"           SUSPEND, POWER_ON, REBOOT, SHUTDOWN, SNAPSHOT, ADD_FIXED_IP,"},{"line_number":523,"context_line":"           POWER_OFF, SHELVE, RESUME, RESTORE, REBUILD)"},{"line_number":524,"context_line":""},{"line_number":525,"context_line":""},{"line_number":526,"context_line":"# TODO(rlrossit): These should be changed over to be a StateMachine enum from"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa6399be_273460dc","line":523,"in_reply_to":"fa6399be_b4ed0bb9","updated":"2016-10-14 15:17:24.000000000","message":"I made a change where I pre-added the needed NotificationAction values so that we avoid merge conflict with each other bumping the version. Please rebase your change top of https://review.openstack.org/#/c/386659/","commit_id":"f9d636be8935c70ca9d7fb115f5df7bb77a145cd"}],"nova/tests/functional/notification_sample_tests/test_instance.py":[{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"f8bb4dfb0ab0ee83a42f0fdc26a72a3ef859b01c","unresolved":false,"context_lines":[{"line_number":371,"context_line":"    def _test_rebuild_server(self, server):"},{"line_number":372,"context_line":"        self.api.post_server_action("},{"line_number":373,"context_line":"            server[\u0027id\u0027], {\u0027rebuild\u0027: {"},{"line_number":374,"context_line":"                \"name\": \"IpPayload\","},{"line_number":375,"context_line":"                \"\"\"\"imageRef\": \"155d900f-4e14-4e4c-a73d-069cbf4541e6\",\"\"\""},{"line_number":376,"context_line":"                \"metadata\": {}"},{"line_number":377,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":11,"id":"ba5da102_69bb31c5","line":374,"updated":"2016-10-27 13:51:05.000000000","message":"name is the server name which is optional in the API, can we remove it?","commit_id":"b6a2856e3220824dcbd98a957c65f04670db785c"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"0d59d3c157f069313c5d8c1a23e5d85ec7bc6d97","unresolved":false,"context_lines":[{"line_number":371,"context_line":"    def _test_rebuild_server(self, server):"},{"line_number":372,"context_line":"        self.api.post_server_action("},{"line_number":373,"context_line":"            server[\u0027id\u0027], {\u0027rebuild\u0027: {"},{"line_number":374,"context_line":"                \"name\": \"IpPayload\","},{"line_number":375,"context_line":"                \"\"\"\"imageRef\": \"155d900f-4e14-4e4c-a73d-069cbf4541e6\",\"\"\""},{"line_number":376,"context_line":"                \"metadata\": {}"},{"line_number":377,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":11,"id":"ba5da102_66d99421","line":374,"in_reply_to":"ba5da102_69bb31c5","updated":"2016-10-27 19:11:17.000000000","message":"yes, at next patch I remove this.","commit_id":"b6a2856e3220824dcbd98a957c65f04670db785c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"f8bb4dfb0ab0ee83a42f0fdc26a72a3ef859b01c","unresolved":false,"context_lines":[{"line_number":372,"context_line":"        self.api.post_server_action("},{"line_number":373,"context_line":"            server[\u0027id\u0027], {\u0027rebuild\u0027: {"},{"line_number":374,"context_line":"                \"name\": \"IpPayload\","},{"line_number":375,"context_line":"                \"\"\"\"imageRef\": \"155d900f-4e14-4e4c-a73d-069cbf4541e6\",\"\"\""},{"line_number":376,"context_line":"                \"metadata\": {}"},{"line_number":377,"context_line":"            }"},{"line_number":378,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":11,"id":"ba5da102_a99e2954","line":375,"updated":"2016-10-27 13:51:05.000000000","message":"As far as I see in the API doc imageRef is mandatory for the rebuild so I don\u0027t know why it is commented out.","commit_id":"b6a2856e3220824dcbd98a957c65f04670db785c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"2b1c1b91778a89650f2216d66ba63a6b166df2af","unresolved":false,"context_lines":[{"line_number":372,"context_line":"        self.api.post_server_action("},{"line_number":373,"context_line":"            server[\u0027id\u0027], {\u0027rebuild\u0027: {"},{"line_number":374,"context_line":"                \"name\": \"IpPayload\","},{"line_number":375,"context_line":"                \"\"\"\"imageRef\": \"155d900f-4e14-4e4c-a73d-069cbf4541e6\",\"\"\""},{"line_number":376,"context_line":"                \"metadata\": {}"},{"line_number":377,"context_line":"            }"},{"line_number":378,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":11,"id":"ba5da102_266311de","line":375,"in_reply_to":"ba5da102_06d29800","updated":"2016-10-28 09:44:10.000000000","message":"The api request is wrongly formatted. this way nova accept the request\n        self.api.post_server_action(\n            server[\u0027id\u0027], {\u0027rebuild\u0027: {\n                \"imageRef\": \"155d900f-4e14-4e4c-a73d-069cbf4541e6\",\n                \"metadata\": {}\n            }\n            }\n        )\nYou might try to trim this down, I haven\u0027t tried that.","commit_id":"b6a2856e3220824dcbd98a957c65f04670db785c"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"0d59d3c157f069313c5d8c1a23e5d85ec7bc6d97","unresolved":false,"context_lines":[{"line_number":372,"context_line":"        self.api.post_server_action("},{"line_number":373,"context_line":"            server[\u0027id\u0027], {\u0027rebuild\u0027: {"},{"line_number":374,"context_line":"                \"name\": \"IpPayload\","},{"line_number":375,"context_line":"                \"\"\"\"imageRef\": \"155d900f-4e14-4e4c-a73d-069cbf4541e6\",\"\"\""},{"line_number":376,"context_line":"                \"metadata\": {}"},{"line_number":377,"context_line":"            }"},{"line_number":378,"context_line":"            }"}],"source_content_type":"text/x-python","patch_set":11,"id":"ba5da102_06d29800","line":375,"in_reply_to":"ba5da102_a99e2954","updated":"2016-10-27 19:11:17.000000000","message":"This patch stay in WIP, I just forged set it.\nThe imageRef is not mandatory (not necessarily) for rebuild. If this value is None, then this just a simple rebuild with original values, data. \nIn functional tests fail the code and we dont know, that why. If the functional tests be good, we know better recognize, the really mandatory values for the rebuild and we can apply this knowledge.","commit_id":"b6a2856e3220824dcbd98a957c65f04670db785c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"2b1c1b91778a89650f2216d66ba63a6b166df2af","unresolved":false,"context_lines":[{"line_number":377,"context_line":"            }"},{"line_number":378,"context_line":"            }"},{"line_number":379,"context_line":"        )"},{"line_number":380,"context_line":"        self._wait_for_state_change(self.api, server,"},{"line_number":381,"context_line":"                                    expected_status\u003d\u0027REBUILDING\u0027)"},{"line_number":382,"context_line":""},{"line_number":383,"context_line":"        self.assertEqual(2, len(fake_notifier.VERSIONED_NOTIFICATIONS))"},{"line_number":384,"context_line":"        self._verify_notification("}],"source_content_type":"text/x-python","patch_set":11,"id":"ba5da102_66826954","line":381,"range":{"start_line":380,"start_character":0,"end_line":381,"end_character":65},"updated":"2016-10-28 09:44:10.000000000","message":"first the instance goes to REBUILD state then it will go to ACTIVE state again. You have to wait first to REBUILD then to ACTIVE to make sure both notification is emitted.","commit_id":"b6a2856e3220824dcbd98a957c65f04670db785c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"e4b7850b19fd65feab5b6a262cf758f61a9fa8f3","unresolved":false,"context_lines":[{"line_number":477,"context_line":"    def _test_rebuild_server_success(self, server):"},{"line_number":478,"context_line":"        post \u003d {"},{"line_number":479,"context_line":"            \u0027rebuild\u0027: {"},{"line_number":480,"context_line":"                \u0027imageRef\u0027: \u0027155d900f-4e14-4e4c-a73d-069cbf4541e6\u0027,"},{"line_number":481,"context_line":"                \u0027metadata\u0027: {}"},{"line_number":482,"context_line":"            }"},{"line_number":483,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":14,"id":"9a629dbe_f1c908ae","line":480,"updated":"2016-11-07 10:57:09.000000000","message":"Could you use a different imageRef to make it more like a real life scenario. E.g. a2459075-d96c-40d5-893e-577ff92e721c","commit_id":"80650c1508f3733af3d7b3efca44e722f03f90e7"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"753fbb242d7698eace27c677a698e939adff47fe","unresolved":false,"context_lines":[{"line_number":477,"context_line":"    def _test_rebuild_server_success(self, server):"},{"line_number":478,"context_line":"        post \u003d {"},{"line_number":479,"context_line":"            \u0027rebuild\u0027: {"},{"line_number":480,"context_line":"                \u0027imageRef\u0027: \u0027155d900f-4e14-4e4c-a73d-069cbf4541e6\u0027,"},{"line_number":481,"context_line":"                \u0027metadata\u0027: {}"},{"line_number":482,"context_line":"            }"},{"line_number":483,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_8b886f24","line":480,"updated":"2016-11-08 08:20:23.000000000","message":"You can use imageRef a2459075-d96c-40d5-893e-577ff92e721c in the rebuild to simulate that the rebuild is called with a different image than what was the original image of the instance.","commit_id":"818e3034afdc75acd7778e29741bba188ac4356c"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"018a195e773218d2ed05a5882a8e894895a94149","unresolved":false,"context_lines":[{"line_number":477,"context_line":"    def _test_rebuild_server_success(self, server):"},{"line_number":478,"context_line":"        post \u003d {"},{"line_number":479,"context_line":"            \u0027rebuild\u0027: {"},{"line_number":480,"context_line":"                \u0027imageRef\u0027: \u0027155d900f-4e14-4e4c-a73d-069cbf4541e6\u0027,"},{"line_number":481,"context_line":"                \u0027metadata\u0027: {}"},{"line_number":482,"context_line":"            }"},{"line_number":483,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_87e90083","line":480,"in_reply_to":"9a629dbe_8b886f24","updated":"2016-11-09 12:22:15.000000000","message":"Done","commit_id":"818e3034afdc75acd7778e29741bba188ac4356c"},{"author":{"_account_id":21239,"name":"Alex Szarka","email":"szarka@inf.u-szeged.hu","username":"xavvior"},"change_message_id":"be0f3a6cf95147f8e7f5cac55b6a5e06b5a0fd62","unresolved":false,"context_lines":[{"line_number":477,"context_line":"    def _test_rebuild_server_success(self, server):"},{"line_number":478,"context_line":"        post \u003d {"},{"line_number":479,"context_line":"            \u0027rebuild\u0027: {"},{"line_number":480,"context_line":"                \u0027imageRef\u0027: \u0027155d900f-4e14-4e4c-a73d-069cbf4541e6\u0027,"},{"line_number":481,"context_line":"                \u0027metadata\u0027: {}"},{"line_number":482,"context_line":"            }"},{"line_number":483,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":15,"id":"9a629dbe_4e9100f0","line":480,"in_reply_to":"9a629dbe_8b886f24","updated":"2016-11-08 10:22:28.000000000","message":"Thanks! Good idea! :)","commit_id":"818e3034afdc75acd7778e29741bba188ac4356c"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"a0404a71809de7ea9eccd63e8415d927f8eef5eb","unresolved":false,"context_lines":[{"line_number":477,"context_line":"    def _test_rebuild_server_success(self, server):"},{"line_number":478,"context_line":"        post \u003d {"},{"line_number":479,"context_line":"            \u0027rebuild\u0027: {"},{"line_number":480,"context_line":"                \u0027imageRef\u0027: \u0027a2459075-d96c-40d5-893e-577ff92e72\u0027,"},{"line_number":481,"context_line":"                \u0027metadata\u0027: {}"},{"line_number":482,"context_line":"            }"},{"line_number":483,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":16,"id":"9a629dbe_8d9835c2","line":480,"updated":"2016-11-09 16:00:36.000000000","message":"You missed the end of the image uuid. This is why the functional test fails. The corret uuid is:\n\na2459075-d96c-40d5-893e-577ff92e721c","commit_id":"be18776680cc0f43b04299fb1ac053eace5de6fd"}]}