RFR: JDK-8215544: SA: Modify ClhsdbLauncher to add sudo privileges to enable MacOS tests on Mach5

Jini George jini.george at oracle.com
Mon Jan 7 16:30:25 UTC 2019


Gentle reminder !

Thanks,
Jini.

On 1/3/2019 11:46 PM, Jini George wrote:
> Hello!
> 
> Requesting reviews for:
> 
> https://bugs.openjdk.java.net/browse/JDK-8215544
> Webrev: http://cr.openjdk.java.net/~jgeorge/8215544/webrev.03/index.html
> 
> The changes here are primarily for modifying the ClhsdbLauncher used for 
> SA tests to check if 'sudo' privileges can be added for executing MacOS 
> tests, and if so, adding these privileges before executing the tests. 
> This is related to the changes which were proposed with:
> 
> http://mail.openjdk.java.net/pipermail/serviceability-dev/2018-July/024373.html 
> 
> 
> for (JDK-8199700: SA: Enable jhsdb jtreg tests for Mac OS X).
> 
> The issue brought out with the proposed changes for JDK-8199700 have 
> been fixed. This patch addresses the case where 'sudo' fails with errors 
> like:
> 
> sudo: no tty present and no askpass program specified
>   or
> sudo: a password is required
> 
> or if 'sudo' waits for a password. In these cases, the tests continue to 
> get skipped.
> 
> Many thanks to Goetz Lindenmaier for helping me with multiple rounds of 
> testing of this in the SAP test farms! With this patch, about 22 SA 
> tests get enabled now on MacOS on Mach5.
> 
> More details at: https://bugs.openjdk.java.net/browse/JDK-8215544
> 
> Thanks,
> Jini.


More information about the serviceability-dev mailing list