Skip to content

Commit

Permalink
[database] Fix database config template for remote Redis instance (#1…
Browse files Browse the repository at this point in the history
…9239)

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 Jun 7, 2024
1 parent a81b7dc commit 04bad02
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 04bad02

Please sign in to comment.