[mellanox]: Fix symlink to QOS config file for MSN2740 platform#1842
Merged
lguohan merged 1 commit intosonic-net:masterfrom Jul 3, 2018
Merged
[mellanox]: Fix symlink to QOS config file for MSN2740 platform#1842lguohan merged 1 commit intosonic-net:masterfrom
lguohan merged 1 commit intosonic-net:masterfrom
Conversation
Signed-off-by: Volodymyr Samotiy <[email protected]>
qiluo-msft
approved these changes
Jul 3, 2018
5 tasks
wen587
added a commit
that referenced
this pull request
Oct 26, 2021
8ea834b [sonic_installer] Change sonic_installer check ASIC mismatch by platforms list (#1836) 9017d99 Fix the option missing in kernel config issue (#1888) 6595ad4 [mlag] fix log print sequence (#1730) e600e1c CLI command to load config in Yang format (#1781) 00948d0 Fix the target db version of portchannel-key test (#1842) 6412fea [sonic-package-manager] remove make_python_identifier (#1801) f738818 [sonic-package-manager] stop service explicitelly before uninstalling package (#1805) d8ee5e9 Remove exec from platform_reboot_plugin call to handle any hang issue. (#1879) 827fcee [chassis][routecheck]filter out the chassis internal interfaces (#1798) 4d732c6 [generic_config_updater] Logging (#1864) 25bb184 [config]: Add loopback interfaces to interface name checker (#1869) c950a55 Validate input of config mirror_session add (#1825) 9ab20fd [show][config] fix the muxcable commands for interface naming mode (#1862) 476b3a4 [multi-asic][cli][chassis-db] Avoid connecting to chassis db for cli commands executed from linecard (#1707)
theasianpianist
pushed a commit
to theasianpianist/sonic-buildimage
that referenced
this pull request
Feb 5, 2022
What I did Refactored cfgmgr Makefile Why I did it To avoid code duplication of common libs How I verified it Build swss Signed-off-by: Nazarii Hnydyn <[email protected]>
taras-keryk
pushed a commit
to taras-keryk/sonic-buildimage
that referenced
this pull request
Apr 28, 2022
What I did The port-channel key migrator was introduced in version 2_0_2 so the expected database version of the test case should be 2_0_2. It was modified to 2_0_3 when the new version was introduced by mistake. This won't fail the test but disable the require its database version to be updated every time a new version is introduced. (Refer sonic-net#1566 and sonic-net#1614 for details) This is to correct it by changing it back to 2_0_2. Signed-off-by: Stephen Sun <[email protected]>
patrickmacarthur
pushed a commit
to patrickmacarthur/sonic-buildimage
that referenced
this pull request
Dec 11, 2025
…andle redis-communication failure case gracefully (sonic-net#1842) #### Why I did it redis-server is considered as critical process in database container. However, currently supervisor-proc-exit-listener will break if redis-server exits. By design, supervisor-proc-exit-listener should 1. periodically logging a critical process is not running if container auto_restart is disabled 2. kill the container if container auto_restart is enabled This PR will help upervisor-proc-exit-listener achieve designed behavior if redis-server exits ##### Work item tracking - Microsoft ADO **(number only)**: #### How I did it #### How to verify it Kill the redis-server process inside database container. Observing logs: ``` 2025 Oct 31 01:25:27.304563 sonic INFO database#supervisord 2025-10-31 01:25:27,303 WARN exited: redis (terminated by SIGKILL; not expected) 2025 Oct 31 01:25:28.309293 sonic WARNING database#supervisor-proc-exit-listener: Unable to retrieve features table from Config DB: Unable to connect to redis - Connection refused(1): Cannot assign requested address 2025 Oct 31 01:26:28.436837 sonic ERR database#supervisor-proc-exit-listener: Process 'redis' is not running in namespace 'host' (1.0 minutes). 2025 Oct 31 01:27:28.559959 sonic ERR database#supervisor-proc-exit-listener: Process 'redis' is not running in namespace 'host' (2.0 minutes). ``` #### Which release branch to backport (provide reason below if selected) <!-- - Note we only backport fixes to a release branch, *not* features! - Please also provide a reason for the backporting below. - e.g. - [x] 202006 --> - [ ] 202205 - [ ] 202211 - [ ] 202305 - [ ] 202311 - [ ] 202405 - [ ] 202411 - [ ] 202505 #### Tested branch (Please provide the tested image version) <!-- - Please provide tested image version - e.g. - [x] 20201231.100 --> - [ ] <!-- image version 1 --> - [ ] <!-- image version 2 --> #### Description for the changelog <!-- Write a short (one line) summary that describes the changes in this pull request for inclusion in the changelog: --> <!-- Ensure to add label/tag for the feature raised. example - PR#2174 under sonic-utilities repo. where, Generic Config and Update feature has been labelled as GCU. --> #### Link to config_db schema for YANG module changes <!-- Provide a link to config_db schema for the table for which YANG model is defined Link should point to correct section on https://github.com/Azure/sonic-buildimage/blob/master/src/sonic-yang-models/doc/Configuration.md --> #### A picture of a cute animal (not mandatory but encouraged)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Volodymyr Samotiy [email protected]
- What I did
Fixed issue when ports are not UP after SONiC start.
- How I did it
Problem is that updategraph cannot find QOS config file (qos.json.j2) and as a result bufferorch cannot fill the
m_port_ready_list_reflist.Fixed symlink to QOS config file for MSN2740 platform in order to point to correct config file.
- How to verify it
- Description for the changelog
Fix symlink to QOS config file for MSN2740 platform