Remove padding and add propagator to CI

Author: ocelotl

Diff for: .github/workflows/test.yml

@@ -23,7 +23,7 @@ jobs:
       fail-fast: false  # ensures the entire test matrix is run, even if one permutation fails
       matrix:
         python-version: [ py35, py36, py37, py38, pypy3 ]
-        package: ["instrumentation", "exporter", "sdkextension"]
+        package: ["instrumentation", "exporter", "sdkextension", "propagator"]
         os: [ ubuntu-latest ]
         include:
           # py35-instrumentation segfaults on 18.04 so we instead run on 20.04

Diff for: propagator/opentelemetry-propagator-opentracing/src/opentelemetry/propagator/opentracing/__init__.py

@@ -73,13 +73,6 @@ def extract(
             and spanid != INVALID_SPAN_ID
             and _valid_spanid.match(spanid) is not None
         ):
-            # FIXME this padding is lost when trace_id is casted to int when a
-            # SpanContext is created. This causes a test case to fail. The
-            # approach coded here attempts to closely follow JS implementation:
-            # https://github.com/open-telemetry/opentelemetry-js-contrib/pull/318/files?file-filters%5B%5D=.js&file-filters%5B%5D=.json&file-filters%5B%5D=.ts&file-filters%5B%5D=dotfile#diff-824ceb4ccf0b4daec4207237a16f2ff4833d941e2a931bd45788f9620b4e8837R91
-            if len(traceid) == 16:
-                traceid = "".join(["0" * 16, traceid])
-
             context = set_span_in_context(
                 DefaultSpan(
                     SpanContext(

Diff for: propagator/opentelemetry-propagator-opentracing/tests/test_opentracing_propagator.py

@@ -270,27 +270,6 @@ def test_extract_trace_id_span_id_sampled_false(self):
         self.assertTrue(span_context.is_remote)
         self.assertEqual(span_context.trace_flags, TraceFlags.DEFAULT)
 
-    def test_extract_padding_trace_id(self):
-        """Test padding of trace_id"""
-
-        span_context = get_current_span(
-            self.opentracing_propagator.extract(
-                carrier_getter,
-                {
-                    OT_TRACE_ID_HEADER: "64fe8b2a57d3eff7",
-                    OT_SPAN_ID_HEADER: "e457b5a2e4d86bd1",
-                    OT_SAMPLED_HEADER: "false",
-                },
-            )
-        ).get_span_context()
-
-        self.assertEqual(
-            hex(span_context.trace_id)[2:], "000000000000000064fe8b2a57d3eff7"
-        )
-        self.assertEqual(hex(span_context.span_id)[2:], "e457b5a2e4d86bd1")
-        self.assertTrue(span_context.is_remote)
-        self.assertEqual(span_context.trace_flags, TraceFlags.DEFAULT)
-
     def test_extract_malformed_trace_id(self):
         """Test extraction with malformed trace_id"""