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. 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? Nir

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. >

> 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 > >

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?

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... >

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> >

On Mon, Aug 3, 2020, 09:55 Ales Musil <amusil(a)redhat.com&gt; wrote: > > > On Mon, Aug 3, 2020 at 8:51 AM Ehud Yonasi <eyonasi(a)redhat.com&gt; wrote: > >> Try again please, I think it will now work. >> > > It does. Thank you. > > >> >> 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. >>>> >>> Ehud, please discuss here changes to vdsm CI before applying them. >>>> 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> >>> >> > > -- > > Ales Musil > > Software Engineer - RHV Network > > Red Hat EMEA <https://www.redhat.com> > > amusil(a)redhat.com IM: amusil > <https://red.ht/sig> > _______________________________________________ > Infra mailing list -- infra(a)ovirt.org > To unsubscribe send an email to infra-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/infra@ovirt.org/message/H7FLG4HKUDN... >