Skip to content

TEZ-4492 : Approach 1 : Update Bower Registry to a different mirror - #284

Merged
abstractdog merged 1 commit into
apache:masterfrom
AnmolSun:TEZ-4492
May 4, 2023
Merged

TEZ-4492 : Approach 1 : Update Bower Registry to a different mirror#284
abstractdog merged 1 commit into
apache:masterfrom
AnmolSun:TEZ-4492

Conversation

@AnmolSun

Copy link
Copy Markdown
Contributor

This approach updates Bowerrc to use bower.herokuapp mirror to avoid Bower Registry CERT_EXPIRE issue

@tez-yetus

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeComment
+0 🆗reexec1m 1sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+1 💚@author0m 0sThe patch does not contain any @author tags.
_ master Compile Tests _
_ Patch Compile Tests _
+1 💚whitespace0m 0sThe patch has no whitespace issues.
_ Other Tests _
+1 💚asflicense0m 52sThe patch does not generate ASF License warnings.
2m 33s
SubsystemReport/Notes
DockerClientAPI=1.42 ServerAPI=1.42 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-284/1/artifact/out/Dockerfile
GITHUB PR#284
Optional Testsdupname asflicense
unameLinux 02f41be7f301 4.15.0-206-generic #217-Ubuntu SMP Fri Feb 3 19:10:13 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/tez.sh
git revisionmaster / 249e017
Max. process+thread count46 (vs. ulimit of 5500)
modulesC: tez-ui U: tez-ui
Console outputhttps://ci-hadoop.apache.org/job/tez-multibranch/job/PR-284/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.

@AnmolSunAnmolSun changed the title TEZ-4492 : Approach 1 forTez UI Build Failing Due to Bower Registry CERT_EXPIRE issueTEZ-4492 : Approach 1 : Update Bower Registry to a different MirrorApr 28, 2023
@AnmolSunAnmolSun changed the title TEZ-4492 : Approach 1 : Update Bower Registry to a different MirrorTEZ-4492 : Approach 1 : Update Bower Registry to a different mirrorApr 28, 2023

@amanraj2520amanraj2520 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 LGTM

@ayushtknayushtkn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The solution is mentioned over here as well
bower/registry#304 (comment)

Another solution is set strict-ssl as false, not sure if it is any better.

@abstractdog can you please bless this, the builds are failing due to this :)

@abstractdog

Copy link
Copy Markdown
Contributor

yay, makes sense, thanks @AnmolSun, I'll merge this once github actions passed

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.

5 participants

@AnmolSun@tez-yetus@abstractdog@ayushtkn@amanraj2520