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

Add ability to create container to host local repo. #100

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dlyle65535
Copy link
Contributor

This is something I find helpful when I'm working in an area with degraded network. The notion is to keep a local copy of the HDP and HDP-UTILS repos and fire up a httpd container to serve them. For ease of use, I register it with the consul instance. I use DOCKER_OPTS to overwrite repoinfo.xml to point to my local instance.

@mhmxs
Copy link
Contributor

mhmxs commented Dec 14, 2015

I tested your change, works well. One more thing missing, a few lines in readme. How and why enable local repo, and an example repoinfo.xml also would be nice. Otherwise no one will use it except you and me :)

@dlyle65535
Copy link
Contributor Author

Sounds good to me. Okay to put the sample repoinfo.xml and hdp_urlinfo.json files at the top level or would you prefer a contrib folder?

@mhmxs
Copy link
Contributor

mhmxs commented Dec 14, 2015

Top level would be fine. Thanks.

@dlyle65535 dlyle65535 force-pushed the local-repo branch 4 times, most recently from b6ab319 to 6d98403 Compare December 15, 2015 22:18
@dlyle65535
Copy link
Contributor Author

Thanks for the feedback!

@mhmxs
Copy link
Contributor

mhmxs commented Feb 12, 2016

Hello @dlyle65535,

I picked up this change, and because there was merge conflicts i moved it to a new branch 07a379d .
Sorry, i missed you updated the readme as i suggested :(.
We update to centos7 so i updated some url to the rigth one. There is an url in repoinfo.xml http://amb-repo.service.consul/HDP-UTILS-1.1.0.20/repos/centos7 but i didn't find any matching url in hdp_utlinfo.json. Is this ok?

@dlyle65535
Copy link
Contributor Author

Hi @mhmxs - I'm going to try to pick this back up today-tomorrow. I'll look it over and get back.

Thanks!

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