Skip to content

Commit

Permalink
Merge pull request #3212 from valohai/ruff-i
Browse files Browse the repository at this point in the history
  • Loading branch information
milas committed Mar 29, 2024
2 parents 0fd79c8 + d50cc42 commit 4c6437d
Show file tree
Hide file tree
Showing 95 changed files with 363 additions and 238 deletions.
3 changes: 1 addition & 2 deletions docker/__init__.py
@@ -1,7 +1,6 @@
from .api import APIClient
from .client import DockerClient, from_env
from .context import Context
from .context import ContextAPI
from .context import Context, ContextAPI
from .tls import TLSConfig
from .version import __version__

Expand Down
6 changes: 1 addition & 5 deletions docker/api/build.py
Expand Up @@ -3,11 +3,7 @@
import os
import random

from .. import auth
from .. import constants
from .. import errors
from .. import utils

from .. import auth, constants, errors, utils

log = logging.getLogger(__name__)

Expand Down
22 changes: 16 additions & 6 deletions docker/api/client.py
Expand Up @@ -8,12 +8,22 @@
import requests.exceptions

from .. import auth
from ..constants import (DEFAULT_NUM_POOLS, DEFAULT_NUM_POOLS_SSH,
DEFAULT_MAX_POOL_SIZE, DEFAULT_TIMEOUT_SECONDS,
DEFAULT_USER_AGENT, IS_WINDOWS_PLATFORM,
MINIMUM_DOCKER_API_VERSION, STREAM_HEADER_SIZE_BYTES)
from ..errors import (DockerException, InvalidVersion, TLSParameterError,
create_api_error_from_http_exception)
from ..constants import (
DEFAULT_MAX_POOL_SIZE,
DEFAULT_NUM_POOLS,
DEFAULT_NUM_POOLS_SSH,
DEFAULT_TIMEOUT_SECONDS,
DEFAULT_USER_AGENT,
IS_WINDOWS_PLATFORM,
MINIMUM_DOCKER_API_VERSION,
STREAM_HEADER_SIZE_BYTES,
)
from ..errors import (
DockerException,
InvalidVersion,
TLSParameterError,
create_api_error_from_http_exception,
)
from ..tls import TLSConfig
from ..transport import UnixHTTPAdapter
from ..utils import check_resource, config, update_headers, utils
Expand Down
15 changes: 8 additions & 7 deletions docker/api/container.py
@@ -1,13 +1,14 @@
from datetime import datetime

from .. import errors
from .. import utils
from .. import errors, utils
from ..constants import DEFAULT_DATA_CHUNK_SIZE
from ..types import CancellableStream
from ..types import ContainerConfig
from ..types import EndpointConfig
from ..types import HostConfig
from ..types import NetworkingConfig
from ..types import (
CancellableStream,
ContainerConfig,
EndpointConfig,
HostConfig,
NetworkingConfig,
)


class ContainerApiMixin:
Expand Down
3 changes: 1 addition & 2 deletions docker/api/exec_api.py
@@ -1,5 +1,4 @@
from .. import errors
from .. import utils
from .. import errors, utils
from ..types import CancellableStream


Expand Down
5 changes: 2 additions & 3 deletions docker/api/network.py
@@ -1,7 +1,6 @@
from ..errors import InvalidVersion
from ..utils import check_resource, minimum_version
from ..utils import version_lt
from .. import utils
from ..errors import InvalidVersion
from ..utils import check_resource, minimum_version, version_lt


class NetworkApiMixin:
Expand Down
3 changes: 1 addition & 2 deletions docker/api/secret.py
@@ -1,7 +1,6 @@
import base64

from .. import errors
from .. import utils
from .. import errors, utils


class SecretApiMixin:
Expand Down
7 changes: 3 additions & 4 deletions docker/api/swarm.py
@@ -1,9 +1,8 @@
import logging
import http.client as http_client
import logging

from .. import errors, types, utils
from ..constants import DEFAULT_SWARM_ADDR_POOL, DEFAULT_SWARM_SUBNET_SIZE
from .. import errors
from .. import types
from .. import utils

log = logging.getLogger(__name__)

Expand Down
3 changes: 1 addition & 2 deletions docker/api/volume.py
@@ -1,5 +1,4 @@
from .. import errors
from .. import utils
from .. import errors, utils


class VolumeApiMixin:
Expand Down
3 changes: 1 addition & 2 deletions docker/auth.py
Expand Up @@ -2,8 +2,7 @@
import json
import logging

from . import credentials
from . import errors
from . import credentials, errors
from .utils import config

INDEX_NAME = 'docker.io'
Expand Down
2 changes: 1 addition & 1 deletion docker/client.py
@@ -1,5 +1,5 @@
from .api.client import APIClient
from .constants import (DEFAULT_TIMEOUT_SECONDS, DEFAULT_MAX_POOL_SIZE)
from .constants import DEFAULT_MAX_POOL_SIZE, DEFAULT_TIMEOUT_SECONDS
from .models.configs import ConfigCollection
from .models.containers import ContainerCollection
from .models.images import ImageCollection
Expand Down
1 change: 1 addition & 0 deletions docker/constants.py
@@ -1,4 +1,5 @@
import sys

from .version import __version__

DEFAULT_DOCKER_API_VERSION = '1.44'
Expand Down
2 changes: 1 addition & 1 deletion docker/context/__init__.py
@@ -1,2 +1,2 @@
from .context import Context
from .api import ContextAPI
from .context import Context
10 changes: 6 additions & 4 deletions docker/context/api.py
Expand Up @@ -2,11 +2,13 @@
import os

from docker import errors
from docker.context.config import get_meta_dir
from docker.context.config import METAFILE
from docker.context.config import get_current_context_name
from docker.context.config import write_context_name_to_docker_config
from docker.context import Context
from docker.context.config import (
METAFILE,
get_current_context_name,
get_meta_dir,
write_context_name_to_docker_config,
)


class ContextAPI:
Expand Down
7 changes: 3 additions & 4 deletions docker/context/config.py
@@ -1,10 +1,9 @@
import os
import json
import hashlib
import json
import os

from docker import utils
from docker.constants import IS_WINDOWS_PLATFORM
from docker.constants import DEFAULT_UNIX_SOCKET
from docker.constants import DEFAULT_UNIX_SOCKET, IS_WINDOWS_PLATFORM
from docker.utils.config import find_config_file

METAFILE = "meta.json"
Expand Down
15 changes: 9 additions & 6 deletions docker/context/context.py
@@ -1,12 +1,15 @@
import os
import json
import os
from shutil import copyfile, rmtree
from docker.tls import TLSConfig

from docker.context.config import (
get_context_host,
get_meta_dir,
get_meta_file,
get_tls_dir,
)
from docker.errors import ContextException
from docker.context.config import get_meta_dir
from docker.context.config import get_meta_file
from docker.context.config import get_tls_dir
from docker.context.config import get_context_host
from docker.tls import TLSConfig


class Context:
Expand Down
4 changes: 2 additions & 2 deletions docker/credentials/__init__.py
@@ -1,8 +1,8 @@
from .store import Store
from .errors import StoreError, CredentialsNotFound
from .constants import (
DEFAULT_LINUX_STORE,
DEFAULT_OSX_STORE,
DEFAULT_WIN32_STORE,
PROGRAM_PREFIX,
)
from .errors import CredentialsNotFound, StoreError
from .store import Store
3 changes: 1 addition & 2 deletions docker/credentials/store.py
Expand Up @@ -4,8 +4,7 @@
import subprocess
import warnings

from . import constants
from . import errors
from . import constants, errors
from .utils import create_environment_dict


Expand Down
2 changes: 1 addition & 1 deletion docker/models/configs.py
@@ -1,5 +1,5 @@
from ..api import APIClient
from .resource import Model, Collection
from .resource import Collection, Model


class Config(Model):
Expand Down
11 changes: 7 additions & 4 deletions docker/models/containers.py
Expand Up @@ -2,16 +2,19 @@
import ntpath
from collections import namedtuple

from .images import Image
from .resource import Collection, Model
from ..api import APIClient
from ..constants import DEFAULT_DATA_CHUNK_SIZE
from ..errors import (
ContainerError, DockerException, ImageNotFound,
NotFound, create_unexpected_kwargs_error
ContainerError,
DockerException,
ImageNotFound,
NotFound,
create_unexpected_kwargs_error,
)
from ..types import HostConfig, NetworkingConfig
from ..utils import version_gte
from .images import Image
from .resource import Collection, Model


class Container(Model):
Expand Down
2 changes: 1 addition & 1 deletion docker/models/networks.py
@@ -1,7 +1,7 @@
from ..api import APIClient
from ..utils import version_gte
from .containers import Container
from .resource import Model, Collection
from .resource import Collection, Model


class Network(Model):
Expand Down
2 changes: 1 addition & 1 deletion docker/models/nodes.py
@@ -1,4 +1,4 @@
from .resource import Model, Collection
from .resource import Collection, Model


class Node(Model):
Expand Down
2 changes: 1 addition & 1 deletion docker/models/secrets.py
@@ -1,5 +1,5 @@
from ..api import APIClient
from .resource import Model, Collection
from .resource import Collection, Model


class Secret(Model):
Expand Down
8 changes: 5 additions & 3 deletions docker/models/services.py
@@ -1,7 +1,9 @@
import copy
from docker.errors import create_unexpected_kwargs_error, InvalidArgument
from docker.types import TaskTemplate, ContainerSpec, Placement, ServiceMode
from .resource import Model, Collection

from docker.errors import InvalidArgument, create_unexpected_kwargs_error
from docker.types import ContainerSpec, Placement, ServiceMode, TaskTemplate

from .resource import Collection, Model


class Service(Model):
Expand Down
1 change: 1 addition & 0 deletions docker/models/swarm.py
@@ -1,5 +1,6 @@
from docker.api import APIClient
from docker.errors import APIError

from .resource import Model


Expand Down
2 changes: 1 addition & 1 deletion docker/models/volumes.py
@@ -1,5 +1,5 @@
from ..api import APIClient
from .resource import Model, Collection
from .resource import Collection, Model


class Volume(Model):
Expand Down
1 change: 1 addition & 0 deletions docker/transport/__init__.py
@@ -1,4 +1,5 @@
from .unixconn import UnixHTTPAdapter

try:
from .npipeconn import NpipeHTTPAdapter
from .npipesocket import NpipeSocket
Expand Down
7 changes: 4 additions & 3 deletions docker/transport/npipeconn.py
@@ -1,13 +1,14 @@
import queue

import requests.adapters
import urllib3
import urllib3.connection

from docker.transport.basehttpadapter import BaseHTTPAdapter

from .. import constants
from .npipesocket import NpipeSocket

import urllib3
import urllib3.connection

RecentlyUsedContainer = urllib3._collections.RecentlyUsedContainer


Expand Down
8 changes: 4 additions & 4 deletions docker/transport/npipesocket.py
@@ -1,12 +1,12 @@
import functools
import time
import io
import time

import win32file
import win32pipe
import pywintypes
import win32event
import win32api
import win32event
import win32file
import win32pipe

cERROR_PIPE_BUSY = 0xe7
cSECURITY_SQOS_PRESENT = 0x100000
Expand Down
15 changes: 8 additions & 7 deletions docker/transport/sshconn.py
@@ -1,19 +1,20 @@
import paramiko
import queue
import urllib.parse
import requests.adapters
import logging
import os
import queue
import signal
import socket
import subprocess
import urllib.parse

from docker.transport.basehttpadapter import BaseHTTPAdapter
from .. import constants

import paramiko
import requests.adapters
import urllib3
import urllib3.connection

from docker.transport.basehttpadapter import BaseHTTPAdapter

from .. import constants

RecentlyUsedContainer = urllib3._collections.RecentlyUsedContainer


Expand Down
8 changes: 4 additions & 4 deletions docker/transport/unixconn.py
@@ -1,12 +1,12 @@
import requests.adapters
import socket

from docker.transport.basehttpadapter import BaseHTTPAdapter
from .. import constants

import requests.adapters
import urllib3
import urllib3.connection

from docker.transport.basehttpadapter import BaseHTTPAdapter

from .. import constants

RecentlyUsedContainer = urllib3._collections.RecentlyUsedContainer

Expand Down

0 comments on commit 4c6437d

Please sign in to comment.