Activity log for bug #1766998

Date Who What changed Old value New value Message
2018-04-25 23:18:55 Tiago Stürmer Daitx bug added bug
2018-04-26 01:33:16 Tiago Stürmer Daitx description I am formatting this as an SRU, but as there is no known regression I will not be subscribing the ubuntu-sru team. I will leave it up to the maintainer to decide if this is worth an SRU (maybe together with bug #1766995). It can also be reviewed later on when openjdk-11 is SRUed to Bionic. [Impact] The fix for bug #1756854 updated the -source and -target from 1.5 to 1.8 to allow the build to succeed with openjdk-9 as well as to keep compatibility with openjdk-11. That change missed the fact that the right way to set the target jdk level in openjdk-9 is to actually use the --release flag, introduced by JEP-247 [1]. By using the --release flag the right bootclasspath is also set, which prevents API compatibility problems when running the compiled jar/classes with older jdks (including the targeted one) [2,3]. Also, while JEP-182 [4] indicated that OpenJDK-11 would only allow a minimum source/target of 1.8, that later changed due to introduction of a 6-month release cycle, which moved the minimum source/target to 1.6 for openjdk-10 [5] and to 1.7 for openjdk-11 [6]. Thus the proposal is to move from "-source 1.8 -target 1.8" to "--release 7" in order to accommodate both changes. Currently ceph does not seem to make use any of the known APIs that can cause such failure when run with openjdk-8, but there is a risk of openjdk-11 introducing additional changes that could change that. [Test Case] * There is no known use case for it yet: of the known API incompatibilities none seems to be used by cephs-java, but that only goes as far as openjdk-9 and openjdk-10. [Regression Potential] * There is no known regression; these changes should actually improve ceph compatibility with openjdk-8 code by using --release to set the right bootclasspath. [Other Info] * openjdk-8 is being kept in the universe pocket for bionic and will be supported until xenial EOL (as it is in xenial's release pocket), thus it is important to guarantee that our packages can also be run and build with openjdk-8. [References] [1] http://openjdk.java.net/jeps/247 [2] https://github.com/plasma-umass/doppio/issues/497#issuecomment-353394008 [3] https://bugs.debian.org/895616 [4] http://openjdk.java.net/jeps/182 [5] https://bugs.openjdk.java.net/browse/JDK-8028563 [6] https://bugs.openjdk.java.net/browse/JDK-8173605 [Impact] The fix for bug #1756854 updated the -source and -target from 1.5 to 1.8 to allow the build to succeed with openjdk-9 as well as to keep compatibility with openjdk-11. That change missed the fact that the right way to set the target jdk level in openjdk-9 is to actually use the --release flag, introduced by JEP-247 [1]. By using the --release flag the right bootclasspath is also set, which prevents API compatibility problems when running the compiled jar/classes with older jdks (including the targeted one) [2,3]. Also, while JEP-182 [4] indicated that OpenJDK-11 would only allow a minimum source/target of 1.8, that later changed due to introduction of a 6-month release cycle, which moved the minimum source/target to 1.6 for openjdk-10 [5] and to 1.7 for openjdk-11 [6]. Thus the proposal is to move from "-source 1.8 -target 1.8" to "--release 7" in order to accommodate both changes. Currently ceph does not seem to make use any of the known APIs that can cause such failure when run with openjdk-8, but there is a risk of openjdk-11 introducing additional changes that could change that. [Test Case]  * There is no known use case for it yet: of the known API incompatibilities none seems to be used by cephs-java, but that only goes as far as openjdk-9 and openjdk-10. In order to check the binary compatibility of the generated jar and jni: $ apt-get install libcephfs-java ceph-test \ openjdk-11-jre-headless # or openjdk-8-jre-headless $ /usr/lib/jvm/java-8-openjdk-amd64/bin/java \ -DCEPH_CONF_FILE=/etc/ceph/ceph.conf \ -Djava.library.path=/usr/lib/jni/ \ -cp /usr/share/java/junit4.jar:/usr/share/java/libcephfs.jar:/usr/share/java/libcephfs-test.jar \ org.junit.runner.JUnitCore \ com.ceph.fs.CephAllTests It should report back: FAILURES!!! Tests run: 27, Failures: 4 The 4 failures are: 1) test_double_mount(com.ceph.fs.CephDoubleMountTest) java.lang.Exception: Unexpected exception, expected<com.ceph.fs.CephAlreadyMountedException> but was<java.io.FileNotFoundException> 2) test_CephMountCreate(com.ceph.fs.CephMountCreateTest) java.io.FileNotFoundException: 3) com.ceph.fs.CephMountTest java.io.FileNotFoundException: 4) com.ceph.fs.CephMountTest java.lang.NullPointerException: @path is null which is caused by the fact that ceph itself is not running nor mounted. [Regression Potential]  * There is no known regression; these changes should actually improve ceph compatibility with openjdk-8 code by using --release to set the right bootclasspath. [Other Info]  * openjdk-8 is being kept in the universe pocket for bionic and will be supported until xenial EOL (as it is in xenial's release pocket), thus it is important to guarantee that our packages can also be run and build with openjdk-8. [References] [1] http://openjdk.java.net/jeps/247 [2] https://github.com/plasma-umass/doppio/issues/497#issuecomment-353394008 [3] https://bugs.debian.org/895616 [4] http://openjdk.java.net/jeps/182 [5] https://bugs.openjdk.java.net/browse/JDK-8028563 [6] https://bugs.openjdk.java.net/browse/JDK-8173605
2018-04-26 01:33:53 Tiago Stürmer Daitx summary ceph: replace -source/-target 1.8 with --release 7 [[SRU] ceph: replace -source/-target 1.8 with --release 7
2018-04-26 01:37:46 Tiago Stürmer Daitx description [Impact] The fix for bug #1756854 updated the -source and -target from 1.5 to 1.8 to allow the build to succeed with openjdk-9 as well as to keep compatibility with openjdk-11. That change missed the fact that the right way to set the target jdk level in openjdk-9 is to actually use the --release flag, introduced by JEP-247 [1]. By using the --release flag the right bootclasspath is also set, which prevents API compatibility problems when running the compiled jar/classes with older jdks (including the targeted one) [2,3]. Also, while JEP-182 [4] indicated that OpenJDK-11 would only allow a minimum source/target of 1.8, that later changed due to introduction of a 6-month release cycle, which moved the minimum source/target to 1.6 for openjdk-10 [5] and to 1.7 for openjdk-11 [6]. Thus the proposal is to move from "-source 1.8 -target 1.8" to "--release 7" in order to accommodate both changes. Currently ceph does not seem to make use any of the known APIs that can cause such failure when run with openjdk-8, but there is a risk of openjdk-11 introducing additional changes that could change that. [Test Case]  * There is no known use case for it yet: of the known API incompatibilities none seems to be used by cephs-java, but that only goes as far as openjdk-9 and openjdk-10. In order to check the binary compatibility of the generated jar and jni: $ apt-get install libcephfs-java ceph-test \ openjdk-11-jre-headless # or openjdk-8-jre-headless $ /usr/lib/jvm/java-8-openjdk-amd64/bin/java \ -DCEPH_CONF_FILE=/etc/ceph/ceph.conf \ -Djava.library.path=/usr/lib/jni/ \ -cp /usr/share/java/junit4.jar:/usr/share/java/libcephfs.jar:/usr/share/java/libcephfs-test.jar \ org.junit.runner.JUnitCore \ com.ceph.fs.CephAllTests It should report back: FAILURES!!! Tests run: 27, Failures: 4 The 4 failures are: 1) test_double_mount(com.ceph.fs.CephDoubleMountTest) java.lang.Exception: Unexpected exception, expected<com.ceph.fs.CephAlreadyMountedException> but was<java.io.FileNotFoundException> 2) test_CephMountCreate(com.ceph.fs.CephMountCreateTest) java.io.FileNotFoundException: 3) com.ceph.fs.CephMountTest java.io.FileNotFoundException: 4) com.ceph.fs.CephMountTest java.lang.NullPointerException: @path is null which is caused by the fact that ceph itself is not running nor mounted. [Regression Potential]  * There is no known regression; these changes should actually improve ceph compatibility with openjdk-8 code by using --release to set the right bootclasspath. [Other Info]  * openjdk-8 is being kept in the universe pocket for bionic and will be supported until xenial EOL (as it is in xenial's release pocket), thus it is important to guarantee that our packages can also be run and build with openjdk-8. [References] [1] http://openjdk.java.net/jeps/247 [2] https://github.com/plasma-umass/doppio/issues/497#issuecomment-353394008 [3] https://bugs.debian.org/895616 [4] http://openjdk.java.net/jeps/182 [5] https://bugs.openjdk.java.net/browse/JDK-8028563 [6] https://bugs.openjdk.java.net/browse/JDK-8173605 [Impact] The fix for bug #1756854 updated the -source and -target from 1.5 to 1.8 to allow the build to succeed with openjdk-9 as well as to keep compatibility with openjdk-11. That change missed the fact that the right way to set the target jdk level in openjdk-9 is to actually use the --release flag, introduced by JEP-247 [1]. By using the --release flag the right bootclasspath is also set, which prevents API compatibility problems when running the compiled jar/classes with older jdks (including the targeted one) [2,3]. Also, while JEP-182 [4] indicated that OpenJDK-11 would only allow a minimum source/target of 1.8, that later changed due to introduction of a 6-month release cycle, which moved the minimum source/target to 1.6 for openjdk-10 [5] and to 1.7 for openjdk-11 [6]. Thus the proposal is to move from "-source 1.8 -target 1.8" to "--release 7" in order to accommodate both changes. Currently ceph does not seem to make use any of the known APIs that can cause such failure when run with openjdk-8, but there is a risk of openjdk-11 introducing additional changes that could change that. [Test Case]  * There is no known use case for it yet: of the known API incompatibilities none seems to be used by cephs-java, but that only goes as far as openjdk-9 and openjdk-10. In order to check the binary compatibility of the generated jar and jni: $ apt-get install libcephfs-java ceph-test \     openjdk-11-jre-headless # or openjdk-8-jre-headless # replace with java-8-openjdk-amd64 for openjdk-8 $ /usr/lib/jvm/java-11-openjdk-amd64/bin/java \     -DCEPH_CONF_FILE=/etc/ceph/ceph.conf \     -Djava.library.path=/usr/lib/jni/ \     -cp /usr/share/java/junit4.jar:/usr/share/java/libcephfs.jar:/usr/share/java/libcephfs-test.jar \     org.junit.runner.JUnitCore \     com.ceph.fs.CephAllTests It should report back: FAILURES!!! Tests run: 27, Failures: 4 The 4 failures are: 1) test_double_mount(com.ceph.fs.CephDoubleMountTest) java.lang.Exception: Unexpected exception, expected<com.ceph.fs.CephAlreadyMountedException> but was<java.io.FileNotFoundException> 2) test_CephMountCreate(com.ceph.fs.CephMountCreateTest) java.io.FileNotFoundException: 3) com.ceph.fs.CephMountTest java.io.FileNotFoundException: 4) com.ceph.fs.CephMountTest java.lang.NullPointerException: @path is null which is caused by the fact that ceph itself is not running nor mounted. [Regression Potential]  * There is no known regression; these changes should actually improve ceph compatibility with openjdk-8 code by using --release to set the right bootclasspath. [Other Info]  * openjdk-8 is being kept in the universe pocket for bionic and will be supported until xenial EOL (as it is in xenial's release pocket), thus it is important to guarantee that our packages can also be run and build with openjdk-8. [References] [1] http://openjdk.java.net/jeps/247 [2] https://github.com/plasma-umass/doppio/issues/497#issuecomment-353394008 [3] https://bugs.debian.org/895616 [4] http://openjdk.java.net/jeps/182 [5] https://bugs.openjdk.java.net/browse/JDK-8028563 [6] https://bugs.openjdk.java.net/browse/JDK-8173605
2018-04-26 01:37:59 Tiago Stürmer Daitx summary [[SRU] ceph: replace -source/-target 1.8 with --release 7 [SRU] ceph: replace -source/-target 1.8 with --release 7
2018-04-26 01:40:01 Tiago Stürmer Daitx description [Impact] The fix for bug #1756854 updated the -source and -target from 1.5 to 1.8 to allow the build to succeed with openjdk-9 as well as to keep compatibility with openjdk-11. That change missed the fact that the right way to set the target jdk level in openjdk-9 is to actually use the --release flag, introduced by JEP-247 [1]. By using the --release flag the right bootclasspath is also set, which prevents API compatibility problems when running the compiled jar/classes with older jdks (including the targeted one) [2,3]. Also, while JEP-182 [4] indicated that OpenJDK-11 would only allow a minimum source/target of 1.8, that later changed due to introduction of a 6-month release cycle, which moved the minimum source/target to 1.6 for openjdk-10 [5] and to 1.7 for openjdk-11 [6]. Thus the proposal is to move from "-source 1.8 -target 1.8" to "--release 7" in order to accommodate both changes. Currently ceph does not seem to make use any of the known APIs that can cause such failure when run with openjdk-8, but there is a risk of openjdk-11 introducing additional changes that could change that. [Test Case]  * There is no known use case for it yet: of the known API incompatibilities none seems to be used by cephs-java, but that only goes as far as openjdk-9 and openjdk-10. In order to check the binary compatibility of the generated jar and jni: $ apt-get install libcephfs-java ceph-test \     openjdk-11-jre-headless # or openjdk-8-jre-headless # replace with java-8-openjdk-amd64 for openjdk-8 $ /usr/lib/jvm/java-11-openjdk-amd64/bin/java \     -DCEPH_CONF_FILE=/etc/ceph/ceph.conf \     -Djava.library.path=/usr/lib/jni/ \     -cp /usr/share/java/junit4.jar:/usr/share/java/libcephfs.jar:/usr/share/java/libcephfs-test.jar \     org.junit.runner.JUnitCore \     com.ceph.fs.CephAllTests It should report back: FAILURES!!! Tests run: 27, Failures: 4 The 4 failures are: 1) test_double_mount(com.ceph.fs.CephDoubleMountTest) java.lang.Exception: Unexpected exception, expected<com.ceph.fs.CephAlreadyMountedException> but was<java.io.FileNotFoundException> 2) test_CephMountCreate(com.ceph.fs.CephMountCreateTest) java.io.FileNotFoundException: 3) com.ceph.fs.CephMountTest java.io.FileNotFoundException: 4) com.ceph.fs.CephMountTest java.lang.NullPointerException: @path is null which is caused by the fact that ceph itself is not running nor mounted. [Regression Potential]  * There is no known regression; these changes should actually improve ceph compatibility with openjdk-8 code by using --release to set the right bootclasspath. [Other Info]  * openjdk-8 is being kept in the universe pocket for bionic and will be supported until xenial EOL (as it is in xenial's release pocket), thus it is important to guarantee that our packages can also be run and build with openjdk-8. [References] [1] http://openjdk.java.net/jeps/247 [2] https://github.com/plasma-umass/doppio/issues/497#issuecomment-353394008 [3] https://bugs.debian.org/895616 [4] http://openjdk.java.net/jeps/182 [5] https://bugs.openjdk.java.net/browse/JDK-8028563 [6] https://bugs.openjdk.java.net/browse/JDK-8173605 [Impact] The fix for bug #1756854 updated the -source and -target from 1.5 to 1.8 to allow the build to succeed with openjdk-9 as well as to keep compatibility with openjdk-11. That change missed the fact that the right way to set the target jdk level in openjdk-9 is to actually use the --release flag, introduced by JEP-247 [1]. By using the --release flag the right bootclasspath is also set, which prevents API compatibility problems when running the compiled jar/classes with older jdks (including the targeted one) [2,3]. Also, while JEP-182 [4] indicated that OpenJDK-11 would only allow a minimum source/target of 1.8, that later changed due to introduction of a 6-month release cycle, which moved the minimum source/target to 1.6 for openjdk-10 [5] and to 1.7 for openjdk-11 [6]. Thus the proposal is to move from "-source 1.8 -target 1.8" to "--release 7" in order to accommodate both changes. Currently ceph does not seem to make use any of the known APIs that can cause such failure when run with openjdk-8, but there is a risk of openjdk-11 introducing additional changes that could change that. [Test Case]  * There is no known use case for it yet: of the known API incompatibilities none seems to be used by cephs-java, but that only goes as far as openjdk-9 and openjdk-10. In order to check the binary compatibility of the generated jar and jni: $ apt-get install libcephfs-java ceph-test \     openjdk-11-jre-headless # or openjdk-8-jre-headless # replace with java-8-openjdk-amd64 for openjdk-8 $ /usr/lib/jvm/java-11-openjdk-amd64/bin/java \     -DCEPH_CONF_FILE=/etc/ceph/ceph.conf \     -Djava.library.path=/usr/lib/jni/ \     -cp /usr/share/java/junit4.jar:/usr/share/java/libcephfs.jar:/usr/share/java/libcephfs-test.jar \     org.junit.runner.JUnitCore \     com.ceph.fs.CephAllTests It should report back: FAILURES!!! Tests run: 27, Failures: 4 The 4 failures are: 1) test_double_mount(com.ceph.fs.CephDoubleMountTest) java.lang.Exception: Unexpected exception, expected<com.ceph.fs.CephAlreadyMountedException> but was<java.io.FileNotFoundException> 2) test_CephMountCreate(com.ceph.fs.CephMountCreateTest) java.io.FileNotFoundException: 3) com.ceph.fs.CephMountTest java.io.FileNotFoundException: 4) com.ceph.fs.CephMountTest java.lang.NullPointerException: @path is null which is caused by the fact that ceph itself is not running nor mounted. The test case results should be the exact same with and without the patch applied, for both openjdk-8 and openjdk-11. This should indicate that the running tests are not being affected by the flag change. [Regression Potential]  * There is no known regression; these changes should actually improve ceph compatibility with openjdk-8 code by using --release to set the right bootclasspath. [Other Info]  * openjdk-8 is being kept in the universe pocket for bionic and will be supported until xenial EOL (as it is in xenial's release pocket), thus it is important to guarantee that our packages can also be run and build with openjdk-8. [References] [1] http://openjdk.java.net/jeps/247 [2] https://github.com/plasma-umass/doppio/issues/497#issuecomment-353394008 [3] https://bugs.debian.org/895616 [4] http://openjdk.java.net/jeps/182 [5] https://bugs.openjdk.java.net/browse/JDK-8028563 [6] https://bugs.openjdk.java.net/browse/JDK-8173605
2018-05-07 14:33:17 Łukasz Zemczak ceph (Ubuntu Bionic): status New Fix Committed
2018-05-07 14:33:19 Łukasz Zemczak bug added subscriber Ubuntu Stable Release Updates Team
2018-05-07 14:33:21 Łukasz Zemczak bug added subscriber SRU Verification
2018-05-07 14:33:26 Łukasz Zemczak tags verification-needed verification-needed-bionic
2018-05-15 20:03:15 Launchpad Janitor ceph (Ubuntu): status New Fix Released
2018-05-24 15:19:48 Tiago Stürmer Daitx tags verification-needed verification-needed-bionic verification-done-bionic
2018-05-24 17:30:58 Launchpad Janitor ceph (Ubuntu Bionic): status Fix Committed Fix Released
2018-05-24 17:31:18 Brian Murray removed subscriber Ubuntu Stable Release Updates Team