eclipse-equinox / p2

Eclipse Public License 2.0
14 stars 39 forks source link

Enhance p2 director with "addJREIU" option. #486

Closed raghucssit closed 5 months ago

raghucssit commented 5 months ago

P2 Director Application fails to install some of the bundles which has requirements on Java-SE 21. The reason for this issue was missing java profile dependency during installation. This new option includes Jre IU as an extra installable Unit.

Fixes https://github.com/eclipse-equinox/p2/issues/484

merks commented 5 months ago

The org.eclipse.equinox.p2.director.app project also needs a version increment.

Please try to consistently used JRE not Jre everywhere.

merks commented 5 months ago

FYI, I kicked off another build here because the last one failed in a way that looked more like a build infrastructure problem.

https://ci.eclipse.org/equinox/job/p2/job/PR-486/

merks commented 5 months ago

The builds keep failing like this:

15:29:30 [ERROR] Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.7-SNAPSHOT:verify (verify) on project org.eclipse.equinox.p2.publisher.eclipse: Execute ApiApplication failed: InvocationTargetException: Error reading manifest for /home/jenkins/agent/workspace/p2_PR-486/.m2/repository/p2/osgi/bundle/bcpg.source/1.77.0/bcpg.source-1.77.0.jar: zip file is empty -> [Help 1]

I think we should give it some time and I can manually kick off the build again later.

github-actions[bot] commented 5 months ago

Test Results

    9 files  ±0      9 suites  ±0   30m 20s :stopwatch: - 1m 50s 2 184 tests ±0  2 180 :white_check_mark: ±0   4 :zzz: ±0  0 :x: ±0  6 642 runs  ±0  6 631 :white_check_mark: ±0  11 :zzz: ±0  0 :x: ±0 

Results for commit 29dc9150. ± Comparison against base commit e262a155.

:recycle: This comment has been updated with latest results.

merks commented 5 months ago

It's still failing today. I'm going to create an "empty" PR to see if this is a general failure for all PRs.

iloveeclipse commented 5 months ago

I guess it simply needs a rebase?

merks commented 5 months ago

@iloveeclipse

It's failing like this:

07:29:34 [ERROR] Failed to execute goal org.eclipse.tycho:tycho-apitools-plugin:4.0.7-SNAPSHOT:verify (verify) on project org.eclipse.equinox.p2.ui.discovery: Execute ApiApplication failed: InvocationTargetException: Error reading manifest for /home/jenkins/agent/workspace/p2_PR-486/.m2/repository/p2/osgi/bundle/bcprov.source/1.77.0/bcprov.source-1.77.0.jar: zip file is empty -> [Help 1]

I'm not sure how a rebase could help with that. I'm testing if it's reproducible using:

https://github.com/eclipse-equinox/p2/pull/488

merks commented 5 months ago

I've also tried to replay the pipeline adding -U as suggested by @HannesWell

raghucssit commented 5 months ago

I guess it simply needs a rebase?

I did rebase just now.. I will check if build succeeds..

iloveeclipse commented 5 months ago

I would assume the local maven cache is broken on Jenkins. @merks : should we just re-create and push a new PR, or is there a way to cleanup all the data in current jenkins workspace?

merks commented 5 months ago

Yes, something broken. My test PR worked. I don't know how to clean the cache. Maybe just some shell script. In any case, I created this new PR with the same content:

https://github.com/eclipse-equinox/p2/pull/489

merks commented 5 months ago

Strange. The build here is referring to the same build as the one for my new PR.

In any case, if the build for my PR passes we can commit from this PR and discard mine. Let's wait and see.

iloveeclipse commented 5 months ago

@merks : many thanks for review and help on the PR!