Skip to content

Conversation

wxsIcey
Copy link
Collaborator

@wxsIcey wxsIcey commented Aug 29, 2025

What this PR does / why we need it?

Upgrade openEuler to 24.03

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

CI passed with new added/existing test.

Signed-off-by: Icey <1790571317@qq.com>
Copy link

👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:‌‌

  • A PR should do only one thing, smaller PRs enable faster reviews.
  • Every PR should include unit tests and end-to-end tests ‌to ensure it works and is not broken by other future PRs.
  • Write the commit message by fulfilling the PR description to help reviewer and future developers understand.

If CI fails, you can run linting and testing checks locally according Contributing and Testing.

@wxsIcey wxsIcey marked this pull request as ready for review August 29, 2025 07:27
source /usr/local/Ascend/ascend-toolkit/set_env.sh && \
source /usr/local/Ascend/nnal/atb/set_env.sh && \
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:/usr/local/Ascend/ascend-toolkit/latest/`uname -i`-linux/devlib && \
export CPLUS_INCLUDE_PATH=$CPLUS_INCLUDE_PATH:/usr/include/c++/12:/usr/include/c++/12/`uname -i`-openEuler-linux && \
Copy link
Collaborator

Choose a reason for hiding this comment

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

This seems just a workaround to specify c++ version manually, just curious Why there are two gcc version here?

Copy link
Collaborator Author

@wxsIcey wxsIcey Aug 29, 2025

Choose a reason for hiding this comment

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

error action: https://github.com/vllm-project/vllm-ascend/actions/runs/17316412493/job/49160001312
root cause: fatal error: 'cstdint' file not found
problem analysis: Taking aarch64 as an example, the c++ header file directory of openEuler22.03 version is in aarch64-linux-gnu (Ubuntu is also in this directory), but 24.03 is changed to aarch64-openEuler-linux, which causes the header file not to be found. Specify additional C++ header file search directories CPLUS_INCLUDE_PATH can solve this problem.

openEuler24.03 only has gcc12, there are no two gcc versions.
related to: #676

@wangxiyuan
Copy link
Collaborator

please update all the openEuler dockerfile. Thanks.

Signed-off-by: Icey <1790571317@qq.com>
@wxsIcey
Copy link
Collaborator Author

wxsIcey commented Aug 30, 2025

please update all the openEuler dockerfile. Thanks.

done

@wangxiyuan wangxiyuan merged commit af62af3 into vllm-project:main Sep 2, 2025
16 checks passed
offline893 pushed a commit to offline893/vllm-ascend that referenced this pull request Sep 16, 2025
### What this PR does / why we need it?
Upgrade openEuler to 24.03

### Does this PR introduce _any_ user-facing change?
N/A

### How was this patch tested?
CI passed with new added/existing test.

- vLLM version: v0.10.1.1
- vLLM main:
vllm-project/vllm@4071c76

---------

Signed-off-by: Icey <1790571317@qq.com>
Signed-off-by: offline0806 <z00858301@china.huawei.com>
wangxiaoteng888 pushed a commit to LCAIZJ/vllm-ascend that referenced this pull request Sep 25, 2025
### What this PR does / why we need it?
Upgrade openEuler to 24.03

### Does this PR introduce _any_ user-facing change?
N/A

### How was this patch tested?
CI passed with new added/existing test.

- vLLM version: v0.10.1.1
- vLLM main:
vllm-project/vllm@4071c76

---------

Signed-off-by: Icey <1790571317@qq.com>
chopper0126 pushed a commit to chopper0126/vllm-ascend that referenced this pull request Sep 26, 2025
### What this PR does / why we need it?
Upgrade openEuler to 24.03

### Does this PR introduce _any_ user-facing change?
N/A

### How was this patch tested?
CI passed with new added/existing test.

- vLLM version: v0.10.1.1
- vLLM main:
vllm-project/vllm@4071c76

---------

Signed-off-by: Icey <1790571317@qq.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.

3 participants