typecheck: add sdk/resources and drop mypy

Signed-off-by: emdneto <[email protected]>

Author: emdneto

dev-requirements.txt

@@ -1,7 +1,6 @@
 pylint==3.3.4
 httpretty==1.1.4
 pyright==1.1.396
-mypy==1.9.0
 sphinx==7.1.2
 sphinx-rtd-theme==2.0.0rc4
 sphinx-autodoc-typehints==1.25.2

mypy-relaxed.ini

@@ -67,7 +67,7 @@
 from json import dumps
 from os import environ
 from types import ModuleType
-from typing import List, MutableMapping, Optional, cast
+from typing import List, Optional, cast
 from urllib import parse
 
 from opentelemetry.attributes import BoundedAttributes
@@ -77,7 +77,10 @@
     OTEL_SERVICE_NAME,
 )
 from opentelemetry.semconv.resource import ResourceAttributes
-from opentelemetry.util._importlib_metadata import entry_points, version
+from opentelemetry.util._importlib_metadata import (
+    entry_points,  # type: ignore[reportUnknownVariableType]
+    version,
+)
 from opentelemetry.util.types import AttributeValue
 
 psutil: Optional[ModuleType] = None
@@ -210,7 +213,7 @@ def create(
                             entry_points(
                                 group="opentelemetry_resource_detector",
                                 name=resource_detector.strip(),
-                            )  # type: ignore
+                            )  # type: ignore[reportUnknownArgumentType]
                         )
                     ).load()()
                 )
@@ -266,8 +269,8 @@ def merge(self, other: "Resource") -> "Resource":
         Returns:
             The newly-created Resource.
         """
-        merged_attributes = self.attributes.copy()  # type: ignore
-        merged_attributes.update(other.attributes)  # type: ignore
+        merged_attributes = dict(self.attributes).copy()
+        merged_attributes.update(other.attributes)
 
         if self.schema_url == "":
             schema_url = other.schema_url
@@ -282,7 +285,7 @@ def merge(self, other: "Resource") -> "Resource":
                 other.schema_url,
             )
             return self
-        return Resource(merged_attributes, schema_url)  # type: ignore
+        return Resource(merged_attributes, schema_url)
 
     def __eq__(self, other: object) -> bool:
         if not isinstance(other, Resource):
@@ -294,16 +297,13 @@ def __eq__(self, other: object) -> bool:
 
     def __hash__(self) -> int:
         return hash(
-            f"{dumps(self._attributes.copy(), sort_keys=True)}|{self._schema_url}"  # type: ignore
+            f"{dumps(self._attributes.copy(), sort_keys=True)}|{self._schema_url}"
         )
 
     def to_json(self, indent: Optional[int] = 4) -> str:
-        attributes: MutableMapping[str, AttributeValue] = dict(
-            self._attributes
-        )
         return dumps(
             {
-                "attributes": attributes,  # type: ignore
+                "attributes": dict(self._attributes),
                 "schema_url": self._schema_url,
             },
             indent=indent,
@@ -334,7 +334,7 @@ class OTELResourceDetector(ResourceDetector):
     # pylint: disable=no-self-use
     def detect(self) -> "Resource":
         env_resources_items = environ.get(OTEL_RESOURCE_ATTRIBUTES)
-        env_resource_map = {}
+        env_resource_map: dict[str, AttributeValue] = {}
 
         if env_resources_items:
             for item in env_resources_items.split(","):
@@ -392,7 +392,7 @@ def detect(self) -> "Resource":
             resource_info[PROCESS_PARENT_PID] = os.getppid()
 
         if psutil is not None:
-            process: psutil_module.Process = psutil.Process()
+            process = psutil.Process()
             username = process.username()
             resource_info[PROCESS_OWNER] = username
 
@@ -483,7 +483,7 @@ def detect(self) -> "Resource":
         )
 
 
-class _HostResourceDetector(ResourceDetector):
+class _HostResourceDetector(ResourceDetector):  # type: ignore[reportUnusedClass]
     """
     The HostResourceDetector detects the hostname and architecture attributes.
     """

mypy.ini

@@ -112,8 +112,6 @@ exclude = [
   "opentelemetry-sdk/src/opentelemetry/sdk/_configuration",
   "opentelemetry-sdk/src/opentelemetry/sdk/_events",
   "opentelemetry-sdk/src/opentelemetry/sdk/_logs",
-  "opentelemetry-sdk/src/opentelemetry/sdk/error_handler",
-  "opentelemetry-sdk/src/opentelemetry/sdk/resources",
   "opentelemetry-sdk/src/opentelemetry/sdk/metrics",
   "opentelemetry-sdk/src/opentelemetry/sdk/trace",
   "opentelemetry-sdk/src/opentelemetry/sdk/util",
@@ -124,4 +122,5 @@ exclude = [
 strict = [
   "opentelemetry-semantic-conventions",
   "opentelemetry-sdk/src/opentelemetry/sdk/environment_variables",
+  "opentelemetry-sdk/src/opentelemetry/sdk/resources",
 ]

opentelemetry-sdk/src/opentelemetry/sdk/resources/__init__.py

@@ -88,7 +88,6 @@ envlist =
 
     spellcheck
     tracecontext
-    mypy,mypyinstalled
     typecheck
     docs
     docker-tests-{otlpexporter,opencensus}
@@ -103,9 +102,6 @@ deps =
   coverage: pytest
   coverage: pytest-cov
 
-  mypy,mypyinstalled: -c {toxinidir}/dev-requirements.txt
-  mypy,mypyinstalled: mypy
-
   api: -r {toxinidir}/opentelemetry-api/test-requirements.txt
 
   sdk: -r {toxinidir}/opentelemetry-sdk/test-requirements.txt
@@ -145,7 +141,6 @@ deps =
 
   propagator-jaeger: -r {toxinidir}/propagator/opentelemetry-propagator-jaeger/test-requirements.txt
 
-  mypyinstalled: -e {toxinidir}/opentelemetry-api
 
   getting-started: -r {toxinidir}/docs/getting_started/tests/requirements.txt
   getting-started: {env:CONTRIB_REPO}\#egg=opentelemetry-util-http&subdirectory=util/opentelemetry-util-http