Skip to content
This repository was archived by the owner on Dec 12, 2024. It is now read-only.

Conversation

@gregw
Copy link
Contributor

@gregw gregw commented Apr 30, 2019

Signed-off-by: Greg Wilkins gregw@webtide.com

Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw gregw requested a review from donmccasland April 30, 2019 11:13
@gregw
Copy link
Contributor Author

gregw commented Apr 30, 2019

Ideally this would be built after GoogleCloudPlatform/openjdk-runtime#196 has been merged, built and released, as without that this runtime is failing on machines with 32GB of memory.
There is also an instability in the smoke tests, so I've not been able to get a clean build locally (on a new dev machine). However, I get the same failures for the previous release, so it is not this upgrade.

gregw added 3 commits May 1, 2019 09:52
Signed-off-by: Greg Wilkins <gregw@webtide.com>
Signed-off-by: Greg Wilkins <gregw@webtide.com>
@gregw
Copy link
Contributor Author

gregw commented May 1, 2019

@donmccasland the smoke tests are now passing for me locally. I cleaned up some of the test classes to give better output, but the main change was to set the heap size so the test images actually start.

edit: still failing on CI... trying a smaller heap size... but GoogleCloudPlatform/openjdk-runtime#196 is a better fix.

Signed-off-by: Greg Wilkins <gregw@webtide.com>
@donmccasland donmccasland merged commit f268574 into master May 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants