Skip to content

[FLINK-41226][build] Bump tcnative to 2.0.81 and run CI on Ubuntu 26.04 - #123

Merged
RocMarshal merged 1 commit into
apache:masterfrom
MartijnVisser:FLINK-41226-openssl-tcnative
Oct 27, 2026
Merged

RocMarshal merged 1 commit into
apache:masterfrom
MartijnVisser:FLINK-41226-openssl-tcnative

Conversation

@MartijnVisser

Copy link
Copy Markdown
Contributor

Purpose of the change

The OpenSSL network benchmark fails since flink-shaded 22.0 and the TLSv1.3 default, and CI then hangs until it's killed after 6h. This supersedes #118 by @snuyanzin, who is co-author, and adds the runner change it was missing.

Brief change log

  • Update flink-shaded to 22.0 and netty-tcnative to 2.0.81
  • Run CI on ubuntu-26.04, since tcnative 2.0.81 needs OpenSSL 3.2+
  • Install libssl3 instead of libssl1.0.0, and note the requirement in the README

Verifying this change

In linux/amd64 containers, StreamNetworkThroughputBenchmarkExecutor hangs on master with ubuntu:24.04. With this change it fails with OPENSSL_3.2.0 not found on ubuntu:24.04 and passes on ubuntu:26.04 (3 runs each), as does the full mvn test -P test-java17.

Impact and documentation

Machines that run these benchmarks need OpenSSL 3.2+ for the OpenSSL parameter.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: Claude Code (Claude Opus 5.5)

- Update flink-shaded to 22.0 and netty-tcnative to 2.0.81, matching Flink master since FLINK-40527
- Run CI on ubuntu-26.04, since tcnative 2.0.81 needs OpenSSL 3.2+ and ubuntu-24.04 ships 3.0.13
- Install libssl3 instead of the libssl1.0.0 package
- Note the OpenSSL 3.2 requirement in the README

Co-authored-by: Sergey Nuyanzin <snuyanzin@gmail.com>
Generated-by: Claude Code (Claude Opus 5.5)
@MartijnVisser

Copy link
Copy Markdown
Contributor Author

@RocMarshal Could you take a look?

@RocMarshal RocMarshal self-assigned this Oct 27, 2026

@RocMarshal RocMarshal left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @MartijnVisser .

@RocMarshal
RocMarshal merged commit 8584a6d into apache:master Oct 27, 2026
1 check passed
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.

2 participants