)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4c1ad2b40f918eea6e36f53cf48053be1a8da79d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e8ac531d_4f7b1876","updated":"2022-07-01 14:15:17.000000000","message":"Leaving +1 instead of +2, in case you want to address a few minor suggestions about the release note content.","commit_id":"80d39f1e6ff613c8a1a24ac2b5ffbdab85cf7d5f"}],"releasenotes/notes/remove-puppet-masquerade-28ae63cfa7f613d6.yaml":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4c1ad2b40f918eea6e36f53cf48053be1a8da79d","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"deprecations:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    This patch removes the \"masquerade-networks-baremetal-puppet\""},{"line_number":5,"context_line":"    deployment file, since it has been replaced by a 100% ansible"},{"line_number":6,"context_line":"    implementation; this removal won\u0027t affect operators, since the"},{"line_number":7,"context_line":"    inclusion of the actual services/masquerade-networks is done from"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ee96431f_2979eb9b","line":4,"range":{"start_line":4,"start_character":4,"end_line":4,"end_character":22},"updated":"2022-07-01 14:15:17.000000000","message":"this is included into the release note content so might be better to avoid mentioning \"this patch\"","commit_id":"80d39f1e6ff613c8a1a24ac2b5ffbdab85cf7d5f"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"697c658e432457ea264e49ff988b73cf3825c281","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"deprecations:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    This patch removes the \"masquerade-networks-baremetal-puppet\""},{"line_number":5,"context_line":"    deployment file, since it has been replaced by a 100% ansible"},{"line_number":6,"context_line":"    implementation; this removal won\u0027t affect operators, since the"},{"line_number":7,"context_line":"    inclusion of the actual services/masquerade-networks is done from"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9a300e60_a3c2168a","line":4,"range":{"start_line":4,"start_character":4,"end_line":4,"end_character":22},"in_reply_to":"ee96431f_2979eb9b","updated":"2022-07-04 06:50:33.000000000","message":"right. Sounds better","commit_id":"80d39f1e6ff613c8a1a24ac2b5ffbdab85cf7d5f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4c1ad2b40f918eea6e36f53cf48053be1a8da79d","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    This patch removes the \"masquerade-networks-baremetal-puppet\""},{"line_number":5,"context_line":"    deployment file, since it has been replaced by a 100% ansible"},{"line_number":6,"context_line":"    implementation; this removal won\u0027t affect operators, since the"},{"line_number":7,"context_line":"    inclusion of the actual services/masquerade-networks is done from"},{"line_number":8,"context_line":"    within the Undercloud deployment only, when we set the \"masquerade\""},{"line_number":9,"context_line":"    parameter to true for selected networks."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c8777521_d6da2619","line":8,"range":{"start_line":7,"start_character":65,"end_line":8,"end_character":10},"updated":"2022-07-01 14:15:17.000000000","message":"can remove one of these ?","commit_id":"80d39f1e6ff613c8a1a24ac2b5ffbdab85cf7d5f"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"697c658e432457ea264e49ff988b73cf3825c281","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    This patch removes the \"masquerade-networks-baremetal-puppet\""},{"line_number":5,"context_line":"    deployment file, since it has been replaced by a 100% ansible"},{"line_number":6,"context_line":"    implementation; this removal won\u0027t affect operators, since the"},{"line_number":7,"context_line":"    inclusion of the actual services/masquerade-networks is done from"},{"line_number":8,"context_line":"    within the Undercloud deployment only, when we set the \"masquerade\""},{"line_number":9,"context_line":"    parameter to true for selected networks."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"2d8033ae_d555361c","line":8,"range":{"start_line":7,"start_character":65,"end_line":8,"end_character":10},"in_reply_to":"c8777521_d6da2619","updated":"2022-07-04 06:50:33.000000000","message":"hmm nope, this is actually correct like this :). Or at least as far as I can remember my English courses. Maybe a bit pedantic, but 100% correct","commit_id":"80d39f1e6ff613c8a1a24ac2b5ffbdab85cf7d5f"}]}