Skip to content

Qos SAI test restructure#1682

Merged
neethajohn merged 2 commits intosonic-net:masterfrom
neethajohn:qos_restructure
May 20, 2020
Merged

Qos SAI test restructure#1682
neethajohn merged 2 commits intosonic-net:masterfrom
neethajohn:qos_restructure

Conversation

@neethajohn
Copy link
Contributor

@neethajohn neethajohn commented May 19, 2020 •
edited by lguohan
Loading

Signed-off-by: Neetha John [email protected]

Description of PR

Organize Qos SAI test vars based on asic type and include port speed and cable length definition. The test depended on pg headroom are specified under portspeed/cable length and all others are specified under the asic.

Type of change

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

How did you verify/test it?

Ran the test on Th and Th2 for one port speed /cable length

@neethajohn
Qos SAI test restructure
bca2914 
Signed-off-by: Neetha John <[email protected]>
stephenxs
stephenxs reviewed May 20, 2020
View reviewed changes
@neethajohn
Rename var
c1b429a 
Signed-off-by: Neetha John <[email protected]>
@neethajohn neethajohn mentioned this pull request May 20, 2020
Merged
3 tasks
@neethajohn neethajohn merged commit f17471c into sonic-net:master May 20, 2020
@neethajohn neethajohn deleted the qos_restructure branch May 20, 2020 22:54
neethajohn added a commit to neethajohn/sonic-mgmt that referenced this pull request Jun 2, 2020
@neethajohn
Qos SAI test restructure (sonic-net#1682)
867ee83 
* Qos SAI test restructure

Signed-off-by: Neetha John <[email protected]>
@neethajohn neethajohn mentioned this pull request Jun 2, 2020
Merged
yxieca pushed a commit that referenced this pull request Jun 6, 2020
@neethajohn
Qos SAI test restructure (#1682) (#1726)
253e225 
* Qos SAI test restructure

Signed-off-by: Neetha John <[email protected]>
shlomibitton pushed a commit to Junchao-Mellanox/sonic-mgmt that referenced this pull request Jul 12, 2020
Squashed commit of the following:
5e7f7a8 
commit d79d500
Author: Xin Wang <[email protected]>
Date:   Sat May 23 11:31:48 2020 +0800

    Fix acl testing for new t1 routes (sonic-net#1688)

    The t1 and t1-lag routes have been changed after replacing static
    routes on VM with exabgp announced routes. This caused the acl
    testing failed. This PR is to fix this issue reported in sonic-net#1603.

    Changes:
    1. Update the acl rules template to use new IP addresses
    2. Update the acl script to use new IP addresses

    Signed-off-by: Xin Wang <[email protected]>

commit c84e699
Author: yvolynets-mlnx <[email protected]>
Date:   Sat May 23 04:41:19 2020 +0300

    [pytest/ntp] Minor fix for NTP test case. (sonic-net#1698)

    Added verification of 'wait_until' result, to easier understand test case failure reason.

    Signed-off-by: Yuriy Volynets <[email protected]>

commit 441ddfa
Author: Vitaliy Senchyshyn <[email protected]>
Date:   Sat May 23 00:40:02 2020 +0300

    [wr_arp][pytest] Fixed wr_arp test (sonic-net#1695)

    * [wr_arp] Fixed wr_arp test

    Signed-off-by: Vitaliy Senchyshyn <[email protected]>

    * Make show ip protocol command working for different sonic images

    * Fixed review comments

    * Made records variable a class memeber

    * Fixed DIP taking from ip route show for sonic master

commit abdb746
Author: Xin Wang <[email protected]>
Date:   Fri May 22 18:45:56 2020 +0800

    [pytest/fib]: Fix the issue of using wrong port announcing IPv6 routes (sonic-net#1697)

    I used wrong port for announcing IPv6 routes in PR sonic-net#1692. This PR is to fix that issue.

    Signed-off-by: Xin Wang <[email protected]>

commit 9e52bba
Author: lguohan <[email protected]>
Date:   Fri May 22 03:45:14 2020 -0700

    [pytest/snmp_cpu]: query nproc when ansible facts does not have vcpu info (sonic-net#1693)

    sometimes, ansible facts does not gather vcpu information. in this case, use nproc to query the cpu number directly.

    Signed-off-by: Guohan Lu <[email protected]>

commit b175f8f
Author: yvolynets-mlnx <[email protected]>
Date:   Fri May 22 02:49:30 2020 +0300

    [pytest/test_drop_counters.py] Fixed IGMP test case (sonic-net#1624)

    Signed-off-by: Yuriy Volynets <[email protected]>

commit 208d02a
Author: Xin Wang <[email protected]>
Date:   Fri May 22 00:23:19 2020 +0800

    Announce vips routes for bgp_multipath_relax testing (sonic-net#1692)

    Previously the vips routes were statically configured in the VMs for
    bgp_multipath_relax testing. After the change of announcing
    routes from exabgp in PTF to VMs, the vips routes were missing.
    The bgp_multipath_relax testing depends on the vips routes failed.

    This change is to announce the vips routes to VMs  that have the
    vips routes configuration. The functions in the fib fixture definition
    file were refactored a little bit.

    Signed-off-by: Xin Wang <[email protected]>

commit 6d7498f
Author: yvolynets-mlnx <[email protected]>
Date:   Thu May 21 11:38:08 2020 +0300

    Fixed fib tests: test_hash_ipv4/test_hash_ipv6 (sonic-net#1683)

    Fixes sonic-net#1679

    Signed-off-by: Yuriy Volynets <[email protected]>

commit f17471c
Author: Neetha John <[email protected]>
Date:   Wed May 20 15:53:44 2020 -0700

    Qos SAI test restructure (sonic-net#1682)

    * Qos SAI test restructure

    Signed-off-by: Neetha John <[email protected]>

commit 87f3a23
Author: Joe LeVeque <[email protected]>
Date:   Wed May 20 10:42:22 2020 -0700

    [pytest][cacl] Remove unused 'testbed_devices' from parameters (sonic-net#1689)

commit bd6e135
Author: Ying Xie <[email protected]>
Date:   Wed May 20 08:11:55 2020 -0700

    [reload config] redirect reload config output to /dev/null (sonic-net#1687)

    config reload command will restart interface-config service, which
    will reset eth0. Which will reset the connection issues config
    reload if the output is coming to stdio/stderr. It in turn could
    cause the pytest to stuck for long time.

    Signed-off-by: Ying Xie <[email protected]>

Signed-off-by: Stepan Blyschak <[email protected]>
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
@yxieca
[202012][swss][utilities] advance submodule heads (sonic-net#7165)
7dc0ae5 
utilities:
* 83f068b 2021-03-22 | Handling error scenario of adding port to Vlan which is part of LAG (sonic-net#1516) (HEAD -> 202012) [Sudharsan Dhamal Gopalarathnam]
* 470e8ce 2021-03-24 | Enable PFCWD only on ports where PFC is enabled (sonic-net#1508) [Andriy Yurkiv]
* 09ef2e0 2021-03-22 | [show][config] add support for setting and displaying switching modes on Y cable (sonic-net#1501) [vdahiya12]
* 0d17d37 2021-03-24 | Warmboot script improvements - timeout exec, disable swss autorestart, remove trap (sonic-net#1495) [Vaibhav Hemant Dixit]
* 2718cd8 2021-03-24 | [show] Fix int status of LAGs, configured as Vlan members (sonic-net#1478) [maksymbelei95]
* cc168fb 2021-03-22 | Fix bug: show vlan config for vlan with no members (sonic-net#1503) [allas-nvidia]

swss:
* 5d8d1fb 2021-03-26 | Revert "Revert "[buffermgr] Support maximum port headroom checking (sonic-net#1607)" (sonic-net#1675)" (sonic-net#1682) (HEAD -> 202012) [Prince Sunny]
* f8df1f8 2021-03-26 | [Dynamic Buffer Calc] Enhance the field checking in table handling (sonic-net#1680) [Stephen Sun]
* 6328c9f 2021-03-22 | [MuxOrch] FDB ageout safety check (sonic-net#1674) [Prince Sunny]
* e1d733e 2021-03-21 | reduce severity of log to info in case of flush on non-existing member (sonic-net#1669) [allas-nvidia]

Signed-off-by: Ying Xie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@stephenxs stephenxs stephenxs approved these changes

@yxieca yxieca yxieca approved these changes

@lguohan lguohan Awaiting requested review from lguohan

Assignees

No one assigned

Labels

Enhancement

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@neethajohn @stephenxs @yxieca