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 Amazon Linux support #124

Merged
merged 1 commit into from
Oct 10, 2014
Merged

Conversation

ryfow
Copy link
Contributor

@ryfow ryfow commented Oct 9, 2014

No description provided.

@garethr
Copy link
Owner

garethr commented Oct 10, 2014

@ryfow I've not had chance to test this, but as it doesn't affect anything else and it looks like you're using this I'm happy to merge. I'll update the supported metadata when I next release to the Forge.

garethr added a commit that referenced this pull request Oct 10, 2014
@garethr garethr merged commit 01c79c3 into garethr:master Oct 10, 2014
@ryfow
Copy link
Contributor Author

ryfow commented Oct 10, 2014

gah. I forgot to update the package_name default code in params.pp. The package_name in Amazon linux is docker. Right now, it defaults to docker-io and you need to override it. I'll submit another pull request.

cegeka-jenkins pushed a commit to cegeka/puppet-docker that referenced this pull request Oct 23, 2017
cegeka-jenkins pushed a commit to cegeka/puppet-docker that referenced this pull request Apr 17, 2024
* Fix systemd escaping issue garethr#124

* Fix test spec/defines/run_spec.rb issue garethr#124
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