-
Notifications
You must be signed in to change notification settings - Fork 334
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 support for multiple flow monitoring protocols and for IPv6 endpoint in existing tests #2592
Conversation
e245ea2
to
3dc4efe
Compare
3dc4efe
to
28c7017
Compare
/retest |
cbba2c9
to
01f76cf
Compare
I think we will soon need to actually run the ipv6 control-plane tests :) Some small nits + questions but otherwise /lgtm |
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.
minor nits, otherwise lgtm. Thanks for working on this.
bf44987
to
e4962b8
Compare
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.
@ricky-rav can you remove the merge commit? You should be able to just rebase your branch to master and add your commit on top.
714fe7c
to
adf3db7
Compare
/retest |
…int in existing netflow test - Existing netflow export test now supports netflow v5, ipfix, sflow. - The original test only validated the export of netflow v5 (legacy netflow) data, which does not support IPv6 by design. In order to validate flow data export in an IPv6 cluster, we can now choose between IPFIX and sflow; also, the test now supports an IPv6 endpoint; - For the time being we're only testing netflow v5 in IPv4 (as before) and sflow in IPv6, because: (1) The test itself is long (~5 minutes); (2) The current implementation of the flow collector container (cloudfare/goflow) doesn't support IPFIX data generated by OVS (cloudflare/goflow#99) Signed-off-by: Riccardo Ravaioli <rravaiol@redhat.com>
adf3db7
to
af04455
Compare
(1) The test itself is long (~5 minutes);
(2) The current implementation of the flow collector container (cloudfare/goflow) doesn't support IPFIX data generated by OVS (Openvswitch ovs-vsctl connection cloudflare/goflow#99)
This patch addresses https://issues.redhat.com/browse/SDN-2059
- How to verify it