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

Allow etcd to be installed through system packages #165

Merged
merged 3 commits into from
Feb 7, 2019

Conversation

ralphje
Copy link
Contributor

@ralphje ralphje commented Sep 17, 2018

In restricted/airgapped systems, using etcd that is to be downloaded from the internet may be hard. In some operating systems, e.g. Debian, etcd-server is available from the package repositories. This commit adds two settings (etcd_install_method and etcd_package_name) to allow installing etcd through system packages.

@scotty-c
Copy link
Contributor

@ralphje we are internally doing some work to increase functionality around etcd configuration. So I am going to leave this PR open and wait to get that work into master. Then we can have a look at this PR then

@davejrt
Copy link
Contributor

davejrt commented Dec 4, 2018

@ralphje apologies for the delay in getting back to you on this. We've had some conversations internally about how to best approach this, and I'd like to see this PR merged. Unfortunately there are some merge conflicts with the recent updates that need to be resolved before I can merge.

Would you consider rebasing and fixing the conflicts?

In restricted/airgapped systems, using etcd that is to be downloaded from the internet may be hard. In some operating systems, e.g. Debian, etcd-server is available from the package repositories. This commit adds two settings (etcd_install_method and etcd_package_name) to allow installing etcd through system packages.
@ralphje
Copy link
Contributor Author

ralphje commented Dec 8, 2018

I have rebased the commit.

@ralphje
Copy link
Contributor Author

ralphje commented Jan 25, 2019

@davejrt So, having rebased my PR, following your comment, I had expected it to have been merged by now. What happened? :(

@davejrt
Copy link
Contributor

davejrt commented Jan 26, 2019

Apologies @ralphje. I believe another PR got merged and then caused another conflict with your changes and I never followed up with you again.

That's on me, and certainly not a reflection of the effort you've made with this PR.

@gregohardy
Copy link
Contributor

LGTM

@gregohardy gregohardy merged commit 0974fac into puppetlabs:master Feb 7, 2019
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.

4 participants