Skip to content

[docker-syncd-*-rpc] Fix issue which prevented supervisord to start#555

Merged
stcheng merged 2 commits intosonic-net:masterfrom
pavel-shirshov:pavelsh/fix_rpc_sup
Apr 29, 2017
Merged

[docker-syncd-*-rpc] Fix issue which prevented supervisord to start#555
stcheng merged 2 commits intosonic-net:masterfrom
pavel-shirshov:pavelsh/fix_rpc_sup

Conversation

@pavel-shirshov
Copy link
Contributor

Previously we had supervisord started after syncd started. But service syncd start was blocking. So supervisord didn't start at all. This patch fixes this.
It runs supervisord in daeomon mode first and then it runs syncd.

@stcheng stcheng merged commit 14578d5 into sonic-net:master Apr 29, 2017
@pavel-shirshov pavel-shirshov deleted the pavelsh/fix_rpc_sup branch April 30, 2017 02:08
yxieca added a commit to yxieca/sonic-buildimage that referenced this pull request Jun 19, 2019
Submodule src/sonic-swss 87f0a5e..869c78a:
  > Add retryCount option for orchagent_restart_check program. (sonic-net#833)

Submodule src/sonic-utilities 5b73b83..a02c4ea:
  > [warm-reboot] Use retryCount option of orchagent_restart_check program (sonic-net#555)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
yxieca added a commit that referenced this pull request Jun 19, 2019
Submodule src/sonic-swss 87f0a5e..869c78a:
  > Add retryCount option for orchagent_restart_check program. (#833)

Submodule src/sonic-utilities 5b73b83..a02c4ea:
  > [warm-reboot] Use retryCount option of orchagent_restart_check program (#555)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
lguohan pushed a commit that referenced this pull request Jul 17, 2019
* ee56d54 2019-07-16 | [fast-reboot] Stop services after killing containers to prevent automatic restart (#572) (HEAD -> master, origin/master, origin/HEAD) [Joe LeVeque]
* c718cab 2019-07-17 | [acl_loader]: Fix show mirror_session error (#580) [JaredLius]
* 2764c6c 2019-07-16 | [Mellanox] Improve the prompt of the Mellanox sdk sniffer command. (#546) [Stephen Sun]
* 5a5b74d 2019-07-10 | Generate interface table to have an entry for default VRF (#525) [Prince Sunny]
* c30e9ff 2019-07-10 | [show] Don't abort in InterfaceAliasConverter ctor if PORT table doesn't exist (#571) [Joe LeVeque]
* 1aac696 2019-07-10 | [fast reboot] stop removing opennsl module before reboot (#560) [Ying Xie]
* 37823d4 2019-07-09 | Modified CLI doc for 201904 release, changed the config interface command, etc., (#561) [kannankvs]
* 77ae6b8 2019-07-03 | [PR template] Add reminder to update Command Line Reference Guide if modifying user-facing commands (#569) [Joe LeVeque]
* dc52015 2019-06-25 | [Sonic-utilities]: Added commands to view acl and port running configs (#557) [Prem Prakash]
* c7f45c3 2019-06-19 | [warm-reboot] Use retryCount option of orchagent_restart_check program (#555) [Jipan Yang]
* c6a08f7 2019-06-17 | [scripts]: add support to db_migrator for non-default unix socket (#551) [Lawrence Lee]
* 98e087f 2019-06-06 | [show]: Edit comment (#549) [Vasant Patil]
* 5667a90 2019-06-06 | [intfutil] Fix error when <interface name> specified in show interface related commands (#548) [Gord Chen]
* b7fcb1f 2019-06-04 | Add buffer pool watermark support in watermarkstat and counterpoll (#521) [Wenda Ni]
* 635dc88 2019-06-03 | [config]: Add config acl add/remove table command (#541) [Shuotian Cheng]
* dcdc922 2019-06-03 | [show]: Add show policer command (#540) [Shuotian Cheng]
* 82ef3ec 2019-06-01 | IP Address validation check (#530) [Tapash Das]
* 1f326ad 2019-05-31 | [show] Remove ability to clear counters from 'show' commands (#518) [Joe LeVeque]
* 669d36d 2019-06-01 | Add SONIC CLI Manual (#533) [kannankvs]
* 7657001 2019-06-01 | 'show vlan config' not displaying the VLAN members, after the clear config and reload with default l2 configuration. (#457) [RAMA CHANDRA REDDY GADDAM]
* 9493883 2019-05-28 | use vlan members (#542) [trzhang-msft]
* 128462a 2019-05-22 | [config]: Add --policer option for mirror session (#537) [Shuotian Cheng]
* 38da7f3 2019-05-22 | [acl_loader]: Add policer attribute in show mirror_session command (#538) [Shuotian Cheng]
* f95ff00 2019-05-22 | [acl_loader]: Do not apply default DENY rule for MIRRORV6 table (#522) [Shuotian Cheng]
* 50b7256 2019-05-21 | [sonic_installer] If asked to install an image which is already installed, simply set as default (#534) [Joe LeVeque]
* d823062 2019-05-20 | [201811] enable DB migrator code (#536) [Ying Xie]
* 7f7f384 2019-05-10 | update scheme variable name (#531) [trzhang-msft]
* c66315a 2019-05-10 | Config interface Loopback0 ip add/del is not working (#443) [Kiran Kumar Kella]
* 430d4b8 2019-05-10 | decode-syseeprom : added new option to show the Product Name. (#496) [Olivier Singla]
Kalimuthu-Velappan pushed a commit to Kalimuthu-Velappan/sonic-buildimage that referenced this pull request Sep 12, 2019
sonic-net#555)

* [warm-reboot] Use retryCount option of orchagent_restart_check program

This helps to reduce the possbility of reply message loss from orchagent within the whole checking period

Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>

* Direct orchagent_restart_check output to /dev/null
madhanmellanox pushed a commit to madhanmellanox/sonic-buildimage that referenced this pull request Mar 23, 2020
* Fix initializePort: only set down status if attr missing
* Refine logging
mssonicbld added a commit that referenced this pull request Dec 7, 2024
…D automatically (#21084)

#### Why I did it
src/sonic-platform-daemons
```
* 60e7224 - (HEAD -> master, origin/test_202411_20, origin/test_202411_100, origin/master, origin/kperumal_202411, origin/HEAD, origin/202411) thermalctld: Add support for fans on non-CPU modules (#555) (9 hours ago) [Patrick MacArthur]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Dec 18, 2024
…D automatically (#21165)

#### Why I did it
src/sonic-platform-daemons
```
* 60e7224 - (HEAD -> 202411, origin/test_202411_20, origin/test_202411_100, origin/kperumal_202411, origin/202411) thermalctld: Add support for fans on non-CPU modules (#555) (11 days ago) [Patrick MacArthur]
```
#### How I did it
#### How to verify it
#### Description for the changelog
VladimirKuk pushed a commit to Marvell-switching/sonic-buildimage that referenced this pull request Jan 21, 2025
…D automatically (sonic-net#21084)

#### Why I did it
src/sonic-platform-daemons
```
* 60e7224 - (HEAD -> master, origin/test_202411_20, origin/test_202411_100, origin/master, origin/kperumal_202411, origin/HEAD, origin/202411) thermalctld: Add support for fans on non-CPU modules (sonic-net#555) (9 hours ago) [Patrick MacArthur]
```
#### How I did it
#### How to verify it
#### Description for the changelog
stepanblyschak pushed a commit to stepanblyschak/sonic-buildimage that referenced this pull request Jan 31, 2025
…ic-platform-common

[submodule][202412] Update submodule sonic-platform-common to the latest HEAD automatically
mssonicbld added a commit that referenced this pull request Apr 24, 2025
… automatically (#22244)

#### Why I did it
src/sonic-platform-common
```
* c51c605 - (HEAD -> master, origin/master, origin/HEAD) Update Amphenol package for mock test (#561) (10 hours ago) [mihirpat1]
* dd6cc64 - Add INNOLIGHT OSFP part for special firmware handling (#559) (7 days ago) [Dylan Godwin]
* 5d90044 - Change the QSFP identifier offset to 0x0 (#558) (13 days ago) [Prince George]
* b536f2c - Support for Custom Amphenol 800G Backplane catridge (#555) (3 weeks ago) [Prince George]
```
#### How I did it
#### How to verify it
#### Description for the changelog
DavidZagury pushed a commit to DavidZagury/sonic-buildimage that referenced this pull request Apr 28, 2025
…est HEAD automatically (sonic-net#1036)

#### Why I did it
src/sonic-platform-common
```
* 7aadc13 - (HEAD -> 202412, origin/202412) Merge pull request #77 from prgeor/amph-bp (10 hours ago) [Riff]
* b1ab4e6 - Support for Custom Amphenol 800G Backplane catridge (sonic-net#555) (11 hours ago) [Prince George]
```
#### How I did it
#### How to verify it
#### Description for the changelog
jianyuewu pushed a commit to jianyuewu/sonic-buildimage that referenced this pull request May 7, 2025
* Support for Custom Amphenol 800G Backplane catridge

* Address comment

* Fix xcvrd crash

* Fix the crash

* Improve code coverage
jianyuewu pushed a commit to jianyuewu/sonic-buildimage that referenced this pull request May 7, 2025
Support for Custom Amphenol 800G Backplane catridge (sonic-net#555)
bobby-nexthop pushed a commit to bobby-nexthop/sonic-buildimage that referenced this pull request Aug 1, 2025
* thermalctld: Add support for fans on non-CPU modules

* Add module fan to unit tests
bobby-nexthop pushed a commit to bobby-nexthop/sonic-buildimage that referenced this pull request Aug 1, 2025
…evice is in detaching mode (sonic-net#546)

* Skip logging the warning, if device is in detaching mode

* Add detach_info table and unittests

* Fix unit tests

* Increase code coverage

* Remove unused header import

* Fix dict get values

* Increase code coverage

* Increase test coverage

* [SmartSwitch] Extend implementation of the DPU chassis daemon. (sonic-net#563)

* Addition of DPU Chassis for thermalctld (sonic-net#564)

* [stormond] Added new dynamic field 'last_sync_time' to STATE_DB (sonic-net#535)

* Added new dynamic field 'last_sync_time' that shows when STORAGE_INFO for disk was last synced to STATE_DB

* Moved 'start' message to actual starting point of the daemon

* Added functions for formatted and epoch time for user friendly time display

* Made changes per prgeor review comments

* Pivot to SysLogger for all logging

* Increased log level so that they are seen in syslogs

* Code coverage improvement

* [lag_id] Add lagid to free_list when LC absent for 30 minutes (sonic-net#542)

When LC is absent for 30 minutes, the database cleanup kicks in. When LagId is released, it needs to be appended to the SYSTEM_LAG_IDS_FREE_LIST

This PR works with the following 2 PRs:
sonic-net/sonic-swss#3303
sonic-net#20369

Signed-off-by: mlok <marty.lok@nokia.com>

* Fixed bug in chassisd causing incorrect number of ASICs in CHASSIS_STATE_DB (sonic-net#560)

Fixed the bug in chassisd due to which incorrect number of ASICs were being pushed to CHASSIS_STATE_DB.

* thermalctld: Add support for fans on non-CPU modules (sonic-net#555)

* thermalctld: Add support for fans on non-CPU modules

* Add module fan to unit tests

* Advanced Azure pipeline to Bookworm (sonic-net#572)

Description
This PR advances the azure pipeline on sonic_platform_daemons from bullseye to bookworm. This fixes the issue where sonic-platform-daemons azp is having some issues due to upgrade to bookworm. See Pipelines - Run 20241210.8 logs for details.

* Take non-CMIS xcvrs out of lpmode in SFF Manager (sonic-net#565)

Description
Fix non-CMIS transceivers in down state by bringing them out of low power mode in the SFF Manager Task.
This is intended to work together with the change in sonic-net#20886.

Motivation and Context
Non-CMIS transceivers were not functioning correctly when put into Low Power mode. So XCVRD now brings them out of lpmode.

How Has This Been Tested?
Loaded an image containing this change alongside the change from sonic-net#20886 on an Arista chassis containing a Clearwater2 linecard.
Verified that without this image some interfaces were in a down state but with the image all interfaces came up as expected.

* Added SmartSwitch support in chassisd and enabling chassisd  (sonic-net#467)

Added SmartSwitch support in chassisd and enabling chassisd

* [chassis][psud] Move the PSU parent information generation to the loop run function from the initialization function (sonic-net#576)

Description
Move the PSU parent information generation to the loop run function from the initialization function

Motivation and Context
Fixes sonic-net#575

How Has This Been Tested?
Tested on Cisco chassis, the PHYSICAL_ENTITY_INFO|PSU * can be re-inserted after thermalctld restart.
And monitored the stated db for memory for hours, works well:

* [chassisd] Address the chassisd crash issue and add UT for it (sonic-net#573)

Description
On Nokia platform, slot name of Supervisor is string "A" instead of a number. Using "int" to convert it could cause issue backtrace. We should use slot value to any checking without any conversion. This will fixes sonic-net#21131

Motivation and Context
Modify the _get_module_info not to convert "slot" to a string value. And also modify the code not to convert slot value to an to do any checking. Just directly use the returned value of get_slot(). Also add UT test_moduleupdater_check_slot_string() to valid it.

How Has This Been Tested?
Tested on 202405 branch


Signed-off-by: mlok <marty.lok@nokia.com>

* Fix a comment

---------

Signed-off-by: mlok <marty.lok@nokia.com>
Co-authored-by: Oleksandr Ivantsiv <oivantsiv@nvidia.com>
Co-authored-by: Gagan Punathil Ellath <gpunathilell@nvidia.com>
Co-authored-by: Ashwin Srinivasan <93744978+assrinivasan@users.noreply.github.com>
Co-authored-by: Marty Y. Lok <76118573+mlok-nokia@users.noreply.github.com>
Co-authored-by: Vivek Verma <137406113+vivekverma-arista@users.noreply.github.com>
Co-authored-by: Patrick MacArthur <pmacarthur@arista.com>
Co-authored-by: Peter Bailey <peterbailey@arista.com>
Co-authored-by: rameshraghupathy <43161235+rameshraghupathy@users.noreply.github.com>
Co-authored-by: Jianquan Ye <jianquanye@microsoft.com>
yuazhe added a commit to yuazhe/sonic-buildimage that referenced this pull request Dec 8, 2025
Signed-off-by: Yuanzhe, Liu <yualiu@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants