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.

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

[Druid] Use a different repository to download sigar artifacts #19780

Merged
merged 1 commit into from
Jun 2, 2023

Conversation

imjalpreet
Copy link
Member

@imjalpreet imjalpreet commented Jun 2, 2023

The old repository(repository.jboss.org) has been throwing 500 Internal Server Errors.

[ERROR] Failed to execute goal on project presto-druid: Could not resolve dependencies for project com.facebook.presto:presto-druid:presto-plugin:0.282-SNAPSHOT: Failed to collect dependencies at org.apache.druid:druid-processing:jar:0.19.0 -> org.apache.druid:druid-core:jar:0.19.0 -> org.hyperic:sigar:jar:1.6.5.132: Failed to read artifact descriptor for org.hyperic:sigar:jar:1.6.5.132: Could not transfer artifact org.hyperic:sigar:pom:1.6.5.132 from/to sigar (https://repository.jboss.org/nexus/content/repositories/thirdparty-uploads/): transfer failed for https://repository.jboss.org/nexus/content/repositories/thirdparty-uploads/org/hyperic/sigar/1.6.5.132/sigar-1.6.5.132.pom, status: 500 Internal Server Error -> [Help 1]

The new repository has been taken from the recent versions of druid. Since druid-24.0.0, the repository has been changed from https://repository.jboss.org/nexus/content/repositories/thirdparty-uploads/ to https://repository.mulesoft.org/nexus/content/repositories/public (apache/druid#12561)

Test plan
Verified that after this change, we are able to download the sigar artifacts from the new repository and the build succeeds.

If release note is NOT required, use:

== NO RELEASE NOTE ==

The old repository(repository.jboss.org) has been throwing 500 Internal Server Errors
@imjalpreet imjalpreet requested a review from a team as a code owner June 2, 2023 09:23
@imjalpreet
Copy link
Member Author

@prestodb/committers @tdcmeehan can someone please review this PR? This will unblock the build failures we are seeing in CI pipeline due to one of the repositories being down.

@tdcmeehan tdcmeehan merged commit 8aab8b0 into prestodb:master Jun 2, 2023
44 checks 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.

None yet

2 participants