]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: Module imported but unused (F401)
authorMichael Fritch <mfritch@suse.com>
Wed, 3 Feb 2021 14:06:06 +0000 (07:06 -0700)
committerMichael Fritch <mfritch@suse.com>
Fri, 5 Feb 2021 18:20:58 +0000 (11:20 -0700)
Signed-off-by: Michael Fritch <mfritch@suse.com>
25 files changed:
src/pybind/mgr/cephadm/__init__.py
src/pybind/mgr/cephadm/migrations.py
src/pybind/mgr/cephadm/module.py
src/pybind/mgr/cephadm/schedule.py
src/pybind/mgr/cephadm/serve.py
src/pybind/mgr/cephadm/services/cephadmservice.py
src/pybind/mgr/cephadm/services/ha_rgw.py
src/pybind/mgr/cephadm/services/iscsi.py
src/pybind/mgr/cephadm/services/nfs.py
src/pybind/mgr/cephadm/services/osd.py
src/pybind/mgr/cephadm/template.py
src/pybind/mgr/cephadm/tests/conftest.py [new file with mode: 0644]
src/pybind/mgr/cephadm/tests/fixtures.py
src/pybind/mgr/cephadm/tests/test_cephadm.py
src/pybind/mgr/cephadm/tests/test_completion.py
src/pybind/mgr/cephadm/tests/test_migration.py
src/pybind/mgr/cephadm/tests/test_osd_removal.py
src/pybind/mgr/cephadm/tests/test_services.py
src/pybind/mgr/cephadm/tests/test_spec.py
src/pybind/mgr/cephadm/tests/test_template.py
src/pybind/mgr/cephadm/tests/test_upgrade.py
src/pybind/mgr/cephadm/upgrade.py
src/pybind/mgr/orchestrator/__init__.py
src/pybind/mgr/orchestrator/module.py
src/pybind/mgr/tox.ini

index f390f18c16aaacde7e142cd485dacdc48135f23c..597d883f7ae399244bae5bfa14719c850c165f1e 100644 (file)
@@ -1,6 +1,10 @@
-import os
+from .module import CephadmOrchestrator
+
+__all__ = [
+    "CephadmOrchestrator",
+]
 
+import os
 if 'UNITTEST' in os.environ:
     import tests
-
-from .module import CephadmOrchestrator
+    __all__.append(tests.__name__)
index a45cdb9a926d2258cc2b82ca779920bbabe30cfd..0b4ac9780264a60664dc89579e4a62e2b20984d0 100644 (file)
@@ -1,5 +1,5 @@
 import logging
-from typing import TYPE_CHECKING, Iterator, Union
+from typing import TYPE_CHECKING, Iterator
 
 from ceph.deployment.service_spec import PlacementSpec, ServiceSpec, HostPlacementSpec
 from cephadm.schedule import HostAssignment
index 85bb7b45b350e46a4fd258a403715ac8f2e306db..40d1be8b039e27237316f7e59bd3a6787fc8e404 100644 (file)
@@ -5,14 +5,13 @@ import re
 import shlex
 from collections import defaultdict
 from configparser import ConfigParser
-from contextlib import contextmanager
 from functools import wraps
 from tempfile import TemporaryDirectory
 from threading import Event
 
 import string
 from typing import List, Dict, Optional, Callable, Tuple, TypeVar, \
-    Any, Set, TYPE_CHECKING, cast, Iterator, NamedTuple, Sequence
+    Any, Set, TYPE_CHECKING, cast, NamedTuple, Sequence
 
 import datetime
 import os
@@ -25,19 +24,19 @@ from ceph.deployment import inventory
 from ceph.deployment.drive_group import DriveGroupSpec
 from ceph.deployment.service_spec import \
     NFSServiceSpec, ServiceSpec, PlacementSpec, assert_valid_host, \
-    HostPlacementSpec, HA_RGWSpec
+    HostPlacementSpec
 from ceph.utils import str_to_datetime, datetime_to_str, datetime_now
 from cephadm.serve import CephadmServe
 from cephadm.services.cephadmservice import CephadmDaemonSpec
 
 from mgr_module import MgrModule, HandleCommandResult, Option
-from mgr_util import create_self_signed_cert, verify_tls, ServerConfigException
+from mgr_util import create_self_signed_cert
 import secrets
 import orchestrator
 from orchestrator import OrchestratorError, OrchestratorValidationError, HostSpec, \
-    CLICommandMeta, OrchestratorEvent, set_exception_subject, DaemonDescription
+    CLICommandMeta, DaemonDescription
 from orchestrator._interface import GenericSpec
-from orchestrator._interface import daemon_type_to_service, service_to_daemon_types
+from orchestrator._interface import daemon_type_to_service
 
 from . import remotes
 from . import utils
@@ -48,7 +47,7 @@ from .services.container import CustomContainerService
 from .services.iscsi import IscsiService
 from .services.ha_rgw import HA_RGWService
 from .services.nfs import NFSService
-from .services.osd import RemoveUtil, OSDRemovalQueue, OSDService, OSD, NotFoundError
+from .services.osd import OSDRemovalQueue, OSDService, OSD, NotFoundError
 from .services.monitoring import GrafanaService, AlertmanagerService, PrometheusService, \
     NodeExporterService
 from .schedule import HostAssignment
@@ -69,7 +68,6 @@ try:
         from remoto.backends import BaseConnection
         BaseConnection.has_connection = remoto_has_connection
     import remoto.process
-    import execnet.gateway_bootstrap
 except ImportError as e:
     remoto = None
     remoto_import_error = str(e)
index 7913c35e6788b957d4243e5116233c15f327ad84..7d2e2308bae615251f54f69f899e1e318b1a4597 100644 (file)
@@ -3,7 +3,7 @@ import random
 from typing import List, Optional, Callable, Iterable, TypeVar, Set
 
 import orchestrator
-from ceph.deployment.service_spec import PlacementSpec, HostPlacementSpec, ServiceSpec
+from ceph.deployment.service_spec import HostPlacementSpec, ServiceSpec
 from orchestrator._interface import DaemonDescription
 from orchestrator import OrchestratorValidationError
 
index 17e279e52005eb639b5c14e1f9b936bbced6badd..3e7b040abb311f9471d90a2e5c4c16d39037d6c9 100644 (file)
@@ -1,4 +1,3 @@
-import datetime
 import json
 import logging
 from collections import defaultdict
index eaa24367128abce3a6fa48479ed29a6d97b7a611..8dfb1501d7f09b5807b35fb6dc51e04fc926e776 100644 (file)
@@ -1,9 +1,7 @@
 import json
 import re
 import logging
-import secrets
 import subprocess
-import collections
 from abc import ABCMeta, abstractmethod
 from typing import TYPE_CHECKING, List, Callable, Any, TypeVar, Generic, \
     Optional, Dict, Any, Tuple, NewType
@@ -13,9 +11,9 @@ from mgr_module import HandleCommandResult, MonCommandFailed
 from ceph.deployment.service_spec import ServiceSpec, RGWSpec
 from ceph.deployment.utils import is_ipv6, unwrap_ipv6
 from orchestrator import OrchestratorError, DaemonDescription
-from orchestrator._interface import daemon_type_to_service, service_to_daemon_types
+from orchestrator._interface import daemon_type_to_service
 from cephadm import utils
-from mgr_util import create_self_signed_cert, ServerConfigException, verify_tls
+from mgr_util import ServerConfigException, verify_tls
 
 if TYPE_CHECKING:
     from cephadm.module import CephadmOrchestrator
index 05dea1eaac84c95faadbd732c29a94d99ea98f24..4a37e006d58b5a256a0685a1df44fef93f9367ee 100644 (file)
@@ -1,12 +1,10 @@
-import json
 import logging
 from typing import List, cast, Tuple, Dict, Any
 
 from ceph.deployment.service_spec import HA_RGWSpec
 
-from orchestrator import DaemonDescription, OrchestratorError
 from .cephadmservice import CephadmDaemonSpec, CephService
-from ..utils import CephadmNoImage, cephadmNoImage, resolve_ip
+from ..utils import resolve_ip
 
 logger = logging.getLogger(__name__)
 
index 159c456a0588ba89b91fe488fa8161edb5468020..98e647bfc9851b66214429dca2ba28170f2e4c30 100644 (file)
@@ -2,10 +2,9 @@ import json
 import logging
 from typing import List, cast
 
-from mgr_module import MonCommandFailed
 from ceph.deployment.service_spec import IscsiServiceSpec
 
-from orchestrator import DaemonDescription, OrchestratorError
+from orchestrator import DaemonDescription
 from .cephadmservice import CephadmDaemonSpec, CephService
 from .. import utils
 
index fdd080e3a3d7d90169d3e5e39e620eac69d6fc40..3a6015982abfa3480d3046aaf921c0cd31a61136 100644 (file)
@@ -1,17 +1,13 @@
 import logging
-from typing import TYPE_CHECKING, Dict, Tuple, Any, List
+from typing import Dict, Tuple, Any, List
 
 from ceph.deployment.service_spec import NFSServiceSpec
 import rados
 
-from orchestrator import OrchestratorError, DaemonDescription
+from orchestrator import DaemonDescription
 
-from cephadm import utils
 from cephadm.services.cephadmservice import AuthEntity, CephadmDaemonSpec, CephService
 
-if TYPE_CHECKING:
-    from cephadm.module import CephadmOrchestrator
-
 logger = logging.getLogger(__name__)
 
 
index 2f3cde6dc6e73ea45ac11f4762617d8f2ca88af9..0e898b6d9db3e44c775c367ad5717890e49a5b9d 100644 (file)
@@ -1,7 +1,7 @@
 import json
 import logging
 from threading import Lock
-from typing import List, Dict, Any, Set, Union, Tuple, cast, Optional, TYPE_CHECKING
+from typing import List, Dict, Any, Set, Tuple, cast, Optional, TYPE_CHECKING
 
 from ceph.deployment import translate
 from ceph.deployment.drive_group import DriveGroupSpec
index c59df3f9fb3a6f47b33362eec601e53d02285804..0d62e587c801eae01d9eca88ec5ea833b0fc4643 100644 (file)
@@ -1,7 +1,7 @@
 import copy
 from typing import Optional, TYPE_CHECKING
 
-from jinja2 import Environment, PackageLoader, select_autoescape, StrictUndefined, Template
+from jinja2 import Environment, PackageLoader, select_autoescape, StrictUndefined
 from jinja2 import exceptions as j2_exceptions
 
 if TYPE_CHECKING:
diff --git a/src/pybind/mgr/cephadm/tests/conftest.py b/src/pybind/mgr/cephadm/tests/conftest.py
new file mode 100644 (file)
index 0000000..e8add2c
--- /dev/null
@@ -0,0 +1,27 @@
+import pytest
+
+from cephadm.services.osd import RemoveUtil, OSD
+from tests import mock
+
+from .fixtures import with_cephadm_module
+
+
+@pytest.fixture()
+def cephadm_module():
+    with with_cephadm_module({}) as m:
+        yield m
+
+
+@pytest.fixture()
+def rm_util():
+    with with_cephadm_module({}) as m:
+        r = RemoveUtil.__new__(RemoveUtil)
+        r.__init__(m)
+        yield r
+
+
+@pytest.fixture()
+def osd_obj():
+    with mock.patch("cephadm.services.osd.RemoveUtil"):
+        o = OSD(0, mock.MagicMock())
+        yield o
index 5c14aee1de411fc06da949caa572d75345642633..8ebbb7cd0ddfcddaee6a3196763e938df1b252e5 100644 (file)
@@ -10,10 +10,8 @@ try:
     from typing import Any, Iterator, List
 except ImportError:
     pass
-import pytest
 
 from cephadm import CephadmOrchestrator
-from cephadm.services.osd import RemoveUtil, OSD
 from orchestrator import raise_if_exception, Completion, HostSpec
 from tests import mock
 
@@ -65,33 +63,13 @@ def with_cephadm_module(module_options=None, store=None):
         yield m
 
 
-@pytest.fixture()
-def cephadm_module():
-    with with_cephadm_module({}) as m:
-        yield m
-
-
-@pytest.fixture()
-def rm_util():
-    with with_cephadm_module({}) as m:
-        r = RemoveUtil.__new__(RemoveUtil)
-        r.__init__(m)
-        yield r
-
-
-@pytest.fixture()
-def osd_obj():
-    with mock.patch("cephadm.services.osd.RemoveUtil"):
-        o = OSD(0, mock.MagicMock())
-        yield o
-
-
 def wait(m, c):
     # type: (CephadmOrchestrator, Completion) -> Any
     m.process([c])
 
     try:
-        import pydevd  # if in debugger
+        # if in debugger
+        import pydevd  # noqa: F401
         in_debug = True
     except ImportError:
         in_debug = False
index fcc953d0730ecd589740838fc2086819d8c7e9ec..dfec2ea03b7affa5260d96885758b36b9cd7cb96 100644 (file)
@@ -7,10 +7,9 @@ import pytest
 from ceph.deployment.drive_group import DriveGroupSpec, DeviceSelection
 from cephadm.serve import CephadmServe
 from cephadm.services.osd import OSD, OSDRemovalQueue
-from cephadm.utils import CephadmNoImage
 
 try:
-    from typing import Any, List
+    from typing import List
 except ImportError:
     pass
 
@@ -21,10 +20,10 @@ from ceph.deployment.service_spec import ServiceSpec, PlacementSpec, RGWSpec, \
 from ceph.deployment.drive_selection.selector import DriveSelection
 from ceph.deployment.inventory import Devices, Device
 from ceph.utils import datetime_to_str, datetime_now
-from orchestrator import ServiceDescription, DaemonDescription, InventoryHost, \
+from orchestrator import DaemonDescription, InventoryHost, \
     HostSpec, OrchestratorError
 from tests import mock
-from .fixtures import cephadm_module, wait, _run_cephadm, match_glob, with_host, \
+from .fixtures import wait, _run_cephadm, match_glob, with_host, \
     with_cephadm_module, with_service, assert_rm_service, _deploy_cephadm_binary
 from cephadm.module import CephadmOrchestrator
 
index 80592ba779dd893aa168008834f1117c389dc081..9480ea591ee40b9e2861917ff6be1105ab69b895 100644 (file)
@@ -1,17 +1,6 @@
-import sys
-import time
-
-
-try:
-    from typing import Any
-except ImportError:
-    pass
-
 import pytest
 
-
-from tests import mock
-from .fixtures import cephadm_module, wait
+from .fixtures import wait
 from ..module import trivial_completion, forall_hosts
 
 
index 049743a1fb51cc8df99d9f25fad727cee18f7f37..4b35440f6cdf6190030b2497761f09574aa5bd80 100644 (file)
@@ -1,13 +1,10 @@
 import json
 
-import pytest
-
 from ceph.deployment.service_spec import PlacementSpec, ServiceSpec, HostPlacementSpec
 from ceph.utils import datetime_to_str, datetime_now
 from cephadm import CephadmOrchestrator
 from cephadm.inventory import SPEC_STORE_PREFIX
-from cephadm.tests.fixtures import _run_cephadm, cephadm_module, wait, with_host
-from orchestrator import OrchestratorError
+from cephadm.tests.fixtures import _run_cephadm, wait, with_host
 from cephadm.serve import CephadmServe
 from tests import mock
 
index f93b2c2c41ac483e97c791f861792fc9f59a8d72..c2bf95b439403b86268f1c89309d9a00967022c4 100644 (file)
@@ -1,8 +1,7 @@
 import json
 
-from cephadm.services.osd import RemoveUtil, OSDRemovalQueue, OSD
+from cephadm.services.osd import OSDRemovalQueue, OSD
 import pytest
-from .fixtures import rm_util, osd_obj, cephadm_module
 from tests import mock
 from datetime import datetime
 
index 4891b81ac00b214e901e93bde309b3506f049a47..0411b3a87b16daa97c5d64a17810ad4867c56c5f 100644 (file)
@@ -3,10 +3,10 @@ import pytest
 from unittest.mock import MagicMock, call
 
 from cephadm.services.cephadmservice import MonService, MgrService, MdsService, RgwService, \
-    RbdMirrorService, CrashService, CephadmService, AuthEntity, CephadmExporter
+    RbdMirrorService, CrashService, CephadmExporter
 from cephadm.services.iscsi import IscsiService
 from cephadm.services.nfs import NFSService
-from cephadm.services.osd import RemoveUtil, OSDRemovalQueue, OSDService, OSD, NotFoundError
+from cephadm.services.osd import OSDService
 from cephadm.services.monitoring import GrafanaService, AlertmanagerService, PrometheusService, \
     NodeExporterService
 from ceph.deployment.service_spec import IscsiServiceSpec
index 586344bee9b51d66a88a0022866cd38b81a1dd86..2018fa2cf3598c128859b5aac49f5c9ed039ebf0 100644 (file)
@@ -9,8 +9,7 @@ import pytest
 import yaml
 
 from ceph.deployment.service_spec import ServiceSpec, NFSServiceSpec, RGWSpec, \
-    IscsiServiceSpec, AlertManagerSpec, HostPlacementSpec, CustomContainerSpec, \
-    HA_RGWSpec
+    IscsiServiceSpec, AlertManagerSpec, HostPlacementSpec, CustomContainerSpec
 
 from orchestrator import DaemonDescription, OrchestratorError
 
index 40b783bdbf9ac68f266dd1eab1ebfbc6d9548008..f67304348603c21e4f762ae5a828ca40d118b8ca 100644 (file)
@@ -2,7 +2,6 @@ import pathlib
 
 import pytest
 
-from cephadm.tests.fixtures import cephadm_module
 from cephadm.template import TemplateMgr, UndefinedError, TemplateNotFoundError
 
 
index 7d4f091e9c5fca70ddda147983ed6f84c34cefdf..228811191598d5407382f81c5acc11615cc20e8b 100644 (file)
@@ -7,7 +7,7 @@ from ceph.deployment.service_spec import ServiceSpec
 from cephadm import CephadmOrchestrator
 from cephadm.upgrade import CephadmUpgrade
 from cephadm.serve import CephadmServe
-from .fixtures import _run_cephadm, wait, cephadm_module, with_host, with_service
+from .fixtures import _run_cephadm, wait, with_host, with_service
 
 
 @mock.patch("cephadm.serve.CephadmServe._run_cephadm", _run_cephadm('{}'))
index 00941f6a66780a4b11c5d7ff3d152aa00782c7cf..f55751c3f305b0a1906b4c898ac5d44655e26416 100644 (file)
@@ -7,7 +7,7 @@ from typing import TYPE_CHECKING, Optional, Dict, List
 import orchestrator
 from cephadm.serve import CephadmServe
 from cephadm.utils import ceph_release_to_major, name_to_config_section, CEPH_UPGRADE_ORDER
-from orchestrator import OrchestratorError, DaemonDescription, daemon_type_to_service, service_to_daemon_types
+from orchestrator import OrchestratorError, DaemonDescription, daemon_type_to_service
 
 if TYPE_CHECKING:
     from .module import CephadmOrchestrator
index 4bcb26cabfa3a2e1b33e2536083300f99941f840..367f7b55d42e5a60e70b6400ce29fca995bbcc87 100644 (file)
@@ -1,8 +1,4 @@
 from __future__ import absolute_import
-import os
-
-if 'UNITTEST' in os.environ:
-    import tests
 
 from .module import OrchestratorCli
 
@@ -17,3 +13,39 @@ from ._interface import \
     OrchestratorEvent, set_exception_subject, \
     InventoryHost, DeviceLightLoc, \
     UpgradeStatusSpec, daemon_type_to_service, service_to_daemon_types
+
+__all__ = [
+    'OrchestratorCli',
+    'Completion',
+    'TrivialReadCompletion',
+    'raise_if_exception',
+    'ProgressReference',
+    'pretty_print',
+    '_Promise',
+    'CLICommand',
+    '_cli_write_command',
+    '_cli_read_command',
+    'CLICommandMeta',
+    'Orchestrator',
+    'OrchestratorClientMixin',
+    'OrchestratorValidationError',
+    'OrchestratorError',
+    'NoOrchestrator',
+    'ServiceDescription',
+    'InventoryFilter',
+    'HostSpec',
+    'DaemonDescription',
+    'OrchestratorEvent',
+    'set_exception_subject',
+    'InventoryHost',
+    'DeviceLightLoc',
+    'UpgradeStatusSpec',
+    'daemon_type_to_service',
+    'service_to_daemon_types',
+
+]
+
+import os
+if 'UNITTEST' in os.environ:
+    import tests
+    __all__.append(tests.__name__)
index 23c4247870b3769cf7f0ecbb8b1b03a3ace7b0d9..16aa8bc425b9101d487217ee855098428a0e2ff1 100644 (file)
@@ -13,14 +13,14 @@ from ceph.deployment.drive_group import DriveGroupSpec, DeviceSelection
 from ceph.deployment.service_spec import PlacementSpec, ServiceSpec
 from ceph.utils import datetime_now
 
-from mgr_util import format_bytes, to_pretty_timedelta, format_dimless
+from mgr_util import to_pretty_timedelta, format_dimless
 from mgr_module import MgrModule, HandleCommandResult, Option
 
 from ._interface import OrchestratorClientMixin, DeviceLightLoc, _cli_read_command, \
     raise_if_exception, _cli_write_command, TrivialReadCompletion, OrchestratorError, \
-    NoOrchestrator, OrchestratorValidationError, NFSServiceSpec, HA_RGWSpec, \
+    NoOrchestrator, OrchestratorValidationError, NFSServiceSpec, \
     RGWSpec, InventoryFilter, InventoryHost, HostSpec, CLICommandMeta, \
-    ServiceDescription, DaemonDescription, IscsiServiceSpec, json_to_generic_spec, GenericSpec
+    ServiceDescription, DaemonDescription, IscsiServiceSpec, json_to_generic_spec
 
 
 def nice_delta(now: datetime.datetime, t: Optional[datetime.datetime], suffix: str = '') -> str:
index d2951143e55427731abd9847df5b5fc21a3034da..ab0380419804fcab1ff585b5dc61a0c99abd4858 100644 (file)
@@ -12,7 +12,6 @@ requires = cython
 max-line-length = 100
 ignore =
     E501,
-    F401,
     F811,
     W503,
 exclude =