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

Add support for loadbalancer member without ports #120

Merged
merged 2 commits into from
Jan 23, 2015

Conversation

ericlaflamme
Copy link

No description provided.

@hunner
Copy link

hunner commented Aug 21, 2014

It looks like http://cbonte.github.io/haproxy-dconv/configuration-1.5.html#bind shows the port being requried. Why would you not want to have a port? Does that even work?

@ericlaflamme
Copy link
Author

The thing is that if you bind multiple ports to listen to and you specified a port for the backend, all ports will redirect to the port specified. If you don't specified any port for the backend, haproxy will send the packet to the port it received it.

Frontend:
frontend something
bind x.x.x.x:25
bind x.x.x.x:110
bind x.x.x.x:135
bind x.x.x.x:139
bind x.x.x.x:587
bind x.x.x.x:5666

backend
server server1 x.x.x.x weight 1 check port 80 inter 2000 rise 5 fall 2

@hunner
Copy link

hunner commented Nov 12, 2014

Oh I see; interesting. Could you update the readme section also, then? https://github.com/puppetlabs/puppetlabs-haproxy#ports Thanks!

@ericlaflamme
Copy link
Author

I have updated the readme.md file, let me know what you think. Thanks :)

@underscorgan
Copy link

@eric-iweb could you squash this down to a single commit? thanks!

underscorgan pushed a commit that referenced this pull request Jan 23, 2015
Add support for loadbalancer member without ports
@underscorgan underscorgan merged commit de12d29 into puppetlabs:master Jan 23, 2015
@underscorgan
Copy link

Thanks @eric-iweb

cegeka-jenkins pushed a commit to cegeka/puppet-haproxy that referenced this pull request Dec 19, 2017
Add support for loadbalancer member without ports
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants