Skip to content

TEZ-4560. Upgrade bouncycastle to 1.77 due to CVE. - #351

Merged
abstractdog merged 1 commit into
apache:masterfrom
slfan1989:TEZ-4560
May 6, 2024
Merged

TEZ-4560. Upgrade bouncycastle to 1.77 due to CVE.#351
abstractdog merged 1 commit into
apache:masterfrom
slfan1989:TEZ-4560

Conversation

@slfan1989

Copy link
Copy Markdown
Contributor

JIRA: TEZ-4560. Upgrade bouncycastle to 1.77 due to CVE.

There are 2 CVE issues in bcprov-jdk15on, CVE-2023-33202 and CVE-2023-33201. We can find more information at the following link:

https://mvnrepository.com/artifact/org.bouncycastle/bcprov-jdk15on/1.70

The link to the CVE is as follows:

CVE-2023-33202
CVE-2023-33201

We can upgrade bcprov-jdk15on to bcprov-jdk18on to address the CVE issues.

@tez-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec12m 23sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
-1 ❌test4tests0m 0sThe patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ master Compile Tests _
+0 🆗mvndep6m 50sMaven dependency ordering for branch
+1 💚mvninstall9m 20smaster passed
+1 💚compile1m 59smaster passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu222.04.1
+1 💚compile1m 58smaster passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~22.04-b06
+1 💚javadoc1m 40smaster passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu222.04.1
+1 💚javadoc1m 25smaster passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~22.04-b06
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall2m 53sthe patch passed
+1 💚compile2m 3sthe patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu222.04.1
+1 💚javac2m 3sthe patch passed
+1 💚compile1m 52sthe patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~22.04-b06
+1 💚javac1m 52sthe patch passed
+1 💚whitespace0m 0sThe patch has no whitespace issues.
+1 💚xml0m 3sThe patch has no ill-formed XML file.
+1 💚javadoc1m 26sthe patch passed with JDK Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu222.04.1
+1 💚javadoc1m 25sthe patch passed with JDK Private Build-1.8.0_402-8u402-ga-2ubuntu1~22.04-b06
_ Other Tests _
+1 💚unit2m 5stez-api in the patch passed.
-1 ❌unit27m 57stez-tests in the patch failed.
-1 ❌unit41m 24sroot in the patch failed.
+1 💚asflicense1m 3sThe patch does not generate ASF License warnings.
119m 3s
ReasonTests
Failed junit teststez.test.TestAMRecovery
tez.test.TestRecovery
tez.test.TestDAGRecovery
tez.test.TestAMRecovery
tez.test.TestRecovery
tez.test.TestDAGRecovery
SubsystemReport/Notes
DockerClientAPI=1.45 ServerAPI=1.45 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-351/1/artifact/out/Dockerfile
GITHUB PR#351
Optional Testsdupname asflicense javac javadoc unit xml compile
unameLinux f3b56d90114b 5.15.0-94-generic #104-Ubuntu SMP Tue Jan 9 15:25:40 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 906059a
Default JavaPrivate Build-1.8.0_402-8u402-ga-2ubuntu1~22.04-b06
Multi-JDK versions/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.22+7-post-Ubuntu-0ubuntu222.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_402-8u402-ga-2ubuntu1~22.04-b06
unithttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-351/1/artifact/out/patch-unit-tez-tests.txt
unithttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-351/1/artifact/out/patch-unit-root.txt
Test Resultshttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-351/1/testReport/
Max. process+thread count1527 (vs. ulimit of 5500)
modulesC: tez-api tez-tests . U: .
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-351/1/console
versionsgit=2.34.1 maven=3.6.3
Powered byApache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@abstractdog

Copy link
Copy Markdown
Contributor

failing unit tests are because of TEZ-4559, this looks good to me

@abstractdog
abstractdog self-requested a review May 6, 2024 07:49
@abstractdog
abstractdog merged commit 071d447 into apache:masterMay 6, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@slfan1989@tez-yetus@abstractdog