On Wednesday, May 04, 2016 03:38:40 PM Eyal Edri wrote:
> Vojtech,
> Any ideas on a test we can add to check-patch.sh (other than building all
> permutations) that can help catch these ?
>
> The original patch didn't fail on unit tests (though I can't see info since
> the build got deleted, we now increased the history for builds so we will
> be able to see it).
> Please let me know once the translations patch is re-submitted,I'd like to
> monitor it and see if CI catch any issues there.
>
> e.
>
Eyal,
I wrote a unit test specifically to catch this particular issue without having
to compile all the permutations. This is what failed the build, I don't
understand why it wasn't run during CI.
> On Wed, May 4, 2016 at 3:30 PM, Vojtech Szocs <vszocs@redhat.com> wrote:
> > ----- Original Message -----
> >
> > > From: "Vojtech Szocs" <vszocs@redhat.com>
> > > To: "Oved Ourfali" <oourfali@redhat.com>
> > > Cc: "devel" <devel@ovirt.org>
> > > Sent: Wednesday, May 4, 2016 2:24:21 PM
> > > Subject: Re: [ovirt-devel] Master ovirt-engine build broken
> > >
> > >
> > >
> > > ----- Original Message -----
> > >
> > > > From: "Vojtech Szocs" <vszocs@redhat.com>
> > > > To: "Oved Ourfali" <oourfali@redhat.com>, "David Caro"
> > > > <dcaroest@redhat.com>
> > > > Cc: "devel" <devel@ovirt.org>
> > > > Sent: Wednesday, May 4, 2016 2:19:09 PM
> > > > Subject: Re: [ovirt-devel] Master ovirt-engine build broken
> > > >
> > > >
> > > >
> > > > ----- Original Message -----
> > > >
> > > > > From: "Oved Ourfali" <oourfali@redhat.com>
> > > > > To: "Vojtech Szocs" <vszocs@redhat.com>
> > > > > Cc: "Marek Libra" <mlibra@redhat.com>, "devel" <devel@ovirt.org>
> > > > > Sent: Wednesday, May 4, 2016 2:07:19 PM
> > > > > Subject: Re: [ovirt-devel] Master ovirt-engine build broken
> > > > >
> > > > > Okay.
> > > > > Makes sense.
> > > > > Thanks Vojtech - please also revert this one.
> > > >
> > > > Trying to revert master patch https://gerrit.ovirt.org/#/c/56720/
> > > > gives me "500 Internal server error" -- am I missing something?
> > > >
> > > > Perhaps oVirt gerrit maintainers can advise here? (adding David)
> > >
> > > Sorry for the noise, I did it manually with git revert:
> > > https://gerrit.ovirt.org/#/c/57031/
> > >
> > > Can someone please ack?
> >
> > Merged, this should fix the master build.
> >
> > > > > Oved
> > > > >
> > > > > On Wed, May 4, 2016 at 3:04 PM, Vojtech Szocs <vszocs@redhat.com>
> >
> > wrote:
> > > > > > (top-posting)
> > > > > >
> > > > > > Hi,
> > > > > >
> > > > > > root cause is wrong number of message parameters (placeholders)
> > > > > > in translated properties files, compared to message definitions
> > > > > > in Java code.
> > > > > >
> > > > > > For example, UIMessages#cpuInfoLabel method has 4 parameters,
> > > > > > but the corresponding message in `UIMessages_de_DE.properties`
> > > > > > only contains 3 placeholders.
> > > > > >
> > > > > > 4.0 String Freeze is May 25 so it's still OK to modify message
> > > > > > definitions in Java code.
> > > > > >
> > > > > > I suggest to simply revert the offending patch:
> > > > > > translations update from zanata ovirt-3.6
> > > > > > https://gerrit.ovirt.org/#/c/56720/
> > > > > >
> > > > > > and create new one which doesn't introduce conflicts.
> > > > > >
> > > > > > We have unit tests to detect these kinds of errors, this is
> > > > > > to ensure the build fails early on (before GWT compilation).
> > > > > >
> > > > > > I'm not sure why `check-patch` job for above mentioned patch
> > > > > > didn't report the problem, since UIMessagesTest should fail,
> > > > > > as Roman wrote below.
> > > > > >
> > > > > > Regards,
> > > > > > Vojtech
> > > > > >
> > > > > >
> > > > > > ----- Original Message -----
> > > > > >
> > > > > > > From: "Marek Libra" <mlibra@redhat.com>
> > > > > > > To: "devel" <devel@ovirt.org>
> > > > > > > Sent: Wednesday, May 4, 2016 12:31:18 PM
> > > > > > > Subject: Re: [ovirt-devel] Master ovirt-engine build broken
> > > > > > >
> > > > > > > There are issues on other keys as well.
> > > > > > > I'm rebuilding for all locales locally now.
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > From: "Marek Libra" <mlibra@redhat.com>
> > > > > > > To: "devel" <devel@ovirt.org>
> > > > > > > Sent: Wednesday, May 4, 2016 11:29:55 AM
> > > > > > > Subject: Re: [ovirt-devel] Master ovirt-engine build broken
> > > > > > >
> > > > > > > The fix is here: https://gerrit.ovirt.org/#/c/57007/
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > From: "Roman Mohr" <rmohr@redhat.com>
> > > > > > > To: "Oved Ourfali" <oourfali@redhat.com>
> > > > > > > Cc: "devel" <devel@ovirt.org>
> > > > > > > Sent: Wednesday, May 4, 2016 10:27:51 AM
> > > > > > > Subject: Re: [ovirt-devel] Master ovirt-engine build broken
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > On Wed, May 4, 2016 at 10:24 AM, Oved Ourfali <
> >
> > oourfali@redhat.com >
> >
> > > > > > wrote:
> > > > > > > Roman - did you build the langs in order to find the issue?
> > > > > > >
> > > > > > > I just ran 'mvn clean verify', so I saw it on my machine. On
> >
> > gerrit
> >
> > > > > > > not
> > > > > >
> > > > > > all
> > > > > >
> > > > > > > of my builds showed that error.
> > > > > > >
> > > > > > > Looking at
> >
> > https://gerrit.ovirt.org/#/q/status:open+project:ovirt-engine+branch:maste
> > r+topic:integration>
> > > > > > > two patches passed and two failed after a topic rebase.
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > Eyal - can we monitor a list of files that i changed will result
> >
> > in
> >
> > > > > > > also
> > > > > > > building all the languages?
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > On Wed, May 4, 2016 at 11:22 AM, Eyal Edri < eedri@redhat.com >
> > > > > > > wrote:
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > right, and we can't enable lang permutation since it takes a
> > > > > > > very
> > > > > > > long
> > > > > >
> > > > > > time
> > > > > >
> > > > > > > to run.
> > > > > > >
> > > > > > > On Wed, May 4, 2016 at 11:07 AM, Tomas Jelinek <
> >
> > tjelinek@redhat.com
> >
> > > > > > wrote:
> > > > > > > ----- Original Message -----
> > > > > > >
> > > > > > > > From: "Oved Ourfali" < oourfali@redhat.com >
> > > > > > > > To: "Tomas Jelinek" < tjelinek@redhat.com >
> > > > > > > > Cc: "Roman Mohr" < rmohr@redhat.com >, "devel" <
> >
> > devel@ovirt.org >,
> >
> > > > > > "Scott
> > > > > >
> > > > > > > > Dickerson" < sdickers@redhat.com >
> > > > > > > > Sent: Wednesday, May 4, 2016 9:55:40 AM
> > > > > > > > Subject: Re: [ovirt-devel] Master ovirt-engine build broken
> > > > > > > >
> > > > > > > > I wonder, how come the CI didn't catch that?
> > > > > > >
> > > > > > > because this happens only when you compile with language
> >
> > permutations
> >
> > > > > > > > On Wed, May 4, 2016 at 10:44 AM, Tomas Jelinek <
> > > > > > > > tjelinek@redhat.com
> > > > > > > >
> > > > > > > > wrote:
> > > > > > > > > hmmm,
> > > > > > > > > regression introduced yesterday by
> > > > > >
> > > > > > https://gerrit.ovirt.org/#/c/56720/
> > > > > >
> > > > > > > > > fix on the way
> > > > > > > > >
> > > > > > > > > ----- Original Message -----
> > > > > > > > >
> > > > > > > > > > From: "Roman Mohr" < rmohr@redhat.com >
> > > > > > > > > > To: "devel" < devel@ovirt.org >
> > > > > > > > > > Sent: Wednesday, May 4, 2016 9:34:53 AM
> > > > > > > > > > Subject: [ovirt-devel] Master ovirt-engine build broken
> > > > > > > > > >
> > > > > > > > > > Hi,
> > > > > > > > > >
> > > > > > > > > > I see failing tests regarding to missing translations
> > > > > > > > > > (e.g.
> > > > > > > > > > [1]).
> > > > > > > > > >
> > > > > > > > > > [...]
> > > > > > > > > >
> > > > > > > > > > Failed tests:
> > > > > > > > > > doTest(org.ovirt.engine.ui.uicompat.UIMessagesTest):
> > > > > > > > > > cpuInfoLabel does not match the number of parameters in
> > > > > > > > > > UIMessages_zh_CN.properties(..)
> > > > > > > > > >
> > > > > > > > > > [...]
> > > > > > > > > >
> > > > > > > > > > Best Regards,
> > > > > > > > > >
> > > > > > > > > > Roman
> > > > > > > > > >
> > > > > > > > > > [1]
> >
> > http://jenkins.ovirt.org/job/ovirt-engine_master_check-patch-fc23-x86_64/4
> > 07/console>
> > > > > > > > > > _______________________________________________
> > > > > > > > > > Devel mailing list
> > > > > > > > > > Devel@ovirt.org
> > > > > > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > > > > > > >
> > > > > > > > > _______________________________________________
> > > > > > > > > Devel mailing list
> > > > > > > > > Devel@ovirt.org
> > > > > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Devel mailing list
> > > > > > > Devel@ovirt.org
> > > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Eyal Edri
> > > > > > > Associate Manager
> > > > > > > RHEV DevOps
> > > > > > > EMEA ENG Virtualization R&D
> > > > > > > Red Hat Israel
> > > > > > >
> > > > > > > phone: +972-9-7692018
> > > > > > > irc: eedri (on #tlv #rhev-dev #rhev-integ)
> > > > > > >
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Devel mailing list
> > > > > > > Devel@ovirt.org
> > > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > > > > >
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Devel mailing list
> > > > > > > Devel@ovirt.org
> > > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > > > > >
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Devel mailing list
> > > > > > > Devel@ovirt.org
> > > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > > > > >
> > > > > > >
> > > > > > > _______________________________________________
> > > > > > > Devel mailing list
> > > > > > > Devel@ovirt.org
> > > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > > > >
> > > > > > _______________________________________________
> > > > > > Devel mailing list
> > > > > > Devel@ovirt.org
> > > > > > http://lists.ovirt.org/mailman/listinfo/devel
> > > >
> > > > _______________________________________________
> > > > Devel mailing list
> > > > Devel@ovirt.org
> > > > http://lists.ovirt.org/mailman/listinfo/devel
> > >
> > > _______________________________________________
> > > Devel mailing list
> > > Devel@ovirt.org
> > > http://lists.ovirt.org/mailman/listinfo/devel
> >
> > _______________________________________________
> > Devel mailing list
> > Devel@ovirt.org
> > http://lists.ovirt.org/mailman/listinfo/devel