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

Dragonwell upgrades to jdk8u212-b04 #33

Closed
yfxhust opened this issue Jun 21, 2019 · 5 comments
Closed

Dragonwell upgrades to jdk8u212-b04 #33

yfxhust opened this issue Jun 21, 2019 · 5 comments
Assignees

Comments

@yfxhust
Copy link
Contributor

yfxhust commented Jun 21, 2019

Dragonwell should follow jdk8u upstream and upgrade to jdk8u212-b04. The current dragonewll state is at jdk8u202-b01 that is out of date.

@yfxhust yfxhust changed the title Dragonwell upgrade to jdk8u212-b04 Dragonwell upgrades to jdk8u212-b04 Jun 21, 2019
@yfxhust yfxhust self-assigned this Jun 21, 2019
@yfxhust
Copy link
Contributor Author

yfxhust commented Jun 21, 2019

The 212u-b04 rebase snapshot test result is as following:

http://ci.jvm.alibaba.net/view/dragonwell8/job/dragonwell.8.test.jtreg.langtools/35/console
http://ci.jvm.alibaba.net/view/dragonwell8/job/dragonwell.8.test.jtreg.hotspot/36/
http://ci.jvm.alibaba.net/view/dragonwell8/job/dragonwell.8.test.jtreg.jdk/36/console

These tests run with my manual built jdk. But the test code is still at 202u.
I hope most of tests can be passed with my manual built jdk and then I push them to branch to verify the final test.

@yfxhust
Copy link
Contributor Author

yfxhust commented Jun 21, 2019

conflict_hs_03084c7aa.diff.log
conflict_jdk_960e45a2e.diff.log
The conflict diff during rebasing upstream is as above.
The conflict resolving is simple and straightforward. The resolved diff is as following.
conflict_jdk_960e45a2e_resloved.diff.log
conflict_hs_03084c7aa_resloved.diff.log

@yfxhust
Copy link
Contributor Author

yfxhust commented Jun 21, 2019

http://ci.jvm.alibaba.net/view/dragonwell8/job/dragonwell.8.test.jtreg.hotspot/36/ failed with the following 5 tests:
gc/arguments/TestCMSHeapSizeFlags.java
gc/arguments/TestG1HeapSizeFlags.java
gc/arguments/TestMinInitialErgonomics.java
gc/arguments/TestParallelHeapSizeFlags.java
gc/arguments/TestSerialHeapSizeFlags.java

Please ignore them. Because the 5 tests failed by stale test code and run with my manual snapshot test. 8u212-b04 has change for these test case code. If I push the 212 code , the tests will success.

@yfxhust
Copy link
Contributor Author

yfxhust commented Jun 22, 2019

http://ci.jvm.alibaba.net/view/dragonwell8/job/dragonwell.8.test.jtreg.jdk/36/console failed several tests. These tests failed because 212u upstream updated test case code.
The ci test still run with 202u test code.
Just ignore them. They will be fixed when I push them to code base and re-run test.

@yfxhust
Copy link
Contributor Author

yfxhust commented Jun 25, 2019

Already push to repo. Close it.

@yfxhust yfxhust closed this as completed Jun 25, 2019
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

No branches or pull requests

1 participant