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

update from azure 201811 #156

Merged
merged 139 commits into from
Apr 30, 2020
Merged

update from azure 201811 #156

merged 139 commits into from
Apr 30, 2020

Conversation

bbinxie
Copy link
Collaborator

@bbinxie bbinxie commented Apr 30, 2020

- 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)

yxieca and others added 30 commits October 18, 2019 16:43
Submodule src/sonic-utilities d315dd7..0cfa942:
  > [neighbor_advertiser]: Adapt to different mirror ACL table names (#703)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
…RESENT" for power off PSU on 201811 (#3598)

* [plugins/psuutil]Align psuutil of 2010 with that of 2100.

* [plugins/psuutil.py]Address comments via using more human-readable code
…adow (#3652)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Submodule src/sonic-swss 2529d79..15652b2:
  > [mirrororch]: Add retry logic when deleting referenced mirror session (#1104)

Submodule src/sonic-utilities 0cfa942..c049e54:
  > [neighbor_advertiser]: Add sleep in setting mirror session and ACL rules (#714)
  > [warm/fast reboot] continue executing when killing docker failed (#713)
  > [ecnconfig] Validate input WRED parameters (#579)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
…3664)

* [device/celestica]: libsensors configuration file for Seastone-DX010

* [device/celestica]: Add hysteresis config for lm75b sensors on Seastone-DX010
- What I did
This fix removes the possibility of 'localhost' entry getting removed from /etc/hosts file by hostname-config service.

Without this change, whenever we change the hostname from 'localhost' to any other name on the config_db.json and reload the config, /etc/hosts file will only have the new hostname on it. But there are multiple sonic utilities (eg: swssconfig) which relies on the hard coded 'localhost' name and they tend to stop working.

- How I did it
Added a new check on hostname-config.sh script to avid blindly deleting the line containing the old hostname from /etc/hosts file. Now it will delete the old hostname only if its not localhost or when the hostname is not changing.

- How to verify it

Bring up SONiC on a device with hostname as localhost
Edit /etc/sonic/config_db.json to update the 'hostname' filed under DEVICE_METADATA from "hostname" : "localhost" --> "hostname" : "sonic"
run config reload -y to reflect the hostname change done on config_db.json file.
cat /etc/hosts and check whether both 127.0.0.1 localhost and 127.0.0.1 sonic entry are present on the file.
ping localhost should work fine.
- Description for the changelog
Make hostname-config service more robust in handling SONiC hostname change from localhost to anything else.
We noticed in tests/production that there is a low probability failure
where /etc/hosts could have some garbage characters before the entry for
local host name. The consequence is that all sudo command would be very
slow. In extreme cases it would prevent some services from starting
properly.

I suspect that the /etc/hosts file might be opened by some process causing
the issue. Editing contents with new file level and replace the whole file
should be safer.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Submodule src/sonic-swss 15652b2..98cfe56:
  > Check RIF/Port exists only for add entries (#1110)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Submodule src/sonic-swss 98cfe56..f09ddb4:
  > [fix] Use the same storm detection condition for queue occupancy non-zero case as the zero case (#1111)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
interfaces-config service configures lo address. If bgp service
starts before lo address is configured, then following config
in zebra will not be applied.

route-map RM_SET_SRC permit 10
 set src 10.1.0.32

The adds a few seconds delay in bgp service start
…it has no reachable tags (#2506)

Signed-off-by: Joe LeVeque <jolevequ@microsoft.com>
Submodule src/sonic-swss f09ddb4..49c9c16:
  > Allow buffer profile apply after init (#1099)
  > [aclorch]: Check for existing mirror table only when creating a new table (#1089)
  > [201811] [portsorch] fix PortsOrch::allPortsReady() returns true when it should not (#1116)
  > Address review comment: remove data member m_entriesCreated, which is introduced for dependancy resolution purpose. (#839)
  > Fix PFC watchdog not getting lossless TC (#876)

Submodule src/sonic-utilities c049e54..2ca1ae1:
  > Add a generic configlet application script (#716)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Submodule src/sonic-swss 49c9c16..1bc989a:
  > Fix pfc watchdog compile error (#1119)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
…ports for mirror tables (#3592) (#3634)

* Update minigraph.py to filter out front-panel ports that are not active
* Update cfggen tests to reflect new behavior

Signed-off-by: Danny Allen <daall@microsoft.com>

* Incorporate PR comments
- Update t0 tests to include additional device neighbors
- Refactor xml parsing logic
* BGPm for 201811 (#3601)

* Feature is downported

* Add monitors to the test minigraphs

* Test

* No pfx filer

* Fix bgp sample

* Quagga requires to activate peer-group before configuration

* Add bgpcfgd and bgpd.peer template

* Catch exception if rendering external template

* Fix tests
* [minigraph.py]: Use default namespace for <Address>
Signed-off-by: Neetha John <nejo@microsoft.com>
Submodule src/sonic-utilities 2ca1ae1..3ed25a4:
  > Do not start pfcwd for M0 devices (#726)
  > Make configlet application script idempotent for updates. (#728)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Revert "Configure buffer profile to all ports (#3561)" (#3628)
Configure buffer profile to all ports (#3561)

This reverts commit 8861cbe.

Signed-off-by: Wenda Ni <wenni@microsoft.com>
This change is intended to fix the issue with dpkg-query during build
process.

The symptom is dpkg-query failed to open package info file, usually
/var/lib/dpkg/updates/000?

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
yxieca and others added 22 commits March 19, 2020 16:08
We believe that the supervisord issue in face of clock rolling backwards
has been addressed. Therefore reverting change 2598 to allow ntp sync
to right clock at the start up time.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
* Add restart configuration of fancontrol for pmon.

* Clean up the default value setting for exitcodes

* Remove the default setting of stopwaitsecs
Submodule src/sonic-utilities e9747899a..f431510ae:
  > [201811][intfutil] set speed to 0 when interface speed is not available (#840)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
Signed-off-by: Wenda Ni <wenni@microsoft.com>
This new FW version includes the following fixes:

SFP thermal shutdown issue

Signed-off-by: Volodymyr Samotiy <volodymyrs@mellanox.com>
admin@sonic:~$ sudo hw-management-wd.sh
Usage: hw-management-wd.sh start [timeout] | stop | tleft | check_reset | help
start - start watchdog
        timeout is optional. Default value will be used in case if it's omitted
        timeout provided in seconds
stop - stop watchdog
tleft - check watchdog timeout left
check_reset - check if previous reset was caused by watchdog
        Prints only in case of watchdog reset
help -this help

Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
New CPLD includes support for watchdog type 3 with maximum timeout 65536 sec.

Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
… VLAN interfaces (#4229)

- Support parsing egress ACLs from minigraph file specified by the "OutAcl" element
- Support attaching ACLs to VLAN interfaces
- What I did
Add configuration to avoid ntpd from panic and exit if the drift between new time and current system time is large.

- How I did it
Added "tinker panic 0" in ntp.conf file.

- How to verify it
[this assumes that there is a valid NTP server IP in config_db/ntp.conf]

Change the current system time to a bad time with a large drift from time in ntp server; drift should be greater than 1000s.
Reboot the device.
Before the fix:
3. upon reboot, ntp-config service comes up fine, ntp service goes to active(exited) state without any error message. This is because the offset between new time (from ntp server) and the current system time is very large, ntpd goes to panic mode and exits. The system continues to show the bad time.

After the fix:
3. Upon reboot, ntp-config comes up fine, ntp services comes up from and stays in active (running) state. The system clock gets synced with the ntp server time.
Submodule src/sonic-utilities f431510ae..d7e8f84cf:
Fix issue of fields overwritten before display (#863)

Signed-off-by: Guohan Lu <lguohan@gmail.com>
Co-authored-by: Guohan Lu <lguohan@gmail.com>
* DellEMC: S6100 CPLD upgrade

* DellEMC: S6100 CPLD upgrade - Retry on failure
* fixes an issue when /host/warmboot/issu_bank.txt is empty/corrupted
switch is not able to over come this and enters continuos reload/reboot
failure.

Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
* [mellanox]: Add SSD FW update tool.

Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>

* [mellanox]: Update SSD tool.

Signed-off-by: Nazarii Hnydyn <nazariig@mellanox.com>
Signed-off-by: Stepan Blyschak <stepanb@mellanox.com>
)

* Include platform info in name.
Get SONiC Version as parameter and use
Make additional tag as optional.
Avoid repetitions by using function.

* Per review comments, make SONIC_VERSION optional and added some comments.

* 1) Added additional params are optional
2) Handle DOCKER_IMAGE_TAG only if given
3) Use BUILD_NUMBER only if SONIC_VERSION not given
4) Tag with SONIC_VERSION if given.

Current behavior is not changed, unless SONIC_VERSION is given.

* Update per review comments
1) Added new args with options
2) Handle PORT possible being empty
3) Exhibit new behavior only if both version & platform are given.

* Drop redundant quotes
Signed-off-by: Danny Allen <daall@microsoft.com>
* [dhcpmon] Filter DHCP O/A Messages of Neighboring Vlans

This code fixes a bug where two or more vlans exist. Cross contamination
happens for DHCP packets Offer/Ack when received on shared northbound links.
The code filters out those packet based on dst IP equal Vlan loopback IP.

signed-off-by: Tamer Ahmed <tamer.ahmed@microsoft.com>
Submodule src/sonic-utilities d7e8f84cf..8c21fc151:
  > [utility] Filter FDB entries (#890)
  > Fix the warm-reboot script to support FRR based warm-reboot (#842)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@bbinxie bbinxie requested a review from tiantianlv April 30, 2020 03:00
@bbinxie bbinxie merged commit e63070a into SONIC-DEV:201811 Apr 30, 2020
mudsut4ke pushed a commit that referenced this pull request Jan 25, 2021
Update sonic-snmpagent submodule to include below commits:
1a2b62a [Namespace]: Fix SAI_ID key used in cpfcIfTable and csqIfQosGroupStatsTable implementation (#138)
d06f00c [pytest/coverage]: add coverage support (#156)
90e9f2e [Namespace]: Simplify sync_d functions to use higher order (#154)
b5815d9 [LLDP]: Modify OID index of LLDPRemTableUpdater MIB (#155)
d5f2b92 [Multiasic]: Provide namespace support for ipNetToMediaPhysAddress (#129)
166c221 [Namespace]: Fix interface counters in RFC 1213 (#145)

Signed-off-by: SuvarnaMeenakshi <sumeenak@microsoft.com>
mudsut4ke pushed a commit that referenced this pull request Jan 25, 2021
…nic-net#6352)

src/sonic-platform-common 9935fca...8664efc (2):

Make sonic_sfp Python2 and Python3 compatible (#157)
[sffbase.py] Fix to make Python 3-compatible (#156)

src/sonic-platform-daemons e6c786b...81318f7 (1):

[psud] Fix issue where PSU Fan info is not updated in State DB (#137)

Fixes sonic-net#6341
mudsut4ke pushed a commit that referenced this pull request May 11, 2021
eff5c1c [thermalctld] No need exit thermalcltd when loading invalid policy file (#172)
5b6d9c0 [syseepromd] Add unit tests; Refactor to allow for greater unit test coverage (#156)
jerseyang pushed a commit that referenced this pull request Jun 4, 2021
…nic-net#6352)

src/sonic-platform-common 9935fca...8664efc (2):

Make sonic_sfp Python2 and Python3 compatible (#157)
[sffbase.py] Fix to make Python 3-compatible (#156)

src/sonic-platform-daemons e6c786b...81318f7 (1):

[psud] Fix issue where PSU Fan info is not updated in State DB (#137)

Fixes sonic-net#6341
jerseyang pushed a commit that referenced this pull request Jun 4, 2021
eff5c1c [thermalctld] No need exit thermalcltd when loading invalid policy file (#172)
5b6d9c0 [syseepromd] Add unit tests; Refactor to allow for greater unit test coverage (#156)
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.