TEZ-4476: Move tez-ui testing from PhantomJS to Headless Chrome by maheshrajus · Pull Request #514 · apache/tez

maheshrajus · 2026-06-29T12:21:53Z

PhantomJS is no longer maintained and has known incompatibilities with
modern OpenSSL (3+).
Replace it with Headless Chrome via testem's built-in browser_args support.

tez-yetus · 2026-06-29T14:04:35Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	4m 0s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	xmllint	0m 0s		xmllint was not available.
+0 🆗	jsonlint	0m 0s		jsonlint 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.
			_ master Compile Tests _
+1 💚	mvninstall	4m 47s		master passed
+1 💚	compile	3m 59s		master passed
+1 💚	javadoc	0m 28s		master passed
			_ Patch Compile Tests _
+1 💚	mvninstall	3m 51s		the patch passed
-1 ❌	codespell	1m 36s	/results-codespell.txt	The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)
+1 💚	compile	3m 58s		the patch passed
+1 💚	javac	3m 58s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	javadoc	0m 25s		the patch passed
			_ Other Tests _
-1 ❌	unit	77m 0s	/patch-unit-root.txt	root in the patch failed.
+1 💚	asflicense	0m 38s		The patch does not generate ASF License warnings.
		102m 9s

Reason	Tests
Failed junit tests	tez.history.TestHistoryParser

Subsystem	Report/Notes
Docker	ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/1/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense javac javadoc codespell detsecrets xmllint jsonlint
uname	Linux cbe42f53ed77 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / `17d1549`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/1/testReport/
Max. process+thread count	1437 (vs. ulimit of 5500)
modules	C: tez-ui U: tez-ui
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/1/console
versions	git=2.43.0 maven=3.9.15 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-06-29T17:37:59Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 11s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 1s		No case conflicting files found.
+0 🆗	detsecrets	0m 1s		detect-secrets was not available.
+0 🆗	xmllint	0m 0s		xmllint was not available.
+0 🆗	jsonlint	0m 0s		jsonlint 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.
			_ master Compile Tests _
+0 🆗	mvndep	0m 36s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 33s		master passed
+1 💚	compile	3m 58s		master passed
+1 💚	javadoc	1m 55s		master passed
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 9s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 40s		the patch passed
+1 💚	codespell	1m 35s		No new issues.
+1 💚	compile	3m 58s		the patch passed
+1 💚	javac	3m 58s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	javadoc	1m 49s		the patch passed
			_ Other Tests _
-1 ❌	unit	73m 39s	/patch-unit-root.txt	root in the patch failed.
-1 ❌	asflicense	1m 30s	/results-asflicense.txt	The patch generated 1 ASF License warnings.
		99m 4s

Subsystem	Report/Notes
Docker	ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/2/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense codespell detsecrets javac javadoc xmllint jsonlint
uname	Linux a3d388f95334 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / `17d1549`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/2/testReport/
Max. process+thread count	1524 (vs. ulimit of 5500)
modules	C: tez-ui . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/2/console
versions	git=2.43.0 maven=3.9.15 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-06-30T04:14:30Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 11s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	xmllint	0m 0s		xmllint was not available.
+0 🆗	jsonlint	0m 0s		jsonlint 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.
			_ master Compile Tests _
+0 🆗	mvndep	0m 34s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 36s		master passed
+1 💚	compile	3m 56s		master passed
+1 💚	javadoc	1m 58s		master passed
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 8s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 51s		the patch passed
+1 💚	codespell	1m 36s		No new issues.
+1 💚	compile	3m 58s		the patch passed
+1 💚	javac	3m 58s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	javadoc	1m 52s		the patch passed
			_ Other Tests _
-1 ❌	unit	72m 49s	/patch-unit-root.txt	root in the patch failed.
+1 💚	asflicense	1m 30s		The patch does not generate ASF License warnings.
		98m 30s

Subsystem	Report/Notes
Docker	ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/3/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense codespell detsecrets javac javadoc xmllint jsonlint
uname	Linux e07e9b03e809 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / `17d1549`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/3/testReport/
Max. process+thread count	1542 (vs. ulimit of 5500)
modules	C: tez-ui . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/3/console
versions	git=2.43.0 maven=3.9.15 codespell=2.4.1
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-06-30T07:10:06Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/4/console in case of problems.

tez-yetus · 2026-06-30T08:53:32Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	1m 29s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	shelldocs	0m 0s		Shelldocs was not available.
+0 🆗	xmllint	0m 0s		xmllint was not available.
+0 🆗	jsonlint	0m 0s		jsonlint 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.
			_ master Compile Tests _
