[ovirt-devel] Moving forward our frontend stack
David Caro
dcaroest at redhat.com
Fri Aug 29 09:24:21 UTC 2014
On 08/29, Martin Betak wrote:
> ----- Original Message -----
> > From: "Vojtech Szocs" <vszocs at redhat.com>
> > To: "Martin Betak" <mbetak at redhat.com>
> > Cc: devel at ovirt.org, "Einav Cohen" <ecohen at redhat.com>, "Alexander Wels" <awels at redhat.com>, "Greg Sheremeta"
> > <gshereme at redhat.com>, "Tomas Jelinek" <tjelinek at redhat.com>, "Lior Vernia" <lvernia at redhat.com>, "Daniel Erez"
> > <derez at redhat.com>
> > Sent: Thursday, August 28, 2014 5:38:04 PM
> > Subject: Re: Moving forward our frontend stack
> >
> > Hey Martin!
> >
> > I've just reviewed your patch, looks good overall.
> >
> > Here are my thoughts:
> >
> > * Jenkins CI fails on patch due to missing dependencies,
> > we need to provide following new dependencies in order
> > to proceed with the upgrade:
> >
> > org.aspectj:aspectjweaver:1.8.2
> > org.aspectj:aspectjrt:1.8.2
> > com.google.gwt:gwt-user:2.6.1
> > com.google.gwt:gwt-dev:2.6.1
> > com.google.gwt:gwt-servlet:2.6.1
> > com.google.gwt:2.6.1
> > org.codehaus.mojo:gwt-maven-plugin:2.6.1
> > com.gwtplatform:gwtp-processors:1.3.1
> > com.gwtplatform:gwtp-mvp-client:1.3.1
> > com.google.gwt.inject:gin:2.1.2
> >
> > Who will take charge of that?
>
> I already spoke to David Caro about the jenkins failure of
> patch http://gerrit.ovirt.org/#/c/32135/. It seems the files
> for appropriate versions are there but the files are corrupted.
>
> Let us hope this can be resolved easier than Node.js :-)
It seems that the pom on jboss repository has a checksum that does not
match the ones there.
http://repository.jboss.org/nexus/content/groups/public-jboss/com/gwtplatform/gwtp/1.3.1/
If you get the gwtp-1.3.1.pom file, you get the hashed:
MD5: 9acbb4e5088825d31e2c8307ae679243
SHA1: eb6e92c012926fb265cf4f359c3d5adca8c5c0f9
While the ones there are:
MD5: c33a1cd1ffa88aa4cec42e03364b0e0c
SHA1: ca6c0155356765fc23f0a557373193f013adfd6c
Any of you have any contact with the guys that maintain that repo (or
know about who can know about?)
I seem to be unable to find any contant in the repo itself...
Anyhow, I've disabled the checksum checking and just pass through, so
it should not complain about unable to download it (it might fail due
to the checksum, it should at least xd).
>
> Best regards,
>
> Martin
>
> >
> > * patch itself looks quite harmless (not too risky)
> >
> > * consolidating Java source & target version across
> > frontend and backend is nice!
> >
> > -> this means we could use Java 7 features
> > also on the frontend (Java/GWT) side
> >
> > * TODO-GWT tags [1] proved to be helpful [2]
> >
> > -> reminder to all UI maintainers to use TODO-GWT
> > tags whenever we have some GWT(P) workaround,
> > so that the future upgrade will be safer w.r.t.
> > existing code
> >
> > [1] https://www.mail-archive.com/devel@ovirt.org/msg00761.html
> > [2]
> > http://gerrit.ovirt.org/#/c/32135/1/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/SearchPanelView.java
> >
> > Regards,
> > Vojtech
> >
> > PS: I went through GWT-Platform release notes,
> > found an interesting new feature in recent GWTP
> > release, worth investigating:
> >
> > https://github.com/ArcBees/GWTP/wiki/Release-Notes
> > #346 : Map more than one name token to presenter
> >
> >
> > ----- Original Message -----
> > > From: "Martin Betak" <mbetak at redhat.com>
> > > To: devel at ovirt.org
> > > Cc: "Vojtech Szocs" <vszocs at redhat.com>, "Einav Cohen" <ecohen at redhat.com>,
> > > "Alexander Wels" <awels at redhat.com>,
> > > "Greg Sheremeta" <gshereme at redhat.com>, "Tomas Jelinek"
> > > <tjelinek at redhat.com>, "Lior Vernia" <lvernia at redhat.com>,
> > > "Daniel Erez" <derez at redhat.com>
> > > Sent: Thursday, August 28, 2014 4:11:10 PM
> > > Subject: Moving forward our frontend stack
> > >
> > > Hello oVirt developers!
> > >
> > > I have prepared patch [1] that upgrades our frontend stack
> > > to use GWT version 2.6.1 (from previous 2.5.1).
> > >
> > > This patch also updates GIN to version 2.1.2 and GWT-P to 1.3.1
> > >
> > > Since GWT 2.6 features support for Java 7 it was possible to increment
> > > language levels of all projects stuck at Java 6 (common, compat,
> > > searchbackend
> > > and entire of frontend).
> > >
> > > To facilitate emitting bytecode compatible with Java 7 also upgrade of
> > > AspectJ was
> > > necessary. This patch upgrades it to AspectJ 1.8 that features even support
> > > for
> > > Java 8 which will save effort when upgrading to GWT 2.7/3.0 in the future.
> > >
> > > Most of the changes in the patch are due to upgrade of GWT-P - i.e.
> > > changing
> > > packages of TokenFormatter and PlaceRequest.
> > >
> > > Overall this patch is *MUCH* simpler than the previous
> > > http://gerrit.ovirt.org/#/c/16739/
> > > which facilitated upgrade from 2.3 to 2.5.1, and hopefully much less risky.
> > >
> > > I have tested draft-compile, debug-mode and also tried to use the resulting
> > > application
> > > manually for some time. So far everything worked (surprisingly well!) and I
> > > have not
> > > detected any defects. Of course I invite anyone to test this patch on his
> > > own
> > > since
> > > it is and upgrade of our core infrastructure.
> > >
> > > That having said I think is comparatively simple and the benefits outweigh
> > > the risks
> > > if this upgrade is done at the beginning of ovirt-3.6 development cycle.
> > >
> > > Reviews, comments and testing are very welcome :-)
> > >
> > > Best regards,
> > >
> > > Martin
> > >
> > > [1] http://gerrit.ovirt.org/#/c/32135/
> > >
> >
--
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/devel/attachments/20140829/de54e917/attachment-0001.sig>
More information about the Devel
mailing list