Skip to content

VS: Check oper_status of the link being brought up instead of Ethernet0#522

Merged
lguohan merged 1 commit intosonic-net:masterfrom
jipanyang:test_nhg
Jun 13, 2018
Merged

VS: Check oper_status of the link being brought up instead of Ethernet0#522
lguohan merged 1 commit intosonic-net:masterfrom
jipanyang:test_nhg

Conversation

@jipanyang
Copy link
Contributor

Signed-off-by: Jipan Yang [email protected]

What I did
Minor fix to the link oper_status check.

Why I did it

How I verified it

jipan@sonic-build:~/igbpatch/vs/sonic-buildimage/src/sonic-swss/tests$ sudo pytest -v --dvsname=vs test_nhg.py
=========================================================================================== test session starts ============================================================================================
platform linux2 -- Python 2.7.12, pytest-3.3.0, py-1.5.2, pluggy-0.6.0 -- /usr/bin/python
cachedir: .cache
rootdir: /home/jipan/igbpatch/vs/sonic-buildimage/src/sonic-swss/tests, inifile:
collected 1 item

test_nhg.py::test_route_nhg PASSED [100%]

======================================================================================== 1 passed in 26.45 seconds =========================================================================================

@lguohan lguohan merged commit ad61242 into sonic-net:master Jun 13, 2018
@jipanyang jipanyang deleted the test_nhg branch February 9, 2019 02:31
EdenGri pushed a commit to EdenGri/sonic-swss that referenced this pull request Feb 28, 2022
…c-net#522)

Default deny rule shall not be applied for neither MIRROR nor MIRRORV6

Signed-off-by: Shu0T1an ChenG <[email protected]>
oleksandrivantsiv pushed a commit to oleksandrivantsiv/sonic-swss that referenced this pull request Mar 1, 2023
Janetxxx pushed a commit to Janetxxx/sonic-swss that referenced this pull request Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants