HADOOP-17756. Increase precommit job timeout from 20 hours to 24 hours. #3090

tasanuma · 2021-06-10T05:34:34Z

hadoop-yetus · 2021-06-10T06:42:43Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	20m 25s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	shelldocs	0m 0s		Shelldocs was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
			_ trunk Compile Tests _
+0 🆗	mvndep	12m 27s		Maven dependency ordering for branch
+1 💚	shadedclient	15m 55s		branch has no errors when building and testing our client artifacts.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 22s		Maven dependency ordering for patch
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	shadedclient	15m 31s		patch has no errors when building and testing our client artifacts.
			_ Other Tests _
+1 💚	asflicense	0m 28s		The patch does not generate ASF License warnings.
		66m 57s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/1/artifact/out/Dockerfile
GITHUB PR	#3090
Optional Tests	dupname asflicense codespell shellcheck shelldocs
uname	Linux c67562f5c308 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/bin/hadoop.sh
git revision	trunk / `7fe02c2`
Max. process+thread count	594 (vs. ulimit of 5500)
modules	C: U:
Console output	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/1/console
versions	git=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered by	Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

virajjasani

+1 (non-binding). Just a thought, before committing, do we want to update just one line in hadoop-main pom.xml to trigger full run? Based on that, we might come to know of new timeout value if 24 hr is also not sufficient.

tasanuma · 2021-06-10T06:57:36Z

@virajjasani Thanks for your review. Agreed with you. Updated the PR.

hadoop-yetus · 2021-06-11T04:10:58Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 51s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	shelldocs	0m 0s		Shelldocs was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
-1 ❌	test4tests	0m 0s		The 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.
			_ trunk Compile Tests _
-1 ❌	mvninstall	39m 18s	/branch-mvninstall-root.txt	root in trunk failed.
+1 💚	compile	22m 46s		trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	compile	19m 10s		trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	mvnsite	25m 3s		trunk passed
+1 💚	javadoc	7m 47s		trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	javadoc	7m 37s		trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	shadedclient	31m 15s		branch has no errors when building and testing our client artifacts.
			_ Patch Compile Tests _
+1 💚	mvninstall	22m 14s		the patch passed
+1 💚	compile	21m 57s		the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	javac	21m 57s		the patch passed
+1 💚	compile	19m 5s		the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	javac	19m 5s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	mvnsite	20m 46s		the patch passed
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	xml	0m 1s		The patch has no ill-formed XML file.
+1 💚	javadoc	7m 41s		the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	javadoc	7m 31s		the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	shadedclient	32m 13s		patch has no errors when building and testing our client artifacts.
			_ Other Tests _
-1 ❌	unit	1014m 19s	/patch-unit-root.txt	root in the patch passed.
+1 💚	asflicense	1m 28s		The patch does not generate ASF License warnings.
		1273m 17s

Reason	Tests
Failed junit tests	hadoop.yarn.server.router.clientrm.TestFederationClientInterceptor
	hadoop.hdfs.server.namenode.ha.TestBootstrapStandby
	hadoop.hdfs.server.namenode.ha.TestEditLogTailer
	hadoop.hdfs.TestDFSShell
	hadoop.hdfs.server.datanode.fsdataset.impl.TestFsVolumeList
	hadoop.hdfs.server.namenode.TestDecommissioningStatusWithBackoffMonitor
	hadoop.hdfs.server.namenode.TestDecommissioningStatus

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/2/artifact/out/Dockerfile
GITHUB PR	#3090
Optional Tests	dupname asflicense codespell shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xml
uname	Linux 97896f3608ce 4.15.0-136-generic #140-Ubuntu SMP Thu Jan 28 05:20:47 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/bin/hadoop.sh
git revision	trunk / `87a7090`
Default Java	Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
Multi-JDK versions	/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
Test Results	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/2/testReport/
Max. process+thread count	2193 (vs. ulimit of 5500)
modules	C: . U: .
Console output	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/2/console
versions	git=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered by	Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

This reverts commit 87a7090.

tasanuma · 2021-06-11T04:38:44Z

The last QA time is 21 hours. I think 24 hours for timeout is appropriate.

virajjasani · 2021-06-11T05:16:58Z

Sounds great, 24 hr should be good enough.

aajisaka

+1

hadoop-yetus · 2021-06-11T22:07:53Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 38s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 1s		codespell was not available.
+0 🆗	shelldocs	0m 1s		Shelldocs was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
-1 ❌	test4tests	0m 0s		The 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.
			_ trunk Compile Tests _
+1 💚	mvninstall	32m 32s		trunk passed
+1 💚	compile	23m 17s		trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	compile	19m 42s		trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	mvnsite	25m 51s		trunk passed
+1 💚	javadoc	7m 33s		trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	javadoc	7m 57s		trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	shadedclient	29m 20s		branch has no errors when building and testing our client artifacts.
			_ Patch Compile Tests _
+1 💚	mvninstall	22m 13s		the patch passed
+1 💚	compile	24m 26s		the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	javac	24m 26s		the patch passed
+1 💚	compile	20m 35s		the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	javac	20m 35s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	mvnsite	22m 48s		the patch passed
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	xml	0m 1s		The patch has no ill-formed XML file.
+1 💚	javadoc	7m 48s		the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04
+1 💚	javadoc	8m 21s		the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
+1 💚	shadedclient	32m 1s		patch has no errors when building and testing our client artifacts.
			_ Other Tests _
+1 💚	unit	792m 36s		root in the patch passed.
+1 💚	asflicense	1m 42s		The patch does not generate ASF License warnings.
		1050m 25s

Subsystem	Report/Notes
Docker	ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/3/artifact/out/Dockerfile
GITHUB PR	#3090
Optional Tests	dupname asflicense codespell shellcheck shelldocs compile javac javadoc mvninstall mvnsite unit shadedclient xml
uname	Linux 3678629e8dd0 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/bin/hadoop.sh
git revision	trunk / `346f63d`
Default Java	Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
Multi-JDK versions	/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
Test Results	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/3/testReport/
Max. process+thread count	3183 (vs. ulimit of 5500)
modules	C: . U: .
Console output	https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3090/3/console
versions	git=2.25.1 maven=3.6.3 shellcheck=0.7.0
Powered by	Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

tasanuma · 2021-06-14T01:18:03Z

Thanks for your reviews, @aajisaka and @virajjasani.

…s. (#3090) Reviewed-by: Akira Ajisaka <aajisaka@apache.org> Reviewed-by: Viraj Jasani <vjasani@apache.org> (cherry picked from commit ae9630f)

…s. (apache#3090) Reviewed-by: Akira Ajisaka <aajisaka@apache.org> Reviewed-by: Viraj Jasani <vjasani@apache.org>

HADOOP-17756. Increase precommit job timeout from 20 hours to 24 hours.

7fe02c2

virajjasani approved these changes Jun 10, 2021

View reviewed changes

modify pom.xml to have QA run for the whole project

87a7090

Revert the last commit

346f63d

This reverts commit 87a7090.

tasanuma requested a review from aajisaka June 11, 2021 04:39

aajisaka approved these changes Jun 11, 2021

View reviewed changes

tasanuma merged commit ae9630f into apache:trunk Jun 14, 2021

tasanuma deleted the HADOOP-17756 branch June 14, 2021 01:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

HADOOP-17756. Increase precommit job timeout from 20 hours to 24 hours. #3090

HADOOP-17756. Increase precommit job timeout from 20 hours to 24 hours. #3090

tasanuma commented Jun 10, 2021

hadoop-yetus commented Jun 10, 2021

virajjasani left a comment •

edited

Loading

tasanuma commented Jun 10, 2021

hadoop-yetus commented Jun 11, 2021

tasanuma commented Jun 11, 2021

virajjasani commented Jun 11, 2021

aajisaka left a comment

hadoop-yetus commented Jun 11, 2021

tasanuma commented Jun 14, 2021

HADOOP-17756. Increase precommit job timeout from 20 hours to 24 hours. #3090

HADOOP-17756. Increase precommit job timeout from 20 hours to 24 hours. #3090

Conversation

tasanuma commented Jun 10, 2021

hadoop-yetus commented Jun 10, 2021

virajjasani left a comment • edited Loading

Choose a reason for hiding this comment

tasanuma commented Jun 10, 2021

hadoop-yetus commented Jun 11, 2021

tasanuma commented Jun 11, 2021

virajjasani commented Jun 11, 2021

aajisaka left a comment

Choose a reason for hiding this comment

hadoop-yetus commented Jun 11, 2021

tasanuma commented Jun 14, 2021

virajjasani left a comment •

edited

Loading