Fixed after CR

Author: shalevr

Diff for: instrumentation/opentelemetry-instrumentation-urllib3/src/opentelemetry/instrumentation/urllib3/__init__.py

@@ -205,7 +205,7 @@ def instrumented_urlopen(wrapped, instance, args, kwargs):
             with _suppress_further_instrumentation():
                 start_time = default_timer()
                 response = wrapped(*args, **kwargs)
-                elapsed_time = (default_timer() - start_time) * 1000
+                elapsed_time = round((default_timer() - start_time) * 1000)
 
             _apply_response(span, response)
             if callable(response_hook):
@@ -227,12 +227,14 @@ def instrumented_urlopen(wrapped, instance, args, kwargs):
                 )
 
             request_size = 0 if body is None else len(body)
+            response_size = int(response.headers.get("Content-Length", 0))
+
             duration_histogram.record(elapsed_time, attributes=metric_labels)
             request_size_histogram.record(
                 request_size, attributes=metric_labels
             )
             response_size_histogram.record(
-                len(response.data), attributes=metric_labels
+                response_size, attributes=metric_labels
             )
 
             return response

Diff for: instrumentation/opentelemetry-instrumentation-urllib3/src/opentelemetry/instrumentation/urllib3/version.py

@@ -12,4 +12,4 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-__version__ = "0.33b0"
+__version__ = "0.32b0"

Diff for: instrumentation/opentelemetry-instrumentation-urllib3/tests/test_urllib3_ip_support.py

@@ -197,7 +197,7 @@ def test_basic_metric_check_client_size_post(self):
                                 self.assertAlmostEqual(
                                     data_point.sum,
                                     client_duration_estimated,
-                                    delta=5000,
+                                    delta=1000,
                                 )
                             self.assertIn(metric.name, expected_metrics)