On Wed, Apr 7, 2021 at 6:45 PM Michal Skrivanek <michal.skriva...@redhat.com> wrote: > > > > > On 7. 4. 2021, at 12:04, Yedidyah Bar David <d...@redhat.com> wrote: > > > > On Tue, Aug 11, 2020 at 11:05 AM Tal Nisan <tni...@redhat.com> wrote: > >> > >> Hi everyone, > >> As you probably know we are now in a mode in which we develop our next > >> zstream version on the master branch as opposed to how we worked before > >> where the master version was dedicated for the next major version. This > >> makes the rapid changes in master to be delivered to customers in a much > >> higher cadence thus affecting stability. > >> Due to that we think it's best that from now on merges in the master > >> branch will be done only by stable branch maintainers after inspecting > >> those closely. > >> > >> What you need to do in order to get your patch merged: > >> - Have it pass Jenkins > >> - Have it get code review +2 > >> - Have it mark verified +1 > >> - It's always encourage to have it tested by OST, for bigger changes it's > >> a must > >> > >> Once you have all those covered, please add me as a reviewer and I'll > >> examine it and merge if everything seems right, if I haven't done it in a > >> timely manner feel free to ping me. > > > > Is the above still the current policy? > > Hi Didi, > well, yes it is. what’s the concern?
No "concern", other than I noticed a few times that people other than Tal merged patches, and yesterday I did the same [1] after getting +1 from Sandro and consulting him, seeing that I have permissions. IIRC I didn't have permissions until recently, so I wondered if anything changed. If the re-granting of permissions was a mistake, let's revert. If it's on purpose, perhaps clarify the situation. [1] https://gerrit.ovirt.org/c/ovirt-engine/+/114130 > I’d love to get to a point when we can automatically gate patches based on > OST, but it’s going slow…so for now it’s still manual. Not sure that's enough, but it would be a step in the right direction. Sometimes patches won't break OST but are still harmful. Did we ever consider going fully to Test-Driven-Development? Not certain if there are studies/methods to calculate/approximate the expected extra work this will require. Assuming you want eventually all developers to also know well-enough OST (in addition to their specific expertise) and be comfortable patching it, it might make sense. Best regards, -- Didi _______________________________________________ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-le...@ovirt.org Privacy Statement: https://www.ovirt.org/privacy-policy.html oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/devel@ovirt.org/message/RAPSZE6V4PW6ZIDXH5F3EBN6XYUABTFO/