Add explicit __all__ to experimental _logs modules

Author: jenshnielsen

opentelemetry-api/src/opentelemetry/_logs/__init__.py

@@ -34,9 +34,7 @@
 .. versionadded:: 1.15.0
 """
 
-# pylint: disable=unused-import
-
-from opentelemetry._logs._internal import (  # noqa: F401
+from opentelemetry._logs._internal import (
     Logger,
     LoggerProvider,
     LogRecord,
@@ -46,13 +44,17 @@
     get_logger_provider,
     set_logger_provider,
 )
-from opentelemetry._logs.severity import (  # noqa: F401
-    SeverityNumber,
-    std_to_otel,
-)
-
-__all__ = []
-for key, value in globals().copy().items():  # type: ignore
-    if not key.startswith("_"):
-        value.__module__ = __name__  # type: ignore
-        __all__.append(key)
+from opentelemetry._logs.severity import SeverityNumber, std_to_otel
+
+__all__ = [
+    "Logger",
+    "LoggerProvider",
+    "LogRecord",
+    "NoOpLogger",
+    "NoOpLoggerProvider",
+    "get_logger",
+    "get_logger_provider",
+    "set_logger_provider",
+    "SeverityNumber",
+    "std_to_otel",
+]

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

@@ -12,9 +12,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# pylint: disable=unused-import
 
-from opentelemetry.sdk._logs._internal import (  # noqa: F401
+from opentelemetry.sdk._logs._internal import (
     LogData,
     Logger,
     LoggerProvider,
@@ -23,8 +22,11 @@
     LogRecordProcessor,
 )
 
-__all__ = []
-for key, value in globals().copy().items():
-    if not key.startswith("_"):
-        value.__module__ = __name__
-        __all__.append(key)
+__all__ = [
+    "LogData",
+    "Logger",
+    "LoggerProvider",
+    "LoggingHandler",
+    "LogRecord",
+    "LogRecordProcessor",
+]

opentelemetry-sdk/src/opentelemetry/sdk/_logs/export/__init__.py

@@ -12,9 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-# pylint: disable=unused-import
-
-from opentelemetry.sdk._logs._internal.export import (  # noqa: F401
+from opentelemetry.sdk._logs._internal.export import (
     BatchLogRecordProcessor,
     ConsoleLogExporter,
     LogExporter,
@@ -23,12 +21,15 @@
 )
 
 # The point module is not in the export directory to avoid a circular import.
-from opentelemetry.sdk._logs._internal.export.in_memory_log_exporter import (  # noqa: F401
+from opentelemetry.sdk._logs._internal.export.in_memory_log_exporter import (
     InMemoryLogExporter,
 )
 
-__all__ = []
-for key, value in globals().copy().items():
-    if not key.startswith("_"):
-        value.__module__ = __name__
-        __all__.append(key)
+__all__ = [
+    "BatchLogRecordProcessor",
+    "ConsoleLogExporter",
+    "LogExporter",
+    "LogExportResult",
+    "SimpleLogRecordProcessor",
+    "InMemoryLogExporter",
+]