Skip to content

[master][Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor#18242

Merged
lguohan merged 1 commit intosonic-net:masterfrom
mlok-nokia:fix-database-chassis
Mar 22, 2024
Merged

[master][Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor#18242
lguohan merged 1 commit intosonic-net:masterfrom
mlok-nokia:fix-database-chassis

Conversation

@mlok-nokia
Copy link
Contributor

Why I did it

redis_chassis process fails to start in the database-chassis on the supervisor card.

Work item tracking
  • Microsoft ADO (number only):

How I did it

The redis and redis_chassis processes in supervisor.conf has been modified to start with 'redis' as user. The modification only added redis and redis group for database containers except database-chassis container. This commit adds code to the docker-database-init.sh to add redis and redis group for the database-chassis to address this issue.

How to verify it

Booting the image on Supervisor. And verify the database-chassis container should be in UP state, and the database-chassis.service is in active(running) state

admin@ixre-cpm-chassis15:~$ docker ps -a
CONTAINER ID   IMAGE                             COMMAND                  CREATED        STATUS        PORTS     NAMES
9691c6111e0f   docker-fpm-frr:latest             "/usr/bin/docker_ini?"   16 hours ago   Up 16 hours             bgp5
...
c7c7159d4e0c   docker-database:latest            "/usr/local/bin/dock?"   31 hours ago   Up 16 hours             database-chassis


admin@ixre-cpm-chassis15:~$ sudo systemctl status database-chassis
? database-chassis.service - database-chassis container
     Loaded: loaded (/lib/systemd/system/database-chassis.service; enabled-runtime; preset: enabled)
     Active: active (running) since Sun 2024-03-03 04:43:12 UTC; 16h ago
    Process: 1640 ExecStartPre=/usr/bin/database.sh start chassisdb (code=exited, status=0/SUCCESS)
   Main PID: 1938 (database.sh)
      Tasks: 10 (limit: 38307)
     Memory: 49.7M
     CGroup: /system.slice/database-chassis.service
             ??1938 /bin/bash /usr/bin/database.sh wait chassisdb
             ??1940 docker wait database-chassis

Notice: journal has been rotated since unit was started, output may be incomplete.
admin@ixre-cpm-chassis15:~$ 


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

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

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)

@mlok-nokia mlok-nokia requested a review from lguohan as a code owner March 3, 2024 21:09
@mlok-nokia
Copy link
Contributor Author

@rlhui @judyjoseph This PR addresses the database-chassis issue in the Supervisor. Please review it.

@mlok-nokia mlok-nokia changed the title [Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor [master][Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor Mar 6, 2024
@mlok-nokia mlok-nokia changed the title [master][Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor [master][master][Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor Mar 6, 2024
@mlok-nokia mlok-nokia changed the title [master][master][Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor [master][Supervisor][datbase-chassis] Fix redis_chassis FATAL issue in database-chassis on Supervisor Mar 6, 2024
@judyjoseph judyjoseph self-assigned this Mar 6, 2024
@judyjoseph judyjoseph requested a review from maipbui March 6, 2024 18:50
@mlok-nokia mlok-nokia force-pushed the fix-database-chassis branch from 4cf195f to 86dcddf Compare March 6, 2024 18:59
…s in database-chassis on Supervisor

Signed-off-by: mlok <[email protected]>
@mlok-nokia
Copy link
Contributor Author

@judyjoseph Please help to merge this PR.

@lguohan lguohan merged commit 2e81eff into sonic-net:master Mar 22, 2024
@mlok-nokia mlok-nokia deleted the fix-database-chassis branch March 29, 2024 16:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants