)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"a6d59675c5d1a8ab589dd1d6899329746a7847d5","unresolved":true,"context_lines":[{"line_number":21,"context_line":"Related-Bug: 2086715"},{"line_number":22,"context_line":"Implements: ironic-vnc-console"},{"line_number":23,"context_line":"Change-Id: I077aa219cc3cefc9fb431132a32a338fdfda246b"},{"line_number":24,"context_line":"Signed-off-by: abhibongale \u003cabhishekbongale@outlook.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6b8d20dc_69f6f595","line":24,"updated":"2026-03-26 22:11:04.000000000","message":"If the following is added then the ironic-tempest-bios-ipmi-autodetect job will show whether the graphical console validation is passing:\n\nDepends-On: https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077","commit_id":"1520351cf39a29be11abd80d2c9f39444a71c655"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"ad98e2f748bf6fb17d6a50064373fd357ec2d300","unresolved":false,"context_lines":[{"line_number":21,"context_line":"Related-Bug: 2086715"},{"line_number":22,"context_line":"Implements: ironic-vnc-console"},{"line_number":23,"context_line":"Change-Id: I077aa219cc3cefc9fb431132a32a338fdfda246b"},{"line_number":24,"context_line":"Signed-off-by: abhibongale \u003cabhishekbongale@outlook.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"ced96788_fef6d004","line":24,"in_reply_to":"6b8d20dc_69f6f595","updated":"2026-03-30 10:27:21.000000000","message":"Done","commit_id":"1520351cf39a29be11abd80d2c9f39444a71c655"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"ece9eb4b29307d3e04c4e10d37edb1b6569dd812","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Related-Bug: 2086715"},{"line_number":23,"context_line":"Implements: ironic-vnc-console"},{"line_number":24,"context_line":"Change-Id: I077aa219cc3cefc9fb431132a32a338fdfda246b"},{"line_number":25,"context_line":"Signed-off-by: abhibongale \u003cabhishekbongale@outlook.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"e13ecaf1_f8821a84","line":25,"updated":"2026-04-06 23:09:17.000000000","message":"Related-Bug: #2147206","commit_id":"72aaa5d895583e52d76f7a402950074c1e841b6e"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"42896a5c5e2e0d33be6dbcb52f4af4ffefdbbb6b","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Related-Bug: 2086715"},{"line_number":23,"context_line":"Implements: ironic-vnc-console"},{"line_number":24,"context_line":"Change-Id: I077aa219cc3cefc9fb431132a32a338fdfda246b"},{"line_number":25,"context_line":"Signed-off-by: abhibongale \u003cabhishekbongale@outlook.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"47592405_cc5db8b6","line":25,"in_reply_to":"e13ecaf1_f8821a84","updated":"2026-04-07 08:27:12.000000000","message":"Done","commit_id":"72aaa5d895583e52d76f7a402950074c1e841b6e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"a6d59675c5d1a8ab589dd1d6899329746a7847d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4c3cdc80_8e74d2c7","updated":"2026-03-26 22:11:04.000000000","message":"Looks good but we need to run the validation tests","commit_id":"1520351cf39a29be11abd80d2c9f39444a71c655"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"e93cb0d06b7c0152415ca568d1479dbba372f026","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"86d10e9d_e7cd58de","updated":"2026-03-31 02:56:57.000000000","message":"https://d2cb2a258655973b4f06-8bf91e93364f14a67a943838d2417a14.ssl.cf1.rackcdn.com/openstack/2f99040ff7694cc0aae830f234be4f20/testr_results.html\n\nThis test is failing because it expects the default interface to be called \u0027fake\u0027 not \u0027fake-graphical\u0027.\n\nI think it would be acceptable for this assert[1] to be changed from an assertEqual to an assertStartsWith.\n\n[1] https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/tests/api/admin/test_nodes.py#L772\n\nI\u0027d suggest proposing a standalone change to do this, and add a Depends-On to this change to pull it in","commit_id":"b3116a40745204ff19e6d9d44335ad4f0ba64644"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"e2677c5fa1f460132352a44b7021303a27f051c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"fa88e525_dd965297","updated":"2026-04-01 08:19:34.000000000","message":"recheck","commit_id":"72aaa5d895583e52d76f7a402950074c1e841b6e"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"757ab6ae9a6b88fa094b2850bcf222ebcb8a3c4e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"38e37ed3_207b9276","updated":"2026-04-14 21:38:26.000000000","message":"https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/982077 shows that this change is correct, re-enables the graphical console scenario test without causing regressions in other jobs.","commit_id":"c2d6761a6f2977e02f28540456467b635202a610"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"2ac864df1534ab3e46c8e68fd052d7562538614d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"29af49e7_7123c281","updated":"2026-04-07 15:41:19.000000000","message":"recheck","commit_id":"c2d6761a6f2977e02f28540456467b635202a610"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"7ea151710a1edb504ba52e8b1e634aa880af8250","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"f458183c_d336c19e","updated":"2026-04-09 07:47:08.000000000","message":"recheck ironic-tempest-uefi-redfish-vmedia POST_FAILURE","commit_id":"c2d6761a6f2977e02f28540456467b635202a610"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"23e442bec2f4192217d58432ce7ef949473b8499","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"1537c341_00f5cf17","updated":"2026-04-08 23:30:29.000000000","message":"recheck unable to access \u0027https://opendev.org/openstack/hacking/\u0027: gnutls_handshake() failed","commit_id":"c2d6761a6f2977e02f28540456467b635202a610"}],"devstack/lib/ironic":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"a4b4af1b00fd581ae12d777da8a015fdc69bc3a3","unresolved":true,"context_lines":[{"line_number":263,"context_line":"if is_service_enabled ir-novnc; then"},{"line_number":264,"context_line":"    # When testing graphical console, use only fake-graphical to ensure it is selected"},{"line_number":265,"context_line":"    # as the default console interface. This prevents Ironic from choosing no-console instead."},{"line_number":266,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake-graphical\"}"},{"line_number":267,"context_line":"else"},{"line_number":268,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake,no-console\"}"},{"line_number":269,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"0ce476e3_546e3251","line":266,"updated":"2026-03-29 21:18:18.000000000","message":"OK we\u0027re in a situation where ironic_tempest_plugin.tests.api.admin.test_nodes.TestHardwareInterfaces requires no-console interface to be enabled, but if it is then it gets chosen instead of fake-graphical in the scenario test. We\u0027re going to need to deal with this by setting IRONIC_DEFAULT_CONSOLE_INTERFACE.\n\nI\u0027d suggest trying the following:\n\n    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake-graphical,fake,no-console\"}\n    if is_service_enabled ir-novnc; then\n      IRONIC_DEFAULT_DEPLOY_INTERFACE\u003d${IRONIC_DEFAULT_DEPLOY_INTERFACE:-fake-graphical}\n    else\n      IRONIC_DEFAULT_DEPLOY_INTERFACE\u003d${IRONIC_DEFAULT_DEPLOY_INTERFACE:-}\n    fi","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"51fbc8554bd355a90103c6343e2ca9531cb54fb8","unresolved":false,"context_lines":[{"line_number":263,"context_line":"if is_service_enabled ir-novnc; then"},{"line_number":264,"context_line":"    # When testing graphical console, use only fake-graphical to ensure it is selected"},{"line_number":265,"context_line":"    # as the default console interface. This prevents Ironic from choosing no-console instead."},{"line_number":266,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake-graphical\"}"},{"line_number":267,"context_line":"else"},{"line_number":268,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake,no-console\"}"},{"line_number":269,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"7d636faf_e25a2eb8","line":266,"in_reply_to":"0ce476e3_546e3251","updated":"2026-03-30 10:17:48.000000000","message":"Acknowledged","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"db1a48a487cf6600992e442c47ffa0fea1757978","unresolved":true,"context_lines":[{"line_number":263,"context_line":"if is_service_enabled ir-novnc; then"},{"line_number":264,"context_line":"    # When testing graphical console, use only fake-graphical to ensure it is selected"},{"line_number":265,"context_line":"    # as the default console interface. This prevents Ironic from choosing no-console instead."},{"line_number":266,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake-graphical\"}"},{"line_number":267,"context_line":"else"},{"line_number":268,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake,no-console\"}"},{"line_number":269,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"db2a1f86_51e1e474","line":266,"in_reply_to":"324b55f0_3722309c","updated":"2026-03-30 19:24:22.000000000","message":"no-console always needs to be enabled for this to pass: https://1f381f2949ecc1bf5cc8-62271a11c17b6e7ed1bc2d4fe711d2ec.ssl.cf5.rackcdn.com/openstack/4a8bc72c086a4cf7aa08c3b608a28b1c/testr_results.html\n\nI think if line #266 is changed to fake-graphical,fake,no-console then that may be enough for both tests to pass","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"d7a77b437421ff7cdcbc7af37bc98817004e257f","unresolved":true,"context_lines":[{"line_number":263,"context_line":"if is_service_enabled ir-novnc; then"},{"line_number":264,"context_line":"    # When testing graphical console, use only fake-graphical to ensure it is selected"},{"line_number":265,"context_line":"    # as the default console interface. This prevents Ironic from choosing no-console instead."},{"line_number":266,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake-graphical\"}"},{"line_number":267,"context_line":"else"},{"line_number":268,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake,no-console\"}"},{"line_number":269,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"324b55f0_3722309c","line":266,"in_reply_to":"5d2912f3_27b3934a","updated":"2026-03-30 15:37:20.000000000","message":"Also latest build: https://zuul.opendev.org/t/openstack/build/f142b71e23c5480b8e9923e05bb61406 \n\nhttps://3ca3ec686f4ad84d08c5-f0dd46c0d8845d18269bbfd0711d9448.ssl.cf5.rackcdn.com/openstack/f142b71e23c5480b8e9923e05bb61406/controller/logs/etc/ironic/ironic_conf.txt\n```\ndefault_console_interface \u003d fake-graphical\nenabled_console_interfaces \u003d fake-graphical,fake,no-console\n```\n\nfailed on 2 reset tests:  MismatchError: \u0027fake\u0027 !\u003d \u0027fake-graphical\u0027\n\nI think I should also add the enabled_console_interfaces inside the if conditions","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"a3677e9da2285cedda06c26fd0c6ad274eaed8ef","unresolved":true,"context_lines":[{"line_number":263,"context_line":"if is_service_enabled ir-novnc; then"},{"line_number":264,"context_line":"    # When testing graphical console, use only fake-graphical to ensure it is selected"},{"line_number":265,"context_line":"    # as the default console interface. This prevents Ironic from choosing no-console instead."},{"line_number":266,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake-graphical\"}"},{"line_number":267,"context_line":"else"},{"line_number":268,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake,no-console\"}"},{"line_number":269,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"5d2912f3_27b3934a","line":266,"in_reply_to":"7d636faf_e25a2eb8","updated":"2026-03-30 13:20:54.000000000","message":"I guess we need use IRONIC_DEFAULT_CONSOLE_INTERFACE instead of IRONIC_DEFAULT_DEPLOY_INTERFACE\n\nbecause build https://zuul.opendev.org/t/openstack/build/13aabd6a792c4d02b92abe342ccb3be2 indicate that?\n\n`++ /opt/stack/ironic/devstack/lib/ironic:configure_ironic_conductor:2008 :   iniset /etc/ironic/ironic.conf DEFAULT enabled_deploy_interfaces fake,direct,ramdisk`\n\nfake,direct,ramdisk are the ones available. AFAIK fake-graphical is a console interface. I guess I will we need to update it to IRONIC_DEFAULT_CONSOLE_INTERFACE.","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"2e03767eac7b24292af36d8c196bd41d24026841","unresolved":false,"context_lines":[{"line_number":263,"context_line":"if is_service_enabled ir-novnc; then"},{"line_number":264,"context_line":"    # When testing graphical console, use only fake-graphical to ensure it is selected"},{"line_number":265,"context_line":"    # as the default console interface. This prevents Ironic from choosing no-console instead."},{"line_number":266,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake-graphical\"}"},{"line_number":267,"context_line":"else"},{"line_number":268,"context_line":"    IRONIC_ENABLED_CONSOLE_INTERFACES\u003d${IRONIC_ENABLED_CONSOLE_INTERFACES:-\"fake,no-console\"}"},{"line_number":269,"context_line":"fi"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"143de740_fc612d6c","line":266,"in_reply_to":"db2a1f86_51e1e474","updated":"2026-03-30 21:00:59.000000000","message":"Done","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"a4b4af1b00fd581ae12d777da8a015fdc69bc3a3","unresolved":true,"context_lines":[{"line_number":280,"context_line":"IRONIC_DEFAULT_BIOS_INTERFACE\u003d${IRONIC_DEFAULT_BIOS_INTERFACE:-}"},{"line_number":281,"context_line":"IRONIC_DEFAULT_BOOT_INTERFACE\u003d${IRONIC_DEFAULT_BOOT_INTERFACE:-}"},{"line_number":282,"context_line":"IRONIC_DEFAULT_CONSOLE_INTERFACE\u003d${IRONIC_DEFAULT_CONSOLE_INTERFACE:-}"},{"line_number":283,"context_line":"IRONIC_DEFAULT_DEPLOY_INTERFACE\u003d${IRONIC_DEFAULT_DEPLOY_INTERFACE:-}"},{"line_number":284,"context_line":"IRONIC_DEFAULT_INSPECT_INTERFACE\u003d${IRONIC_DEFAULT_INSPECT_INTERFACE:-}"},{"line_number":285,"context_line":"IRONIC_DEFAULT_MANAGEMENT_INTERFACE\u003d${IRONIC_DEFAULT_MANAGEMENT_INTERFACE:-}"},{"line_number":286,"context_line":"IRONIC_DEFAULT_NETWORK_INTERFACE\u003d${IRONIC_DEFAULT_NETWORK_INTERFACE:-}"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"cc840f09_ec79bc91","line":283,"updated":"2026-03-29 21:18:18.000000000","message":"This is now set above, so it can be deleted here","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"},{"author":{"_account_id":37924,"name":"Abhishek Bongale","display_name":"abongale","email":"abhishekbongale@outlook.com","username":"abhibongale"},"change_message_id":"51fbc8554bd355a90103c6343e2ca9531cb54fb8","unresolved":false,"context_lines":[{"line_number":280,"context_line":"IRONIC_DEFAULT_BIOS_INTERFACE\u003d${IRONIC_DEFAULT_BIOS_INTERFACE:-}"},{"line_number":281,"context_line":"IRONIC_DEFAULT_BOOT_INTERFACE\u003d${IRONIC_DEFAULT_BOOT_INTERFACE:-}"},{"line_number":282,"context_line":"IRONIC_DEFAULT_CONSOLE_INTERFACE\u003d${IRONIC_DEFAULT_CONSOLE_INTERFACE:-}"},{"line_number":283,"context_line":"IRONIC_DEFAULT_DEPLOY_INTERFACE\u003d${IRONIC_DEFAULT_DEPLOY_INTERFACE:-}"},{"line_number":284,"context_line":"IRONIC_DEFAULT_INSPECT_INTERFACE\u003d${IRONIC_DEFAULT_INSPECT_INTERFACE:-}"},{"line_number":285,"context_line":"IRONIC_DEFAULT_MANAGEMENT_INTERFACE\u003d${IRONIC_DEFAULT_MANAGEMENT_INTERFACE:-}"},{"line_number":286,"context_line":"IRONIC_DEFAULT_NETWORK_INTERFACE\u003d${IRONIC_DEFAULT_NETWORK_INTERFACE:-}"}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"23902b28_1773bc6f","line":283,"in_reply_to":"cc840f09_ec79bc91","updated":"2026-03-30 10:17:48.000000000","message":"Acknowledged","commit_id":"2168803a9d50d46c855c9d09d09236b8b2e81c5c"}]}
