Skip to content

Commit

Permalink
Revert "Revert "Open 8001 port as one that will be used for nanny-con…
Browse files Browse the repository at this point in the history
…tact-address""

This reverts commit 1f9be32.
  • Loading branch information
oshadura committed Feb 17, 2023
1 parent 78de622 commit d50c81d
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions coffea_casa/coffea_casa.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
DEFAULT_SCHEDULER_PORT = 8786
DEFAULT_DASHBOARD_PORT = 8787
DEFAULT_CONTAINER_PORT = 8786
#DEFAULT_NANNY_PORT = 8001
DEFAULT_NANNY_PORT = 8001

# Security settings for Dask scheduler
# REMOVE ME (backward compatibity for now)
Expand Down Expand Up @@ -71,7 +71,7 @@ def __init__(self,
scheduler_options=None,
scheduler_port=DEFAULT_SCHEDULER_PORT,
dashboard_port=DEFAULT_DASHBOARD_PORT,
#nanny_port=DEFAULT_NANNY_PORT,
nanny_port=DEFAULT_NANNY_PORT,
**job_kwargs):
"""
Parameters
Expand Down Expand Up @@ -118,7 +118,7 @@ def __init__(self,
worker_image=worker_image,
scheduler_port=scheduler_port,
dashboard_port=dashboard_port,
#nanny_port=nanny_port,
nanny_port=nanny_port,
)
# Instantiate args and parameters from parent abstract class security=security
super().__init__(**job_kwargs)
Expand All @@ -132,7 +132,7 @@ def _modify_job_kwargs(cls,
scheduler_options=None,
scheduler_port=DEFAULT_SCHEDULER_PORT,
dashboard_port=DEFAULT_DASHBOARD_PORT,
#nanny_port=DEFAULT_NANNY_PORT
nanny_port=DEFAULT_NANNY_PORT
):
job_config = job_kwargs.copy()
input_files = []
Expand Down Expand Up @@ -190,10 +190,9 @@ def _modify_job_kwargs(cls,
"docker_image": worker_image or dask.config.get(f"jobqueue.{cls.config_name}.worker-image")
},
{
#"container_service_names": "dask,nanny",
#"dask_container_port": DEFAULT_CONTAINER_PORT,
#"nanny_container_port": nanny_port,
"container_service_names": "dask",
"container_service_names": "dask,nanny",
"dask_container_port": DEFAULT_CONTAINER_PORT,
"nanny_container_port": DEFAULT_NANNY_PORT,
"dask_container_port": DEFAULT_CONTAINER_PORT,
},
{"transfer_input_files": files},
Expand Down

0 comments on commit d50c81d

Please sign in to comment.