Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUILD] Fix the java.net.BindException when testing with Github Action #2451

Merged
merged 2 commits into from
Apr 9, 2021
Merged

[BUILD] Fix the java.net.BindException when testing with Github Action #2451

merged 2 commits into from
Apr 9, 2021

Conversation

pan3793
Copy link
Member

@pan3793 pan3793 commented Apr 9, 2021

@github-actions github-actions bot added the INFRA label Apr 9, 2021
@rymurr
Copy link
Contributor

rymurr commented Apr 9, 2021

Cc @snazy

@aokolnychyi
Copy link
Contributor

This makes sense to me.

@RussellSpitzer @rdblue, could you take a look?

@pan3793
Copy link
Member Author

pan3793 commented Apr 9, 2021

Spark switched 127.0.0.1 to localhost, see apache/spark#32102

@pan3793
Copy link
Member Author

pan3793 commented Apr 9, 2021

GA passed, both on JDK8 and JDK11, hope we can get it in ASAP to recover the development progress.

@RussellSpitzer
Copy link
Member

apache/spark#32096 (comment) - Underlying issue is hypothesized to be the dns resolving properties of Inet.getLocalAddress.

@RussellSpitzer
Copy link
Member

Our Test Hive code already uses localhost, so that should be fine.

@RussellSpitzer RussellSpitzer merged commit b102843 into apache:master Apr 9, 2021
@RussellSpitzer
Copy link
Member

Thanks @pan3793 !

@pan3793 pan3793 deleted the ci branch April 10, 2021 09:05
rymurr pushed a commit to rymurr/nessie that referenced this pull request Apr 11, 2021
Fix flaky Spark tests
ca501a4 
see apache/iceberg#2451
@rymurr rymurr mentioned this pull request Apr 11, 2021
Merged
rymurr added a commit to projectnessie/nessie that referenced this pull request Apr 12, 2021
@rymurr
Fix flaky Spark tests (#1071)
9d4e80a 
see apache/iceberg#2451
@lhotari
Copy link
Member

lhotari commented Apr 14, 2021

Root cause: reverse name lookups for the current host name is broken in GitHub Actions Runner VMs: actions/runner-images#3185
builds@apache.org mailing list thread:
https://lists.apache.org/thread.html/r40b2da182952f21a3b94696e6f1bf8594e7ff16e1c5979fe8b2d51c8%40%3Cbuilds.apache.org%3E

@snazy
Copy link
Member

snazy commented Apr 14, 2021

Interesting, maybe that's also causing the new-ish CI failures (#2475)

snazy added a commit to snazy/iceberg that referenced this pull request Apr 14, 2021
@snazy
Try reverse DNS lookup workaround for Ubunut GitHub runners
2cda7cd 
See apache#2451 (comment)
See https://lists.apache.org/thread.html/r40b2da182952f21a3b94696e6f1bf8594e7ff16e1c5979fe8b2d51c8%40%3Cbuilds.apache.org%3E
@snazy snazy mentioned this pull request Apr 14, 2021
Closed
snazy added a commit to snazy/iceberg that referenced this pull request Apr 14, 2021
@snazy
Try reverse DNS lookup workaround for Ubunut GitHub runners
8d6299b 
See apache#2451 (comment)
See https://lists.apache.org/thread.html/r40b2da182952f21a3b94696e6f1bf8594e7ff16e1c5979fe8b2d51c8%40%3Cbuilds.apache.org%3E
snazy added a commit to snazy/iceberg that referenced this pull request Apr 14, 2021
@snazy
Try reverse DNS lookup workaround for Ubunut GitHub runners
2a0b99a 
See apache#2451 (comment)
See https://lists.apache.org/thread.html/r40b2da182952f21a3b94696e6f1bf8594e7ff16e1c5979fe8b2d51c8%40%3Cbuilds.apache.org%3E

Fixes apache#2475
@snazy snazy mentioned this pull request Apr 14, 2021
Merged
RussellSpitzer pushed a commit that referenced this pull request Apr 14, 2021
@snazy
[INFRA] Workaround for Reverse DNS lookup on Ubuntu GitHub runners (#…
6f0ecad 
…2480)

See #2451 (comment)
See https://lists.apache.org/thread.html/r40b2da182952f21a3b94696e6f1bf8594e7ff16e1c5979fe8b2d51c8%40%3Cbuilds.apache.org%3E

Fixes #2475
stevenzwu pushed a commit to stevenzwu/iceberg that referenced this pull request Jul 28, 2021
@pan3793 @aokolnychyi
[BUILD] Fix the java.net.BindException when testing with Github Action (
7e95f0b 
apache#2451)

[BUILD] Fix the java.net.BindException when testing with Github Action

Switches to using localhost from 127.0.0.1 for the Spark driver bind address in CI tests
stevenzwu pushed a commit to stevenzwu/iceberg that referenced this pull request Jul 28, 2021
@snazy @aokolnychyi
[INFRA] Workaround for Reverse DNS lookup on Ubuntu GitHub runners (a…
1e34b66 
…pache#2480)

See apache#2451 (comment)
See https://lists.apache.org/thread.html/r40b2da182952f21a3b94696e6f1bf8594e7ff16e1c5979fe8b2d51c8%40%3Cbuilds.apache.org%3E

Fixes apache#2475
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RussellSpitzer RussellSpitzer RussellSpitzer approved these changes

Assignees
No one assigned
Labels
INFRA
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@pan3793 @rymurr @aokolnychyi @RussellSpitzer @lhotari @snazy