)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"47fa8d589af2bc200da458a7a5ff30d086ab2c7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f7080628_79d92fb0","updated":"2023-04-03 01:17:39.000000000","message":"Need to fix stable/yoga backport first.","commit_id":"e2c43bf759fefbec5b5e0844a4167d9fb0376635"}],"manifests/pxe.pp":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"507c17c7ed49238adc7413790b59aeddd73b717e","unresolved":true,"context_lines":[{"line_number":297,"context_line":"    ensure    \u003d\u003e \u0027file\u0027,"},{"line_number":298,"context_line":"    seltype   \u003d\u003e \u0027tftpdir_t\u0027,"},{"line_number":299,"context_line":"    owner     \u003d\u003e $::ironic::params::user,"},{"line_number":300,"context_line":"    group     \u003d\u003e $::ironic::params::group,"},{"line_number":301,"context_line":"    mode      \u003d\u003e \u00270744\u0027,"},{"line_number":302,"context_line":"    source    \u003d\u003e \"${::ironic::params::grub_efi_file}\","},{"line_number":303,"context_line":"    backup    \u003d\u003e false,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"b775375e_e062790b","line":300,"updated":"2023-04-03 02:12:02.000000000","message":"Should we stick with the hard-coded \u0027ironic\u0027 for owner and group, like the others file directives in pxe.pp?","commit_id":"0841f20f4228f406623c1f79ee231d3b1fce3535"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"50423316dae7b318d909199951c510958b9697ed","unresolved":false,"context_lines":[{"line_number":297,"context_line":"    ensure    \u003d\u003e \u0027file\u0027,"},{"line_number":298,"context_line":"    seltype   \u003d\u003e \u0027tftpdir_t\u0027,"},{"line_number":299,"context_line":"    owner     \u003d\u003e $::ironic::params::user,"},{"line_number":300,"context_line":"    group     \u003d\u003e $::ironic::params::group,"},{"line_number":301,"context_line":"    mode      \u003d\u003e \u00270744\u0027,"},{"line_number":302,"context_line":"    source    \u003d\u003e \"${::ironic::params::grub_efi_file}\","},{"line_number":303,"context_line":"    backup    \u003d\u003e false,"}],"source_content_type":"text/x-puppet","patch_set":2,"id":"6e899e66_66fcd71a","line":300,"in_reply_to":"b775375e_e062790b","updated":"2023-04-03 02:22:38.000000000","message":"Yeah that was part of refactoring work during yoga so we should use hard-coded ones here.","commit_id":"0841f20f4228f406623c1f79ee231d3b1fce3535"}],"spec/classes/ironic_drivers_pxe_spec.rb":[{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"507c17c7ed49238adc7413790b59aeddd73b717e","unresolved":true,"context_lines":[{"line_number":23,"context_line":"describe \u0027ironic::drivers::pxe\u0027 do"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  let :default_params do"},{"line_number":26,"context_line":"    { :uefi_ipxe_bootfile_name        \u003d\u003e \u0027snponly.efi\u0027,"},{"line_number":27,"context_line":"    }"},{"line_number":28,"context_line":"  end"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-ruby","patch_set":2,"id":"8555e8e1_b3694d2c","line":26,"updated":"2023-04-03 02:12:02.000000000","message":"Should this be added back?\n\n  :uefi_pxe_bootfile_name         \u003d\u003e \u0027bootx64.efi\u0027,","commit_id":"0841f20f4228f406623c1f79ee231d3b1fce3535"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"50423316dae7b318d909199951c510958b9697ed","unresolved":false,"context_lines":[{"line_number":23,"context_line":"describe \u0027ironic::drivers::pxe\u0027 do"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"  let :default_params do"},{"line_number":26,"context_line":"    { :uefi_ipxe_bootfile_name        \u003d\u003e \u0027snponly.efi\u0027,"},{"line_number":27,"context_line":"    }"},{"line_number":28,"context_line":"  end"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-ruby","patch_set":2,"id":"4ea2184a_14bd030c","line":26,"in_reply_to":"8555e8e1_b3694d2c","updated":"2023-04-03 02:22:38.000000000","message":"not really because we are not using these values and that\u0027s why I removed these in newer branches. I\u0027ve updated the change to actually use these values.","commit_id":"0841f20f4228f406623c1f79ee231d3b1fce3535"}]}