-
Notifications
You must be signed in to change notification settings - Fork 272
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
[Proposal] Support AL2023/NodeJS18 for OpenSearch/Dashboards Releases #1563
Comments
I really don't know enough about this @peterzhuamazon. What are pros/cons? |
Now, with this issue on the horizon, we need to consider how to proceed with the changes.
This implies:
|
Tagging @vamshin @seanneumann @AMoo-Miki @ananzh @jmazanec15 for visibility. |
For the Node 18 upgrade, we're targeting 2.8. This is important as Node 14 no longer will receive security updates (as of April 2023). Is there an ask to delay to solve the above? |
KNN support on the native code compilation:
Seems like if we want to switch from CentOS7 to CentOS8/RockyLinux8, we will need to remove support of CentOS7 and AL2 from our support matrix on our website as well, since CentOS8/RockyLinux8 is the baseline of node18 with glibc 2.28.
And if we compile knn on 2.28 then the lib will crash on glibc that is lower than 2.28. cc: @krisfreedain |
@vamshin @jmazanec15 Can you please provide your inputs as well? |
I would agree @peterzhuamazon - if we test on CentOS8/RockyLinux8 the compatibility matrix needs to be updated to accurately reflect that for the community |
Hi @peterzhuamazon we will bump the node.js to version |
@peterzhuamazon want to clarify, you are saying that node 18 requires 2.28, correct? Then doesn't this break compatibility with AL2 and centos 7? So, regardless of k-NN libs, upgrading to Node 18 would require us to remove centos 7 and AL2 from the compatibility matrix. If this is not the case, we could also elect to compile k-NN libs in separate docker image to maintain compatibility. |
@peterzhuamazon not sure if you saw this opensearch-project/OpenSearch-Dashboards#3601 (comment) unlike chromium, it is possible to do a custom build of nodejs on centos 7 to lower glibc dependencies (I also saw musl builds somewhere, which can be another possibility to workaround this issue). I've not looked into performance or other issues might be caused by downgrading glibc, but if OS upgrade is difficult maybe this would be easier? |
Did anyone confirm node 18 with AL2? @seanneumann We noticed some glibc related compatibility issues. Not sure if it's an isolated incident. |
That is likely not an option because CentOS7 is going to be out of support in 1 year, and we are also on track to move to al2023 anyway. Manually rebuild nodejs18 would add another dep on our side, which is not going to scale on the long run. Thanks. |
The cmake upgrade to 3.23.3 version in #3706 was initially setup as a follow up in the al2023 upgrade/nodejs upgrade, tho now it seems like a 2.9.0 dependencies thus it needs to be updated soon. Thanks. |
AL2023/NodeJS18 on 2.10.0 Release:Required Steps
Optional Steps
|
What is the status of the separate pipeline to test node 14 backward compatibility? There are going to be users using old linux versions who can't go past node 14 yet, and it would be useful to know what fixes we can do so as to not unnecessarily break things for those users. |
@wbeckler We do not have plans to create a separate pipeline in public Jenkins infra for EOL versions of Node. If needed, this needs to be tackled at repo level. |
Have quick talk with OSD @AMoo-Miki about core, and we are planning to switch 2.x to 18.16.0 next week to start testing on the 2.10.0 build with newer nodejs. Thanks. |
….16.0 version and update changelog of 2.9.0/Unreleased Sections (#4948) * Bump 2.10.0 to use nodejs 18.16.0 version * Update 2.9.0 and 2.x changelog for unreleased 2.10.0 version Issues resolved: opensearch-project/opensearch-build#1563 opensearch-project/opensearch-build#3743 Signed-off-by: Peter Zhu <[email protected]>
The majority of the tasks in this issue is resolved and implemented. Thanks. |
With the announcement of Amazon Linux 2023 Preview available for testing, we want to setup some notes here for the support of AL2023 once it is fully released to public.
https://aws.amazon.com/about-aws/whats-new/2021/11/preview-amazon-linux-2022/
https://aws.amazon.com/about-aws/whats-new/2023/03/amazon-linux-2023/
We welcome more discussion on this.
Thanks.
The text was updated successfully, but these errors were encountered: