CI succeeded but gerrit does not show +1

David Caro dcaro at redhat.com
Wed Sep 2 09:15:33 UTC 2015


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 <dcaro at redhat.com> 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 <amureini at redhat.com>
> > wrote:
> > >
> > > > forwarding to infra at ovirt.org, just to make sure someone picks it up
> > > > ---------- Forwarded message ----------
> > > > From: Nir Soffer <nsoffer at redhat.com>
> > > > Date: Tue, Sep 1, 2015 at 9:59 PM
> > > > Subject: CI succeeded but gerrit does not show +1
> > > > To: David Caro <dcaroest at redhat.com>
> > > > Cc: Dan Kenigsberg <danken at redhat.com>, Idan Shaby <ishaby at redhat.com
> > >,
> > > > Allon Mureinik <amureini at redhat.com>
> > > >
> > > >
> > > > 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
> > > > Infra at ovirt.org
> > > > 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
> > > Infra at ovirt.org
> > > 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: dcaro at redhat.com
> > 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: dcaro at redhat.com
Web: www.redhat.com
RHT Global #: 82-62605
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20150902/a6ce4337/attachment.sig>


More information about the Infra mailing list