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 Trellis docs for resolving local_path on Windows/ansible-local #62

Closed
QWp6t opened this issue Dec 28, 2016 · 3 comments
Closed

Add Trellis docs for resolving local_path on Windows/ansible-local #62

QWp6t opened this issue Dec 28, 2016 · 3 comments

Comments

@QWp6t
Copy link
Member

QWp6t commented Dec 28, 2016

Some genius wrote this:

The correct way to resolve this is to set your local_path in your wordpress_sites.yml file.

https://github.com/roots/trellis/blob/master/group_vars/production/wordpress_sites.yml#L11

I'm guessing that it's currently ../site, which is resolving to /vagrant/../site, which is /site, which doesn't exist. You should change that to something like /srv/www/example.com/current.

Alternatively, you can move your Vagrantfile one level up so that it sits alongside your trellis and site folders. This is how we used to do it in roots-example-project. Just make sure to change your ANSIBLE_PATH in Vagrantfile to something like File.join(__dir__, 'trellis') [ref]. This will put your site files in /vagrant/site and your trellis stuff in /vagrant/trellis. Now local_path: ../site will resolve to /vagrant/trellis/../site, which is /vagrant/site, which exists.

https://discourse.roots.io/t/trellis-bedrock-sage-on-windows-issue-with-build-before-yml/7825/6

The latter suggestion of moving Vagrantfile up one level has the added benefit of making the settings work cross-platform.

@swalkinshaw
Copy link
Member

@QWp6t @fullyint is this relevant anymore after recent Windows updates/docs?

Ref: #64

@QWp6t
Copy link
Member Author

QWp6t commented Jan 1, 2017

Yeah. In fact, I think roots/trellis#705 might have even made things worse in this regard insofar that there doesn't appear to be a cross-platform way to resolve local_path like there used to be. So this might need to be taken to the trellis repo since it could possibly be considered a regression.

@fullyint
Copy link
Contributor

Closing. Addressed in roots/trellis#725

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

No branches or pull requests

3 participants