)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"04a783c39ce7a069ea9cd54a0b4b9b99b57e2e6b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     b29c73db (Merge \"Remove MAAS integration\")"},{"line_number":2,"context_line":"Author:     jgilaber \u003cjgilaber@redhat.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2026-09-16 15:18:17 +0200"},{"line_number":4,"context_line":"Commit:     jgilaber \u003cjgilaber@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7af8d388_7d52b075","line":1,"updated":"2026-09-25 06:51:33.000000000","message":"The commit subject \u0027Fix baremetal collector notification_endpoints returns None\u0027 is 59 characters, exceeding the documented 50-character maximum. Its trailing clause \u0027returns None\u0027 also reads as a declarative statement rather than imperative phrasing, so the subject mixes an imperative verb (\u0027Fix\u0027) with a descriptive clause instead of describing the action taken. Both defects land in the permanent change history, where review tooling cannot catch them.\n\n**Severity**: SUGGESTION | **Confidence**: 0.85\n\n**Impact**: Subjects longer than 50 characters are truncated in Gerrit change listings, git log --oneline output, and notification emails, weakening the scanability of the permanent history. The length also conflicts with an explicit project rule marked ALL REQUIRED, so the change as authored fails the project\u0027s documented message format even though the code is correct.\n\n**Recommendation**:\nAmend the subject to 50 characters or fewer with imperative phrasing, for example \u0027Return [] from baremetal notification_endpoints\u0027 (47 characters). Keep the existing body, Closes-Bug, Change-Id, Generated-By, and Signed-off-by footers unchanged; amend the same commit rather than adding a new one.","commit_id":"551d950035d4cce5fee7b261b4ca08754e8db1a6"},{"author":{"_account_id":28006,"name":"teim-ci","display_name":"teim-ci","email":"ci@seanmooney.info","username":"ci-sean-mooney","status":"this is a third-party ci account run by sean-k-mooney on irc\nhosted at zuul.teim.app"},"tag":"autogenerated:zuul:automatic-ci","change_message_id":"e9cba3bd450e2c98a52215f584157a009bfbe500","unresolved":false,"context_lines":[{"line_number":1,"context_line":"Parent:     b29c73db (Merge \"Remove MAAS integration\")"},{"line_number":2,"context_line":"Author:     jgilaber \u003cjgilaber@redhat.com\u003e"},{"line_number":3,"context_line":"AuthorDate: 2026-09-16 15:18:17 +0200"},{"line_number":4,"context_line":"Commit:     jgilaber \u003cjgilaber@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"301e6ee0_bc6ff6ff","line":1,"updated":"2026-09-24 17:02:16.000000000","message":"The reviewed commit subject \u0027Fix baremetal collector notification_endpoints returns None\u0027 measures 59 characters, exceeding the supplied OpenStack guideline of fewer than 50 characters, and states the defect in indicative mood (\u0027returns\u0027) instead of the required imperative form. The commit body and footers (Closes-Bug, Change-Id, Signed-off-by) are compliant; only the subject line deviates.\n\n**Severity**: SUGGESTION | **Confidence**: 0.85\n\n**Impact**: Minor, non-functional: the permanent change history entry deviates from the project\u0027s commit-subject convention. Long subjects wrap or truncate in `git log --oneline`, Gerrit lists, and generated release summaries, slightly reducing scannability of the fix in history.\n\n**Recommendation**:\nWhen amending for the next patchset, shorten the subject to at most 50 characters in imperative mood, for example \u0027Return [] from baremetal notification_endpoints\u0027 (47 chars) or \u0027Fix baremetal notification_endpoints None return\u0027 (48 chars). Keep the body and all existing footers unchanged.","commit_id":"551d950035d4cce5fee7b261b4ca08754e8db1a6"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"5e61dbd28331ef7bd674529711ee297512af7afd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"c8f7e875_b830d54b","updated":"2026-09-24 12:46:48.000000000","message":"Thanks Joan","commit_id":"551d950035d4cce5fee7b261b4ca08754e8db1a6"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"cafd279b76c17b0ca083b43f0ee3f1b5402d3e7d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d73f1ba5_8a5ab4fe","updated":"2026-09-24 16:51:57.000000000","message":"recheck","commit_id":"551d950035d4cce5fee7b261b4ca08754e8db1a6"},{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"32b16805a65e9361170daea0fa916672c2fa4c3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ea5610f7_9ca51d30","updated":"2026-09-25 06:42:02.000000000","message":"recheck","commit_id":"551d950035d4cce5fee7b261b4ca08754e8db1a6"}]}