Skip to content

HDDS-10228. Intermittent failure downloading bzip2 from sourceware.org#6111

Merged
adoroszlai merged 2 commits into
apache:masterfrom
adoroszlai:HDDS-10228
Jan 30, 2024
Merged

HDDS-10228. Intermittent failure downloading bzip2 from sourceware.org#6111
adoroszlai merged 2 commits into
apache:masterfrom
adoroszlai:HDDS-10228

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Fix intermittent failure downloading bzip2 due to Network is unreachable.

The fix is similar to HDDS-10157: download from GitHub in CI. The tarballs from these two sources are a bit different: contents are in bzip2-bzip2-1.0.8/ in the one from GitHub, bzip2-1.0.8/ in the original one. So an additional move Ant task renames the GitHub-specific dir to the original one. failonerror=false ensures it passes with the original tarball, too.

https://issues.apache.org/jira/browse/HDDS-10228

How was this patch tested?

Tested locally for the original URL:

# install dependencies
$ mvn -DskipTests -am -pl :hdds-rocks-native clean install
...

# test
$ mvn -Drocks_tools_native -pl :hdds-rocks-native clean generate-sources
...
[INFO] BUILD SUCCESS

$ ls -1 hadoop-hdds/rocks-native/target/bzip2
bzip2-1.0.8
bzip2-v1.0.8.tar.gz

CI (with the CI-specific URL):
https://github.com/adoroszlai/ozone/actions/runs/7681371921/job/20934591236

Copy link
Copy Markdown
Contributor

@hemantk-12 hemantk-12 left a comment

Choose a reason for hiding this comment

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

Thanks @adoroszlai for working on this.

LGTM.

@adoroszlai adoroszlai merged commit 7579bd0 into apache:master Jan 30, 2024
@adoroszlai adoroszlai deleted the HDDS-10228 branch January 30, 2024 10:24
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Thanks @hemantk-12 for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants