SDiFI / masdif

Manager for Spoken Dialog Framework
Apache License 2.0
1 stars 0 forks source link

Fix Sidekiq PostgreSQL connection #19

Closed lumpidu closed 1 year ago

lumpidu commented 1 year ago

The cleanup job execution for TTS fails, because the Sidekiq container cannot connect to PostgreSQL database. This is due to the missing env. variables necessary for accessing the PostgreSQL host as defined in config/database.yml. The connection therefore defaults to Unix domain sockets, where no connection is possible.

Also use redis://redis:6379/1 instead of redis://masdif_redis:6379/1 as Redis connection, because the former uses the Docker service name, the latter the customized Docker container name, which might change more likely in case we switch to default container names assigned by Docker.

Also fix triggering deployment when PR is merged into main. It should only be triggered in case a tag v... is pushed.

This closes #18 This closes #20