Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Download Kafka from a different server #625

Merged
merged 1 commit into from
Nov 29, 2019
Merged

Download Kafka from a different server #625

merged 1 commit into from
Nov 29, 2019

Conversation

suminb
Copy link
Contributor

@suminb suminb commented Nov 19, 2019

It appears http://redrockdigimark.com/apachemirror/kafka is no longer operational, and thus docker build for apache-kafka:1.0.0 produces the following error message:

Step 6/12 : ADD http://redrockdigimark.com/apachemirror/kafka/${KAFKA_VERSION}/${FILE_NAME} /opt/
ADD failed: failed to GET http://redrockdigimark.com/apachemirror/kafka/1.0.0/kafka_2.12-1.0.0.tgz with status 500 Internal Server Error: <!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>500 Internal Server Error</title>
</head><body>
<h1>Internal Server Error</h1>
<p>The server encountered an internal error or
misconfiguration and was unable to complete
your request.</p>
<p>Please contact the server administrator at
 info@redrockdigimark.com to inform them of the time this error occurred,
 and the actions you performed just before this error.</p>
<p>More information about this error may be available
in the server error log.</p>
<hr>
<address>Apache/2.4.29 (Ubuntu) Server at redrockdigimark.com Port 443</address>
</body></html>

This PR alters the download URL so that it fetches the file from the official mirror site. Switching from plain HTTP to secure HTTPS would be a nice addition ;-)

kamalcph
kamalcph approved these changes Nov 29, 2019
View reviewed changes
Copy link
Contributor

@kamalcph kamalcph left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks, @suminb for fixing this!

@kamalcph kamalcph merged commit 977f484 into hortonworks:master Nov 29, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@kamalcph kamalcph kamalcph approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@suminb @kamalcph