search

Oefenweb / ansible-postfix

Ansible role to set up postfix in Debian-like systems
MIT License
174 stars 83 forks source link

Make inet_interfaces configurable #13

Closed md5 closed 9 years ago

tersmitten commented 9 years ago

@md5 Can you explain the last commit?

md5 commented 9 years ago

@tersmitten When I run vagrant up, the Ansible provisioner fails since it can't find the postfix role. It's able to find the ansible-postfix role, however. I figured that change was fine since tests/test.yml also uses ansible-postfix (presumably for the same reason).

md5 commented 9 years ago

All this assumes that the user's checkout is called ansible-postfix.

tersmitten commented 9 years ago

@md5 For me it just work fine. We use this for all our roles. Do you have the latest version of Vagrant and Virtualbox?

tersmitten commented 9 years ago

All this assumes that the user's checkout is called ansible-postfix

That explains it. We always check it out as postfix. Can you please revert that particular change?

md5 commented 9 years ago

@tersmitten I've dropped that commit.

What do you think about doing something to make vagrant up work out of the box? As far as I can tell, the Vagrantfile is only meant for testing, so it would be cool if it could work without being reliant on the directory name of the cloned repository.

tersmitten commented 9 years ago

I will figure it out

md5 commented 9 years ago

Sounds good.

If you don't mind syncing a new tag to Ansible Galaxy once this is merged, that would be very helpful.

tersmitten commented 9 years ago

Done

md5 commented 9 years ago

Thanks!