On 09/02, Anton Marchukov wrote: > Hello David, All. > > As I see patch set 2 was a no code change and we exclude those from > check_patch. I can include it unless you had a better idea in mind.
Yes, we have to change the vdsm flag config to carry over the CI flag on no code change (that will apply only to new patches though). For this one the maintainer can override it (set the flag himself). > > Anton. > > On Wed, Sep 2, 2015 at 9:50 AM, David Caro <[email protected]> wrote: > > > > > Anton: If this is not yet solved ping me, I know the issue and how to > > solve it > > > > On 09/01, Eyal Edri wrote: > > > anton, > > > can you check? > > > > > > e. > > > > > > On Tue, Sep 1, 2015 at 10:11 PM, Allon Mureinik <[email protected]> > > wrote: > > > > > > > forwarding to [email protected], just to make sure someone picks it up > > > > ---------- Forwarded message ---------- > > > > From: Nir Soffer <[email protected]> > > > > Date: Tue, Sep 1, 2015 at 9:59 PM > > > > Subject: CI succeeded but gerrit does not show +1 > > > > To: David Caro <[email protected]> > > > > Cc: Dan Kenigsberg <[email protected]>, Idan Shaby <[email protected] > > >, > > > > Allon Mureinik <[email protected]> > > > > > > > > > > > > Hi David, > > > > > > > > For some reason gerrit will not show CI +1 for this patch: > > > > https://gerrit.ovirt.org/45585 > > > > > > > > The regular build after posting the patch: > > > > http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc22-x86_64/274/ > > > > > > > > When this build finished, gerrit added a comment: > > > > Patch Set 1: Continuous-Integration+1 > > > > Build Successful > > > > > > > > But we don't see the status in the main patch view: > > > > > > > > Code-Review > > > > +2 Nir Soffer > > > > +1 Allon Mureinik > > > > Continuous-Integration > > > > Verified > > > > +1 Idan Shaby > > > > > > > > triggered another build manually: > > > > http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc22-x86_64/294/ > > > > > > > > This build succeeded, but no change in gerrit. > > > > > > > > Seems like a bug in gerrit. Can you restart the system or force it > > > > to update? > > > > > > > > Nir > > > > > > > > > > > > _______________________________________________ > > > > Infra mailing list > > > > [email protected] > > > > http://lists.ovirt.org/mailman/listinfo/infra > > > > > > > > > > > > > > > > > -- > > > Eyal Edri > > > Supervisor, RHEV CI > > > EMEA ENG Virtualization R&D > > > Red Hat Israel > > > > > > phone: +972-9-7692018 > > > irc: eedri (on #tlv #rhev-dev #rhev-integ) > > > > > _______________________________________________ > > > Infra mailing list > > > [email protected] > > > http://lists.ovirt.org/mailman/listinfo/infra > > > > > > -- > > David Caro > > > > Red Hat S.L. > > Continuous Integration Engineer - EMEA ENG Virtualization R&D > > > > Tel.: +420 532 294 605 > > Email: [email protected] > > Web: www.redhat.com > > RHT Global #: 82-62605 > > -- David Caro Red Hat S.L. Continuous Integration Engineer - EMEA ENG Virtualization R&D Tel.: +420 532 294 605 Email: [email protected] Web: www.redhat.com RHT Global #: 82-62605
pgpe99UTbFYXv.pgp
Description: PGP signature
_______________________________________________ Infra mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/infra
