Skip to content

HCD-25: Add Netty native epoll libs for ARM#2421

Merged
emerkle826 merged 1 commit into
mainfrom
HCD-25-main
May 14, 2026
Merged

HCD-25: Add Netty native epoll libs for ARM#2421
emerkle826 merged 1 commit into
mainfrom
HCD-25-main

Conversation

@emerkle826
Copy link
Copy Markdown

This patch adds the linux-aarch_64 native epoll libs so that epoll can be used on Linux ARM machines

What is the issue

Native Netty Epoll libraries for Linux ARM64/aarch_64 are missing from the build

What does this PR fix and why was it fixed

Adds the Linux ARM64 native Netty Epoll libraries to the dependency list

This patch adds the linux-aarch_64 native epoll libs
so that epoll can be used on Linux ARM machines
@github-actions
Copy link
Copy Markdown

Checklist before you submit for review

  • This PR adheres to the Definition of Done
  • Make sure there is a PR and ticket in the CNDB project updating the Converged Cassandra version
  • Use NoSpamLogger for log lines that may appear frequently in the logs
  • Verify test results on Butler
  • Test coverage for new/modified code is > 80%
  • Proper code formatting
  • Proper title for each commit staring with the project-issue number, like CNDB-1234
  • Each commit has a meaningful description
  • Each commit is not very long and contains related changes
  • Renames, moves and reformatting are in distinct commits
  • All new files should contain the IBM copyright header instead of the Apache License one (no DataStax copyright any longer)

@emerkle826
Copy link
Copy Markdown
Author

Do I need to create the associated CNDB ticket/PR for this change? If I understand correctly, CNDB brings in its own Netty dependency
https://github.com/riptano/cndb/blob/main/pom.xml#L251-L255
and explicitly excludes the one from CC:
https://github.com/riptano/cndb/blob/main/pom.xml#L559-L577

@JeremiahDJordan
Copy link
Copy Markdown
Member

This won't affect the CNDB build, so we can probably skip the cndb PR here.

@emerkle826
Copy link
Copy Markdown
Author

This won't affect the CNDB build, so we can probably skip the cndb PR here.

Thanks JD.

@emerkle826
Copy link
Copy Markdown
Author

Also as an fyi, it looks like these libs have been a part of main-5.0 for some time:
https://github.com/datastax/cassandra/blob/main-5.0/.build/cassandra-deps-template.xml#L185-L189
And the commit that cleaned up the unnecessary Netty libs actuially explicitly added the epoll stuff for linux_aarch_64: aa84b19

@sonarqubecloud
Copy link
Copy Markdown

@plpesvc-ds
Copy link
Copy Markdown

❌ Build ds-cassandra-pr-gate/PR-2421 rejected by Butler


4 regressions found
See build details here


Found 4 new test failures

Test Explanation Runs Upstream
snapshot_test.TestSnapshot.test_snapshot_and_restore_with_nodetool_import (offheap-bti-scm-hcd_1) REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorCompaction100dTest.testZeroOrOneToManyCompaction[version=ec enableNVQ=false] REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorSiftSmallTest.testSiftSmall[version=fa enableNVQ=false enableFused=false] REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorTypeTest.lwtTest[version=ca enableNVQ=false enableFused=false] REGRESSION 🔴 0 / 30

Found 4 known test failures

@emerkle826
Copy link
Copy Markdown
Author

@JeremiahDJordan Ok to merge this? Butle shows 4 potential regressions, but I don't think adding the Netty epoll libs has anything to do with them.

@emerkle826 emerkle826 merged commit d4e3ed4 into main May 14, 2026
3 of 5 checks passed
@emerkle826 emerkle826 deleted the HCD-25-main branch May 14, 2026 18:12
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