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

net/haproxy: release 2.8 #772

Merged
merged 3 commits into from
Aug 24, 2018
Merged

net/haproxy: release 2.8 #772

merged 3 commits into from
Aug 24, 2018

Conversation

fraenki
Copy link
Member

@fraenki fraenki commented Aug 2, 2018

New features

Bugfixes

Enhancements

  • use "reload" instead of "restart" RC action
  • if "reload" fails, also issue a "restart" command (required when enabling seamless reloads)
  • start progress animation (spinner) earlier when applying settings

@fraenki fraenki added the feature Adding new functionality label Aug 2, 2018
@fraenki fraenki self-assigned this Aug 2, 2018
@fraenki
Copy link
Member Author

fraenki commented Aug 2, 2018

With seamless reloads enabled

$ ab -r -c 10 -n100000 http://opnsense-haproxy-test/
Concurrency Level: 10
Time taken for tests: 25.575 seconds
Complete requests: 100000
Failed requests: 0

Without seamless reloads

$ ab -r -c 10 -n100000 http://opnsense-haproxy-test/
Concurrency Level: 10
Time taken for tests: 25.719 seconds
Complete requests: 100000
Failed requests: 20

@fraenki
Copy link
Member Author

fraenki commented Aug 2, 2018

I've submitted a patch for the FreeBSD port, because the seamless reload feature requires a small modification to the RC script:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=230305

@fraenki fraenki requested a review from fichtner August 2, 2018 20:26
@fraenki fraenki changed the title net/haproxy: release 2.8 [WIP] net/haproxy: release 2.8 Aug 2, 2018
Copy link
Member

@fichtner fichtner left a comment

Choose a reason for hiding this comment

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

This requires HAProxy 1.8.13? Is the FreeBSD modification already included or do we need to pull this into our tree as well?

@fraenki
Copy link
Member Author

fraenki commented Aug 3, 2018

@fichtner, HAProxy 1.8.13 is recommended because it fixes a related bug. The FreeBSD modifications are not yet included, we need to pull this into our tree or wait until it gets committed upstream.

@fichtner
Copy link
Member

fichtner commented Aug 3, 2018 via email

@fraenki fraenki force-pushed the haproxy_224 branch 2 times, most recently from a803b48 to fc8353f Compare August 6, 2018 21:27
@fraenki fraenki changed the title [WIP] net/haproxy: release 2.8 net/haproxy: release 2.8 Aug 6, 2018
@fraenki fraenki added the upstream Third party issue label Aug 16, 2018
@fraenki
Copy link
Member Author

fraenki commented Aug 24, 2018

@fichtner The upstream patch was commited in the FreeBSD ports tree. I'll merge this now, please pull in the required pieces in the ports repo too. :)

@fraenki fraenki merged commit ae6955b into opnsense:master Aug 24, 2018
@fichtner
Copy link
Member

Ack. Will do when I’m back in September. Thanks for the FreeBSD submission 😊

@fichtner
Copy link
Member

fichtner commented Sep 3, 2018

FYI 697c98b8 -- we're all good so 2.8 will be in 18.7.2. Thanks!

@fraenki fraenki deleted the haproxy_224 branch September 4, 2018 13:39
@ma-karai
Copy link

are there plans to import existing haproxy configs?

@fraenki
Copy link
Member Author

fraenki commented Sep 10, 2018

@bott0r Please open a new issue for unrelated requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Adding new functionality upstream Third party issue
Development

Successfully merging this pull request may close these issues.

3 participants