)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"dbcbc80791ebf49d8364fac5ff1123de6dbd1f7f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e75b3f25_701e649b","updated":"2023-05-27 07:55:52.000000000","message":"I\u0027m not actually sure which merge mode will be correct / accepted by zuul, possibly some iteration on this is needed.","commit_id":"19a52011a639769deb8bcbaceb4ae05da1ad8a20"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e8cd4da7978abf271ad6145725ed736691b144c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"dcbc61d4_86589e86","updated":"2023-05-31 05:21:38.000000000","message":"Well the \"faithful\" part is really just speculation, if a project allows different merge modes, like either squash or rebase, that decision is made by the person who actually merges the patch and any prior guessing is just that.\n\nAnyway, the only thing I care about is getting rid of the config error. Can we just drop the projects from configuration completely for the time being, then? Since in the current state no jobs are running for them anyhow?","commit_id":"69800fce72f9bcfa834f6fd030395b4067e7ad80"},{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"61f265ee8ac32bb5518324889e7c1d7a1f38df76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"67f23497_923ab7da","updated":"2023-05-31 17:04:08.000000000","message":"+2 with a note that this may impact OpenDev\u0027s ansible devel test job runs. I\u0027m not sure if merge mode is important when there isn\u0027t much speculative state. I think the only cases would be if we tried to have system-config changes depends-on ansible PRs.","commit_id":"bdaba25653be6af71f1fd4dd49e1346e1c6536ba"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"8973dc80e9865d980ccb1d87638580e9d996a65b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5184bcb0_df75982e","updated":"2023-06-01 09:15:39.000000000","message":"Going ahead and testing this, should be an easy revert if it doesn\u0027t work out.","commit_id":"bdaba25653be6af71f1fd4dd49e1346e1c6536ba"}],"zuul.d/projects.yaml":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"27b44ef668b3cea99ef8f2c62f6d4fb942637e91","unresolved":true,"context_lines":[{"line_number":26,"context_line":"    default-branch: devel"},{"line_number":27,"context_line":"    # Note: Specifying the merge-mode shouldn\u0027t be necessary since we"},{"line_number":28,"context_line":"    # do not gate this project, but zuul checks it anyway and generates"},{"line_number":29,"context_line":"    # a config error on mismatch"},{"line_number":30,"context_line":"    merge-mode: merge"},{"line_number":31,"context_line":"    third-party-check:"},{"line_number":32,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"e821d6c9_532c7938","line":29,"updated":"2023-05-30 19:23:45.000000000","message":"This comment is misleading.  It is important for Zuul to know the merge mode regardless of whether it gates a project, which is why it notes the mismatch.  Including this comment here could end up misleading other users as to the importance of this.\n\nAlso, this is the default and current value, so this change isn\u0027t going to change anything.","commit_id":"69800fce72f9bcfa834f6fd030395b4067e7ad80"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"e8cd4da7978abf271ad6145725ed736691b144c9","unresolved":true,"context_lines":[{"line_number":26,"context_line":"    default-branch: devel"},{"line_number":27,"context_line":"    # Note: Specifying the merge-mode shouldn\u0027t be necessary since we"},{"line_number":28,"context_line":"    # do not gate this project, but zuul checks it anyway and generates"},{"line_number":29,"context_line":"    # a config error on mismatch"},{"line_number":30,"context_line":"    merge-mode: merge"},{"line_number":31,"context_line":"    third-party-check:"},{"line_number":32,"context_line":"      jobs:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"cd172b02_c714b129","line":29,"in_reply_to":"e821d6c9_532c7938","updated":"2023-05-31 05:21:38.000000000","message":"So why does zuul generate a config error on this patch for any other option I tried, but none for this variant?","commit_id":"69800fce72f9bcfa834f6fd030395b4067e7ad80"}]}