-
Notifications
You must be signed in to change notification settings - Fork 300
Conversation
Can one of the admins verify this patch? |
I am really looking forward to getting this merged. Any ETA on this issue, please? |
I haven't had a chance to run it through any sort of testing other than the small integration test suite @mikejennings did above. I don't expect any issues but there may be some subtle template changes that will need to be documented. If ya'll have successfully run this with a real world workload, please do comment here that it worked without issue for you, if several people run it without issue I'm more inclined to merge it before the next release. Otherwise it may be a bit before I get some time to setup some test cases and run them, probably after the next release (which will include the latest 1.8.x). |
We run it in production on a Marathon cluster + HAProxy 1.9 with around 80 apps without issues. |
We can confirm that HAProxy versions 1.9.5 & 1.9.6 work fine with Marathon-lb master commit id 5ebb622. This works with GRPC if you setup as per HAProxy documentation. DC/OS version: 1.10.8 Steps to achieve this:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for reporting everyone. I bumped it to 1.9.6 just so we get the latest 1.9.x build in the next release.
Test output of 1.9.6:
|
updated, tests passed.