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.

Sign up for GitHub

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

Jump to bottom

Bind the service on all available interface #60

Closed
olbat opened this issue Sep 28, 2015 · 5 comments
Closed

Bind the service on all available interface #60

olbat opened this issue Sep 28, 2015 · 5 comments
Labels
enhancement New feature or request

Comments

@olbat
Copy link

olbat commented Sep 28, 2015

ATM it's not possible to disable the bind parameter of the configuration file, it's also not possible to specify multiple binding address using the recipe. So if you want your server to listen to every available network interfaces, it's not possible ...
@jarshwah
Copy link

+1 to this. With later versions of redis you can specify multiple bind => "ipddr1 ipddr2".

@siliconalchemy
Copy link

You can do this to listen on all interfaces:
bind => '0.0.0.0'

@olbat
Copy link
Author

olbat commented Jan 9, 2016

Okay, but that does not really solve the problem ...
Imagine that I have 4 network interfaces on my machine and I only want the service to be exposed on two of them.

@siliconalchemy
Copy link

Well it solves the issue you describe and the title of the issue, but yes it doesn't solve binding multiple selective interfaces, which would be a good enhancement.

@petems
Copy link
Member

petems commented Nov 30, 2016

Ok, should be easy enough to implement, simply change the $bind variable to an array, loop through the array for multiple bindings 👍

@petems petems added the enhancement New feature or request label Nov 30, 2016
petems added a commit to petems/puppet-redis that referenced this issue Jul 6, 2017
@petems
Allows $bind parameter to be set as an array
068f92c 
* Allows setting bind to multiple IP addresses
* Closes voxpupuli#60
alexjfisher added a commit to alexjfisher/puppet-redis that referenced this issue Mar 11, 2019
@alexjfisher
Support binding to all interfaces
c60550d 
`bind` now takes an array (of IP addresses).
To bind to all interfaces, use `bind => []`.

Due to the new parameter validation, if you were previously using a
single string to specify multiple IP addresses, you will need to change
this to an array.

Fixes voxpupuli#257
Fixes voxpupuli#60
@alexjfisher alexjfisher mentioned this issue Mar 11, 2019
Merged
alexjfisher added a commit to alexjfisher/puppet-redis that referenced this issue Mar 11, 2019
@alexjfisher
Support binding to all interfaces
fafe534 
`bind` now takes an array (of IP addresses).
To bind to all interfaces, use `bind => []`.

Due to the new parameter validation, if you were previously using a
single string to specify multiple IP addresses, you will need to change
this to an array.

Fixes voxpupuli#257
Fixes voxpupuli#60
alexjfisher added a commit to alexjfisher/puppet-redis that referenced this issue Mar 11, 2019
@alexjfisher
Support binding to all interfaces
9cb74b0 
`bind` now takes an array (of IP addresses).
To bind to all interfaces, use `bind => []`.

Due to the new parameter validation, if you were previously using a
single string to specify multiple IP addresses, you will need to change
this to an array.

Fixes voxpupuli#257
Fixes voxpupuli#60
alexjfisher added a commit to alexjfisher/puppet-redis that referenced this issue Mar 12, 2019
@alexjfisher
Support binding to all interfaces
d94f060 
`bind` now takes an array (of IP addresses).
To bind to all interfaces, use `bind => []`.

Due to the new parameter validation, if you were previously using a
single string to specify multiple IP addresses, you will need to change
this to an array.

Fixes voxpupuli#257
Fixes voxpupuli#60
cegeka-jenkins pushed a commit to cegeka/puppet-redis that referenced this issue Feb 16, 2021
@alexjfisher
Support binding to all interfaces
c370a0c 
`bind` now takes an array (of IP addresses).
To bind to all interfaces, use `bind => []`.

Due to the new parameter validation, if you were previously using a
single string to specify multiple IP addresses, you will need to change
this to an array.

Fixes voxpupuli#257
Fixes voxpupuli#60
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@olbat @jarshwah @petems @siliconalchemy