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 IPFIX fixes from Asumu to 2017.08 release #1211

Merged
merged 11 commits into from Aug 27, 2017

Conversation

Projects
None yet
3 participants
@wingo
Contributor

wingo commented Aug 18, 2017

This branch adds fixes for #1201 and #1208.

takikawa and others added some commits Aug 1, 2017

Fix pcap output for ipfix probe
In/out apps need to be properly separated for that to work
Fix app configuration in ipfix probe
When ARP is used (non-pcap cases), the output app should also
feedback into the arp app so that arp replies are actually
accepted.
Add an ipfix test script that uses a collector
Tests the ipfix probe program with some pcap input
with nfcapd from the nfdump suite. Checks to make sure
the expected flows are received by the collector.
Merge pull request #1201 from takikawa/ipfix-nfcapd-test
Add an integration test for ipfix probe using an ipfix collector
Improve ipfix collector test
Runs the test with both v9 and v10 modes. Better error
checking and minor refactoring as well.
@lukego

This comment has been minimized.

Show comment
Hide comment
@lukego

lukego Aug 18, 2017

Member

Merging as soon as Hydra has a munch. Good to have proper coverage on wingo-next now :).

Member

lukego commented Aug 18, 2017

Merging as soon as Hydra has a munch. Good to have proper coverage on wingo-next now :).

@lukego lukego self-assigned this Aug 18, 2017

@lukego lukego merged commit dad3438 into next Aug 27, 2017

1 check passed

SnabbDoc Documentation as single HTML file
Details

lukego added a commit that referenced this pull request Aug 27, 2017

@lukego

This comment has been minimized.

Show comment
Hide comment
@lukego

lukego Aug 27, 2017

Member

Groovalicious! Merged. Sorry about the wait.

Member

lukego commented Aug 27, 2017

Groovalicious! Merged. Sorry about the wait.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment