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

[hostcfgd] [202012] Fixed the brief blackout in hostcfgd using SubscriberStateTable #19

Closed
wants to merge 5 commits into from

Conversation

vivekrnv
Copy link
Owner

@vivekrnv vivekrnv commented Nov 5, 2021

Why I did it

Backporting https://github.com/Azure/sonic-buildimage/pull/8861 to 202012

How I did it

How to verify it

tests/determine-reboot-cause_test.py .........                                                                                                                       [ 34%]
tests/procdockerstatsd_test.py .                                                                                                                                     [ 38%]
tests/caclmgrd/caclmgrd_dhcp_test.py ......                                                                                                                          [ 61%]
tests/hostcfgd/hostcfgd_test.py ..........                                                                                                                           [100%]
Nov  5 19:37:26.999907 r-lionfish-16 INFO /hostcfgd: ConfigDB connect success
Nov  5 19:37:27.000092 r-lionfish-16 INFO /hostcfgd: KdumpCfg init...
Nov  5 19:37:27.026653 r-lionfish-16 INFO /hostcfgd: Waiting for systemctl to finish initialization
Nov  5 19:37:27.042503 r-lionfish-16 INFO /hostcfgd: systemctl has finished initialization -- proceeding ...
Nov  5 19:37:27.093703 r-lionfish-16 INFO /hostcfgd: Kdump handler...
Nov  5 19:37:27.093924 r-lionfish-16 INFO /hostcfgd: Kdump global configuration update
Nov  5 19:37:27.799234 r-lionfish-16 INFO hostcfgd[234919]: kdump is already disabled
Nov  5 19:37:27.799324 r-lionfish-16 INFO hostcfgd[234919]: Kdump is already disabled
Nov  5 19:37:28.212487 r-lionfish-16 INFO hostcfgd[234961]: Kdump configuration has been updated in the startup configuration
Nov  5 19:37:28.659445 r-lionfish-16 INFO hostcfgd[234976]: Kdump configuration has been updated in the startup configuration
Nov  5 19:37:28.691903 r-lionfish-16 INFO /hostcfgd: Feature swss is enabled and started
Nov  5 19:37:28.711435 r-lionfish-16 INFO /hostcfgd: Feature lldp is enabled and started
Nov  5 19:37:28.730893 r-lionfish-16 INFO /hostcfgd: Running cmd: 'sudo systemctl unmask sflow.service'
Nov  5 19:37:28.754107 r-lionfish-16 INFO hostcfgd[235002]: Removed /etc/systemd/system/sflow.service.
Nov  5 19:37:29.180023 r-lionfish-16 INFO /hostcfgd: Running cmd: 'sudo systemctl enable sflow.service'
Nov  5 19:37:29.193978 r-lionfish-16 INFO hostcfgd[235021]: Created symlink /etc/systemd/system/sonic.target.wants/sflow.service → /lib/systemd/system/sflow.service.
Nov  5 19:37:29.621868 r-lionfish-16 INFO /hostcfgd: Running cmd: 'sudo systemctl start sflow.service'
Nov  5 19:37:30.100161 r-lionfish-16 INFO /hostcfgd: Feature sflow is enabled and started
Nov  5 19:37:30.117581 r-lionfish-16 INFO /hostcfgd: Feature radv is enabled and started
Nov  5 19:37:30.135637 r-lionfish-16 INFO /hostcfgd: Feature teamd is enabled and started
Nov  5 19:37:30.152337 r-lionfish-16 INFO /hostcfgd: Feature what-just-happened is enabled and started
Nov  5 19:37:30.169778 r-lionfish-16 INFO /hostcfgd: Feature snmp is enabled and started
Nov  5 19:37:30.186031 r-lionfish-16 INFO /hostcfgd: Feature telemetry is enabled and started
Nov  5 19:37:30.203852 r-lionfish-16 INFO /hostcfgd: Feature pmon is enabled and started
Nov  5 19:37:30.221953 r-lionfish-16 INFO /hostcfgd: Feature dhcp_relay is enabled and started
Nov  5 19:37:30.239453 r-lionfish-16 INFO /hostcfgd: Feature bgp is enabled and started
Nov  5 19:37:30.256187 r-lionfish-16 INFO /hostcfgd: Feature mgmt-framework is enabled and started
Nov  5 19:37:30.273000 r-lionfish-16 INFO /hostcfgd: Feature database is enabled and started
Nov  5 19:37:30.290031 r-lionfish-16 INFO /hostcfgd: Feature syncd is enabled and started
Nov  5 19:37:30.306905 r-lionfish-16 INFO /hostcfgd: Feature nat is stopped and disabled

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>

d03ba4fa [202012] [portstat, intfstat] added rates and utilization  (sonic-net#1812)
499ad3f4 [config reload] Fix config reload failure due to sonic.target job cancellation (sonic-net#1814)
96d658c2 [202012][sonic installer] Add swap setup support (sonic-net#1815)
a9c69702 platform pre-check for reboot in 202012 branch (sonic-net#1788)
0e0478ba Unify the number format in the ourput of portstat and pfcstat in all cases (sonic-net#1795)
2d1e00ed [ecnconfig] Fix exception seen during display and add unit tests (sonic-net#1784) (sonic-net#1789)
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
@vivekrnv vivekrnv closed this Nov 10, 2021
vivekrnv pushed a commit that referenced this pull request Jan 18, 2022
* [BFN] Updated platform APIs impl

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* Extended BFN platform SFP APIs implementation

* Update sfp.py

* [BFN] Extended SFP platform plugin implementation

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* [BFN] Extended Fans platform plugin implementation

* [BFN] divided classes Fan and  FanDrawer into 2 files

* Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com>

What I did
	Add get_model() function
	Add get_low_critical_threshold() function
	Change __get(...) function.
How I did it
	Differnece from previous implementation of __get(...) function is return real value or -9999.9 if value is not provided by thrift API

* Add get_presence() function and revised __get() function

Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com>

* [BFN] Updated PSU platform APIs impl

Signed-off-by: Dmytro Lytvynenko <dmytrox.lytvynenko@intel.com>

* Added BFN PSU cache (#9)

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* [BFN]  Fans and Fantray platform APIs update (#7)

* [BFN] Updated SFP platform APIs (#10)

Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>

* [BFN] Updated platform API for thermal (#8)

* Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com>

* Revert "[BFN]  Fans and Fantray platform APIs update (#7)" (#11)

This reverts commit c62a733.

* Add support health monitor system (#15)

Signed-off-by: Petro Bratash <petrox.bratash@intel.com>

* Update chassis.py

* [BFN] Updated FANs and FAN Tray platform API (#14)

* Fix fix_alignment (#17)

Signed-off-by: Petro Bratash <petrox.bratash@intel.com>

* [BFN] Improvement show environment (#16)

* Added PSU temperature skip into platform.json (#18)

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* Do not skip psud on Newport

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* [BFN] fix fan status from Not OK to Ok (#19)

* [BFN] Updated SFP platform plugin (#13)

Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>

* [DPB] Fix typo for Ethernet0 2x200G[100G,40G] breakout mode (#21)

Signed-off-by: Mykola Gerasymenko <mykolax.gerasymenko@intel.com>

* [barefoot] Tmp fix vendor_rev (#22)

Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>

* Fixed python issues in sonic_platform/fan_drawer.py

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* Updated fan_drawer.py

* Fixing trailing white spaces in fan_drawer.py

* [BFN] Fix thrift for SFPs API

Signed-off-by: Volodymyr Boyko <volodymyrx.boiko@intel.com>

* In platform.json, replaced 'false' with '0' to workaround ast.literal_eval() issue

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* [Newport] Thermal manager  (#23)

* Signed-off-by: Vadym Yashchenko <vadymx.yashchenko@intel.com>

* Revert "In platform.json, replaced 'false' with '0' to workaround ast.literal_eval() issue"

This reverts commit 1e73127.

* Removed 'controllable' options from platform.json to fix factory default config generation

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

* Update thermal_manager.py

* Migrated SFP plugin to sonic_xcvr API (#30)

Signed-off-by: Andriy Kokhan <andriyx.kokhan@intel.com>

Co-authored-by: KostiantynYarovyiBf <kostiantynx.yarovyi@intel.com>
Co-authored-by: Vadym Yashchenko <vadymx.yashchenko@intel.com>
Co-authored-by: Dmytro Lytvynenko <dmytrox.lytvynenko@intel.com>
Co-authored-by: Volodymyr Boiko <volodymyrx.boiko@intel.com>
Co-authored-by: Petro Bratash <petrox.bratash@intel.com>
Co-authored-by: Mykola Gerasymenko <mykolax.gerasymenko@intel.com>
vivekrnv pushed a commit that referenced this pull request Nov 8, 2022
sonic-host-services submodule update with following changes

6eac2d3 Merge pull request #19 from judyjoseph/macsec_feature_enable
31c6108 Sync has_per_asic_scope attribute to config_db in all namespaces for multi-asic
185547f Add UT to improve coverage, for namespace config update
7c3aca0 macsec_supported info is part of DEVICE_RUNTIME_METADATA itself.
vivekrnv added a commit that referenced this pull request Dec 6, 2022
2fbe729 disable cfg dynamic change (#25)
13d0805 Use github code scanning instead of LGTM (#26)
1e846f6 Fix packet range check for relay-reply packets (#21)
4d19e13 (work/master, master) Add unittest infrastructure (#5)
7f4fdab fix packet range check issue (#20)
257ecdf Add client packet UDP header length check (#19)

Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
vivekrnv added a commit that referenced this pull request Dec 6, 2022
2fbe729 disable cfg dynamic change (#25)
13d0805 Use github code scanning instead of LGTM (#26)
1e846f6 Fix packet range check for relay-reply packets (#21)
4d19e13 Add unittest infrastructure (#5)
7f4fdab fix packet range check issue (#20)
257ecdf Add client packet UDP header length check (#19)

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
vivekrnv pushed a commit that referenced this pull request Dec 19, 2022
Added below commits:
9b30690 jcaiMR Fri Dec 16 fix handleSwssNotification crash in dhcp6relay (#28)
047afb7 jcaiMR Wed Dec 14 14:08:58 2022 +0800 Fix multiple vlan issue (#27)
ff6bec3 Vivek Thu Dec 8 09:44:15 2022 -0800 Made the Error log informative (#22)
2fbe729 jcaiMR Wed Nov 30 14:41:53 2022 +0800 disable cfg dynamic change (#25)
13d0805 Liu Shilong Wed Nov 30 10:54:11 2022 +0800 Use github code scanning instead of LGTM (#26)
1e846f6 kellyyeh Wed Nov 23 14:36:02 2022 -0800 Fix packet range check for relay-reply packets (#21)
4d19e13 kellyyeh Thu Nov 17 10:04:53 2022 -0800 Add unittest infrastructure (#5)
7f4fdab jcaiMR Fri Nov 11 14:47:51 2022 +0800 fix packet range check issue (#20)
257ecdf kellyyeh Thu Nov 3 11:34:11 2022 -0700 Add client packet UDP header length check (#19)
vivekrnv pushed a commit that referenced this pull request Mar 13, 2023


advance dhcp relay for 202211

4bf1868 - (HEAD, origin/master, origin/HEAD, master) fix relay-reply dhcpv6 packet counter issue (add support for a7050 qx32 platform #29) (2 weeks ago) [jcaiMR]
9b30690 - fix handleSwssNotification crash in dhcp6relay (Add libnl-nf-3-200 to docker-team #28) (4 weeks ago) [jcaiMR]
047afb7 - Fix multiple vlan issue (Failure trying to run: chroot /sonic-buildimage/fsroot mount -t proc proc /proc #27) (4 weeks ago) [jcaiMR]
ff6bec3 - Made the Error log informative (add python-tenjin as build dependency for p4-switch #22) (5 weeks ago) [Vivek]
2fbe729 - disable cfg dynamic change (p4: fix build dependency for python-p4c-bm #25) (6 weeks ago) [jcaiMR]
13d0805 - Use github code scanning instead of LGTM (Removed sx-libnl from Mellanox containers dependencies. #26) (6 weeks ago) [Liu Shilong]
1e846f6 - Fix packet range check for relay-reply packets (update sonic-swss and p4-switch submodule to fix docker sonic p4 bug #21) (7 weeks ago) [kellyyeh]
4d19e13 - Add unittest infrastructure (Cavium customization for docker containers #5) (8 weeks ago) [kellyyeh]
7f4fdab - fix packet range check issue (Makefile: add build dependency for python-p4c-bm #20) (9 weeks ago) [jcaiMR]
257ecdf - Add client packet UDP header length check (change port_config.ini directory for s6000 #19) (2 months ago) [kellyyeh]
vivekrnv added a commit that referenced this pull request Sep 11, 2024
vivekrnv added a commit that referenced this pull request Dec 6, 2024
* Update to Linux 6.1.94

Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>

* Integrate HW-MGMT 7.0040.1008 Changes (#18)

* Integrate HW-MGMT 7.0040.1011 Changes (#19)

* Update sonic-linux-kernel submodule (#20)

Signed-off-by: Vivek Reddy <vkarri@nvidia.com>

---------

Signed-off-by: Saikrishna Arcot <sarcot@microsoft.com>
Signed-off-by: Vivek Reddy <vkarri@nvidia.com>
Co-authored-by: Vivek <vivekreddykarri98@gmail.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