+0 🆗	mvndep	0m 34s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 35s		master passed
+1 💚	compile	4m 0s		master passed
+1 💚	javadoc	1m 55s		master passed
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 9s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 49s		the patch passed
+1 💚	codespell	1m 35s		No new issues.
+1 💚	compile	3m 58s		the patch passed
+1 💚	javac	3m 58s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	hadolint	0m 2s		No new issues.
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	javadoc	1m 54s		the patch passed
			_ Other Tests _
+1 💚	unit	76m 26s		root in the patch passed.
+1 💚	asflicense	1m 30s		The patch does not generate ASF License warnings.
		103m 30s

Subsystem	Report/Notes
Docker	ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/4/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense codespell detsecrets hadolint shellcheck shelldocs javac javadoc xmllint jsonlint
uname	Linux 016c58a2ad3d 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / `17d1549`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/4/testReport/
Max. process+thread count	2069 (vs. ulimit of 5500)
modules	C: tez-ui . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/4/console
versions	git=2.43.0 maven=3.9.15 hadolint=2.14.0 codespell=2.4.1 shellcheck=0.11.0
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

maheshrajus · 2026-06-30T10:35:41Z

@abstractdog @ayushtkn
Could you please review the PR at your convenience?
Thank you !

abstractdog · 2026-06-30T12:44:34Z

thanks for this patch @maheshrajus, a few notes:

after removing phantomjs, we can try to build tez-ui also on arm64:

tez/.github/workflows/build.yml

Line 53 in 17d1549

&& '-pl !tez-ui' || '' }}
checked out pr, ran mvn clean install in tez-ui and I saw a browser window running the tests:

while it was awesome to see, I'm expecting a headless browser to not display at all, does it make sense?

after install, I got this diff:

-"moment@>= 2.29.4":
-  version "2.29.4"
-  resolved "https://registry.yarnpkg.com/moment/-/moment-2.29.4.tgz#3dbe052889fe7c1b2ed966fcb3a77328964ef108"
+"moment@>= 2.6.0":
+  version "2.30.1"
+  resolved "https://registry.yarnpkg.com/moment/-/moment-2.30.1.tgz#f8c91c07b7a786e30c59926df530b4eac96974ae"

maheshrajus · 2026-06-30T12:49:16Z

@abstractdog Thank you for the review.
Let me check and address the review comments.

maheshrajus · 2026-06-30T13:11:45Z

2. checked out pr, ran mvn clean install in tez-ui and I saw a browser window running the tests:

@abstractdog Initially the changes related to headless present in my first commit. After that while fixing ui-tests it got overwritten. Let me fix and run locally and verify it. thanks !

tez-yetus · 2026-06-30T17:12:33Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/5/console in case of problems.

tez-yetus · 2026-07-01T03:35:25Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/6/console in case of problems.

tez-yetus · 2026-07-01T09:06:08Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/7/console in case of problems.

tez-yetus · 2026-07-01T17:08:53Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/8/console in case of problems.

tez-yetus · 2026-07-01T20:36:10Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 13s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	shelldocs	0m 0s		Shelldocs was not available.
+0 🆗	xmllint	0m 0s		xmllint was not available.
+0 🆗	jsonlint	0m 0s		jsonlint was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 2 new or modified test files.
			_ master Compile Tests _
+0 🆗	mvndep	0m 34s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 37s		master passed
+1 💚	compile	4m 1s		master passed
+1 💚	javadoc	1m 56s		master passed
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 9s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 53s		the patch passed
+1 💚	codespell	1m 46s		No new issues.
+1 💚	jshint	109m 35s		No new issues.
+1 💚	compile	3m 57s		the patch passed
+1 💚	javac	3m 57s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	hadolint	0m 2s		No new issues.
-1 ❌	markdownlint	0m 3s	/results-markdownlint.txt	The patch generated 11 new + 51 unchanged - 0 fixed = 62 total (was 51)
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	javadoc	1m 52s		the patch passed
			_ Other Tests _
-1 ❌	unit	71m 41s	/patch-unit-root.txt	root in the patch failed.
+1 💚	asflicense	1m 29s		The patch does not generate ASF License warnings.
		207m 18s

Reason	Tests
Failed junit tests	tez.history.TestHistoryParser

Subsystem	Report/Notes
Docker	ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/8/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense codespell detsecrets hadolint shellcheck shelldocs javac javadoc xmllint markdownlint jsonlint jshint
uname	Linux 82067fc2e13b 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / `8fde353`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/8/testReport/
Max. process+thread count	1479 (vs. ulimit of 5500)
modules	C: tez-ui . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/8/console
versions	git=2.43.0 maven=3.9.15 hadolint=2.14.0 codespell=2.4.1 jshint=2.13.6 markdownlint=0.46.0 shellcheck=0.11.0
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-07-02T06:01:51Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/9/console in case of problems.

tez-yetus · 2026-07-02T09:32:49Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 14s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+0 🆗	shelldocs	0m 0s		Shelldocs was not available.
+0 🆗	xmllint	0m 1s		xmllint was not available.
+0 🆗	jsonlint	0m 1s		jsonlint was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 2 new or modified test files.
			_ master Compile Tests _
+0 🆗	mvndep	0m 39s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 39s		master passed
+1 💚	compile	3m 57s		master passed
+1 💚	javadoc	1m 57s		master passed
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 9s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 46s		the patch passed
+1 💚	codespell	1m 35s		No new issues.
+1 💚	jshint	109m 16s		No new issues.
+1 💚	compile	3m 57s		the patch passed
+1 💚	javac	3m 57s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	hadolint	0m 1s		No new issues.
+1 💚	markdownlint	0m 3s		No new issues.
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	javadoc	1m 54s		the patch passed
			_ Other Tests _
+1 💚	unit	75m 54s		root in the patch passed.
+1 💚	asflicense	1m 27s		The patch does not generate ASF License warnings.
		211m 2s

Subsystem	Report/Notes
Docker	ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/9/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense codespell detsecrets hadolint shellcheck shelldocs javac javadoc xmllint markdownlint jsonlint jshint
uname	Linux 23ec5b3d53ba 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / `8fde353`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/9/testReport/
Max. process+thread count	1486 (vs. ulimit of 5500)
modules	C: tez-ui . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/9/console
versions	git=2.43.0 maven=3.9.15 hadolint=2.14.0 codespell=2.4.1 jshint=2.13.6 markdownlint=0.46.0 shellcheck=0.11.0
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

tez-yetus · 2026-07-02T10:25:41Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/10/console in case of problems.

maheshrajus · 2026-07-02T10:25:52Z

after removing phantomjs, we can try to build tez-ui also on arm64:

@abstractdog Can we handle this in separate JIRA ?

maheshrajus · 2026-07-02T10:27:16Z

@abstractdog I fixed your review comments and verified locally + in yetus CI env.
Could you please review the PR at your convenience?
Thank you !

tez-yetus · 2026-07-02T14:02:08Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	4m 49s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 1s		No case conflicting files found.
+0 🆗	detsecrets	0m 1s		detect-secrets was not available.
+0 🆗	shelldocs	0m 1s		Shelldocs was not available.
+0 🆗	xmllint	0m 0s		xmllint was not available.
+0 🆗	jsonlint	0m 0s		jsonlint was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	test4tests	0m 0s		The patch appears to include 2 new or modified test files.
			_ master Compile Tests _
+0 🆗	mvndep	0m 38s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 31s		master passed
+1 💚	compile	4m 2s		master passed
+1 💚	javadoc	1m 54s		master passed
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 9s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 44s		the patch passed
+1 💚	codespell	1m 35s		No new issues.
+1 💚	jshint	109m 6s		No new issues.
+1 💚	compile	4m 4s		the patch passed
+1 💚	javac	4m 4s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	hadolint	0m 0s		No new issues.
+1 💚	markdownlint	0m 3s		No new issues.
+1 💚	shellcheck	0m 0s		No new issues.
+1 💚	javadoc	1m 54s		the patch passed
			_ Other Tests _
+1 💚	unit	77m 5s		root in the patch passed.
+1 💚	asflicense	1m 29s		The patch does not generate ASF License warnings.
		216m 31s

Subsystem	Report/Notes
Docker	ClientAPI=1.55 ServerAPI=1.55 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/10/artifact/out/Dockerfile
Optional Tests	dupname compile unit asflicense codespell detsecrets hadolint shellcheck shelldocs javac javadoc xmllint markdownlint jsonlint jshint
uname	Linux 70d447d0c557 5.15.0-181-generic #191-Ubuntu SMP Fri May 22 19:09:02 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	tez-personality.sh
git revision	master / `d53e67d`
Default Java	Eclipse Adoptium-21.0.11+10-LTS
Test Results	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/10/testReport/
Max. process+thread count	1467 (vs. ulimit of 5500)
modules	C: tez-ui . U: .
Console output	https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/10/console
versions	git=2.43.0 maven=3.9.15 hadolint=2.14.0 codespell=2.4.1 jshint=2.13.6 markdownlint=0.46.0 shellcheck=0.11.0
Powered by	Apache Yetus 0.15.1 https://yetus.apache.org

This message was automatically generated.

abstractdog · 2026-07-04T09:43:14Z

@abstractdog I fixed your review comments and verified locally + in yetus CI env. Could you please review the PR at your convenience? Thank you !

looks good so far, I have only one concern: on a vanilla ubuntu server, when chromium wasn't installed, the build simply failed by:

