)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"3075950be8ebdb4235c34b6750f67256e3aa4cff","unresolved":true,"context_lines":[{"line_number":11,"context_line":"to make changes to tox.ini."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Another change is to add the [stable-only] tag, it could help to better"},{"line_number":14,"context_line":"track backports."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I553fdda626b8afb8949d5ae9fc170caf87d31032"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"40220e6b_fdfa02d6","line":14,"updated":"2021-03-23 20:41:20.000000000","message":"I guess you added this to explain the change in line 40 of the script, but it\u0027s actually a fix for cases where a patch becomes a merge patch in zuul tests [1].\n\nsome explanation for the script:\n* it checks if the pushed patch on a stable/* branch contains \u0027cherry-picked from\u0027 line, which helps to see if a patch is really a backport, as it should on stable branches\n* it also checks if the \u0027cherry-picked from\u0027 lines contains hashes that are from patches that have merged, which is useful to see whether a patch is cherry-picked from the latest patch set\n* some cases it can happen that a fix (e.g. CI related change) is a \u0027new\u0027 patch, thus not a bug fix backport. In this case the patch has to contain the tag \u0027[stable-only]\u0027 in the commit message (e.g. [2]), so that this check script ignores the lack of the \u0027cherry-picked from\u0027 line.\n\nHope this helps a bit to understand the aim of the check-cherry-picks.sh.\n\n[1] Ia725346b65dd5e2f16aa049c74b45d99e22b3524\n[2] I749dfd4e2b97fd28734ef9ce7db180b0d0576af1","commit_id":"5b1deb775af53de9d72b1ed73e4664069b3a8d4f"},{"author":{"_account_id":26458,"name":"Brin Zhang","email":"zhangbailin@inspur.com","username":"zhangbailin"},"change_message_id":"be1fcb03645d23be998fc2e2be05c71a489a0ff5","unresolved":true,"context_lines":[{"line_number":11,"context_line":"to make changes to tox.ini."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Another change is to add the [stable-only] tag, it could help to better"},{"line_number":14,"context_line":"track backports."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I553fdda626b8afb8949d5ae9fc170caf87d31032"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a7d91524_37a16551","line":14,"in_reply_to":"40220e6b_fdfa02d6","updated":"2021-03-24 11:55:44.000000000","message":"Thansk for your explanation, I would like to add these commnet to this commit message, but it will be looks redundancy. This is let me know why we need this check-cherry-picks shell and which it can be help us check what happened if we backport a patch for stable/* branch. I will add the comment for the [stable-only] tag.","commit_id":"5b1deb775af53de9d72b1ed73e4664069b3a8d4f"}]}