Skip to content

Commit

Permalink
updated scale number and removed traffic check for failed route due to (
Browse files Browse the repository at this point in the history
openconfig#2640)

OTG issue openconfig#2639
Also added ARP check to make sure ARP is resolved before Traffic is
sent.

"This code is a Contribution to the OpenConfig Feature Profiles project ("Work") made under the Google Software Grant and Corporate Contributor License Agreement ("CLA") and governed by the Apache License 2.0. No other rights or licenses in or to any of Nokia's intellectual property are granted for any other purpose. This code is provided on an "as is" basis without any warranties of any kind."
  • Loading branch information
bkreddy143 authored and frasieroh committed Feb 21, 2024
1 parent 16920b0 commit 212b726
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ import (
"github.com/openconfig/featureprofiles/internal/deviations"
"github.com/openconfig/featureprofiles/internal/fptest"
"github.com/openconfig/featureprofiles/internal/gribi"
"github.com/openconfig/featureprofiles/internal/otgutils"
"github.com/openconfig/gribigo/fluent"
"github.com/openconfig/ondatra"
"github.com/openconfig/ondatra/gnmi"
Expand Down Expand Up @@ -69,7 +70,7 @@ var (
vendorSpecRoutecount = map[ondatra.Vendor]uint32{
ondatra.ARISTA: 2500000,
ondatra.JUNIPER: 2500000,
ondatra.NOKIA: 1600000,
ondatra.NOKIA: 2600000,
}
dutPort1 = attrs.Attributes{
Desc: "dutPort1",
Expand Down Expand Up @@ -310,7 +311,6 @@ func sendTraffic(t *testing.T, args *testArgs) {
v4 := flow1ipv4.Packet().Add().Ipv4()
v4.Src().SetValue(atePort1.IPv4)
v4.Dst().Increment().SetStart(fibPassedDstRoute)

flow2ipv4 := args.otgConfig.Flows().Add().SetName("Flow2")
flow2ipv4.Metrics().SetEnable(true)
flow2ipv4.TxRx().Device().
Expand All @@ -326,19 +326,21 @@ func sendTraffic(t *testing.T, args *testArgs) {
v4Flow2.Dst().Increment().SetStart(fibFailedDstRoute)

args.otg.PushConfig(t, args.otgConfig)
time.Sleep(2 * time.Minute)
args.otg.StartProtocols(t)

otgutils.WaitForARP(t, args.ate.OTG(), args.otg.FetchConfig(t), "IPv4")
t.Logf("Starting traffic")
args.otg.StartTraffic(t)
time.Sleep(15 * time.Second)
t.Logf("Stop traffic")
args.otg.StopTraffic(t)

verifyTraffic(t, args, flow1ipv4.Name(), !wantLoss)

if !deviations.GRIBISkipFIBFailedTrafficForwardingCheck(args.dut) {
verifyTraffic(t, args, flow2ipv4.Name(), wantLoss)
}
/*
if !deviations.GRIBISkipFIBFailedTrafficForwardingCheck(args.dut) {
verifyTraffic(t, args, flow2ipv4.Name(), wantLoss)
}
*/
}

func verifyTraffic(t *testing.T, args *testArgs, flowName string, wantLoss bool) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ platform_exceptions: {
vendor: NOKIA
}
deviations: {
explicit_gribi_under_network_instance: true
explicit_port_speed: true
explicit_interface_in_default_vrf: true
interface_enabled: true
Expand Down

0 comments on commit 212b726

Please sign in to comment.