-
Notifications
You must be signed in to change notification settings - Fork 82
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
new raw blocks: listen, frontend, backend #652
Conversation
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.
Thank you for your contribution!
Can you please add some tests? Then we can run it through our CI to see if everything works as expected.
And please drop the merge commit, we prefer a linear history :) |
I've added some tests. @maxmoehl Could you please check my code? PS. I'm already using this in my dev https://github.com/kinjelom/haproxy-boshrelease/releases/tag/v13.3.1%2B2.8.9 |
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.
Approving for CI
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
Unit-tests and acceptance-test are green
I went ahead and cleaned up the history. @b1tamara please approve again, I'll enable auto-merge to get it in. |
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
thank you 👍 |
Proper formatting of indented multiline configurations, new raw blocks: listen, frontend, backend.