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

Enable (re-)running vagrant provision #246

Merged
merged 1 commit into from Mar 12, 2016

Conversation

@aneeshusa
Copy link
Member

aneeshusa commented Mar 12, 2016

Vagrant will pass the -C flag to the bootstrap script when Salt is
already installed (and only needs to be configured). Implement this
flag for our bootstrap script so that running vagrant provision after
vagrant up, or vagrant provision repeatedly, works correctly.


This change is Review on Reviewable

Vagrant will pass the -C flag to the bootstrap script when Salt is
already installed (and only needs to be configured). Implement this
flag for our bootstrap script so that running vagrant provision after
vagrant up, or vagrant provision repeatedly, works correctly.
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Mar 12, 2016

@bors-servo r+

Thanks! I had been wondering why vagrant provision didn't work :-)

@bors-servo
Copy link
Contributor

bors-servo commented Mar 12, 2016

📌 Commit 45ddf1b has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Mar 12, 2016

Testing commit 45ddf1b with merge fe4cfba...

bors-servo added a commit that referenced this pull request Mar 12, 2016
Enable (re-)running vagrant provision

Vagrant will pass the -C flag to the bootstrap script when Salt is
already installed (and only needs to be configured). Implement this
flag for our bootstrap script so that running vagrant provision after
vagrant up, or vagrant provision repeatedly, works correctly.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/246)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 12, 2016

☀️ Test successful - travis

@bors-servo bors-servo merged commit 45ddf1b into servo:master Mar 12, 2016
1 of 2 checks passed
1 of 2 checks passed
homu Testing commit 45ddf1b with merge fe4cfba...
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@aneeshusa
Copy link
Member Author

aneeshusa commented Mar 12, 2016

I updated the wiki to mention this as a useful command.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.