Re: [ovirt-devel] Re: [VDSM] Trouble merging patch, related to jenkins whitelist or new RHEL CI?

Try again please, I think it will now work.

On Mon, Aug 3, 2020 at 9:47 AM Ales Musil <amusil(a)redhat.com&gt; wrote: > > > On Mon, Aug 3, 2020 at 8:30 AM Ehud Yonasi <eyonasi(a)redhat.com&gt; wrote: > >> Hey, >> That's my fault there, I've added the label to prepare for the rhel >> tests on ovirt and forgot to add the maintainers group. >> >> Could you verify you can add it now? >> > > Unfortunately not. It still has the same message/error. > > >> >> Thanks, >> Ehud. >> >> On Mon, Aug 3, 2020 at 8:43 AM Ales Musil <amusil(a)redhat.com&gt; wrote: >> >>> >>> >>> On Mon, Aug 3, 2020 at 2:24 AM Germano Veit Michel <germano(a)redhat.com&gt; >>> wrote: >>> >>>> >>>> >>>> On Sat, Aug 1, 2020 at 7:11 AM Nir Soffer <nsoffer(a)redhat.com&gt; wrote: >>>> >>>>> I could not merge: >>>>> https://gerrit.ovirt.org/c/109402/ >>>>> >>>>> Although it was approved and verified, and got +1 from Continuous >>>>> Integration. >>>>> >>>>> There is a new "RHEL-Continuous-Integration", which does not run with >>>>> this change, >>>>> even when I trigger the tests manually with "ci test". >>>>> >>>>> I tried to add +1 for "RHEL-Continuous-Integration" but this is not >>>>> possible, I see: >>>>> >>>>> RHEL-Continuous-Integration You don't have permission to edit >>>>> this label. >>>>> >>>> >>> I cannot merge anything as well. I wonder why maintainers don't get the >>> permission >>> for the new flag? Apparently the automation for this is not working >>> yet. >>> >>> >>>> >>>>> So finally I downloaded the patch and pushed it manually. >>>>> >>>>> I think this patch will fix the problem: >>>>> https://gerrit.ovirt.org/#/c/110576/ >>>>> >>>>> But I need CI experts to review this. Since we have 50 projects that >>>>> need this, >>>>> this probably should be fixed elsewhere, and inherited by all >>>>> projects. >>>>> >>>>> I know that Germano was not able to trigger tests because he was >>>>> missing in the >>>>> jenkins whitelist, but this was fixed last week. >>>>> >>>>> Germano, maybe just to check that everything works for you, you can >>>>> post some >>>>> trivial patch? >>>>> >>>> Build and test are working for me now after contacting infra. >>>> >>>> >>>> >>>>> >>>>> Nir >>>>> >>>>> >>> >>> -- >>> >>> Ales Musil >>> >>> Software Engineer - RHV Network >>> >>> Red Hat EMEA <https://www.redhat.com> >>> >>> amusil(a)redhat.com IM: amusil >>> <https://red.ht/sig> >>> _______________________________________________ >>> Devel mailing list -- devel(a)ovirt.org >>> To unsubscribe send an email to devel-leave(a)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/GFIT7NPOYQU... >>> >> > > -- > > Ales Musil > > Software Engineer - RHV Network > > Red Hat EMEA <https://www.redhat.com> > > amusil(a)redhat.com IM: amusil > <https://red.ht/sig> >