)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32704,"name":"Alfredo Garcia","email":"alfrgarc@redhat.com","username":"alfrgarc"},"change_message_id":"d297cd9d3d80e23383c4a5b3a5c6c8e436c61f7a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b383f63c_8022cf18","updated":"2026-05-04 12:50:34.000000000","message":"Good patch, it\u0027s also good that you commented out those lines for ironic-tempest-bios-ipmi-autodetect instead of deleting them so we can remember to re-enable this in the future when the job becomes stable","commit_id":"426e8a9f3334773b3329840d06676f742dbd081e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a28884684b47ed9389f571d4283ab794b98b56e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"74b4706f_17dc6102","updated":"2026-05-04 19:02:23.000000000","message":"this is ironic job which should be mark non voting there. I mean we can make it non voting here but would like to know if it is running voting in ironic but failing here","commit_id":"7d16b857fae6e30a71c8a3a85e07f66a2216fe90"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b7a34c0ab6c9e337acd81a8ae9bff974aec59f20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c672069f_6420aecd","in_reply_to":"31e40966_9780acdf","updated":"2026-05-04 19:22:20.000000000","message":"I see, I think, in that case we should make it non-voting in ironic as many place we run it for example, tempest, devstack etc.","commit_id":"7d16b857fae6e30a71c8a3a85e07f66a2216fe90"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"2cb1afd42ee703a6d54798029dc0a1957b2784bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"31e40966_9780acdf","in_reply_to":"74b4706f_17dc6102","updated":"2026-05-04 19:18:13.000000000","message":"It seems to be failing in both:\n\nhttps://zuul.opendev.org/t/openstack/builds?job_name\u003dironic-tempest-bios-ipmi-autodetect\u0026project\u003dopenstack/tempest\n\nhttps://zuul.opendev.org/t/openstack/builds?job_name\u003dironic-tempest-bios-ipmi-autodetect\u0026project\u003dopenstack/ironic\n\nThanks for adding Ironic to the bug, I just got tired of running re-check with the same outcome.","commit_id":"7d16b857fae6e30a71c8a3a85e07f66a2216fe90"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2a359b57247932c4e23576d9a4b97cf98443906c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"70b7dcee_a5b6cd6e","in_reply_to":"88faf345_8239df10","updated":"2026-05-04 19:42:28.000000000","message":"Technically, if you mark it non-voting in the job definition, then it will not block Tempest or other gates that use it.  here https://opendev.org/openstack/ironic/src/branch/master/zuul.d/ironic-jobs.yaml#L1130\n\nIf you are marking only in ironic check and gate pipeline then yes we need to mark non voting everywhere it is used. That is why my suggestion is to mark it as non-voting in the definition, so that we do not need to update its usage now or once it is fixed.","commit_id":"7d16b857fae6e30a71c8a3a85e07f66a2216fe90"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"e6f1515d2540871fb6800df1d954eab282869307","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"88faf345_8239df10","in_reply_to":"c672069f_6420aecd","updated":"2026-05-04 19:35:45.000000000","message":"Ack, let me do that, but I think we\u0027d still need to mark non-voting here as well, and also remove from gate queue.","commit_id":"7d16b857fae6e30a71c8a3a85e07f66a2216fe90"}],"zuul.d/project.yaml":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a0ba29d30f88e4fec838df6567483356434f1a3e","unresolved":true,"context_lines":[{"line_number":120,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":121,"context_line":"        - tempest-full-preprov-creds:"},{"line_number":122,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":123,"context_line":"        # NOTE(haleyb): Restore to voting and put back in gate queue below"},{"line_number":124,"context_line":"        # once a fix for this merges:"},{"line_number":125,"context_line":"        # https://bugs.launchpad.net/tempest/+bug/2150896"},{"line_number":126,"context_line":"        - ironic-tempest-bios-ipmi-autodetect:"},{"line_number":127,"context_line":"            voting: false"},{"line_number":128,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":129,"context_line":"        - openstack-tox-bashate:"},{"line_number":130,"context_line":"            irrelevant-files: *tempest-irrelevant-files-2"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fb02505_930195af","line":127,"range":{"start_line":123,"start_character":0,"end_line":127,"end_character":25},"updated":"2026-05-04 19:43:58.000000000","message":"This job is used as ironic testing in many place for example tempest, devstack etc and instead of marking non voting on usage side, I will suggest to mark it in the definition so that it can be controlled from single place:  https://opendev.org/openstack/ironic/src/branch/master/zuul.d/ironic-jobs.yaml#L1130","commit_id":"3be9659e418e6abc71666a69b35abcfd3853bb52"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"68761e07f8fee4f3a8d9822a9a0138366c24662f","unresolved":true,"context_lines":[{"line_number":120,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":121,"context_line":"        - tempest-full-preprov-creds:"},{"line_number":122,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":123,"context_line":"        # NOTE(haleyb): Restore to voting and put back in gate queue below"},{"line_number":124,"context_line":"        # once a fix for this merges:"},{"line_number":125,"context_line":"        # https://bugs.launchpad.net/tempest/+bug/2150896"},{"line_number":126,"context_line":"        - ironic-tempest-bios-ipmi-autodetect:"},{"line_number":127,"context_line":"            voting: false"},{"line_number":128,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":129,"context_line":"        - openstack-tox-bashate:"},{"line_number":130,"context_line":"            irrelevant-files: *tempest-irrelevant-files-2"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"70af1160_433b55b7","line":127,"range":{"start_line":123,"start_character":0,"end_line":127,"end_character":25},"in_reply_to":"6adb9fb7_b524f5a9","updated":"2026-05-04 20:04:17.000000000","message":"++, thanks but worth to send it on ML to avoid unnecessary running test again until it si fixed","commit_id":"3be9659e418e6abc71666a69b35abcfd3853bb52"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"096ed98ad6825855722b92f32311848c443ef52d","unresolved":true,"context_lines":[{"line_number":120,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":121,"context_line":"        - tempest-full-preprov-creds:"},{"line_number":122,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":123,"context_line":"        # NOTE(haleyb): Restore to voting and put back in gate queue below"},{"line_number":124,"context_line":"        # once a fix for this merges:"},{"line_number":125,"context_line":"        # https://bugs.launchpad.net/tempest/+bug/2150896"},{"line_number":126,"context_line":"        - ironic-tempest-bios-ipmi-autodetect:"},{"line_number":127,"context_line":"            voting: false"},{"line_number":128,"context_line":"            irrelevant-files: *tempest-irrelevant-files"},{"line_number":129,"context_line":"        - openstack-tox-bashate:"},{"line_number":130,"context_line":"            irrelevant-files: *tempest-irrelevant-files-2"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"6adb9fb7_b524f5a9","line":127,"range":{"start_line":123,"start_character":0,"end_line":127,"end_character":25},"in_reply_to":"9fb02505_930195af","updated":"2026-05-04 20:00:33.000000000","message":"Well my change did get a response from Julia that this should fix the issue:\n\nhttps://review.opendev.org/c/openstack/ironic/+/987239\n\nIf I see that pass consistently I\u0027ll do some rechecks here and drop this change.","commit_id":"3be9659e418e6abc71666a69b35abcfd3853bb52"}]}