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.

Sign up for GitHub

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

Jump to bottom

TestTraceflow/testTraceflowEgress/egressFromRemoteNode failing consistently on dual-stack testbed #4502

Closed
antoninbas opened this issue Dec 20, 2022 · 1 comment · Fixed by #4529
Closed

TestTraceflow/testTraceflowEgress/egressFromRemoteNode failing consistently on dual-stack testbed #4502

antoninbas opened this issue Dec 20, 2022 · 1 comment · Fixed by #4529
Assignees
@Atish-iaf
Labels
area/ops/traceflow Issues or PRs related to the Traceflow feature area/test/e2e Issues or PRs related to Antrea specific end-to-end testing. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test.

Comments

@antoninbas
Copy link
Contributor

Describe the bug
The TestTraceflow/testTraceflowEgress/egressFromRemoteNode e2e test is failing consistently on the dual-stack testbed.

=== RUN   TestTraceflow/testTraceflowEgress/egressFromRemoteNode
    traceflow_test.go:2252: Latest Traceflow status: {Running  2022-12-20 21:44:51 +0000 UTC 7 [{antrea-ipv6-9-1  1671572693 [{SpoofGuard  Forwarded     0    } {Egress  ForwardedToEgressNode    egress-wgkqd 0    } {Forwarding Output Forwarded     0    }]}] <nil>}
    traceflow_test.go:2439: Error: Get Traceflow failed: timed out waiting for the condition

The test was recently added as part of #3926
@antoninbas antoninbas added the kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test. label Dec 20, 2022
@antoninbas
Copy link
Contributor Author

Assigning @Atish-iaf who contributed the test

@antoninbas antoninbas added area/test/e2e Issues or PRs related to Antrea specific end-to-end testing. area/ops/traceflow Issues or PRs related to the Traceflow feature labels Dec 20, 2022
@antoninbas antoninbas mentioned this issue Dec 20, 2022
Merged
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Jan 3, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
05777e5 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6 on dual-stack testbed.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
@Atish-iaf Atish-iaf mentioned this issue Jan 3, 2023
Merged
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Jan 6, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
d361404 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6 on dual-stack testbed.

Add verification of tunnelDstIP in observation traceflow e2e tests.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Jan 7, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
41bf4f9 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6 on dual-stack testbed.

Add verification of tunnelDstIP in observation of traceflow e2e tests.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Feb 6, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
6131384 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Add verification of tunnelDstIP in observation of traceflow e2e tests.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
@Atish-iaf Atish-iaf mentioned this issue Feb 17, 2023
Closed
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Mar 2, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
c616b76 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Add verification of tunnelDstIP in observation of traceflow e2e tests.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Mar 2, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
ef262de 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Add verification of tunnelDstIP in observation of traceflow e2e tests.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Mar 2, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
94c2b34 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
Atish-iaf added a commit to Atish-iaf/antrea that referenced this issue Mar 3, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack
2d51095 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
@tnqn tnqn closed this as completed in #4529 Mar 3, 2023
tnqn pushed a commit that referenced this issue Mar 3, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack (#4529)
9c95110 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Fixes #4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
GraysonWu pushed a commit to GraysonWu/antrea that referenced this issue Mar 3, 2023
@Atish-iaf @GraysonWu
Fix tunnelDstIP field in traceflow on dual-stack (antrea-io#4529)
583f62f 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
luolanzone pushed a commit to luolanzone/antrea that referenced this issue Apr 11, 2023
@Atish-iaf @luolanzone
Fix tunnelDstIP field in traceflow on dual-stack
a67aaf8 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
tnqn pushed a commit that referenced this issue Apr 11, 2023
@luolanzone
Fix tunnelDstIP field in traceflow on dual-stack (#4840)
cbe3a8b 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Fixes #4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
jainpulkit22 pushed a commit to urharshitha/antrea that referenced this issue Apr 28, 2023
@Atish-iaf
Fix tunnelDstIP field in traceflow on dual-stack (antrea-io#4529)
9debb37 
On dual-stack testbed, Inter-Node traceflow observation for
IPv4 packet in encap mode has empty tunnelDstIP field because
isIPv6 variable is set.

Change the logic to set isIPv6 variable based on the packet
IPv4 or IPv6.

Fixes antrea-io#4502

Signed-off-by: Kumar Atish <atish.iaf@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Atish-iaf Atish-iaf

Labels
area/ops/traceflow Issues or PRs related to the Traceflow feature area/test/e2e Issues or PRs related to Antrea specific end-to-end testing. kind/failing-test Categorizes issue or PR as related to a consistently or frequently failing test.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix tunnelDstIP field in traceflow on dual-stack Atish-iaf/antrea
2 participants
@antoninbas @Atish-iaf