Nir, I will.
On Mon, Aug 3, 2020 at 10:17 AM Nir Soffer <nsoffer(a)redhat.com> wrote:
On Mon, Aug 3, 2020, 09:55 Ales Musil <amusil(a)redhat.com> wrote:
>
>
> On Mon, Aug 3, 2020 at 8:51 AM Ehud Yonasi <eyonasi(a)redhat.com> 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> wrote:
>>
>>>
>>>
>>> On Mon, Aug 3, 2020 at 8:30 AM Ehud Yonasi <eyonasi(a)redhat.com> 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>
wrote:
>>>>
>>>>>
>>>>>
>>>>> On Mon, Aug 3, 2020 at 2:24 AM Germano Veit Michel <
>>>>> germano(a)redhat.com> wrote:
>>>>>
>>>>>>
>>>>>>
>>>>>> On Sat, Aug 1, 2020 at 7:11 AM Nir Soffer
<nsoffer(a)redhat.com>
>>>>>> 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...
>