-
-
Notifications
You must be signed in to change notification settings - Fork 25
Allow setting a custom instance name in Lima #515
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
Conversation
70054fd to
47fa754
Compare
|
Thanks for the pull request @dalepgrant . Hope @swalkinshaw can have a look see one of these days. Would really help getting macOS VMs to work with multiple sites on one virtual machine. |
|
@dalepgrant for testing this change how should I do this ? Local Here is what I did: So it started the wrong vm and then did a So does not seem to work for me. P.S. This is my Update: I decided to create the config file: Delete existing VMs (if needed) Start with your custom binary BUT, when I tried that it seemed to ignore the and not just use the existing vm |
|
Sanity check: This file should have something like: |
|
As discussed on discord I got it working using # Trellis CLI example config file
# https://roots.io/trellis/docs/cli/#configuration
#
# database_app: sequel-ace
# open:
# admin: "https://mysite.com/wp/wp-admin"
vm:
instance_name: "imagewize.com"Works really well. |
47fa754 to
abf7259
Compare
abf7259 to
d75de75
Compare
|
Thank you 🚀 |
|
@swalkinshaw thanks for the merger & @dalepgrant thanks again for the pull request! et voilà! |
Relates to #514
A WIP but I've run out of time today, hopefully will pick this back up this week. Pushing up as I'm not even sure this is the right approach or wanted functionality, any feedback welcomed.Edit: Have been using this locally for a couple of weeks, haven't hit any issues. Opening this up for review & approval after a nudge in Discourse from @jasperf.