[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [cross-project-issues-dev] [epp-dev] Photon aggregation jobs are stuck



On Fri, May 18, 2018 at 10:38 AM, MikaÃl Barbero <mikael.barbero@xxxxxxxxxxxxxxxxxxxxxx> wrote:
As platform is now building with Tycho 1.2.0-snapshot, the issues we see in the p2 repo files probably come from the different versions of p2.

As suggested by Alexander, I'm currently building a new version of the aggregator with the latest p2 from platform M8 (https://git.eclipse.org/r/#/c/122911/)

I'll start an aggregation build with this new version of the aggregator (https://git.eclipse.org/r/#/c/122913/) shortly after.

After a bit of investigation aggregator MirrorGenerator does some really ugly things to p2 metadata like ignoring/removing the new checksums so despite the fact that in p2 and Tycho as a result we have moved to publish sha-256 checksums the mirror application tries to remove them (and doesn't succeed in that even). The following patch https://git.eclipse.org/r/#/c/122914/ should do the intended action but it's plain wrong to modify artifacts like that. So here is the fix this time but from now on "The only contributions I'll make to aggregator will be aimed at replacing it by Tycho" .
Â

--
MikaÃl Barbero -ÂEclipse Foundation
IT Services - Release Engineering
ð (+33) 642 028 039
ð @mikbarbero

Le 18 mai 2018 Ã 09:18, Mickael Istria <mistria@xxxxxxxxxx> a Ãcrit :



On Fri, May 18, 2018 at 8:01 AM, MikaÃl Barbero <mikael.barbero@eclipse-foundation.org> wrote:
Any idea how it could be fixed?

Nope, but I think you should file a bug and link it to https://bugs.eclipse.org/bugs/show_bug.cgi?id=423715 .
Comparing with the Platform M7 repo, It looks like that aggregator does create a new artifact, with a new md5 checksum (which is already questionable as the idea behind p2 is more to reuse artifacts and metadata) and has copied the download.checksum.* metadata from Platform (which is strange too, as other hash says it's apparently a different artifact) and moreover, those are metadata from the packed artifact while the artifact here is not packed!


--
Mickael Istria
Eclipse IDE developer, for Red Hat Developers
_______________________________________________
cross-project-issues-dev mailing list
cross-project-issues-dev@eclipse.org
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/cross-project-issues-dev


_______________________________________________
epp-dev mailing list
epp-dev@xxxxxxxxxxx
To change your delivery options, retrieve your password, or unsubscribe from this list, visit
https://dev.eclipse.org/mailman/listinfo/epp-dev



--
Alexander Kurtakov
Red Hat Eclipse Team