-
-
Notifications
You must be signed in to change notification settings - Fork 718
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
Add the heartbeat_interval variable to scheduler Parameters #4083
Comments
Correct, right now heartbeat intervals are adaptive based on the number of workers, and there is no option to make this configurable. In principle having a configuration value that defaulted to "auto" which would imply the current behavior seems sensible to me. Short term, you could also change your logging level. I tend to use INFO by default. |
Thanks @mrocklin , in case I would set logging INFO I am reading here https://docs.dask.org/en/latest/configuration.html and see that is possible to set with YAML, with system ENV or inline.
So I would pass the logging to scheduler:
But not sure if is possible and how :) |
I'm not that familiar with docker-compose unfortunately. Does it allow you
to add files or specify environment variables? If so, then I would use one
of those.
…On Mon, Aug 31, 2020 at 2:46 AM garanews ***@***.***> wrote:
Thanks @mrocklin <https://github.com/mrocklin> , in case I would set
logging INFO I am reading here
https://docs.dask.org/en/latest/configuration.html and see that is
possible to set with YAML, with system ENV or inline.
I am wondering if is possible to pass it as command.
For example, this is docker-compose of worker:
worker01:
build:
context: .
dockerfile: ./compose/local/dask/Dockerfile
args:
- http_proxy
- https_proxy
image: daskdev/dask
hostname: dask-worker
command:
[
"dask-worker",
"--preload",
"/tmp/daskworkerinit.py",
"--nthreads",
"1",
"--nprocs",
"1",
"--memory-limit",
"3G",
"tcp://scheduler:8786",
]
So I would pass the logging to scheduler:
scheduler:
build:
context: .
dockerfile: ./compose/local/dask/Dockerfile
args:
- http_proxy
- https_proxy
image: daskdev/dask
hostname: dask-scheduler
ports:
- "8786:8786"
- "8787:8787"
command: ["dask-scheduler"]
networks:
- elastic
But not sure if is possible and how :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#4083 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACKZTDNMZG4DHCRPE6AMSDSDNWRFANCNFSM4QOGAP7A>
.
|
I would probably start with passing in the the logging level as an env var with docker compose. That would be something like:
Note: I used our new handy conversion tool to help with this: https://docs.dask.org/en/latest/configuration.html#conversion-utility Thanks @jacobtomlinson |
thanks, I passed that and I see it is set inside the container, but still logging the debug messages... So question is how to avoid these messages? The 2 variables I set previously don't seem to work: worker01_1 | DEBUG 2020-09-03 09:54:16,758 worker 363 140492157343552 Heartbeat: tcp://172.18.0.7:43423 |
Are you setting the logging level to |
ehm, I just using docker image daskdev/dask so nothing touch about debug logging. |
Hmm, I just tried the latest from https://github.com/dask/dask-docker and I only have INFO output. Is there some in your |
I found it! I had DEBUG level in base.py : So now finally I do not have hearbeat log anymore! Thanks @quasiben for your time! |
Thanks @garanews -- closing |
With the #3896 it was tuned the heartbeat_interval and correlated to number of workers:
I think it would be useful have the heartbeat_interval as a Parameter.
With few workers the heartbeat messages are flooding everything and is hard to debug code:
The text was updated successfully, but these errors were encountered: