)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14228,"name":"Clif Houck","email":"me@clifhouck.com","username":"clif_h"},"change_message_id":"2583cdd0e3ce2be755f03e3c14561643d135d4b1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f1ea9c79_cf2a6fb0","updated":"2026-05-06 17:23:23.000000000","message":"This may need to be re-worked in light of https://bugs.launchpad.net/ironic-python-agent/+bug/2148310/comments/10 This initial patch was written prior to that comment. Leaving up for now since it\u0027s a pretty small patch as it stands and should only take a few minutes for an initial review.","commit_id":"51932a8b10aba90447082d4b00344eaba8c6e3bd"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"dda0bfabf96def05530c7a894e5e08643f42e6d0","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bf74352c_86e21c84","in_reply_to":"abb4053a_2f3ff832","updated":"2026-05-07 11:57:20.000000000","message":"Yes, this (missed this comment)","commit_id":"51932a8b10aba90447082d4b00344eaba8c6e3bd"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"893b2d182754833ee8b4e50ff91045554753f4a1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"abb4053a_2f3ff832","in_reply_to":"f1ea9c79_cf2a6fb0","updated":"2026-05-06 18:54:18.000000000","message":"This could work, but it does block out users who have an image with UEFI loader artifacts. In such a case, I\u0027d wire this in very close to the issue and not at the front end of prepare_image. In otherwords, in install_bootloader.","commit_id":"51932a8b10aba90447082d4b00344eaba8c6e3bd"},{"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":"0e5faeb311fc6dcddf17d3c857634914bea08798","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dd7d57e3_e48014b1","updated":"2026-05-20 17:33:47.000000000","message":"release note","commit_id":"0166df6a9036ada6add0d67f513eee92f38606dd"}],"ironic_python_agent/config.py":[{"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":"3ffae0059e98bacc8afd80ed9d904ca4ef55020c","unresolved":true,"context_lines":[{"line_number":439,"context_line":"                     \u0027because the bootc command inside of the ramdisk \u0027"},{"line_number":440,"context_line":"                     \u0027comes from the supplied image to be deployed.\u0027),"},{"line_number":441,"context_line":"    cfg.BoolOpt(\u0027enable_partition_images\u0027,"},{"line_number":442,"context_line":"                default\u003dFalse,"},{"line_number":443,"context_line":"                help\u003d\u0027Enables the \"partition\" image type. Disabled by default \u0027"},{"line_number":444,"context_line":"                     \u0027due to security risks.\u0027),"},{"line_number":445,"context_line":"]"}],"source_content_type":"text/x-python","patch_set":1,"id":"f84652cf_6d12b4b6","line":442,"updated":"2026-05-06 18:59:19.000000000","message":"Should we make this enablable via kernel command line or conductor? It seems like asking anyone using existing partition image support to build a custom IPA might be too heavy, but obviously there\u0027s added security risk to having it remotely toggled.","commit_id":"51932a8b10aba90447082d4b00344eaba8c6e3bd"}],"ironic_python_agent/extensions/standby.py":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"b598c4521637e1dc2177fb18ce6cdd42509e457c","unresolved":true,"context_lines":[{"line_number":1011,"context_line":"        \"\"\""},{"line_number":1012,"context_line":"        LOG.debug(\u0027Preparing image %s\u0027, image_info[\u0027id\u0027])"},{"line_number":1013,"context_line":""},{"line_number":1014,"context_line":"        if not CONF.enable_partition_images \\"},{"line_number":1015,"context_line":"           and image_info.get(\u0027image_type\u0027) \u003d\u003d \u0027partition\u0027:"},{"line_number":1016,"context_line":"            LOG.info(\u0027Partition images are disabled, refusing to prepare \u0027"},{"line_number":1017,"context_line":"                     \u0027image %s\u0027, image_info[\u0027id\u0027])"}],"source_content_type":"text/x-python","patch_set":1,"id":"a378bb70_4524a79e","line":1014,"updated":"2026-05-07 11:55:45.000000000","message":"Please don\u0027t disable all partition images, it has too high of an impact. As discussed on the bug, we only need to disable running grub by default.","commit_id":"51932a8b10aba90447082d4b00344eaba8c6e3bd"}]}
