adding logging format lint check (#4525)

* adding logging format lint check

* changes from ruff format command

* reapplying format changes changes

---------

Co-authored-by: Riccardo Magliocchetti <[email protected]>

Author: thogar-computer

exporter/opentelemetry-exporter-otlp-proto-common/src/opentelemetry/exporter/otlp/proto/common/_internal/metrics_encoder/__init__.py

@@ -117,8 +117,9 @@ def _get_temporality(
                 _logger.warning(
                     "Unrecognized OTEL_EXPORTER_METRICS_TEMPORALITY_PREFERENCE"
                     " value found: "
-                    f"{otel_exporter_otlp_metrics_temporality_preference}, "
-                    "using CUMULATIVE"
+                    "%s, "
+                    "using CUMULATIVE",
+                    otel_exporter_otlp_metrics_temporality_preference,
                 )
             instrument_class_temporality = {
                 Counter: AggregationTemporality.CUMULATIVE,

exporter/opentelemetry-exporter-otlp-proto-grpc/src/opentelemetry/exporter/otlp/proto/grpc/exporter.py

@@ -125,7 +125,8 @@ def _read_file(file_path: str) -> Optional[bytes]:
             return file.read()
     except FileNotFoundError as e:
         logger.exception(
-            f"Failed to read file: {e.filename}. Please check if the file exists and is accessible."
+            "Failed to read file: %s. Please check if the file exists and is accessible.",
+            e.filename,
         )
         return None
 

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

@@ -130,8 +130,7 @@ def __exit__(self, exc_type, exc_value, traceback):
                 # pylint: disable=broad-exception-caught
                 except Exception as error_handling_error:
                     logger.exception(
-                        "%s error while handling error"
-                        " %s by error handler %s",
+                        "%s error while handling error %s by error handler %s",
                         error_handling_error.__class__.__name__,
                         exc_value.__class__.__name__,
                         error_handler_class.__name__,

pyproject.toml

@@ -77,6 +77,7 @@ select = [
   "PLC", # pylint convention
   "PLE", # pylint error
   "Q",   # flake8-quotes
+  "G",   # https://docs.astral.sh/ruff/rules/#flake8-logging-format-g
 ]
 
 ignore = [