)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f1e9de59fcc248da1eb6b3c2e976c236ecce0e8b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9da11ed2_38085ef9","updated":"2022-08-01 14:59:37.000000000","message":"recheck Werkzeug breakage","commit_id":"651a09aee05768a995248e75c141ed9c80b96cc9"},{"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":"76fbb37096ad14f2fd509be7e18c9dce09b3fad3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"43ecf10d_9622a592","updated":"2022-08-16 22:21:37.000000000","message":"Please fix Ruby\u0027s concern in this patch since it is likely to be backported.","commit_id":"7854b8f7b70952a6b00336e2072cd1dafef9db8b"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"8d98e5b62bf349c1768674e2c9861669f40fe824","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"16e916b6_94720ddb","updated":"2022-08-15 15:16:43.000000000","message":"Thanks for fixing this Julia. I\u0027m not sure if we want to update the release note. (If not, LMK and I\u0027ll +2).\n\nI\u0027m not sure if we want to backport this. It might be dangerous to backport. Thinking of the case where someone has things configured/working for eg yoga (and they are using the \u0027default\u0027, pxe). If this is backported, will someone think to look at the reno, or will they just update their yoga to the latest, which will break them, cuz the new \u0027default\u0027 will be ipxe?","commit_id":"7854b8f7b70952a6b00336e2072cd1dafef9db8b"},{"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":"adf7967944905a197bfb810d00fb3ea29a0dd0bb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"44111381_7cc132c4","updated":"2022-08-25 21:11:53.000000000","message":"One concern, just with the BC+1 -- I\u0027m pretty sure sure we shouldn\u0027t backport this, as it represents a significant change in behavior for existing deployers. \n\nI know the existing behavior is bad/unexplainable as to why it\u0027s setup that way, but we also shouldn\u0027t punish deployers who have come to rely on this strange behavior.","commit_id":"c921c077d482eba20ff1864b52eafe5b26d0caa2"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"7734a67a728cc1670b4f7cb62a14e50b0d4ad9ff","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b1bc7921_55764047","updated":"2022-08-19 17:10:27.000000000","message":"recheck metalsmith job failed with curl: (22) The requested URL returned error: 503 ","commit_id":"c921c077d482eba20ff1864b52eafe5b26d0caa2"}],"releasenotes/notes/fix-ilo-boot-interface-order-238a2da9933cf28c.yaml":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d3713eaecb795e73bc87b286b36e3c39e8a8a6c8","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes the default boot interface order for the ``ilo`` hardware type"},{"line_number":5,"context_line":"    where previously it would prefer ``pxe`` over ``ipxe``. This created"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2de1c206_e2d7e40c","line":2,"updated":"2022-08-01 15:26:53.000000000","message":"Not sure I\u0027d call it a feature, especially if you\u0027re going to backport it.. But it does have an upgrade impact.","commit_id":"651a09aee05768a995248e75c141ed9c80b96cc9"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"5dfba839f9ba45bbfa6b76f40f0ebb2cbc79ad02","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Fixes the default boot interface order for the ``ilo`` hardware type"},{"line_number":5,"context_line":"    where previously it would prefer ``pxe`` over ``ipxe``. This created"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4f559400_a9e78379","line":2,"in_reply_to":"2de1c206_e2d7e40c","updated":"2022-08-01 18:18:33.000000000","message":"Ahh, right, I do need to backport this too! I\u0027ll fix the reno!","commit_id":"651a09aee05768a995248e75c141ed9c80b96cc9"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"8d98e5b62bf349c1768674e2c9861669f40fe824","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    Fixes the default boot interface order for the ``ilo`` hardware type"},{"line_number":5,"context_line":"    where previously it would prefer ``pxe`` over ``ipxe``. This created"},{"line_number":6,"context_line":"    inconsistencies for operators using multiple hardware types, where"},{"line_number":7,"context_line":"    both interfaces were enabled enabled in the deployment."},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Operators who are upgrading should be aware that a bug was discovered"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"03ac7a6f_2f255332","line":7,"updated":"2022-08-15 15:16:43.000000000","message":"nit s/enabled enabled/enabled/","commit_id":"7854b8f7b70952a6b00336e2072cd1dafef9db8b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f013c554c389b5d273fcf193170ea717c3eb6952","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    Fixes the default boot interface order for the ``ilo`` hardware type"},{"line_number":5,"context_line":"    where previously it would prefer ``pxe`` over ``ipxe``. This created"},{"line_number":6,"context_line":"    inconsistencies for operators using multiple hardware types, where"},{"line_number":7,"context_line":"    both interfaces were enabled enabled in the deployment."},{"line_number":8,"context_line":"upgrade:"},{"line_number":9,"context_line":"  - |"},{"line_number":10,"context_line":"    Operators who are upgrading should be aware that a bug was discovered"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b7e30b9f_9e1ed27b","line":7,"in_reply_to":"03ac7a6f_2f255332","updated":"2022-08-17 16:51:21.000000000","message":"Done","commit_id":"7854b8f7b70952a6b00336e2072cd1dafef9db8b"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"8d98e5b62bf349c1768674e2c9861669f40fe824","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    this may, or may not have happened. Some users may have been relying"},{"line_number":16,"context_line":"    upon this incorrect behavior, and explicitly setting a ``boot_interface``"},{"line_number":17,"context_line":"    on a node level, or by using a ``[DEFAULT]default_boot_interface``"},{"line_number":18,"context_line":"    parameter in ironic.conf."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"5cea1398_bdd4ab40","line":18,"updated":"2022-08-15 15:16:43.000000000","message":"wanted to check here. The above about explicitly setting the boot_interface per node or via config, is referring to someone that wants ipxe? If so, should we be explicit that if someone wants the \u0027buggy\u0027 behaviour, that they should do that explicit setting too?","commit_id":"7854b8f7b70952a6b00336e2072cd1dafef9db8b"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"f013c554c389b5d273fcf193170ea717c3eb6952","unresolved":true,"context_lines":[{"line_number":15,"context_line":"    this may, or may not have happened. Some users may have been relying"},{"line_number":16,"context_line":"    upon this incorrect behavior, and explicitly setting a ``boot_interface``"},{"line_number":17,"context_line":"    on a node level, or by using a ``[DEFAULT]default_boot_interface``"},{"line_number":18,"context_line":"    parameter in ironic.conf."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8929fa07_a05b7db8","line":18,"in_reply_to":"5cea1398_bdd4ab40","updated":"2022-08-17 16:51:21.000000000","message":"I think that is what I was suggesting.","commit_id":"7854b8f7b70952a6b00336e2072cd1dafef9db8b"}]}