[Starlingx-discuss] Malformed Depends-On tag can cause unexpected merge

Eslimi, Dariush Dariush.Eslimi at windriver.com
Tue Nov 12 18:41:59 UTC 2019


Is it possible to add a zuul check to look for this malformed tag ? and also warn the committer about use of deprecated tags?

Dariush

From: Penney, Don [mailto:Don.Penney at windriver.com]
Sent: November-12-19 1:15 PM
To: 'starlingx-discuss at lists.starlingx.io' (starlingx-discuss at lists.starlingx.io) <starlingx-discuss at lists.starlingx.io>
Subject: [Starlingx-discuss] Malformed Depends-On tag can cause unexpected merge

Hi all,

Please be careful when adding dependencies to your commit messages to ensure the formatting is correct. If it is not, then Gerrit/Zuul may not properly handle the dependency and allow a commit to merge before the dependency is merged. It appears that a space is mandatory after the "Depends-On:" tag.

Proper formatting:
Depends-On: https://review.opendev.org/XXXXXX

Improper:
Depends-On:https://review.opendev.org/XXXXXX

We saw updates merge this morning prematurely, apparently due to a commit message that was missing this space after the Depends-On tag.

As a side note, I believe the syntax of "Depends-On: CHANGE-ID" is deprecated, maybe due to the potential for CHANGE-ID matching multiple reviews. The suggested syntax is to use the Gerrit review URL.

Cheers,
Don.


Don Penney, Developer, Wind River

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.starlingx.io/pipermail/starlingx-discuss/attachments/20191112/51bf1657/attachment.html>


More information about the Starlingx-discuss mailing list