)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6ac5eb0cdd1673ee9230047e19f4543862ce39df","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"44dc6c82_4c732ad7","updated":"2024-11-15 09:02:51.000000000","message":"I would suggest to still merge this patch for documentation purposes, even if the actual deletion needs to be done by hand (which I volunteer to do). we do have the same data in place for ocata and queens, would be weird to have a gap in between","commit_id":"c02c4247da3232490c6a7963cfa1a014d5fb2bc2"},{"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":"090b2f28efe3c93a55285e73e326030488b2e747","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"32fe7a0e_4a1b634e","updated":"2023-10-20 16:47:34.000000000","message":"So you\u0027re suggesting that, for EOL, we manually tag pike-eol, delete the branch, and never update the release documentation to say that it existed?","commit_id":"c02c4247da3232490c6a7963cfa1a014d5fb2bc2"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9a4b3dfff450b09abc1c37607cef6aac63bbd12d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d9e87d80_93e3e0ca","updated":"2023-10-20 13:49:58.000000000","message":"There are more problem with this:\n- relmgt tooling skips processing any change against an EOL\u0027d series (thus, no tagging will happen if we merge this)\n- we are changing history a bit by adding new deliverable files in old branches\n\nThere were short discussions whether some manual cleanup would help with these kind of issues (as there are multiple open branches that should have been deleted), probably that would be the simplest solution.","commit_id":"c02c4247da3232490c6a7963cfa1a014d5fb2bc2"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"c1bac26ce9e44b80fcdf955ff397bc7f702e6796","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"153aaca8_e7cbba12","updated":"2023-10-17 14:37:50.000000000","message":"Usually we avoid adding a deliverable to a stable branch after the final release of that series, but in this case, the stable branch exist where the deliverable doesn\u0027t exists, so this patch LGTM","commit_id":"c02c4247da3232490c6a7963cfa1a014d5fb2bc2"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"55d5220e2d97592a5aef6b829fa5cebbd4fc8994","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a60c06e7_b236cf0b","updated":"2024-11-22 07:38:40.000000000","message":"overriding ptl-approved flag for eol branch","commit_id":"c02c4247da3232490c6a7963cfa1a014d5fb2bc2"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"599e90d1d0cd9563781da8d158a003f1b60607f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"0cd0b523_c192a798","in_reply_to":"32fe7a0e_4a1b634e","updated":"2023-10-20 17:20:34.000000000","message":"Yepp.","commit_id":"c02c4247da3232490c6a7963cfa1a014d5fb2bc2"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"266d1f16aa72536d4932f1d9ff00762f891c66db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"081176c3_ee2cb9ea","in_reply_to":"44dc6c82_4c732ad7","updated":"2024-11-15 10:05:25.000000000","message":"thanks Jens for taking care of this. I\u0027d rather not change history, but if you do the manual steps then i\u0027m OK with this to clean up finally. Next time i would add a comment in the yaml that \"this deliverable was not part of the release originally rather added it later for documentation purposes\".","commit_id":"c02c4247da3232490c6a7963cfa1a014d5fb2bc2"}]}
