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

Vendor deploy_helper module + Ansible 2 requirement #478

Merged
merged 3 commits into from
Feb 6, 2016

Conversation

swalkinshaw
Copy link
Member

This PR makes Ansible 2.x a requirement for Trellis.

In order to do that we needed to fix #469. This meant temporarily "vendoring"/copying in the upstream deploy_helper module into the repo.

This module is now part of Ansible in modules-extras
Temporary fix until Ansible releases a version which includes this
version of the `deploy_helper` module.
@fullyint
Copy link
Contributor

fullyint commented Feb 5, 2016

Looks good to me!
Provision and deploy worked for me on 2.0.0.2, even with space in path to Ansible files.
Excited for this!

@deegs
Copy link

deegs commented Feb 5, 2016

Thank you for this! This fixed issue #469 for me. Using Ansible 2.0.0.2

swalkinshaw added a commit that referenced this pull request Feb 6, 2016
Vendor deploy_helper module + Ansible 2 requirement
@swalkinshaw swalkinshaw merged commit f44c1a4 into master Feb 6, 2016
@swalkinshaw swalkinshaw deleted the remove-f500-deploy-module branch February 6, 2016 01:03
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.

Finalizing deploys broken on Ansible 2.x
3 participants