Skip to content

[VRF] Modify some errors for test_bgp_with_loopback in test_vrf.py#1445

Merged
lguohan merged 1 commit intosonic-net:masterfrom
Minkang-Tsai:master
Mar 18, 2020
Merged

[VRF] Modify some errors for test_bgp_with_loopback in test_vrf.py#1445
lguohan merged 1 commit intosonic-net:masterfrom
Minkang-Tsai:master

Conversation

@Minkang-Tsai
Copy link
Copy Markdown
Contributor

@Minkang-Tsai Minkang-Tsai commented Mar 13, 2020

Description of PR

  1. Access the asn parameter from /etc/sonic/constants.yml instead of /etc/sonic/deployment_id_asn_map.yml.
  2. Don't remove bgp associated with default VRF when there are bgp associated with non default VRF. This is FRR's restriction(FRRouting/frr@dd5868c).

Type of change

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

How did you verify/test it?

Run test_bgp_with_loopback again.

============================= test session starts ==============================
platform linux2 -- Python 2.7.12, pytest-4.6.7, py-1.8.0, pluggy-0.13.1 -- /usr/bin/python
cachedir: .pytest_cache
ansible: 2.8.7
rootdir: /var/sonic/sonic-mgmt/tests, inifile: pytest.ini
plugins: ansible-2.2.2
collecting ... collected 1 item

test_vrf_bgp.py::TestVrfLoopbackIntf::test_bgp_with_loopback PASSED      [100%]

=============================== warnings summary ===============================
test_vrf_bgp.py::TestVrfLoopbackIntf::test_bgp_with_loopback
  /usr/local/lib/python2.7/dist-packages/pytest_ansible/module_dispatcher/v28.py:70: UserWarning: provided hosts list is empty, only localhost is available
    warnings.warn("provided hosts list is empty, only localhost is available")

-- Docs: https://docs.pytest.org/en/latest/warnings.html
==================== 1 passed, 1 warnings in 211.35 seconds ====================

Description of PR
1. Access the asn parameter from /etc/sonic/constants.yml instead of /etc/sonic/deployment_id_asn_map.yml.
2. Don't remove bgp associated with default VRF when there are bgp associated with non default VRF. This is
   FRR's restriction.

Type of change
 Bug fix
[] Testbed and Framework(new/improvement)
[] Test case(new/improvement)

How did you verify/test it?
Run test_bgp_with_loopback again.
@lguohan
Copy link
Copy Markdown
Contributor

lguohan commented Mar 13, 2020

can you post test results?

@Minkang-Tsai
Copy link
Copy Markdown
Contributor Author

@lguohan I had updated.

Comment thread tests/test_vrf.py
@lguohan lguohan merged commit 007ab24 into sonic-net:master Mar 18, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
…or 202012 (sonic-net#7105)

sonic-swss
* [nbrmgrd] added function to parse IP address from APP_DB (sonic-net#1672)
* [MUX/PFCWD] Use in_ports for acls instead of seperate ACL table (sonic-net#1670)
* [mux] VS test for neigh, route and fdb (sonic-net#1656)
* [Dynamic buffer calc] Bug fix: Remove PGs from an administratively down port. (sonic-net#1652)
* spell check fixes (sonic-net#1630)

sonic-utilities
* [reboot]: Stop mux before reboot on dual ToR (sonic-net#1500)
* [config] Disable/enable container monitoring when stopping/starting services (sonic-net#1499)
* Add 'show' and 'clear' command for PG drop (sonic-net#1461)
* [CLI][techsupport] Add NOOP option for commands that did not have that option (sonic-net#1445)
* [202012][reload] Improve reload by using sonic.target (sonic-net#1509)

Signed-off-by: Danny Allen <daall@microsoft.com>
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.

3 participants