Building..Built project successfully. Stored in "/home/abstractdog/apache/tez/tez-ui/src/main/webapp/tmp/class-tests_dist-NhEe6O0O.tmp".
not ok 1 Chrome - Browser "scripts/chrome-launcher.sh --headless --disable-gpu --no-sandbox --disable-dev-shm-usage --no-default-browser-check --no-first-run --ignore-certificate-errors --test-type <url>" exited unexpectedly.

if the build process cannot take care of fetching a chrome or chromium, then a preinstalled chrome is a hard dependency of a mvn clean install command, which doesn't sound good to me
could the build process take care of the same automatically?

maheshrajus · 2026-07-06T09:07:31Z

if the build process cannot take care of fetching a chrome or chromium, then a preinstalled chrome is a hard dependency of a mvn clean install command, which doesn't sound good to me
could the build process take care of the same automatically?

@abstractdog Thank you for the review. I have not taken care of chromium, Let me check and fix it.

tez-yetus · 2026-07-08T09:16:58Z

(!) A patch to the testing environment has been detected.
Re-executing against the patched versions to perform further tests.
The console is at https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-514/11/console in case of problems.

maheshrajus · 2026-07-08T09:23:41Z

@abstractdog I have addressed your review comments and tested the changes in the environments listed below. Everything is working as expected.
Please let me know if I missed anything or if I should verify this in any other environment. Thanks !

macOS with & without system chrome, 
Ubuntu 22.04 – no system Chrome
Ubuntu 22.04 – system Chrome installed
Debian 12 – no system Chrome
Amazon Linux 2023 – no system Chrome

maheshrajus force-pushed the TEZ-4476 branch from 4976441 to 6fef20c Compare June 30, 2026 02:35

maheshrajus force-pushed the TEZ-4476 branch from 27e10d9 to 5711577 Compare July 1, 2026 03:34

maheshrajus force-pushed the TEZ-4476 branch from 5711577 to 6512952 Compare July 1, 2026 09:05

maheshrajus changed the title ~~TEZ-4476: Move tez-ui testing from PhantomJS to Headless Chrome~~ [wIP] TEZ-4476: Move tez-ui testing from PhantomJS to Headless Chrome Jul 1, 2026

maheshrajus force-pushed the TEZ-4476 branch from 6512952 to dabef30 Compare July 1, 2026 17:08

maheshrajus force-pushed the TEZ-4476 branch from dabef30 to 5d2c4ba Compare July 2, 2026 06:01

maheshrajus changed the title ~~[wIP] TEZ-4476: Move tez-ui testing from PhantomJS to Headless Chrome~~ TEZ-4476: Move tez-ui testing from PhantomJS to Headless Chrome Jul 2, 2026

maheshrajus added 5 commits July 2, 2026 15:53

TEZ-4476: Move tez-ui testing from PhantomJS to Headless Chrome

ad8fa78

Fix codespell issues

dd8c9fc

tez-ui test fix

0fa7a39

Review comments fix:1

d876e57

Fix markdownlint issues in tez-ui/README.md

46da530

maheshrajus force-pushed the TEZ-4476 branch from 5d2c4ba to 46da530 Compare July 2, 2026 10:25

Review comments fix:2

53ccf15

Uh oh!

Conversation

maheshrajus commented Jun 29, 2026

Uh oh!

tez-yetus commented Jun 29, 2026

Uh oh!

tez-yetus commented Jun 29, 2026

Uh oh!

tez-yetus commented Jun 30, 2026

Uh oh!

tez-yetus commented Jun 30, 2026

Uh oh!

tez-yetus commented Jun 30, 2026

Uh oh!

maheshrajus commented Jun 30, 2026

Uh oh!

abstractdog commented Jun 30, 2026

Uh oh!

maheshrajus commented Jun 30, 2026

Uh oh!

maheshrajus commented Jun 30, 2026

Uh oh!

tez-yetus commented Jun 30, 2026

Uh oh!

tez-yetus commented Jul 1, 2026

Uh oh!

tez-yetus commented Jul 1, 2026

Uh oh!

tez-yetus commented Jul 1, 2026

Uh oh!

tez-yetus commented Jul 1, 2026

Uh oh!

tez-yetus commented Jul 2, 2026

Uh oh!

tez-yetus commented Jul 2, 2026

Uh oh!

tez-yetus commented Jul 2, 2026

Uh oh!

maheshrajus commented Jul 2, 2026

Uh oh!

maheshrajus commented Jul 2, 2026

Uh oh!

tez-yetus commented Jul 2, 2026

Uh oh!

abstractdog commented Jul 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

maheshrajus commented Jul 6, 2026

Uh oh!

tez-yetus commented Jul 8, 2026

Uh oh!

maheshrajus commented Jul 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

abstractdog commented Jul 4, 2026 •

edited

Loading