Skip to content

[snmp_facts.py]: do not query memory oid for eos switch#1473

Merged
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:snmpfa
Mar 20, 2020
Merged

[snmp_facts.py]: do not query memory oid for eos switch#1473
lguohan merged 1 commit intosonic-net:masterfrom
lguohan:snmpfa

Conversation

@lguohan
Copy link
Contributor

@lguohan lguohan commented Mar 20, 2020

eos switch does not support the two memory oids

Signed-off-by: Guohan Lu gulv@microsoft.com

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • [] Testbed and Framework(new/improvement)
  • [] Test case(new/improvement)

Approach

How did you do it?

How did you verify/test it?

============================= test session starts ==============================
platform linux2 -- Python 2.7.12, pytest-4.6.9, py-1.8.1, pluggy-0.13.1 -- /usr/bin/python
  chedir: .pytest_cache
ansible: 2.8.7
rootdir: /data/tests, inifile: pytest.ini
plugins: ansible-2.2.2
collecting ... collected 1 item

test_nbr_health.py::test_neighbors_health FAILED                         [100%]
================================================== test session starts ==================================================
platform linux2 -- Python 2.7.12, pytest-4.6.9, py-1.8.1, pluggy-0.13.1 -- /usr/bin/python
cachedir: .pytest_cache
ansible: 2.8.7
rootdir: /data/tests, inifile: pytest.ini
plugins: ansible-2.2.2
collected 7 items                                                                                                       

snmp/test_snmp_cpu.py::test_snmp_cpu PASSED                                                                       [ 14%]
snmp/test_snmp_interfaces.py::test_snmp_interfaces PASSED                                                         [ 28%]
snmp/test_snmp_lldp.py::test_snmp_lldp PASSED                                                                     [ 42%]
snmp/test_snmp_pfc_counters.py::test_snmp_pfc_counters PASSED                                                     [ 57%]

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

eos switch does not support the two memory oids

Signed-off-by: Guohan Lu <gulv@microsoft.com>
@lguohan lguohan merged commit a9e9ac4 into sonic-net:master Mar 20, 2020
@lguohan lguohan deleted the snmpfa branch March 20, 2020 09:59
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
d03c6ccc90b92d9319c500a8adcf727a9fa5609b (HEAD -> 201911, origin/201911) [intfsorch] Init proxy_arp variable while adding router interface. (sonic-net#1473)
9e7c0bc8e3f2c7c5422f2f8a2c6498f659dcdf84 [drop counters] Clarify log messages for initial counter setup (sonic-net#1445)
da8ac754fa1c36f9bb7ba1210017f915f339cfe0 Create vnet tunnel map only if it doesn't exist (sonic-net#1482)

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Including the following changes:
[bitmap_vnet] Remove BMTOR implementation (sonic-net#1496)
[intfsorch] Init proxy_arp variable while adding router interface. (sonic-net#1473)
[drop counters] Clarify log messages for initial counter setup (sonic-net#1445)

Signed-off-by: Liat Grozovik <liatg@nvidia.com>
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.

1 participant