)]}'
{"ironic/drivers/modules/redfish/boot.py":[{"author":{"_account_id":35555,"name":"Aaron S","display_name":"as0","email":"as3310@protonmail.com","username":"as0"},"change_message_id":"b0afec50606eb5646f6b1cbce5585dacc7eccd2d","unresolved":true,"context_lines":[{"line_number":387,"context_line":"                continue"},{"line_number":388,"context_line":""},{"line_number":389,"context_line":"        if v_media.inserted:"},{"line_number":390,"context_line":"            if v_media.image !\u003d boot_url:"},{"line_number":391,"context_line":"                # A different image is mounted in this slot; leave it alone"},{"line_number":392,"context_line":"                # and look for a free device to use instead."},{"line_number":393,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":2,"id":"72d419e6_b79bb1d2","line":390,"updated":"2026-06-01 14:29:39.000000000","message":"IMO ejecting any vmedia in all slots would be preferable? As having vmedia mounted in another slot could cause issues as part of Ironic provisioning or other operations?","commit_id":"84a50f67460a79650dff29e7d3c05ec443156df3"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"559454f867930bfeac17fbe8eaee104dc9609cfd","unresolved":true,"context_lines":[{"line_number":387,"context_line":"                continue"},{"line_number":388,"context_line":""},{"line_number":389,"context_line":"        if v_media.inserted:"},{"line_number":390,"context_line":"            if v_media.image !\u003d boot_url:"},{"line_number":391,"context_line":"                # A different image is mounted in this slot; leave it alone"},{"line_number":392,"context_line":"                # and look for a free device to use instead."},{"line_number":393,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":2,"id":"b4dbfddd_d3581e49","line":390,"in_reply_to":"47766ee0_aa328217","updated":"2026-06-05 14:04:48.000000000","message":"Yeah, I think it would be good to just eject anything detected in general.","commit_id":"84a50f67460a79650dff29e7d3c05ec443156df3"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"e254709a5bc824454db47acc5bd7f74fd6363276","unresolved":true,"context_lines":[{"line_number":387,"context_line":"                continue"},{"line_number":388,"context_line":""},{"line_number":389,"context_line":"        if v_media.inserted:"},{"line_number":390,"context_line":"            if v_media.image !\u003d boot_url:"},{"line_number":391,"context_line":"                # A different image is mounted in this slot; leave it alone"},{"line_number":392,"context_line":"                # and look for a free device to use instead."},{"line_number":393,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":2,"id":"47766ee0_aa328217","line":390,"in_reply_to":"72d419e6_b79bb1d2","updated":"2026-06-01 15:55:43.000000000","message":"++","commit_id":"84a50f67460a79650dff29e7d3c05ec443156df3"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"63075f30e0d0d3e2eccfc33173aa5ffaa3b14e5c","unresolved":false,"context_lines":[{"line_number":387,"context_line":"                continue"},{"line_number":388,"context_line":""},{"line_number":389,"context_line":"        if v_media.inserted:"},{"line_number":390,"context_line":"            if v_media.image !\u003d boot_url:"},{"line_number":391,"context_line":"                # A different image is mounted in this slot; leave it alone"},{"line_number":392,"context_line":"                # and look for a free device to use instead."},{"line_number":393,"context_line":"                continue"}],"source_content_type":"text/x-python","patch_set":2,"id":"3dff417e_78df5f14","line":390,"in_reply_to":"b4dbfddd_d3581e49","updated":"2026-06-08 12:28:00.000000000","message":"Acknowledged","commit_id":"84a50f67460a79650dff29e7d3c05ec443156df3"},{"author":{"_account_id":36770,"name":"cid","display_name":"cid","email":"cid@gr-oss.io","username":"cidelight","status":"@gr-oss upstream: Doing good IRONIC things..."},"change_message_id":"65a2d6a3e1ec6b91db9be3936d3b676374434552","unresolved":true,"context_lines":[{"line_number":357,"context_line":"        acceptable_id \u003d None"},{"line_number":358,"context_line":""},{"line_number":359,"context_line":"    # NOTE(cid): some BMCs (observed on Dell iDRAC) keep reporting media as"},{"line_number":360,"context_line":"    # inserted for a short window after a successful eject. Since the ISO URL"},{"line_number":361,"context_line":"    # is derived from the node UUID, that stale URL matches the one we are"},{"line_number":362,"context_line":"    # about to insert, so trusting it would skip the insert and the node would"},{"line_number":363,"context_line":"    # boot with nothing mounted. Eject any media from every usable slot of"},{"line_number":364,"context_line":"    # this device type and re-insert into the first slot; the retry below"}],"source_content_type":"text/x-python","patch_set":3,"id":"baa8fde6_2c860685","line":361,"range":{"start_line":360,"start_character":60,"end_line":361,"end_character":35},"updated":"2026-06-10 02:45:55.000000000","message":"Why do I feel the need to want to reduce the amount of claim this phrase makes?","commit_id":"1069c0f1e03aeb1f2e74eea14a2b48a462d98710"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"ec44185301b9bee6f85466172433ae7ef4f03990","unresolved":true,"context_lines":[{"line_number":362,"context_line":"    # about to insert, so trusting it would skip the insert and the node would"},{"line_number":363,"context_line":"    # boot with nothing mounted. Eject any media from every usable slot of"},{"line_number":364,"context_line":"    # this device type and re-insert into the first slot; the retry below"},{"line_number":365,"context_line":"    # (story 2008504) covers an eject still in progress edge case."},{"line_number":366,"context_line":"    try:"},{"line_number":367,"context_line":"        _eject_vmedia_from_resource(task, resource, boot_device)"},{"line_number":368,"context_line":"    except sushy.exceptions.SushyError as exc:"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a82cd9e_c739f2d4","line":365,"updated":"2026-06-09 22:32:26.000000000","message":"This change is likely fine as-is, but a further enhancement might be putting some kind of identifier in the URL so we can tell if it\u0027s [current media] or [old media]. This may not be possible or have security implications, but might be worth a look to harden up this code.","commit_id":"1069c0f1e03aeb1f2e74eea14a2b48a462d98710"}]}
