-
Notifications
You must be signed in to change notification settings - Fork 122
Can't recursively update tensorrt_llm submodule 3rdparty directory links #717
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@kaiyux Could you take a look at this to confirm? |
+1, encountering the same issue when trying to build the docker image. |
Sorry for the late reply, is the issue persist on the latest main branch? |
I believe the issue has been resolved on |
Yeah it's resolved in main now. |
When cloning the
main
branch, and running:The 3rdparty directory in tensorrt_llm submodule has links that cannot be resolved. I tried manually following the links in the repo and could not resolve them either.
I believe the cause of this issue is that the latest PR #715 (10 hours ago as of writing) changed the tensorrt_llm submodule version to one with a 3rdparty dierctory that references an internal or private GitLab instance within NVIDIA's network, judging from the error saying
Could not resolve host: gitlab-master.nvidia.com
. These are the 3rdparty links in the in tensorrt_llm submodule of the current tensorrtllm_backend main branch, which I can't resolve in my browser either. Whereas I can resolve the tensorrt_llm main branch 3rdparty submodules just fine (the tensorrtllm_backend:main's tensorrt_llm submodule is not tensorrt_llm:main).As a workaround I will clone the v0.17.0 release branch, which has a tensorrt_llm version with 3rdparty submodules I can resolve.
Error log:
The text was updated successfully, but these errors were encountered: