--AA9g+nFNFPYNJKiL
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On 01/28 16:17, Eyal Edri wrote:
David,
Is it something manual we need to change per version or we can add
something automatic to work on any 3.6.X branch only (exluding the 3.6
branch)?
Can you summarize the complete checks that should be done on the hooks
side? Because it's starting to be a bit of a mess as we are changing
small things each time without the complete knowledge (for example,
the target release was being checked before when moving to MODIFIED,
and that was commented out, and there's a hook to warn if the patch is
not merged yet on a previous branch, like not being merged on 3.6 if
it's on 3.6.x)
=20
e.
=20
On Thu, Jan 28, 2016 at 4:00 PM, Tal Nisan <tnisan(a)redhat.com> wrote:
=20
> This was the behavior in 3.6.2, don't know why it changed
>
> On Thu, Jan 28, 2016 at 3:57 PM, Eyal Edri <eedri(a)redhat.com> wrote:
>
>> so maybe we'll stop moving bugs to MODIFIED if they are merged on stab=
le
>> branch (i.e ovirt-engine-3.6) and only if its merged into a
version br=
anch
>> (i.e ovirt-engine-3.6.3) ?
>>
>> On Thu, Jan 28, 2016 at 3:52 PM, Tal Nisan <tnisan(a)redhat.com> wrote:
>>
>>> It makes life easier for everyone when automation moves the bug to
>>> MODIFIED, if the work is not done then the developer can always move =
it
>>> back to POST like we sometimes do.
>>> In any way, when a 3.6.3 bug is merged in 3.6.3 it should be moved to
>>> MODIFIED, not a second before
>>>
>>>
>>> On Thu, Jan 28, 2016 at 12:23 PM, Yedidyah Bar David
<didi(a)redhat.com>
>>> wrote:
>>>
>>>> On Thu, Jan 28, 2016 at 10:59 AM, Eyal Edri <eedri(a)redhat.com>
wrote:
>>>> >
>>>> >
>>>> > On Thu, Jan 28, 2016 at 10:23 AM, Yedidyah Bar David
<didi(a)redhat.=
com
>>>> >
>>>> > wrote:
>>>> >>
>>>> >> On Thu, Jan 28, 2016 at 10:08 AM, Eyal Edri
<eedri(a)redhat.com>
>>>> wrote:
>>>> >> > Maybe we should change the logic on moving bugs to
MODIFIED,
>>>> >> > Now that we moved to the version branch.
>>>> >> >
>>>> >> > Tal - any thoughts on a logic that SHOULD move bugs to
MODIFIED
>>>> that
>>>> >> > we'll
>>>> >> > be sure its OK?
>>>> >>
>>>> >> Repeating myself from a previous discussion: I think we should
ne=
ver
>>>> move
>>>> >> bugs from POST to MODIFIED. Only a developer can know if
it's ind=
eed
>>>> >> ready,
>>>> >> or another patch is still needed but was not yet pushed. I
think =
it's
>>>> >> safer
>>>> >> to miss bugs left on POST although they could have been moved
to
>>>> MODIFIED,
>>>> >> than the opposite - move to MODIFIED bugs that actually require
m=
ore
>>>> work.
>>>> >>
>>>> >
>>>> > You're optimistic that bugs will not stay on POST, we've
done it in
>>>> the
>>>> > past.
>>>> > This has a bigger risk of bugs getting left behind on POST and
>>>> increasing
>>>> > the amount of
>>>> > bugs that are not verified on time.
>>>> >
>>>> > Its the easiest solution to drop this bot, but i belive it will do
>>>> more
>>>> > damage than good.
>>>>
>>>> Perhaps, then, run it once a day and move to MODIFIED only if last p=
atch
>>>> was merged at least X hours ago? Hopefully
developers will not wait =
more
>>>> than X before pushing an additional patch for the
same bug if needed.
>>>>
>>>> Of course, we then also have to fix the issue starting current threa=
d.
>>>>
>>>> >
>>>> >>
>>>> >> >
>>>> >> > e.
>>>> >> >
>>>> >> > On Thu, Jan 28, 2016 at 10:03 AM, Tal Nisan
<tnisan(a)redhat.com>
>>>> wrote:
>>>> >> >>
>>>> >> >> After merging a patch for a 3.6.3 bug on the
ovirt-engine-3.6
>>>> branch
>>>> >> >> the
>>>> >> >> bug is moved to MODIFIED, since the 3.6.3 branch was
opened
>>>> yesterday
>>>> >> >> the
>>>> >> >> bug should stay in POST until merged in
ovirt-engine-3.6.3 bra=
nch
>>>> as
>>>> >> >> well
>>>> >> >>
>>>> >> >>
>>>> >> >> _______________________________________________
>>>> >> >> Infra mailing list
>>>> >> >> Infra(a)ovirt.org
>>>> >> >>
http://lists.ovirt.org/mailman/listinfo/infra
>>>> >> >>
>>>> >> >
>>>> >> >
>>>> >> >
>>>> >> > --
>>>> >> > Eyal Edri
>>>> >> > Associate Manager
>>>> >> > EMEA ENG Virtualization R&D
>>>> >> > Red Hat Israel
>>>> >> >
>>>> >> > phone: +972-9-7692018
>>>> >> > irc: eedri (on #tlv #rhev-dev #rhev-integ)
>>>> >> >
>>>> >> > _______________________________________________
>>>> >> > Infra mailing list
>>>> >> > Infra(a)ovirt.org
>>>> >> >
http://lists.ovirt.org/mailman/listinfo/infra
>>>> >> >
>>>> >>
>>>> >>
>>>> >>
>>>> >> --
>>>> >> Didi
>>>> >
>>>> >
>>>> >
>>>> >
>>>> > --
>>>> > Eyal Edri
>>>> > Associate Manager
>>>> > EMEA ENG Virtualization R&D
>>>> > Red Hat Israel
>>>> >
>>>> > phone: +972-9-7692018
>>>> > irc: eedri (on #tlv #rhev-dev #rhev-integ)
>>>>
>>>>
>>>>
>>>> --
>>>> Didi
>>>>
>>>
>>>
>>
>>
>> --
>> Eyal Edri
>> Associate Manager
>> EMEA ENG Virtualization R&D
>> Red Hat Israel
>>
>> phone: +972-9-7692018
>> irc: eedri (on #tlv #rhev-dev #rhev-integ)
>>
>
>
=20
=20
--=20
Eyal Edri
Associate Manager
EMEA ENG Virtualization R&D
Red Hat Israel
=20
phone: +972-9-7692018
irc: eedri (on #tlv #rhev-dev #rhev-integ)
--=20
David Caro
Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D
Tel.: +420 532 294 605
Email: dcaro(a)redhat.com
IRC: dcaro|dcaroest@{freenode|oftc|redhat}
Web:
www.redhat.com
RHT Global #: 82-62605
--AA9g+nFNFPYNJKiL
Content-Type: application/pgp-signature; name="signature.asc"
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
iQEcBAEBAgAGBQJWsOUhAAoJEEBxx+HSYmnDKqIH/RjqeEgHXC4OuKIM6t2M36ge
oBW226IzslN9fqkdJBrwC6zmEOoX5G7DgvNOo0Lths/Y2lijuLgPcecU8ralFXs/
ePAK/0OE6k9oR8gn3sYzBUurgrC9SzxF+/FrtHv8Bj0FFvVDaSI12uY81FGvumCN
HAQzd31ZXo2eTIXiLsQYduvSFhgmQZrtlVo67pIYTve0TJfgIHGExIE9N8WhDsn/
MEwpaNLeFHjyiHtHSvwY+v0UvoAFr+c/m6OBVSmoFWG0dHkJvUMLEYnP19enLwDD
/1Ck38AJzmDziwHPMFxT+k0BAv5wG8TYeVYBMJBkzivjgOw7UZRRxy48qfSgGKY=
=weAi
-----END PGP SIGNATURE-----
--AA9g+nFNFPYNJKiL--