Skip to content
This repository has been archived by the owner on Dec 20, 2023. It is now read-only.

add redir requirement to docs #185

Merged
merged 3 commits into from
Feb 5, 2018
Merged

add redir requirement to docs #185

merged 3 commits into from
Feb 5, 2018

Conversation

vdloo
Copy link
Member

@vdloo vdloo commented Feb 5, 2018

extra dependency needed after #182

extra requirement needed after #182
@andreas-bytenl
Copy link
Contributor

Is this because we added host_ip: "0.0.0.0"? Is that part needed? I know I personally wouldn't want my Vagrant boxes to listen on all interfaces of my host machine.

@vdloo
Copy link
Member Author

vdloo commented Feb 5, 2018

It is because previously no portforwarding was done for the lxc provider at all. Now the default behavior is to forward 80 and 3306, but yes good point I can imagine that binding on all ipv4 interfaces might not be desirable. But to disable it you could add default_ports: false to your local.xml to disable it like before. I'll add that to the docs.

in case you do not want to bind the default forwarded ports on 0.0.0.0
@vdloo
Copy link
Member Author

vdloo commented Feb 5, 2018

by the way, this was the default behavior before for virtualbox already:

host_ip (string) - The IP on the host you want to bind the forwarded port to. If not specified, it will be bound to every IP. By default, this is empty.

https://www.vagrantup.com/docs/networking/forwarded_ports.html

@@ -14,6 +14,8 @@ If you encounter any issues relating to `vagrant-lxc` and not `hypernode-vagrant
pacman -S rsync dnsmasq git lxc vagrant --needed --noconfirm
```

You will also need to install [redir](https://linux.die.net/man/1/redir). Alternatively you could disable [port forwarding](https://www.vagrantup.com/docs/networking/forwarded_ports.html) by adding `default_ports: false` to your `local.xml`.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

xml 🙈

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😬

@vdloo vdloo merged commit 17c9b69 into master Feb 5, 2018
@vdloo vdloo deleted the add-redir-to-docs branch February 5, 2018 12:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants