Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update the target platform for Orbit m2 #1233

Merged
merged 1 commit into from
Jul 26, 2023

Conversation

merks
Copy link
Contributor

@merks merks commented Jul 25, 2023

@merks merks force-pushed the issue-1199-m2 branch 4 times, most recently from ad8df20 to a9f17ab Compare July 26, 2023 07:51
@merks
Copy link
Contributor Author

merks commented Jul 26, 2023

@iloveeclipse

I'm ready to merge this now, but I noticed that the verification build here does not actually run any tests. That concerns me a bit because of the changes to org.junit. When I merge this, I want to immediately start a nightly integration build. Could you please help me with that? I.e., could you squash and merge this, start a build, and post a link to the build here please?

@laeubi
Copy link
Contributor

laeubi commented Jul 26, 2023

@merks if you want the test to run (but keep in mind this can take long) you need to remove (temporary maybe with another PR) the -DskipTests flag from the Jenkinsfile.

@merks
Copy link
Contributor Author

merks commented Jul 26, 2023

@laeubi

That's a good idea! Thanks!!

@iloveeclipse

I shall try that first!

@merks
Copy link
Contributor Author

merks commented Jul 26, 2023

@laeubi

I see the tests are running now, so your idea was a good one! So far I see nothing failing that suggest target platform changes are bad...

@merks
Copy link
Contributor Author

merks commented Jul 26, 2023

Well, a bunch of tests ran but I don't think everything is well configured to run tests when it fails like this eventually:

11:14:38  [INFO] Command line:
11:14:38  	[/opt/tools/java/openjdk/jdk-17/latest/bin/java, -Dosgi.noShutdown=false, -Dosgi.os=linux, -Dosgi.ws=gtk, -Dosgi.arch=x86_64, -Xmx1G, -Djdt.default.test.compliance=1.8, -DDetectVMInstallationsJob.disabled=true, ${tycho.surefire.argLine}, -Dosgi.clean=true, -ea, -jar, /home/jenkins/agent/workspace/atform.releng.aggregator_PR-1233/equinox/bundles/org.eclipse.equinox.launcher/target/org.eclipse.equinox.launcher-1.6.500-SNAPSHOT.jar, -data, /home/jenkins/agent/workspace/atform.releng.aggregator_PR-1233/eclipse.jdt.core/org.eclipse.jdt.core.tests.model/target/work/data, -install, /home/jenkins/agent/workspace/atform.releng.aggregator_PR-1233/eclipse.jdt.core/org.eclipse.jdt.core.tests.model/target/work, -configuration, /home/jenkins/agent/workspace/atform.releng.aggregator_PR-1233/eclipse.jdt.core/org.eclipse.jdt.core.tests.model/target/work/configuration, -application, org.eclipse.tycho.surefire.osgibooter.headlesstest, -testproperties, /home/jenkins/agent/workspace/atform.releng.aggregator_PR-1233/eclipse.jdt.core/org.eclipse.jdt.core.tests.model/target/surefire.properties]
11:14:38  Error: Could not find or load main class ${tycho.surefire.argLine}
11:14:38  Caused by: java.lang.ClassNotFoundException: ${tycho.surefire.argLine}

This is not related to target platform content.

I will revert the changes to run tests and assume all is good...

@merks merks merged commit 1a5c07c into eclipse-platform:master Jul 26, 2023
2 checks passed
@merks merks deleted the issue-1199-m2 branch July 26, 2023 10:37
@merks
Copy link
Contributor Author

merks commented Jul 26, 2023

@iloveeclipse

It would probably be good to kick off a nightly build...

@iloveeclipse
Copy link
Member

HeikoKlare added a commit to HeikoKlare/eclipse.platform that referenced this pull request Jul 26, 2023
The target platform has been updated with Lucene 9.7.0. This adapts the
according index compatibility test case.

See eclipse-platform/eclipse.platform.releng.aggregator#1233
@HeikoKlare
Copy link
Contributor

The Lucene version update makes the according index compatility test in platform fail in current builds such as https://ci.eclipse.org/platform/job/eclipse.platform/job/PR-579/4/testReport/

Is it a proper way to simply adapt the compatibility test case? eclipse-platform/eclipse.platform#595

@merks
Copy link
Contributor Author

merks commented Jul 26, 2023

@HeikoKlare

It might be proper to delete the test rather than update it each time Lucene updates simply to match the Lucene version yet again. I.e., I think the point was to be sure that we pre-index using the same version of Lucene as we have in the target platform, but I don't think we do that at all anymore.

HeikoKlare added a commit to HeikoKlare/eclipse.platform that referenced this pull request Jul 28, 2023
The target platform has been updated with Lucene 9.7.0. This adapts the
according index compatibility test case to not rely on a specific Lucene
version.

See eclipse-platform/eclipse.platform.releng.aggregator#1233
HeikoKlare added a commit to HeikoKlare/eclipse.platform that referenced this pull request Jul 28, 2023
The target platform has been updated with Lucene 9.7.0. This adapts the
according index compatibility test case to not rely on a specific Lucene
version.

See eclipse-platform/eclipse.platform.releng.aggregator#1233
Fixes eclipse-platform#601
akurtakov pushed a commit to eclipse-platform/eclipse.platform that referenced this pull request Jul 31, 2023
The target platform has been updated with Lucene 9.7.0. This adapts the
according index compatibility test case to not rely on a specific Lucene
version.

See eclipse-platform/eclipse.platform.releng.aggregator#1233
Fixes #601
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants