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

Add derby tests to the set of External Tests running at Adopt #400

Merged
merged 1 commit into from
May 2, 2018

Conversation

Mesbah-Alam
Copy link
Contributor

@Mesbah-Alam Mesbah-Alam commented Apr 26, 2018

Add derby tests to the set of External Tests running at Adopt
Signed-off-by: Mesbah Alam Mesbah_Alam@ca.ibm.com

@Mesbah-Alam Mesbah-Alam self-assigned this Apr 26, 2018
@Mesbah-Alam
Copy link
Contributor Author

Related issue: #172

@Mesbah-Alam Mesbah-Alam force-pushed the addDerbyTests branch 2 times, most recently from ab79451 to 9f086b1 Compare April 27, 2018 15:30
@Mesbah-Alam
Copy link
Contributor Author

Mesbah-Alam commented Apr 27, 2018

We can not use derby from its master branch (trunk) as it requires Java 9 or above to build :
/https://urv-jenkins.canlab.ibm.com:9095/view/SV_Tests/job/test.personal/105/console

17:00:33 /derby/build.xml:182: Compiler level must be Java 9 or later.

We need to checkout from a branch that supports Java 8 and above, which is 10.14-- as stated in the downloads page.

New personal build submitted with derby 10.14 : https://urv-jenkins.canlab.ibm.com:9095/view/SV_Tests/job/test.personal/106/console

<command>docker run --rm -v $(JDK_HOME):/java adoptopenjdk-derby-test:latest ; \
$(TEST_STATUS)</command>
<subsets>
<subset>SE80</subset>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you drop this subset, then do you not get past the "needs Java9+" error msg?

Copy link
Contributor Author

@Mesbah-Alam Mesbah-Alam Apr 30, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @ShelleyLambert , yes we would, but I was under the impression that we want to test Java 8 with the external tests, due to the fact that we have only Java 8 builds for external tests at Adopt (openjdk8_hs* and openjdk8_j9*). Please correct me if I am wrong.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As per our slack conversation just now, we can add openjdk9* and openjdk10* builds at any time, I just had not gotten to it yet :)

So for now, you could pull material from HEAD for Derby, and run them against Java9 SDKs

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I am on it. Thanks for clarifying.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated Derby test scripts to pull Derby from HEAD. Tested here: https://urv-jenkins.canlab.ibm.com:9095/view/SV_Tests/job/test.personal/135/console

@Mesbah-Alam Mesbah-Alam force-pushed the addDerbyTests branch 8 times, most recently from 59d710c to 126ff05 Compare May 2, 2018 15:00
@Mesbah-Alam Mesbah-Alam removed the request for review from mamatha-jv May 2, 2018 16:59
Signed-off-by: Mesbah Alam <Mesbah_Alam@ca.ibm.com>
@Mesbah-Alam Mesbah-Alam merged commit 65628cf into adoptium:master May 2, 2018
@Mesbah-Alam Mesbah-Alam deleted the addDerbyTests branch May 29, 2018 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants