Skip to content

Install wget to fix image build#231

Merged
wangxiyuan merged 2 commits intovllm-project:mainfrom
Yikun:fix-docker
Mar 4, 2025
Merged

Install wget to fix image build#231
wangxiyuan merged 2 commits intovllm-project:mainfrom
Yikun:fix-docker

Conversation

@Yikun
Copy link
Copy Markdown
Member

@Yikun Yikun commented Mar 3, 2025

What this PR does / why we need it?

Install wget to fix image build

Does this PR introduce any user-facing change?

No

How was this patch tested?

CI passed

@Yikun
Install wget to fix image build
ee0cccf 
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
@Yikun Yikun mentioned this pull request Mar 3, 2025
Merged
@Yikun
Enable set -ex and add extra
bce1a49 
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
@Yikun Yikun marked this pull request as draft March 3, 2025 14:20
@Yikun Yikun marked this pull request as ready for review March 3, 2025 15:23
@Yikun
Copy link
Copy Markdown
Member Author

Yikun commented Mar 4, 2025 

#20 [linux/amd64  9/10] RUN bash /workspace/vllm-ascend/pta_install.sh
#20 7.277 Processing ./torch_npu-2.5.1.dev20250226-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
#20 [linux/amd64  9/10] RUN bash /workspace/vllm-ascend/pta_install.sh
#20 13.22       Successfully uninstalled torch-2.6.0
#20 26.06 Successfully installed torch-2.5.1+cpu torch-npu-2.5.1.dev20250226

#23 [linux/arm64  9/10] RUN bash /workspace/vllm-ascend/pta_install.sh
#23 7.242 + pip install ./torch_npu-2.5.1.dev20250226-cp310-cp310-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
#23 11.41 Processing ./torch_npu-2.5.1.dev20250226-cp310-cp310-manylinux_2_17_aarch64.manylinux2014_aarch64.whl
#23 122.1 Successfully installed torch-2.5.1 torch-npu-2.5.1.dev20250226

@Yikun
Copy link
Copy Markdown
Member Author

Yikun commented Mar 4, 2025

Confirm it works

wangxiyuan pushed a commit that referenced this pull request Mar 4, 2025
@Yikun
[v0.7.3] Install wget to fix image build (#232)
72c8f9c 
### What this PR does / why we need it?
Install `wget` to fix image build

Backport #231

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

### How was this patch tested?
CI passed

---------

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
@wangxiyuan wangxiyuan merged commit 839dac8 into vllm-project:main Mar 4, 2025
ttanzhiqiang pushed a commit to ttanzhiqiang/vllm-ascend that referenced this pull request Apr 27, 2025
@Yikun
Install wget to fix image build (vllm-project#231)
c40f9ee 
### What this PR does / why we need it?

Install `wget` to fix image build

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

### How was this patch tested?
CI passed

---------

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
yangzhe-2026 pushed a commit to yangzhe-2026/vllm-ascend that referenced this pull request May 6, 2026
@Yikun
Install wget to fix image build (vllm-project#231)
5aeea67 
### What this PR does / why we need it?

Install `wget` to fix image build

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

### How was this patch tested?
CI passed

---------

Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wangxiyuan wangxiyuan wangxiyuan approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Yikun @wangxiyuan