Skip to content

[healthd] Start system-health after interfaces-config#46

Closed
vivekrnv wants to merge 1 commit intomasterfrom
intf_cfg_dep_healthd
Closed

[healthd] Start system-health after interfaces-config#46
vivekrnv wants to merge 1 commit intomasterfrom
intf_cfg_dep_healthd

Conversation

@vivekrnv
Copy link
Owner

@vivekrnv vivekrnv commented Apr 21, 2023

Why I did it

interfaces-config service restarts networking service, which in-turn results in loopback interface address is being removed and reassigned back

If the system-health happens to start during that instance expections and logs like this are seen:

Apr 15 18:14:49.357869 r-panther-20 ERR healthd: update system status exception:Unable to connect to redis: Cannot assign requested address
Apr 15 18:14:49.429778 r-panther-20 ERR healthd: subscribe_statedb exited- Unable to connect to redis: Cannot assign requested address
Apr 15 18:14:52.218594 r-panther-20 ERR healthd: system_service_Map_base::at
Apr 15 18:14:52.219714 r-panther-20 ERR healthd: system_service_Map_base::at
Apr 15 18:14:55.218636 r-panther-20 ERR healthd: system_service_Map_base::at
Apr 15 18:14:55.218722 r-panther-20 ERR healthd: system_service_Map_base::at

How I did it

Start system-health after interfaces-config.service

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

Signed-off-by: Vivek Reddy Karri <vkarri@nvidia.com>
@vivekrnv vivekrnv force-pushed the intf_cfg_dep_healthd branch from 598218b to 341108e Compare April 21, 2023 16:46
@vivekrnv vivekrnv closed this Apr 25, 2023
vivekrnv pushed a commit that referenced this pull request Feb 10, 2024
…ly (sonic-net#18046)

#### Why I did it
src/dhcprelay
```
* 363fa06 - (HEAD -> master, origin/master, origin/HEAD) Skip vlans with no dhcpv6 server configured (#46) (8 hours ago) [kellyyeh]
```
#### How I did it
#### How to verify it
#### Description for the changelog
vivekrnv pushed a commit that referenced this pull request Mar 24, 2025
…lly (sonic-net#731)

#### Why I did it
src/sonic-swss
```
* b87da49 - (HEAD -> 202412, origin/202412) Set Port UPDATE_DSCP attribute when TC_TO_DSCP map is attached (#44) (9 hours ago) [mssonicbld]
* 8c99374 - sonic-swss: Code changes for WRED and ECN statistics (#46) (9 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
vivekrnv pushed a commit that referenced this pull request Mar 24, 2025
… automatically (sonic-net#768)

#### Why I did it
src/sonic-platform-common
```
* 44faed9 - (HEAD -> 202412, origin/202412) [code sync] Merge code from sonic-net/sonic-platform-common:202411 to 202412 (#46) (21 hours ago) [mssonicbld]
```
#### How I did it
#### How to verify it
#### Description for the changelog
vivekrnv pushed a commit that referenced this pull request Apr 10, 2025
…ly (sonic-net#21743)

#### Why I did it
src/sonic-stp
```
* 29eb387 - (HEAD -> master, origin/master, origin/HEAD) Fixing compilaion error for marvell_armhf (#47) (9 hours ago) [Divya Kumaran Chandralekha]
* b7da558 - Merge pull request #44 from wajahatrazi/stp_co_pr (3 days ago) [Divya Kumaran Chandralekha]
|\ 
| failure_prs.log skip_prs.log 47e1e77 - Merge branch 'master' into stp_co_pr (3 days ago) [Divya Kumaran Chandralekha]
| |\ 
| |/ 
|/| 
* | 6e4a94f - Merge pull request #46 from divyachandralekha/pvst_pr_1 (4 days ago) [Divya Kumaran Chandralekha]
|\ \ 
| failure_prs.log skip_prs.log | aa5e824 - libevent explicit building dependency removed (4 days ago) [Divya Chandralekha]
|/ / 
* | 9345e5a - Merge pull request #43 from omaaartamer/fix/alignment-warnings (5 days ago) [Divya Kumaran Chandralekha]
* | eedb772 - [stp] Add memory alignment padding to structures (9 weeks ago) [omaaartamer]
* | 128e53d - fix: resolve STP alignment warnings (9 weeks ago) [omaaartamer]
 / 
* 30febfa - Fixing changes in the code (9 weeks ago) [Wajahat Razi]
```
#### How I did it
#### How to verify it
#### Description for the changelog
vivekrnv pushed a commit that referenced this pull request Aug 19, 2025
…sonic-net#23654)

#### Why I did it
src/dhcpmon
```
* 1cb6ced - (HEAD -> master, origin/master, origin/HEAD) Update clear_counter_timeout to fix clear counter issue (#45) (19 hours ago) [Yaqiang Zhu]
* 848304e - [build] Update to use libyang3 (#46) (4 days ago) [Yaqiang Zhu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
vivekrnv pushed a commit that referenced this pull request Nov 20, 2025
…tically (sonic-net#23416)

#### Why I did it
src/sonic-dash-api
```
* aa3fc48 - (HEAD -> master, origin/master, origin/HEAD) [ci] Automated agent pool migration for branch master (#46) (3 days ago) [yijingyan2]
```
#### How I did it
#### How to verify it
#### Description for the changelog
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