PG headroom change for th2 #3600

wendani · 2019-10-14T19:34:38Z

Verified with sonic-net/sonic-mgmt#1156

Signed-off-by: Wenda Ni [email protected]

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Wenda Ni <[email protected]>

lguohan · 2019-10-14T20:17:56Z

can you change 40m as well?

Signed-off-by: Wenda Ni <[email protected]>

abdosi · 2021-06-09T00:12:16Z

@wendani do you remember what was the context of this change ? Why we increased headroom here ?

wendani · 2021-06-09T01:02:01Z

@wendani do you remember what was the context of this change ? Why we increased headroom here ?

I do not remember. Most likely come from rdma team recommendation/requirement. @baiwei0427

baiwei0427 · 2021-06-09T01:11:17Z

@wendani do you remember what was the context of this change ? Why we increased headroom here ?

I do not remember. Most likely come from rdma team recommendation/requirement. @baiwei0427

@wendani I did some calculations and found these numbers were too large.

…lly (#22378) #### Why I did it src/sonic-swss ``` * e0b2ac61 - (HEAD -> master, origin/master, origin/HEAD) swss: Align STP structures and remove GCC diagnostic pragmas (#3440) (3 hours ago) [Omar Tamer] * 120ed90a - [fgnhgorch] Fix incorrect state DB updates due to unordered field-val… (#3566) (3 hours ago) [InspurSDN] * 1552e340 - Move timestamps out of counter table to avoid update too frequently (#3585) (6 hours ago) [Stephen Sun] * 73e534c3 - Initialize the last fec ber computed values if not found (#3604) (27 hours ago) [Prince George] * 504daa19 - [az] Install vpp artifacts (#3609) (27 hours ago) [Kamil Cudnik] * bc750a5a - [orchagent] Add value validation for RouteOrch, NhgOrch and Srv6Orch (#3596) (4 days ago) [Yuqing Zhao(Alibaba Inc)] * d1fb9f7f - [DASH] Dash tunnel/PL NSG implementation (#3543) (4 days ago) [Lawrence Lee] * ffc865e6 - Fix a pipeline issue due to dependency in docker-sonic-vs (#3603) (4 days ago) [Dashuai Zhang] * f7ca83d2 - Fix the missed port status notifications issue (#3595) (7 days ago) [Vivek] * 5592fbe0 - [portsorch] pass oper status at hostif creation (#3563) (7 days ago) [Stepan Blyshchak] * 7bcd21c7 - [orchagent] Fix issue: typo in high BER (#3600) (8 days ago) [Junchao-Mellanox] * 3e1f8d8f - [portmgrd] regression: prevent runtime exception (crash) in configuring portchannel at boot (#3432) (8 days ago) [Brad House] * 492a85ae - [DASH] Make DASH vnet orch bulk logic consistent (#3554) (2 weeks ago) [Lawrence Lee] * 822ab913 - Handle outbound_direction_lookup for DASH_APPLIANCE_TABLE (#3571) (3 weeks ago) [prabhataravind] * fe9b89e4 - [DVS] Remove framework package during DVS build (#3577) (3 weeks ago) [Lawrence Lee] * 596d88c1 - Added Change for given Route ECMP to fallback on Default Route ECMP (#3389) (3 weeks ago) [abdosi] * d35c1c77 - Initialize number of counters to 0 before querying the capabilities (#3546) (5 weeks ago) [Stephen Sun] * 0e0685ae - Fix the switch level Voq counter to use switch_id Oid instead of counter name for PACKET_INTEGRITY counter (#3521) (5 weeks ago) [saksarav-nokia] * ee38b39e - [BFD] Change capability check to get. (#3559) (5 weeks ago) [dypet] * 45fdedb9 - Prevent lossless profile creation for 0m cable (#3548) (5 weeks ago) [Jianyue Wu] ``` #### How I did it #### How to verify it #### Description for the changelog

…lly (sonic-net#22378) #### Why I did it src/sonic-swss ``` * e0b2ac61 - (HEAD -> master, origin/master, origin/HEAD) swss: Align STP structures and remove GCC diagnostic pragmas (sonic-net#3440) (3 hours ago) [Omar Tamer] * 120ed90a - [fgnhgorch] Fix incorrect state DB updates due to unordered field-val… (sonic-net#3566) (3 hours ago) [InspurSDN] * 1552e340 - Move timestamps out of counter table to avoid update too frequently (sonic-net#3585) (6 hours ago) [Stephen Sun] * 73e534c3 - Initialize the last fec ber computed values if not found (sonic-net#3604) (27 hours ago) [Prince George] * 504daa19 - [az] Install vpp artifacts (sonic-net#3609) (27 hours ago) [Kamil Cudnik] * bc750a5a - [orchagent] Add value validation for RouteOrch, NhgOrch and Srv6Orch (sonic-net#3596) (4 days ago) [Yuqing Zhao(Alibaba Inc)] * d1fb9f7f - [DASH] Dash tunnel/PL NSG implementation (sonic-net#3543) (4 days ago) [Lawrence Lee] * ffc865e6 - Fix a pipeline issue due to dependency in docker-sonic-vs (sonic-net#3603) (4 days ago) [Dashuai Zhang] * f7ca83d2 - Fix the missed port status notifications issue (sonic-net#3595) (7 days ago) [Vivek] * 5592fbe0 - [portsorch] pass oper status at hostif creation (sonic-net#3563) (7 days ago) [Stepan Blyshchak] * 7bcd21c7 - [orchagent] Fix issue: typo in high BER (sonic-net#3600) (8 days ago) [Junchao-Mellanox] * 3e1f8d8f - [portmgrd] regression: prevent runtime exception (crash) in configuring portchannel at boot (sonic-net#3432) (8 days ago) [Brad House] * 492a85ae - [DASH] Make DASH vnet orch bulk logic consistent (sonic-net#3554) (2 weeks ago) [Lawrence Lee] * 822ab913 - Handle outbound_direction_lookup for DASH_APPLIANCE_TABLE (sonic-net#3571) (3 weeks ago) [prabhataravind] * fe9b89e4 - [DVS] Remove framework package during DVS build (sonic-net#3577) (3 weeks ago) [Lawrence Lee] * 596d88c1 - Added Change for given Route ECMP to fallback on Default Route ECMP (sonic-net#3389) (3 weeks ago) [abdosi] * d35c1c77 - Initialize number of counters to 0 before querying the capabilities (sonic-net#3546) (5 weeks ago) [Stephen Sun] * 0e0685ae - Fix the switch level Voq counter to use switch_id Oid instead of counter name for PACKET_INTEGRITY counter (sonic-net#3521) (5 weeks ago) [saksarav-nokia] * ee38b39e - [BFD] Change capability check to get. (sonic-net#3559) (5 weeks ago) [dypet] * 45fdedb9 - Prevent lossless profile creation for 0m cable (sonic-net#3548) (5 weeks ago) [Jianyue Wu] ``` #### How I did it #### How to verify it #### Description for the changelog

PG headroom change for th2

0832ae3

Signed-off-by: Wenda Ni <[email protected]>

wendani requested review from lguohan, 04diiguyi, yxieca, baiwei0427 and haseebniaz and removed request for 04diiguyi October 14, 2019 19:34

wendani added the Request for 201811 Branch label Oct 14, 2019

lguohan approved these changes Oct 15, 2019

View reviewed changes

Address comment: apply pg headroom change to 40m profile

0ba94e0

Signed-off-by: Wenda Ni <[email protected]>

wendani force-pushed the 7260_pg_master branch from e1dd85d to 0ba94e0 Compare October 15, 2019 02:30

wendani mentioned this pull request Oct 15, 2019

PG headroom change for th2 sonic-net/sonic-mgmt#1156

Merged

lguohan merged commit 0d2aa7f into sonic-net:master Oct 15, 2019

yxieca pushed a commit that referenced this pull request Oct 15, 2019

[devices]: PG headroom change for Arista 7260 (#3600)

241b6fa

Signed-off-by: Wenda Ni <[email protected]>

yxieca added the Included in 201811 Branch label Oct 15, 2019

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

PG headroom change for th2 #3600

PG headroom change for th2 #3600

wendani commented Oct 14, 2019 •

edited

Loading

lguohan commented Oct 14, 2019

abdosi commented Jun 9, 2021

wendani commented Jun 9, 2021

baiwei0427 commented Jun 9, 2021

PG headroom change for th2 #3600

PG headroom change for th2 #3600

Conversation

wendani commented Oct 14, 2019 • edited Loading

lguohan commented Oct 14, 2019

abdosi commented Jun 9, 2021

wendani commented Jun 9, 2021

baiwei0427 commented Jun 9, 2021

wendani commented Oct 14, 2019 •

edited

Loading