https://bugzilla.redhat.com/show_bug.cgi?id=1567221
Bug ID: 1567221 Summary: Issue checking should ensure required flags are present, even on newer patchsets in gerrit Product: GlusterFS Version: mainline Component: project-infrastructure Assignee: b...@gluster.org Reporter: srang...@redhat.com CC: b...@gluster.org, gluster-infra@gluster.org Smoke initially failed stating (Doc/Spec)Approved is missing: https://build.gluster.org/job/comment-on-issue/6270/console On re-submission it passed: https://build.gluster.org/job/comment-on-issue/6271/console It should not have, but as the check for the required flags are only on new issues in the commit message when the patchset is not the first, it avoids that check and passes smoke. As in this line, https://github.com/gluster/glusterfs-patch-acceptance-tests/blob/master/github/handle_github.py#L110 should be out of the new issues in commit message check, and should be run for all issues in the commit message. This inadvertently provides a workaround, that if smoke is rerun it will pass :) -- You are receiving this mail because: You are on the CC list for the bug. Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=BqCIktzHMe&a=cc_unsubscribe _______________________________________________ Gluster-infra mailing list Gluster-infra@gluster.org http://lists.gluster.org/mailman/listinfo/gluster-infra