-
Notifications
You must be signed in to change notification settings - Fork 459
[Image] Upgrade openEuler to 24.03 #2631
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
Conversation
Signed-off-by: Icey <1790571317@qq.com>
👋 Hi! Thank you for contributing to the vLLM Ascend project. The following points will speed up your PR merge:
If CI fails, you can run linting and testing checks locally according Contributing and Testing. |
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 && \ |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
please update all the openEuler dockerfile. Thanks. |
done |
### 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>
### 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>
### 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>
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.