Skip to content

Commit

Permalink
[database] Fix database config template for remote Redis instance (so…
Browse files Browse the repository at this point in the history
…nic-net#19239)

The local Redis daemon should not run for a remote database.

How I did it
Do not generate a supervisord configuration entry for the remote database instances.

How to verify it
Compile and run the DPU image. The database container on the DPU should run Redis daemons only for local DB instances.
  • Loading branch information
oleksandrivantsiv authored and arun1355492 committed Jul 26, 2024
1 parent a4bcf43 commit e94d9c3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion dockers/docker-database/database_config.json.j2
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
,"remote_redis":{
"hostname" : "{{REMOTE_DB_IP}}",
"port" : {{REMOTE_DB_PORT}},
"unix_socket_path": ""
"unix_socket_path": "",
"persistence_for_warm_boot" : "yes"
}
{% endif %}
Expand Down

0 comments on commit e94d9c3

Please sign in to comment.