[Development] Suggestion to add labels when changing API
Konstantin Tokarev
annulen at yandex.ru
Fri Dec 8 17:10:41 CET 2017
08.12.2017, 18:50, "Oswald Buddenhagen" <oswald.buddenhagen at qt.io>:
> On Fri, Dec 08, 2017 at 04:15:10PM +0100, Sergio Ahumada wrote:
>> On 08.12.2017 15:42, Adam Treat wrote:
>> > Relying upon qt5 submodule pins is the problem. The underlying issue is
>> > atomicity of commits. Oswald is right.
>> >
>> > We need to have a way to provide atomic commits across modules at least
>> > the CI should see these as atomic and integrate accordingly.
>> >
>>
>> what about trying to enable gerrit topic's feature again for cross-repo
>> changes?
>
> from the ci perspective, that's both pointless (because the grouping can
> be achieved temporally by just staging the changes at the same time) and
> insufficient (because the system currently just won't do atomic
> integrations).
>
> if you want to use the topic name as the label that has been originally
> proposed, regardless of the CI system, then there is nothing to fix:
> the ability to set the topic has been available forever (it just does
> not have any magic effect on integration).
> the problem with such "soft-labeling" is of course that it's laborious
> and terribly easy to mess up.
>
> fwiw, openstack's zuul recognizes explicit dependency markers in commit
> message footers, which is kind of a middle ground (it's formalized, so
> it can be checked).
If git hashes were used instead of Change-Id's, it would be useful to store
this information in git commits (in case of future need for bisecting)
i'm not a particular fan of their gerrit<=>ci
> integration approach to start with, though.
> _______________________________________________
> Development mailing list
> Development at qt-project.org
> http://lists.qt-project.org/mailman/listinfo/development
--
Regards,
Konstantin
More information about the Development
mailing list