)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38495,"name":"Ayo Edwin Kayode","display_name":"Ayo","email":"edwinkayodeayo@gmail.com","username":"ayok"},"change_message_id":"0f3f9dbff8fa3c1d05f5e91bcae353445d339223","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ed8f32f7_fd1978c4","updated":"2025-11-14 15:12:37.000000000","message":"if i\u0027m understanding this change correctly what it does is to check for out of band data and if its there it skips the process of using ipmi tool to collect hardware data? and move to the next step?","commit_id":"1218ae7c7b0d9e2ae84f4f7e0556a27ac8c39168"},{"author":{"_account_id":38495,"name":"Ayo Edwin Kayode","display_name":"Ayo","email":"edwinkayodeayo@gmail.com","username":"ayok"},"change_message_id":"3982860034da04da1ec6ab1770c3ee9b3b4f7084","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2f3551c0_9dfa67d2","in_reply_to":"b55db8bf_138d7ba4","updated":"2025-11-17 09:48:52.000000000","message":"The log message would definitely help during troubleshooting. Once the initial connection works, most people wouldn’t immediately suspect the BMC address if errors happen later especially since it’s uncommon for a BMC address to change. So having that log would save time.","commit_id":"1218ae7c7b0d9e2ae84f4f7e0556a27ac8c39168"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"843448bf7b6cc2b30f90b8451e32b3f9242825e0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b55db8bf_138d7ba4","in_reply_to":"eb3b95c8_11034383","updated":"2025-11-17 08:48:27.000000000","message":"that\u0027s a good question and a valid concern, although I think it will be handled correctly: if the BMC address change for any reason but it\u0027s not updated in the ironic configuration, the oob connection will just fail, the change needs to be applied manually anyway.\nI will add a log message to make it clear when the BMC detection is skipped to help with any troubleshooting.","commit_id":"1218ae7c7b0d9e2ae84f4f7e0556a27ac8c39168"},{"author":{"_account_id":38495,"name":"Ayo Edwin Kayode","display_name":"Ayo","email":"edwinkayodeayo@gmail.com","username":"ayok"},"change_message_id":"c9b7fa13c3cd3359a511ff0c5892d144ead3c5c9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"eb3b95c8_11034383","in_reply_to":"ed8f32f7_fd1978c4","updated":"2025-11-14 15:25:50.000000000","message":"will it pose an issue if any change is made to the bmc address or hardware spec and it skips the collection step as an initial out of band connection was made?","commit_id":"1218ae7c7b0d9e2ae84f4f7e0556a27ac8c39168"},{"author":{"_account_id":38495,"name":"Ayo Edwin Kayode","display_name":"Ayo","email":"edwinkayodeayo@gmail.com","username":"ayok"},"change_message_id":"135940670e738933fbe1a834d8666a9f6ade74af","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"30442f85_f2ccff0e","updated":"2025-11-17 09:59:23.000000000","message":"I’m still thinking in aws lol, I thought the update would be in some way automated","commit_id":"614ff04ce177a0895aa91a6362d50ddb2a6c4ed7"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"78094e5ea18503aa27676839efedb32d16b62f6e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4a774f01_9009a9e4","updated":"2025-11-19 08:16:01.000000000","message":"recheck ansible galaxy shenanighans","commit_id":"4f815d84c8d9e090239dac4e7eaf1e832171dde2"}],"ironic/api/controllers/v1/ramdisk.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"73f2128b4053ae7f05d8a314cc965ed81d97a241","unresolved":true,"context_lines":[{"line_number":54,"context_line":"            \u0027redfish\u0027, \u0027idrac-redfish\u0027,"},{"line_number":55,"context_line":"            \u0027ilo\u0027, \u0027ilo5\u0027, \u0027ilo6\u0027,"},{"line_number":56,"context_line":"            \u0027irmc\u0027"},{"line_number":57,"context_line":"        )"},{"line_number":58,"context_line":"        if skip_bmc_detect:"},{"line_number":59,"context_line":"            LOG.debug(\u0027Skipping BMC detection for node %(node)s with \u0027"},{"line_number":60,"context_line":"                      \u0027out-of-band management interface %(interface)s\u0027,"}],"source_content_type":"text/x-python","patch_set":5,"id":"e08ee4e6_b2bc25fc","line":57,"updated":"2025-11-21 20:39:17.000000000","message":"so everything but ipmi. I guess that is okay. :\\","commit_id":"4f815d84c8d9e090239dac4e7eaf1e832171dde2"}]}
