Skip to content

[FLINK-39787][build] Map runner hostname to loopback to avoid SSL SAN mismatch#117

Merged
MartijnVisser merged 2 commits into
apache:masterfrom
MartijnVisser:FLINK-39787-fix-benchmarks
May 28, 2026
Merged

[FLINK-39787][build] Map runner hostname to loopback to avoid SSL SAN mismatch#117
MartijnVisser merged 2 commits into
apache:masterfrom
MartijnVisser:FLINK-39787-fix-benchmarks

Conversation

@MartijnVisser
Copy link
Copy Markdown
Contributor

GitHub-hosted Azure runners assign a VM hostname whose reverse DNS returns an internal cloudapp.net FQDN. flink-runtime's SSL test certificate (local127.keystore) lists only 'localhost', '127.0.0.1', and '::1' as SANs, so the SSL handshake in
StreamNetworkThroughputBenchmark fails and the benchmark hangs until GitHub kills the job at the 6h timeout.

Prepend a loopback mapping for the runner hostname to /etc/hosts so InetAddress.getLocalHost() (used by StreamNetworkBenchmarkEnvironment to bind the NettyServer) returns 127.0.0.1; reverse DNS on that IP then yields 'localhost', which matches the certificate.

… mismatch

GitHub-hosted Azure runners assign a VM hostname whose reverse DNS
returns an internal cloudapp.net FQDN. flink-runtime's SSL test
certificate (local127.keystore) lists only 'localhost', '127.0.0.1',
and '::1' as SANs, so the SSL handshake in
StreamNetworkThroughputBenchmark fails and the benchmark hangs until
GitHub kills the job at the 6h timeout.

Prepend a loopback mapping for the runner hostname to /etc/hosts so
InetAddress.getLocalHost() (used by StreamNetworkBenchmarkEnvironment
to bind the NettyServer) returns 127.0.0.1; reverse DNS on that IP
then yields 'localhost', which matches the certificate.
Copy link
Copy Markdown
Contributor

@alpinegizmo alpinegizmo left a comment

Choose a reason for hiding this comment

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

LGTM

@MartijnVisser MartijnVisser merged commit 8882010 into apache:master May 28, 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