[JIRA] (OVIRT-1448) Enable devs to specifiy patch dependencies for OST
Barak Korren (oVirt JIRA)
jira at ovirt-jira.atlassian.net
Sun Jun 18 15:06:54 UTC 2017
[ https://ovirt-jira.atlassian.net/browse/OVIRT-1448?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=32405#comment-32405 ]
Barak Korren commented on OVIRT-1448:
-------------------------------------
Here are some docs about how this is done in OpenStack:
https://docs.openstack.org/infra/manual/developers.html#cross-repository-dependencies
The basic Idea is that developers add a '{{Depends-On:}}' header to the commit message follows by a 40-char Gerrit change ID.
One implication of this is that the same change ID can point to multiple changes (same change in different branches), so we will need to query Gerrit and find all actual changes with that ID.
> Enable devs to specifiy patch dependencies for OST
> --------------------------------------------------
>
> Key: OVIRT-1448
> URL: https://ovirt-jira.atlassian.net/browse/OVIRT-1448
> Project: oVirt - virtualization made easy
> Issue Type: Improvement
> Components: oVirt CI
> Reporter: Barak Korren
> Assignee: infra
> Labels: change-queue
>
> We have an issue with our system ATM where if there are patches for different projects that depend on one another, the system is unaware of that dependency.
> What typically happens in this scenario is that sending the dependent patch makes the experimental test fail and keep failing until the other patch is also merged.
> The change queue will handle this better, but the typical behaviour for it would be to reject both patches, unless they are somehow coordinated to make it into the same test.
> The change queue core code already includes the ability to track and understand dependencies between changes. What is missing is the ability for developers to specify theses dependencies.
> We would probably want to adopt OpenStack's convention here.
--
This message was sent by Atlassian JIRA
(v1000.1068.0#100050)
More information about the Infra
mailing list