Skip to content

Commit

Permalink
Modified TC5 final port up check as timing was causing flaky results (#…
Browse files Browse the repository at this point in the history
…3662)

* modifed TC5 final port up check as timing was causing flaky results

* Update README.md

---------

Co-authored-by: Pramod Maurya <[email protected]>
  • Loading branch information
mastarkey and self-maurya authored Jan 6, 2025
1 parent 2d5bb89 commit 835c4de
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 13 deletions.
26 changes: 14 additions & 12 deletions feature/interface/holdtime/otg_tests/holdtime_test/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -49,21 +49,23 @@ Verify oper-state behaviour
* oper-status last change time has NOT changed
* Stop sending Ethernet Remote Fault (RF) from OTG port-1

## Config Parameter Coverage
## OpenConfig Path and RPC Coverage

* /interfaces/interface/hold-time/config/up
* /interfaces/interface/hold-time/config/down
The below yaml defines the OC paths and RPC intended to be covered by this test.

## Telemetry Parameter Coverage
```yaml
paths:
/interfaces/interface/hold-time/config/up:
/interfaces/interface/hold-time/config/down:
/interfaces/interface/state/oper-status:
/interfaces/interface/state/last-change:

* /interfaces/interface/hold-time/config/up
* /interfaces/interface/hold-time/config/down
* /interfaces/interface/state/oper-status
* /interfaces/interface/state/last-change

## Protocol/RPC Parameter Coverage

None
rpcs:
gnmi:
gNMI.Get:
gNMI.Set:
gNMI.Subscribe:
```
## Minimum DUT Platform Requirement
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -610,7 +610,7 @@ func TestTC5ShortDOWN(t *testing.T) {

t.Run("Verify port status UP", func(t *testing.T) {
t.Log("re-verify that the interface state is still up")
verifyPortsStatus(t, dut, "UP", 10)
verifyPortsStatus(t, dut, "UP", 30)

})
}

0 comments on commit 835c4de

Please sign in to comment.