Skip to content

[CI] Uninstall triton in dockerfile#298

Merged
Yikun merged 2 commits intovllm-project:mainfrom
wangxiyuan:fix_ci
Mar 11, 2025
Merged

[CI] Uninstall triton in dockerfile#298
Yikun merged 2 commits intovllm-project:mainfrom
wangxiyuan:fix_ci

Conversation

@wangxiyuan
Copy link
Copy Markdown
Collaborator

@wangxiyuan wangxiyuan commented Mar 11, 2025

triton doesn't work with ascend. We should make sure it's uninstalled in dockerfile

Related: #291

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Comment thread Dockerfile Outdated
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
@Yikun
Copy link
Copy Markdown
Member

Yikun commented Mar 11, 2025

#19 [linux/amd64  8/11] RUN python3 -m pip uninstall -y triton
#19 0.826 Found existing installation: triton 3.2.0
#19 0.846 Uninstalling triton-3.2.0:
#19 1.446   Successfully uninstalled triton-3.2.0
#23 [linux/arm64 8/11] RUN python3 -m pip uninstall -y triton 

2061#23 4.250 WARNING: Skipping triton as it is not installed.

@Yikun Yikun merged commit 9450e98 into vllm-project:main Mar 11, 2025
Yikun added a commit to Yikun/vllm-ascend that referenced this pull request Mar 12, 2025
triton doesn't work with ascend. We should make sure it's uninstalled in
dockerfile


Related: vllm-project#291

---------

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Co-authored-by: Yikun Jiang <yikunkero@gmail.com>
Yikun added a commit to Yikun/vllm-ascend that referenced this pull request Mar 12, 2025
triton doesn't work with ascend. We should make sure it's uninstalled in
dockerfile

Related: vllm-project#291

---------

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Co-authored-by: Yikun Jiang <yikunkero@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
wangxiyuan added a commit that referenced this pull request Mar 12, 2025
### What this PR does / why we need it?

The triton doesn't work with ascend. We should make sure it's
uninstalled in dockerfile

Backport: #298
Closes: #291

### Does this PR introduce _any_ user-facing change?
NO

### How was this patch tested?
CI passed

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Co-authored-by: wangxiyuan <wangxiyuan1007@gmail.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Mar 18, 2025
triton doesn't work with ascend. We should make sure it's uninstalled in
dockerfile

Related: vllm-project#291

---------

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Co-authored-by: Yikun Jiang <yikunkero@gmail.com>
Signed-off-by: angazenn <zengyanjia@huawei.com>
@wangxiyuan wangxiyuan deleted the fix_ci branch March 20, 2025 07:08
ttanzhiqiang pushed a commit to ttanzhiqiang/vllm-ascend that referenced this pull request Apr 27, 2025
triton doesn't work with ascend. We should make sure it's uninstalled in
dockerfile


Related: vllm-project#291

---------

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Co-authored-by: Yikun Jiang <yikunkero@gmail.com>
Angazenn pushed a commit to Angazenn/vllm-ascend that referenced this pull request Oct 21, 2025
triton doesn't work with ascend. We should make sure it's uninstalled in
dockerfile


Related: vllm-project#291

---------

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Co-authored-by: Yikun Jiang <yikunkero@gmail.com>
yangzhe-2026 pushed a commit to yangzhe-2026/vllm-ascend that referenced this pull request May 6, 2026
triton doesn't work with ascend. We should make sure it's uninstalled in
dockerfile


Related: vllm-project#291

---------

Signed-off-by: wangxiyuan <wangxiyuan1007@gmail.com>
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Co-authored-by: Yikun Jiang <yikunkero@gmail.com>
Sign up for free to 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.

2 participants