Bug 1357421 - Fail smoke tests if cherry-picked bugs contain the old git-tags
Summary: Fail smoke tests if cherry-picked bugs contain the old git-tags
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: GlusterFS
Classification: Community
Component: project-infrastructure
Version: mainline
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
Assignee: bugs@gluster.org
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-07-18 07:51 UTC by Niels de Vos
Modified: 2020-03-12 12:51 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-03-12 12:51:22 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:


Attachments (Terms of Use)

Description Niels de Vos 2016-07-18 07:51:02 UTC
Description of problem:
The commit message of patches should not have incorrect tags. This is common to see when backports are done through the Gerrit webui. Many seem to ignore the http://gluster.readthedocs.io/en/latest/Developer-guide/Backport-Guidelines/ where it is clearly mentioned what changes to the commit message are required.

Backports may rely on other changes, or may have been modified in order to match the older version of the code. Keeping the git-tags from the backported patch is suggested, but indention is required to make the distinction between the old and the new (inserted by Gerrit on merging) onced.

Version-Release number of selected component (if applicable):
all

How reproducible:
Backport a change without taking the Backport Guidelines into account.


Actual results:
For example http://review.gluster.org/#/c/13863/1

Expected results:
http://review.gluster.org/#/c/13863/2

Additional info:
A new Jenkins job that is part of the smoke test would do. The following git-tags in the commit message should be prohibited:

Reviewed-on:
Smoke:
CentOS-regression:
NetBSD-regression:

A basic script that fails when the commit message contains any of these:

MATCHES=$(git log -1 --format=email | grep -c \
    -e '^Reviewed-on:' \
    -e '^Smoke:' \
    -e '^CentOS-regression:' \
    -e '^NetBSD-regression:')

exit ${MATCHES}

Comment 2 Worker Ant 2020-03-12 12:51:22 UTC
This bug is moved to https://github.com/gluster/project-infrastructure/issues/23, and will be tracked there from now on. Visit GitHub issues URL for further details


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