Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions dask-gateway-server/dask_gateway_server/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
from .backends import Backend
from .routes import default_routes
from .traitlets import Application, Type
from .utils import AccessLogger, LogFormatter, classname, normalize_address, run_main
from .utils import AccessLogger, classname, normalize_address, run_main


class GenerateConfig(Application):
Expand Down Expand Up @@ -145,8 +145,6 @@ def _default_address(self):
def _validate_address(self, proposal):
return normalize_address(proposal.value)

_log_formatter_cls = LogFormatter

classes = List([Backend, Authenticator])

@catch_config_error
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ async def do_start_cluster(self, cluster):
async def do_stop_cluster(self, cluster):
scheduler = self.schedulers.pop(cluster.name)

await scheduler.close(fast=True)
await scheduler.close()
scheduler.stop()

workdir = cluster.state.get("workdir")
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
from ...utils import (
AccessLogger,
FrozenAttrDict,
LogFormatter,
RateLimiter,
TaskPool,
normalize_address,
Expand Down Expand Up @@ -284,8 +283,6 @@ def _validate_proxy_prefix(self, proposal):
config=True,
)

_log_formatter_cls = LogFormatter

# Fail if the config file errors
raise_config_file_errors = True

Expand Down
17 changes: 16 additions & 1 deletion dask-gateway-server/dask_gateway_server/traitlets.py
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,22 @@
from traitlets import Unicode
from traitlets.config import Application

# Override default values for logging
# We replace the class of the default formatter used via a configuration change.
#
# References:
#
# - Traitlets' Application.logging_config defaults:
# https://github.com/ipython/traitlets/blob/e2c731ef72dd41d4be527d4d93dd87ccc409830d/traitlets/config/application.py#L229-L256
# - Python official schema for Application.logging_config:
# https://docs.python.org/3/library/logging.config.html#logging-config-dictschema
#
Application.logging_config = {
"formatters": {
"console": {
"class": "dask_gateway_server.utils.LogFormatter",
},
},
}
Application.log_level.default_value = "INFO"
Application.log_format.default_value = (
"%(log_color)s[%(levelname)1.1s %(asctime)s.%(msecs).03d "
Expand Down
3 changes: 2 additions & 1 deletion dask-gateway-server/dask_gateway_server/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -175,10 +175,11 @@ def classname(cls):


class LogFormatter(ColoredFormatter):
def __init__(self, fmt=None, datefmt=None):
def __init__(self, fmt=None, datefmt=None, style=None):
super().__init__(
fmt=fmt,
datefmt=datefmt,
style=style,
reset=False,
log_colors={
"DEBUG": "blue",
Expand Down
2 changes: 1 addition & 1 deletion dask-gateway-server/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
aiohttp
colorlog
cryptography
traitlets
traitlets>=5.2.2.post1
6 changes: 3 additions & 3 deletions dask-gateway/requirements.txt
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
# These are dependencies for installing the dask-gateway package.
#
aiohttp
click
dask>=2.2.0
distributed>=2021.01.1
click>=8.1.3
dask>=2022.1.0
distributed>=2022.1.0,!=2022.5.1,!=2022.5.2
pyyaml
tornado
2 changes: 1 addition & 1 deletion tests/test_db_backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -469,7 +469,7 @@ async def test_cluster_fails_after_connect():
async with gateway.new_cluster() as cluster:
# Kill scheduler
scheduler = g.gateway.backend.schedulers[cluster.name]
await scheduler.close(fast=True)
await scheduler.close()
scheduler.stop()

# Gateway notices and cleans up cluster in time
Expand Down