Fix CI: add CHANGELOG entry, apply ruff format, fix pylint variable name

- Add CHANGELOG.md entry for the 413 splitting feature
- Apply ruff format to source files (line wrapping adjustments)
- Rename loop variable 'i' to 'idx' to satisfy pylint naming convention

Author: Krishnachaitanyakc

CHANGELOG.md

@@ -12,6 +12,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+- `opentelemetry-exporter-otlp-proto-http`: Handle HTTP 413 (Payload Too Large) responses in trace and log exporters by splitting the batch in half and retrying each half recursively
+  ([#5032](https://github.com/open-telemetry/opentelemetry-python/pull/5032))
 - `opentelemetry-sdk`: Add file configuration support with YAML/JSON loading, environment variable substitution, and schema validation against the vendored OTel config JSON schema
   ([#4898](https://github.com/open-telemetry/opentelemetry-python/pull/4898))
 - Fix intermittent CI failures in `getting-started` and `tracecontext` jobs caused by GitHub git CDN SHA propagation lag by installing contrib packages from the already-checked-out local copy instead of a second git clone

exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/_log_exporter/__init__.py

@@ -227,9 +227,7 @@ def _export_batch(
                         "Payload too large (%d log records), splitting into two batches",
                         len(batch),
                     )
-                    first = self._export_batch(
-                        list(batch[:mid]), deadline_sec
-                    )
+                    first = self._export_batch(list(batch[:mid]), deadline_sec)
                     if first != LogRecordExportResult.SUCCESS:
                         return LogRecordExportResult.FAILURE
                     second = self._export_batch(

exporter/opentelemetry-exporter-otlp-proto-http/src/opentelemetry/exporter/otlp/proto/http/trace_exporter/__init__.py

@@ -220,9 +220,7 @@ def _export_batch(
                         "Payload too large (%d spans), splitting into two batches",
                         len(spans),
                     )
-                    first = self._export_batch(
-                        list(spans[:mid]), deadline_sec
-                    )
+                    first = self._export_batch(list(spans[:mid]), deadline_sec)
                     if first != SpanExportResult.SUCCESS:
                         return SpanExportResult.FAILURE
                     second = self._export_batch(

exporter/opentelemetry-exporter-otlp-proto-http/tests/test_proto_span_exporter.py

@@ -465,14 +465,14 @@ def test_413_recursive_splitting(self, mock_post):
         resp_ok.status_code = 200
 
         spans = []
-        for i in range(4):
+        for idx in range(4):
             spans.append(
                 _Span(
-                    f"span{i}",
+                    f"span{idx}",
                     context=Mock(
                         **{
                             "trace_state": {},
-                            "span_id": 10217189687419569865 + i,
+                            "span_id": 10217189687419569865 + idx,
                             "trace_id": 67545097771067222548457157018666467027,
                         }
                     ),
@@ -582,4 +582,3 @@ def test_413_deadline_expired_returns_failure(self, mock_post, mock_time):
                 for record in warning.records
             )
         )
-