search

MattDGTL / sonarr-radarr-queue-cleaner

A simple script using Sonarr and Radarr API
63 stars 25 forks source link

Docker API_TIMEOUT flag does nothing #9

Open thierrybla opened 1 year ago

thierrybla commented 1 year ago

Hi,

When I change the flag to API_TIMEOUT=3600 (1 hour) it does not change when the docker is run it will cooldown for 10 minutes and not 1 hour as stated in the README.

ggoupy commented 1 year ago

I believe it does change the timeout, but in the code the log message:

logging.info('Finished running media-tools script. Sleeping for 10 minutes.')

should be replaced by: 

logging.info(f'Finished running media-tools script. Sleeping for {API_TIMEOUT} seconds.')

EDIT: My bad, you also have to change ENV TIMEOUT=<N> to ENV API_TIMEOUT=<N> in the Dockerfile

kjames2001 commented 1 year ago

I believe it does change the timeout, but in the code the log message:

logging.info('Finished running media-tools script. Sleeping for 10 minutes.')

should be replaced by:

logging.info(f'Finished running media-tools script. Sleeping for {API_TIMEOUT} seconds.')

EDIT: My bad, you also have to change ENV TIMEOUT=<N> to ENV API_TIMEOUT=<N> in the Dockerfile

Thanks for the correct answer! will fix it on my folk, and send a pr.