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 pull[bot] committed Jun 25, 2024
1 parent e1d4819 commit ac6e7ad
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 ac6e7ad

Please sign in to comment.