search

GoogleCloudPlatform / jetty-runtime

Google Cloud Platform Jetty Docker image
Apache License 2.0
44 stars 33 forks source link

upgrade to jetty-9.4.18 #258

Closed gregw closed 5 years ago

gregw commented 5 years ago

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

gregw commented 5 years ago

Ideally this would be built after https://github.com/GoogleCloudPlatform/openjdk-runtime/pull/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 commented 5 years ago

@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 https://github.com/GoogleCloudPlatform/openjdk-runtime/pull/196 is a better fix.