)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"9e949bfc0f1e0f6e97820180e59a55328b819e67","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Github automatically uses the commit_title of a PR as the title of the"},{"line_number":10,"context_line":"resulting commit message in git, so we should not duplicate it into the"},{"line_number":11,"context_line":"body text."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Dr. Jens Harbott \u003charbott@osism.tech\u003e"},{"line_number":14,"context_line":"Change-Id: Iff47d08f5be6afe182a22d21ed3863348ffc16bb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"2541b0c8_ab38365c","line":11,"updated":"2022-03-22 15:08:43.000000000","message":"Can this be configured? For us what I\u0027m seeing as the merge commit is the following:\n\n  Merge pull request #\u003cno\u003e from \u003corga\u003e/\u003cproject\u003e\n\n  \u003cPR title\u003e\n\n  \u003cPR body\u003e\n\n  \u003cZuul reviewed-by msg\u003e","commit_id":"c47b0b8ad832721a1e304215d35201a2136b19bd"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"9bf4e7cb80348eea9438c72530d02c7afaf1168f","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Github automatically uses the commit_title of a PR as the title of the"},{"line_number":10,"context_line":"resulting commit message in git, so we should not duplicate it into the"},{"line_number":11,"context_line":"body text."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Dr. Jens Harbott \u003charbott@osism.tech\u003e"},{"line_number":14,"context_line":"Change-Id: Iff47d08f5be6afe182a22d21ed3863348ffc16bb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"631c3784_fea8699f","line":11,"in_reply_to":"0931555a_deab970a","updated":"2022-03-23 06:49:16.000000000","message":"Sorry, missed your comment where you mention that you use merge-rebase. I\u0027m just wondering how this works with Zuul as I don\u0027t see that merge-rebase is supported: https://zuul-ci.org/docs/zuul/latest/config/project.html#attr-project.merge-mode","commit_id":"c47b0b8ad832721a1e304215d35201a2136b19bd"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"51687f899234d9eef68684b6ebfad6ae97a89be7","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Github automatically uses the commit_title of a PR as the title of the"},{"line_number":10,"context_line":"resulting commit message in git, so we should not duplicate it into the"},{"line_number":11,"context_line":"body text."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Dr. Jens Harbott \u003charbott@osism.tech\u003e"},{"line_number":14,"context_line":"Change-Id: Iff47d08f5be6afe182a22d21ed3863348ffc16bb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dcf1fa9f_ac5b81c2","line":11,"in_reply_to":"127031f7_bf7c1cb5","updated":"2022-03-22 15:31:25.000000000","message":"Which merge-mode are you using? There is one other option we could consider, which is taking the title and explicitly submitting it with the merge request as commit_title, maybe that changes the result.\n\nCan you link to a PR where you see that or is it all private repos? Could you try to reproduce with a public repo otherwise?","commit_id":"c47b0b8ad832721a1e304215d35201a2136b19bd"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"4957212342f4f4d25c29a72ae1cac9abcbb447dd","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Github automatically uses the commit_title of a PR as the title of the"},{"line_number":10,"context_line":"resulting commit message in git, so we should not duplicate it into the"},{"line_number":11,"context_line":"body text."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Dr. Jens Harbott \u003charbott@osism.tech\u003e"},{"line_number":14,"context_line":"Change-Id: Iff47d08f5be6afe182a22d21ed3863348ffc16bb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"127031f7_bf7c1cb5","line":11,"in_reply_to":"2541b0c8_ab38365c","updated":"2022-03-22 15:10:18.000000000","message":"To clarify: this is on GitHub Enterprise","commit_id":"c47b0b8ad832721a1e304215d35201a2136b19bd"},{"author":{"_account_id":27582,"name":"Simon Westphahl","email":"simon.westphahl@bmw.de","username":"simon.westphahl"},"change_message_id":"9779f07d87a40482cbc5b36fd951d370e3fe4a2f","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Github automatically uses the commit_title of a PR as the title of the"},{"line_number":10,"context_line":"resulting commit message in git, so we should not duplicate it into the"},{"line_number":11,"context_line":"body text."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Signed-off-by: Dr. Jens Harbott \u003charbott@osism.tech\u003e"},{"line_number":14,"context_line":"Change-Id: Iff47d08f5be6afe182a22d21ed3863348ffc16bb"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"0931555a_deab970a","line":11,"in_reply_to":"dcf1fa9f_ac5b81c2","updated":"2022-03-23 06:15:36.000000000","message":"Unfortunately it\u0027s a private GH instance. We are using the default \"merge-resolve\" mode in Zuul with the \"merge\" mode on GH side.\n\nAre you using squash-merge?","commit_id":"c47b0b8ad832721a1e304215d35201a2136b19bd"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"181ecbd4f02b8f752720fef92f2442ec8e2a812d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c5d4b07d_a5e213f5","updated":"2022-03-22 15:32:33.000000000","message":"Also seems someone else submitted a similar patch https://review.opendev.org/c/zuul/zuul/+/809200 , so I\u0027m not the only one seeing this.","commit_id":"c47b0b8ad832721a1e304215d35201a2136b19bd"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"4ab9b27c3de7ab9d61c401a54a2dd46d89171459","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cfbdee31_fbe23cb1","updated":"2022-03-22 17:49:39.000000000","message":"I did some testing and what you see happens when merging with a merge commit while my case is with merge-rebase. squash mode is handled even different. this needs more thought and testing","commit_id":"c47b0b8ad832721a1e304215d35201a2136b19bd"}]}
