Changes to utilize default +UseContainerSupport in new releases of Java

[imranzunzani]

Updates in run-java.sh to use latest VM provisions and switches for running Java in containers.
-XX:InitialRAMPercentage
-XX:MaxRAMPercentage
-XX:MinRAMPercentage

[imranzunzani]

@rhuss,
Could you please review?
These are changes to run-java.sh as per new features in the JDK.
We discussed about these over #46

[rhuss]

Oops. Sorry for the delay, the notification slipped through somehow ;-(

You must not change run-java.sh directly, but send a PR to https://github.com/fabric8io-images/run-java-sh The files here a auto-generated with a framework called fish-pepper.

So the steps would be:

• Send a PR against --> https://github.com/fabric8io-images/run-java-sh/blob/master/fish-pepper/run-java-sh/fp-files/run-java.sh
• We release run-java.sh
• Update the dependency here to this new version
• Run fish-pepper (which will fan out to all the images)
• Commit and create a PR here.

Sorry for the delayed response ...

[imranzunzani]

Hi @rhuss,
I created the PR but the vm_test(s) are failing owing to assertions for +UseParallelGC which is not required with the new switches and the tests are also not recognising the new options.
Could you please take a look and let me know what should be changed regarding these old assertions?

[rhuss]

Hi @rhuss,
I created the PR but the vm_test(s) are failing owing to assertions for +UseParallelGC which is not required with the new switches and the tests are also not recognising the new options.
Could you please take a look and let me know what should be changed regarding these old assertions?

Yes, I can do but I can't promise a timeframe as I have currently quite a bit on my table (sorry). Please ping me again, when I should come back to you within the next weeks.

However, maybe @astefanutti could jump in ?

[imranzunzani]

@rhuss,
It took me some time and digging to figure out the test logic and relations among the artefacts for CI (Tests).
I have made changes, so that the tests are now in sync with the new features and switches of the JDK.
Could you please help by reviewing the PR or by asking someone to review it, and then merge if things look good?
By the way, the new logic will work only with version > 8u212 which means that JDK 7 won't work, but since the Tests were for JDK 8 and JDK 11, I believe it is alright.

[imranzunzani]

@rhuss,
Just reminding you.
Please take a look.

[Paramchoudhary]

Good job