Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docker API_TIMEOUT flag does nothing #9

Open
thierrybla opened this issue Jun 12, 2023 · 2 comments
Open

Docker API_TIMEOUT flag does nothing #9

thierrybla opened this issue Jun 12, 2023 · 2 comments

Comments

@thierrybla
Copy link

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
Copy link

ggoupy commented Jun 18, 2023

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
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants