Details
-
Bug
-
Resolution: Done
-
P1: Critical
-
None
-
production
-
None
-
11bc5f7f3a639c4919a26c26966963caaa5f63b9 (qt/qtqa/dev)
Description
The Pick-to: commit message parser in cherry-pick bot uses a regex to ensure that the footer sits at the beginning of a line, but it fails to parse this correctly when parsing commit messages returned as a result of a REST query to gerrit.
--
This appears to be due to the rest responses from gerrit being encoded in plaintext, and newline characters "\n" are interpreted as string literals. This needs to be corrected for, and the bot's parser regex updated to look for either start of line with "^" or the string literal "\n".
This actually appears to be due to using invalid functions to determine the length and content of the set of branches returned by the parser.
Attachments
For Gerrit Dashboard: QTQAINFRA-3794 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
303630,4 | Cherry-pick bot: Fix commit message parser for string literal "\n" | dev | qt/qtqa | Status: MERGED | +2 | 0 |
313580,2 | Cherry-pick bot: Fix interpreting of parent Pick-to footer | dev | qt/qtqa | Status: MERGED | +2 | 0 |