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

Support for dual protocol #132

Merged
merged 4 commits into from
Jun 1, 2016
Merged

Conversation

dave-burke
Copy link
Contributor

@dave-burke dave-burke commented Jun 1, 2016

Let me know if there's anything you'd like me to tweak. The test works for me locally after changing the value of $IMG to my local image and skipping the iptables config, but I didn't run it as part of the full suite on Travic CI (not sure if I can? I've never used Travis before). [edit: I see Travis CI ran when I submitted the pull request. Sweet!]
#85

Generated arguments will be added only if matching arguments were not
specified by the user. User arguments will be placed after generated
arguments. This allows the user to override any generated configuration
values.
@kylemanna
Copy link
Owner

Hmm, for some reason it doesn't appear that the other tests are running. Perhaps something with my travis-ci config or test script is broken. I'll have to investigate this.

@kylemanna kylemanna self-assigned this Jun 1, 2016
@kylemanna kylemanna merged commit a5b9ade into kylemanna:master Jun 1, 2016
@kylemanna
Copy link
Owner

Fixed travis-ci, everything looks great now.

 ____________               ___________
< it worked! >             < both ways! >
 ------------               ------------
        \   ^__^        ^__^   /
     \  (oo)\______/(oo)  /
        (__)\      /(__)
                ||w---w||
                ||     ||

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants