)]}'
{".zuul.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"36a2e7c36d47bb62d651896a05392fc5d810ff98","unresolved":false,"context_lines":[{"line_number":86,"context_line":"              volume_backed_live_migration: true"},{"line_number":87,"context_line":"              block_migration_for_live_migration: true"},{"line_number":88,"context_line":"              block_migrate_cinder_iscsi: true"},{"line_number":89,"context_line":"    post-run: playbooks/nova-live-migration/post-run.yaml"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"# TODO(lyarwood): The following jobs need to be written as part of the"},{"line_number":92,"context_line":"# migration to zuulv3 before nova-live-migration can be removed:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1f621f24_969f61b3","line":89,"range":{"start_line":89,"start_character":4,"end_line":89,"end_character":12},"updated":"2020-10-30 14:47:17.000000000","message":"can we run this as \u0027run\u0027 phase? with run-evacuate-hook role (same way it was done in devstack-tempest-ipv6 job where ipv6 checks are injected before run tests - https://github.com/openstack/tempest/blob/b439c974faab1d593ce14ecb3cbfd5467b0005cf/playbooks/devstack-tempest-ipv6.yaml).\n\nmaking it in run phase will centralize the test results and with post-run phase will end up the failing job as POST_FAILURE if any test fails which make confusion by seeing the zuul log that there are testr_results.html having all test passing but evacuate test fail.","commit_id":"c676b37c0048262aa13c6540496124ee58693368"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"69c98dcde233f352d7f792fab73e76685ecd7e38","unresolved":false,"context_lines":[{"line_number":86,"context_line":"              volume_backed_live_migration: true"},{"line_number":87,"context_line":"              block_migration_for_live_migration: true"},{"line_number":88,"context_line":"              block_migrate_cinder_iscsi: true"},{"line_number":89,"context_line":"    post-run: playbooks/nova-live-migration/post-run.yaml"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"# TODO(lyarwood): The following jobs need to be written as part of the"},{"line_number":92,"context_line":"# migration to zuulv3 before nova-live-migration can be removed:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1f621f24_a3380aa0","line":89,"range":{"start_line":89,"start_character":4,"end_line":89,"end_character":12},"in_reply_to":"1f621f24_969f61b3","updated":"2020-11-06 10:55:23.000000000","message":"That would require us to either copy or call the temptest/playbooks/devstack-tempest.yaml playbook right?\n\nI\u0027m not a huge fan of that, while I appreciate the post-run section failure might be confusing at first it should be obvious from the job output where we have failed etc.","commit_id":"c676b37c0048262aa13c6540496124ee58693368"}]}