)]}'
{".zuul.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d9f88402b8bf54565c1c46e59b800a5b09096f91","unresolved":true,"context_lines":[{"line_number":226,"context_line":"            $NEUTRON_CONF:"},{"line_number":227,"context_line":"              nova:"},{"line_number":228,"context_line":"                live_migration_events: True"},{"line_number":229,"context_line":"    post-run: playbooks/nova-live-migration/post-run.yaml"},{"line_number":230,"context_line":""},{"line_number":231,"context_line":"- job:"},{"line_number":232,"context_line":"    name: nova-live-migration-ceph"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"b19e4fa4_bfd74ea1","line":229,"range":{"start_line":229,"start_character":0,"end_line":229,"end_character":57},"updated":"2023-09-01 21:17:16.000000000","message":"as we are disabling cinder here, we need to disable the evacuate bfv server test in this job which are: \n\nhttps://github.com/openstack/nova/blob/88c73e2931dcb276982612dbbdc3b849d4977c16/roles/run-evacuate-hook/files/test_negative_evacuate.sh#L36\n\nhttps://github.com/openstack/nova/blob/88c73e2931dcb276982612dbbdc3b849d4977c16/roles/run-evacuate-hook/files/test_evacuate.sh#L58","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"476eac37776a0a2164b34274beeadf0a6a27bbeb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"eaf7d53c_3b788873","updated":"2023-09-01 21:13:39.000000000","message":"Yeah, I think we just need to make that post run not do the bfv part if there\u0027s no cinder.","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"7b37606ed16b9877c82eff08d7fb0932ec1800a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"748797cc_fa3db45c","updated":"2023-09-01 21:08:36.000000000","message":"evacuate-bfv-test failing here is valid failure as cinder is disabled","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ca729dee4d8a4498a97ab1864a639cd612151ac3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"158065bb_d355b131","updated":"2023-09-01 19:08:42.000000000","message":"recheck evacuate test failing for connecting to libvirt/host \nhttps://zuul.opendev.org/t/openstack/build/475dab49cd33437a96ba81437779b875/log/controller/logs/screen-n-cpu.txt#14442","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"c0ba25b793c60c543fdc0d5753436c7c780ea331","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"63e76946_aa99352c","updated":"2023-09-02 12:03:11.000000000","message":"recheck nova-ovs-hybrid-plug \n\nhttps://zuul.openstack.org/builds?job_name\u003dnova-ovs-hybrid-plug\u0026result\u003dPOST_FAILURE\u0026result\u003dFAILURE\u0026result\u003dRETRY_LIMIT\u0026result\u003dNODE_FAILURE\u0026skip\u003d0\n\n`nova.exception.HypervisorUnavailable: Connection to the hypervisor is broken on host`\n https://zuul.opendev.org/t/openstack/build/40424718c3524bb78ca22cb9d5293b6c/log/controller/logs/screen-n-cpu.txt#14946\n\nhttps://zuul.opendev.org/t/openstack/build/b4cd9d137dd544e7ac6199d14c2d2221/log/controller/logs/screen-n-cpu.txt?severity\u003d4#14317","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"d5b6f3e10bd8af5bf44b0fbe5fde13f76bab0c9a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2cbf3ab1_3dad69ba","updated":"2023-09-01 20:59:06.000000000","message":"seems like this job is failing on same error again","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2f1edb4024ddc80d857f62641fcdc1fbd38f4b87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a9e25b14_27acec70","updated":"2023-09-01 19:07:37.000000000","message":"this lgtm, cinder and swift are disabled in all nodes. there are other failure in this job but those are not related to this change. \n\nhttps://zuul.opendev.org/t/openstack/build/475dab49cd33437a96ba81437779b875/log/controller/logs/screen-n-cpu.txt#14442","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b1b9d544c7eee64829dc2f71e5906c989e831bca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"2b54c1a4_64d80262","updated":"2023-09-01 17:11:23.000000000","message":"this shoudl work but it can also wait till monday os +2\n\nfeel free to merge if it passes but the imporant patch is the one below this","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b64470fbc251f1dfa1f415719b4d5860ab5b8337","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"aef7eeae_f3e51b36","in_reply_to":"63e76946_aa99352c","updated":"2023-09-02 16:51:49.000000000","message":"@Amit, this valid failure and due to this change. re check would not help. Dan and I discussed it in IRC on Friday and commented here. Sean can see on Monday if we can disable the bfv test in this job because that need cinder.","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"be6faef54232199fc619bfe1f0db8c8280dafe72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"129466a4_8ada2430","in_reply_to":"eaf7d53c_3b788873","updated":"2023-09-01 21:20:22.000000000","message":"yeah. or separating those in separate scripts itself will be easy","commit_id":"c99f9291b78c78bb32f8cefe60678ef680764ffe"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b2c9d9ed6394c98db871bb1e019862a21517ffe8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"398909da_04511e93","updated":"2023-09-13 17:40:09.000000000","message":"ill look at this later or tomorrow but at first glance it looks reasonable but i have not looked at thsi properly so no +/- votes","commit_id":"5455a655a3ef6462a3617d9f8f86bc241f18be68"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a998141f9aa4608bc47610308cd0a050c27a5370","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"37cf0d04_41382e0c","updated":"2023-09-14 19:40:52.000000000","message":"recheck ceph job failing for ssh timeout which is not related to this change. nova-ovs-hybrid-plug job passing.","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"000f13849ea2ebc2f9a8b8071848dc8455eb1e6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"c8d1ca4f_b4a98493","updated":"2023-09-15 14:05:42.000000000","message":"recheck failed migration with trunk known unstable test","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"a5c18ecdb95b8b9a69877cbc8ebd5381a6e99f15","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"39795e15_0e5f567a","updated":"2023-09-14 16:55:48.000000000","message":"recheck grenade failed, obviously unrelated to this non-grenade job change","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"c512e5e8c54f8ab05e5166ec96e2b9b42f86db7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1dd2484d_67f99b17","updated":"2023-09-16 17:10:33.000000000","message":"recheck test_live_migration_with_trunk test skip is fixed","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"56d4d5aca143341cc64bd4b4fe9393085def5959","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"f5c19e3c_d69cf35e","updated":"2023-09-15 17:30:36.000000000","message":"test_live_migration_with_trunk is failing again for tempest bug in ceph job, let\u0027s wait for tempest fix to merge bfore we re trigger CI here","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2183540b313bca79632218e76d866f7ff14c9b54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5686381d_601b2e3d","updated":"2023-09-15 14:28:07.000000000","message":"the hybrid plug job passed on the last run and the change looks good so +2w","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"421c5ace7cffa323eeb06e99b18da3b36c6e8a31","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e01f1826_28065f34","in_reply_to":"064ceec2_97830747","updated":"2023-09-15 17:29:28.000000000","message":"fixing that in https://review.opendev.org/c/openstack/tempest/+/895496","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ccedbf301ef96a029921a22a128e935e652522b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"064ceec2_97830747","in_reply_to":"7cb18dd4_b02317ab","updated":"2023-09-15 17:06:22.000000000","message":"\"tempest.lib.exceptions.InvalidParam: Invalid Parameter passed: 2024160/2033887 must be a valid launchpad number\"\n\nit is tempest bug in marking this known unstable test skip decorator. Ithought we skiped this test in https://review.opendev.org/c/openstack/tempest/+/895167/1/tempest/api/compute/admin/test_live_migration.py#261\n\nbut syntax error for unstable_test() decorator stop making it non voting.let me fix that","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2183540b313bca79632218e76d866f7ff14c9b54","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"7cb18dd4_b02317ab","in_reply_to":"c8d1ca4f_b4a98493","updated":"2023-09-15 14:28:07.000000000","message":"for what its worth test_live_migration_with_trunk is tested i the hybrid job and it only unstable with ovn.\n\nthe workflow on the neutron side is diffetn because how trunk ports are implemnted for ml2/ovs is diffent then for ml2/ovn.\n\nso if this continues to be a probalem we coud temporally skip it with ml2/ovn and still have some coverage with ml2/ovs\n\nthat said they will hopefully correct the issue in neutron sooner rather then later.","commit_id":"c486cc89dc11c2698cbf19fe8b2e477666331dee"}],"roles/run-evacuate-hook/files/test_evacuate.sh":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"457902ff1436092cae1715fa09337a27ba2183c5","unresolved":true,"context_lines":[{"line_number":47,"context_line":"    evacuate_and_wait_for_active evacuate-bfv-test"},{"line_number":48,"context_line":"fi"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# Make sure the servers moved."},{"line_number":51,"context_line":"for server in evacuate-test evacuate-bfv-test; do"},{"line_number":52,"context_line":"    host\u003d$(openstack server show ${server} -f value -c OS-EXT-SRV-ATTR:host)"},{"line_number":53,"context_line":"    if [[ ${host} !\u003d ${CONTROLLER_HOSTNAME} ]]; then"},{"line_number":54,"context_line":"        echo \"Unexpected host ${host} for server ${server} after evacuate.\""},{"line_number":55,"context_line":"        exit 7"},{"line_number":56,"context_line":"    fi"},{"line_number":57,"context_line":"done"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"# Cleanup test servers"},{"line_number":60,"context_line":"openstack server delete --wait evacuate-test"},{"line_number":61,"context_line":"if [ \"$have_cinder\" ]; then"}],"source_content_type":"text/x-sh","patch_set":6,"id":"d24755ff_8c30941c","line":58,"range":{"start_line":50,"start_character":0,"end_line":58,"end_character":0},"updated":"2023-09-13 18:17:46.000000000","message":"we need to put these also in cinder check block otherwise it is failing here\n\nhttps://zuul.opendev.org/t/openstack/build/5efc0aaf88874c45b2eb68d8e9cf4a0d/console","commit_id":"5455a655a3ef6462a3617d9f8f86bc241f18be68"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"b2c9d9ed6394c98db871bb1e019862a21517ffe8","unresolved":true,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"# Cleanup test servers"},{"line_number":60,"context_line":"openstack server delete --wait evacuate-test"},{"line_number":61,"context_line":"if [ \"$have_cinder\" ]; then"},{"line_number":62,"context_line":"    openstack server delete --wait evacuate-bfv-test"},{"line_number":63,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":6,"id":"82140863_8c343ea7","line":61,"range":{"start_line":61,"start_character":0,"end_line":61,"end_character":27},"updated":"2023-09-13 17:40:09.000000000","message":"i would be tempted to just do the qury again but sure  this works too i guess","commit_id":"5455a655a3ef6462a3617d9f8f86bc241f18be68"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"2183540b313bca79632218e76d866f7ff14c9b54","unresolved":false,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"# Cleanup test servers"},{"line_number":60,"context_line":"openstack server delete --wait evacuate-test"},{"line_number":61,"context_line":"if [ \"$have_cinder\" ]; then"},{"line_number":62,"context_line":"    openstack server delete --wait evacuate-bfv-test"},{"line_number":63,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":6,"id":"83a31c6f_c78bd7f2","line":61,"range":{"start_line":61,"start_character":0,"end_line":61,"end_character":27},"in_reply_to":"0e73de5f_f752ff62","updated":"2023-09-15 14:28:07.000000000","message":"Ack","commit_id":"5455a655a3ef6462a3617d9f8f86bc241f18be68"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"75e418968f2ac3cc69e81db931e1fe5848712205","unresolved":true,"context_lines":[{"line_number":58,"context_line":""},{"line_number":59,"context_line":"# Cleanup test servers"},{"line_number":60,"context_line":"openstack server delete --wait evacuate-test"},{"line_number":61,"context_line":"if [ \"$have_cinder\" ]; then"},{"line_number":62,"context_line":"    openstack server delete --wait evacuate-bfv-test"},{"line_number":63,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":6,"id":"0e73de5f_f752ff62","line":61,"range":{"start_line":61,"start_character":0,"end_line":61,"end_character":27},"in_reply_to":"82140863_8c343ea7","updated":"2023-09-13 17:41:52.000000000","message":"Yeah, just trying to save a couple seconds and the potential for an API call failure.","commit_id":"5455a655a3ef6462a3617d9f8f86bc241f18be68"}],"roles/run-evacuate-hook/files/test_negative_evacuate.sh":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"cd68d30ef57205f199f88413102ef4932087bad9","unresolved":true,"context_lines":[{"line_number":33,"context_line":"}"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"evacuate_and_wait_for_error evacuate-test"},{"line_number":36,"context_line":"if openstack endpoint list | grep cinder; then"},{"line_number":37,"context_line":"    evacuate_and_wait_for_error evacuate-bfv-test"},{"line_number":38,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":5,"id":"564e8c58_c622777e","line":38,"range":{"start_line":36,"start_character":0,"end_line":38,"end_character":2},"updated":"2023-09-13 16:17:55.000000000","message":"+1, and you need to do this in test_evacuate.sh[1] also as that is also called after negative test[2]\n\n[1] https://github.com/openstack/nova/blob/53012f1c55072c42ced267a2b1adef0a669d9f45/roles/run-evacuate-hook/files/test_evacuate.sh#L58\n\n[2] https://github.com/openstack/nova/blob/53012f1c55072c42ced267a2b1adef0a669d9f45/roles/run-evacuate-hook/tasks/main.yaml#L90","commit_id":"4e1d8369f5b25a81f6e850ca0250c8ca03929266"}]}