Bug 1618407 - Smoke jobs fail when commit messages have commented BZ or other meta information
Summary: Smoke jobs fail when commit messages have commented BZ or other meta information
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: GlusterFS
Classification: Community
Component: project-infrastructure
Version: mainline
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
: 1633487 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-08-16 14:53 UTC by Shyamsundar
Modified: 2018-10-03 04:26 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2018-10-03 04:18:55 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Embargoed:


Attachments (Terms of Use)

Description Shyamsundar 2018-08-16 14:53:00 UTC
Description of problem:

Smoke fails for the following commits as it picks up the incorrect (commented out) BZ# to validate, (the following are examples)
- https://review.gluster.org/c/glusterfs/+/20752/1//COMMIT_MSG
  - https://build.gluster.org/job/bugzilla-post/7377/console
- https://review.gluster.org/c/glusterfs/+/20750/1//COMMIT_MSG
  - https://build.gluster.org/job/bugzilla-post/7374/console

BTW, I think this happens for github issue reference as well, not finding an example ready at hand.

Expected results:
In the past lines starting with '>' were treated as comments in the commit message and ignored for parsing, I assume this is broken and needs correction.

Additional info:
There was a discussion in the devel list to drop this information, https://lists.gluster.org/pipermail/gluster-devel/2017-June/053140.html

But, I guess that is stale for this conversation/bug. Posted it here, to state, if we are deliberately failing smoke due to these extra comments, then the bug possibly needs to be closed and we need to work on the bug #1428047 instead.

Comment 2 Nigel Babu 2018-10-03 04:26:26 UTC
*** Bug 1633487 has been marked as a duplicate of this bug. ***


Note You need to log in before you can comment on or make changes to this bug.