Skip to content

[tests/config]: Add protection to test_config.py for adapt different topo structure#1525

Merged
lguohan merged 1 commit intosonic-net:masterfrom
Blueve:dev/jika/config
Apr 2, 2020
Merged

[tests/config]: Add protection to test_config.py for adapt different topo structure#1525
lguohan merged 1 commit intosonic-net:masterfrom
Blueve:dev/jika/config

Conversation

@Blueve
Copy link
Copy Markdown
Collaborator

@Blueve Blueve commented Apr 2, 2020

  • [tests/config]: Add protection to test_config.py for adapt different topo structure

Description of PR

Summary:
The test will simply been skipped if there are no portchannel or portchannel has no member

Type of change

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

Approach

How did you do it?

Check portchannel and portchannel member before execute test

How did you verify/test it?

Run test locally and it pass/skip

Any platform specific information?

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

Documentation

…topo structure

- The test will simply been skipped if there are no portchannel or portchannel has no member
@lguohan lguohan merged commit 0b1c96a into sonic-net:master Apr 2, 2020
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
* d58ba82 2021-04-05 | Fix fast-reboot fdb filtering for IPv6 addresses (sonic-net#1548) (github/201811) [Vaibhav Hemant Dixit]
* cc7ad9e 2021-03-24 | Fast reboot: check if the files are present before accessing (sonic-net#1526) [Vaibhav Hemant Dixit]
* 410323a 2021-03-24 | [201811] Add missing variable DEVPATH to fast-reboot script (sonic-net#1525) [Vaibhav Hemant Dixit]
* eb5023a 2021-03-21 | [201811] soft-reboot (sonic-net#1487) [Sujin Kang]
* 3f10540 2020-12-08 | [fast-reboot] Fix fast-reboot when NDP entries are present (sonic-net#1295) (HEAD -> 201811) [shlomibitton]

Signed-off-by: Ying Xie <ying.xie@microsoft.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.

2 participants