Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix links of debian git server #1755

Merged
merged 1 commit into from
Jun 1, 2018

Conversation

qiluo-msft
Copy link
Collaborator

@qiluo-msft qiluo-msft commented May 31, 2018

Will update this PR ASAP.~~~

The personal repo is removed in this PR: https://github.com/Azure/sonic-buildimage/pull/1795

Signed-off-by: Qi Luo <qiluo-msft@users.noreply.github.com>
@qiluo-msft qiluo-msft merged commit a6d2d77 into sonic-net:master Jun 1, 2018
@qiluo-msft qiluo-msft deleted the qiluo/fix-debian-git branch June 1, 2018 01:38
@jleveque jleveque mentioned this pull request Jun 1, 2018
lguohan pushed a commit that referenced this pull request Jun 1, 2018
yurypm added a commit to yurypm/sonic-buildimage that referenced this pull request Jun 4, 2018
Porting of sonic-net#1755 to stretch branch
cytsai0409 pushed a commit to cytsai0409/sonic-buildimage that referenced this pull request Jun 8, 2018
lguohan pushed a commit that referenced this pull request Jun 9, 2018
* Fix links of debian git server (#1755)

* [dockers] Prevent apt-get from installing suggested and recommended packages by default (#1666)

* [cfggen]: Fix build by fixing pyangbind version (#1633)

* [Ingrasys] Add platform support for S9280-64X with Barefoot ASIC

* Revert "[Ingrasys] Add platform support for S9280-64X with Barefoot ASIC"

This reverts commit 29b9d90.

Separate S9280-64X platform driver files into a new PR
zhenggen-xu pushed a commit to zhenggen-xu/sonic-buildimage that referenced this pull request Oct 17, 2019
volodymyrsamotiy added a commit to volodymyrsamotiy/sonic-buildimage that referenced this pull request May 28, 2021
dad1287 [tunneldecaporch] Set default MTU for the overlay loopback interface (sonic-net#1756)
1bc94d1 [orchagent] Fix typo in PortsOrch::initPortSupportedSpeeds (sonic-net#1755)
a44e651 [nhg]: Add support for weight in nexthop group member. (sonic-net#1752)
5c625b2 [Bulk mode] Limit the size of bulker (sonic-net#1744)
d1cd0fd Fix error msg due to not supported "SAI_SWITCH_ATTR_MAX_NUMBER_OF_TEMP_SENSORS"  attributes (sonic-net#1745)
278770d [sub intf] Fix kernel side processing to enslave sub interface to non-default vrf (sonic-net#1521)
031f536 support flush FDB entries per port and per vlan (sonic-net#1064)
3629d70 [sonic-swss] Add port auto negotiation support to swss (sonic-net#1714)
7c6ebb1 [fix] Use the same storm detection condition for queue occupancy non-zero case as the zero case (sonic-net#1111)
fb06c32 [fabricportsorch] Add fabric support (sonic-net#1459)

Signed-off-by: Volodymyr Samotiy <volodymyrs@nvidia.com>
liat-grozovik pushed a commit that referenced this pull request May 31, 2021
dad1287 [tunneldecaporch] Set default MTU for the overlay loopback interface (#1756)
1bc94d1 [orchagent] Fix typo in PortsOrch::initPortSupportedSpeeds (#1755)
a44e651 [nhg]: Add support for weight in nexthop group member. (#1752)
5c625b2 [Bulk mode] Limit the size of bulker (#1744)
d1cd0fd Fix error msg due to not supported "SAI_SWITCH_ATTR_MAX_NUMBER_OF_TEMP_SENSORS"  attributes (#1745)
278770d [sub intf] Fix kernel side processing to enslave sub interface to non-default vrf (#1521)
031f536 support flush FDB entries per port and per vlan (#1064)
3629d70 [sonic-swss] Add port auto negotiation support to swss (#1714)
7c6ebb1 [fix] Use the same storm detection condition for queue occupancy non-zero case as the zero case (#1111)
fb06c32 [fabricportsorch] Add fabric support (#1459)

Signed-off-by: Volodymyr Samotiy <volodymyrs@nvidia.com>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
dad1287 [tunneldecaporch] Set default MTU for the overlay loopback interface (sonic-net#1756)
1bc94d1 [orchagent] Fix typo in PortsOrch::initPortSupportedSpeeds (sonic-net#1755)
a44e651 [nhg]: Add support for weight in nexthop group member. (sonic-net#1752)
5c625b2 [Bulk mode] Limit the size of bulker (sonic-net#1744)
d1cd0fd Fix error msg due to not supported "SAI_SWITCH_ATTR_MAX_NUMBER_OF_TEMP_SENSORS"  attributes (sonic-net#1745)
278770d [sub intf] Fix kernel side processing to enslave sub interface to non-default vrf (sonic-net#1521)
031f536 support flush FDB entries per port and per vlan (sonic-net#1064)
3629d70 [sonic-swss] Add port auto negotiation support to swss (sonic-net#1714)
7c6ebb1 [fix] Use the same storm detection condition for queue occupancy non-zero case as the zero case (sonic-net#1111)
fb06c32 [fabricportsorch] Add fabric support (sonic-net#1459)

Signed-off-by: Volodymyr Samotiy <volodymyrs@nvidia.com>
judyjoseph added a commit that referenced this pull request Sep 2, 2021
sonic-swss

73f6f68 [Flex Counters] Delay flex counters even if tables are present in the DB (#1877)
5edb9e5 [buffer orch] Bugfix: Don't query counter SAI_BUFFER_POOL_STAT_XOFF_ROOM_WATERMARK_BYTES on a pool where it is not supported (#1857)
fce0c60 [crm] Fix for Issue #8036 (#1829)

sonic-utilities

2630ac1 [Fast-reboot] Set flex counters delay indicator to prevent flex counters enablement after fast-reboot (#1768)
606f1b1 [portstat pfcstat] Unify the packet number format in the output of portstat and pfcstat in all cases (#1755)
2c6a15e [ecnconfig] Fix exception seen during display and add unit tests (#1784)
9b1995e Fix logic in RIF counters print (#1732)

sonic-swss-comon

3e7b81f Add a new field for FLEX_COUNTER_TABLE to indicate delay for flex counters (#523)
lguohan pushed a commit that referenced this pull request Sep 9, 2021
To include sonic_installer swap setup changes

171eb4f [sonic_installer] Add swap setup support (#1787)
6483b0b QOS fieldvalue refernce ABNF format to string (#1626)
8d16eb5 [Fast-reboot] Set flex counters delay indicator to prevent flex counters enablement after fast-reboot (#1768)
1bc24ca [portstat pfcstat] Unify the packet number format in the output of portstat and pfcstat in all cases (#1755)
d7900dd [config][generic-update] Implementing patch sorting (2nd attempt) (#1794)
7ed9a91 [show] Allow system with no ports in config db run without errors (#1754)

Signed-off-by: Longxiang Lyu <lolv@microsoft.com>
theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022
taras-keryk pushed a commit to taras-keryk/sonic-buildimage that referenced this pull request Apr 28, 2022
…rtstat and pfcstat in all cases (sonic-net#1755)

#### What I did

Unify the packet number format in the output of "portstat" and "pfcstat" for all cases.  
In some cases, the packet numbers in the output of these two commands are formatted with a comma, in some cases not. 
This is because only when the numbers are treated by function `ns_diff`, it will format the numbers with commas:  `return '{:,}'.format(max(0, new - old))` , but `ns_diff` is not called in all cases.

**for example, packet numbers in the output are NOT formatted with commas:**

```
pfcstat

root@r-qa-sw-eth-2133:/home/admin# pfcstat
    Port Rx    PFC0    PFC1    PFC2    PFC3    PFC4    PFC5    PFC6    PFC7
-----------  ------  ------  ------  ------  ------  ------  ------  ------
Ethernet120       0       0       0       0       0       0       0       0
Ethernet124  137407       0   45659   45660       0   45662       0       0 <-----FORMAT OF DATA BEFORE clear counters

portstat
portstat
      IFACE    STATE     RX_OK    RX_BPS    RX_UTIL    RX_ERR    RX_DRP    RX_OVR     TX_OK    TX_BPS    TX_UTIL    TX_ERR    TX_DRP    TX_OVR
-----------  -------  --------  --------  ---------  --------  --------  --------  --------  --------  ---------  --------  --------  --------
  Ethernet0        X         0       N/A        N/A         0         0         0         0       N/A        N/A         0         0         0
  Ethernet4        U  12804864       N/A        N/A         0         0         0         9       N/A        N/A         0         0         0
  Ethernet8        X         0       N/A        N/A         0         0         0         0       N/A        N/A         0         0         0
Ethernet120        U         1       N/A        N/A         0         0         0        11       N/A        N/A         0         0         0
Ethernet124        U         1       N/A        N/A         0         0         0  12363470       N/A        N/A         0    429517         0

```
**packet numbers in the output are formatted with commas:**

```
pfcstat
Last cached time was 2021-08-02 10:35:32.725158
    Port Rx    PFC0    PFC1    PFC2    PFC3    PFC4    PFC5    PFC6    PFC7
-----------  ------  ------  ------  ------  ------  ------  ------  ------
Ethernet120       0       0       0       0       0       0       0       0
Ethernet124  25,007       0  25,006  25,006       0  25,007       0       0 <-----DIFFERENT FORMAT AFTER COUNTER CLEAR COMMAND

portstat
Last cached time was 2021-08-02 10:35:47.829677
      IFACE    STATE      RX_OK        RX_BPS    RX_UTIL    RX_ERR    RX_DRP    RX_OVR      TX_OK       TX_BPS    TX_UTIL    TX_ERR    TX_DRP    TX_OVR
-----------  -------  ---------  ------------  ---------  --------  --------  --------  ---------  -----------  ---------  --------  --------  --------
  Ethernet0        X          0      0.00 B/s      0.00%         0         0         0          0     0.00 B/s      0.00%         0         0         0
  Ethernet4        U  1,570,775   862.60 MB/s     69.01%         0         0         0          0   28.29 KB/s      0.00%         0         0         0
Ethernet120        U          0      0.00 B/s      0.00%         0         0         0          0     0.00 B/s      0.00%         0         0         0
Ethernet124        U          0  1406.45 KB/s      0.11%         0         0         0  1,488,765  817.56 MB/s     65.40%         0    83,040         0 <----DIFFERENT PRESENTATION FORMAT OF NUMBERS
```

#### How I did it

Add a new function `format_number_with_comma` to format the packet numbers with comma, this function will be called in case `ns_diff` is not applicable.

Update the unitest to cover this new change.

#### How to verify it

execute portstat and pfcstat to check the output whether the number format is expected. 

#### Previous command output (if the output of a command-line utility has changed)

```
pfcstat

root@r-qa-sw-eth-2133:/home/admin# pfcstat
    Port Rx    PFC0    PFC1    PFC2    PFC3    PFC4    PFC5    PFC6    PFC7
-----------  ------  ------  ------  ------  ------  ------  ------  ------
Ethernet116       0       0       0       0       0       0       0       0
Ethernet120       0       0       0       0       0       0       0       0
Ethernet124  137407       0   45659   45660       0   45662       0       0 

portstat
      IFACE    STATE     RX_OK    RX_BPS    RX_UTIL    RX_ERR    RX_DRP    RX_OVR     TX_OK    TX_BPS    TX_UTIL    TX_ERR    TX_DRP    TX_OVR
-----------  -------  --------  --------  ---------  --------  --------  --------  --------  --------  ---------  --------  --------  --------
  Ethernet0        X         0       N/A        N/A         0         0         0         0       N/A        N/A         0         0         0
  Ethernet4        U    804864       N/A        N/A         0         0         0         9       N/A        N/A         0         0         0
Ethernet120        U         1       N/A        N/A         0         0         0        11       N/A        N/A         0         0         0
Ethernet124        U         1       N/A        N/A         0         0         0    363470       N/A        N/A         0    429517         0

```

#### New command output (if the output of a command-line utility has changed)

```
pfcstat
root@r-qa-sw-eth-2133:/home/admin# pfcstat
    Port Rx    PFC0    PFC1    PFC2    PFC3    PFC4    PFC5    PFC6    PFC7
-----------  ------  ------  ------  ------  ------  ------  ------  ------
Ethernet116       0       0       0       0       0       0       0       0
Ethernet120       0       0       0       0       0       0       0       0
Ethernet124 137,407       0  45,659  45,660       0  45,662       0       0 

portstat
Last cached time was 2021-08-02 10:35:47.829677
      IFACE    STATE     RX_OK    RX_BPS    RX_UTIL    RX_ERR    RX_DRP    RX_OVR     TX_OK    TX_BPS    TX_UTIL    TX_ERR    TX_DRP    TX_OVR
-----------  -------  --------  --------  ---------  --------  --------  --------  --------  --------  ---------  --------  --------  --------
  Ethernet0       X         0       N/A        N/A         0         0         0         0       N/A        N/A         0         0         0
  Ethernet4       U   804,864       N/A        N/A         0         0         0         9       N/A        N/A         0         0         0
Ethernet120       U         1       N/A        N/A         0         0         0        11       N/A        N/A         0         0         0
Ethernet124       U         1       N/A        N/A         0         0         0   363,470       N/A        N/A         0   429,517         0
```
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