[Gluster-infra] [Bug 1567221] New: Issue checking should ensure required flags are present , even on newer patchsets in gerrit
bugzilla at redhat.com
bugzilla at redhat.com
Fri Apr 13 15:16:36 UTC 2018
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: bugs at gluster.org
Reporter: srangana at redhat.com
CC: bugs at gluster.org, gluster-infra at 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
More information about the Gluster-infra
mailing list