Revert "chore(profiling): add process_tags (#15226)" (#15363)

Right now process tags are set as a tag. However as profiler is using
libdatadog, it should be added as a root field and not a tag if we want
to not break anything.

This PR reverts the changes I made on profiling.

Author: dubloom

ddtrace/profiling/profiler.py

@@ -12,7 +12,6 @@
 from ddtrace import config
 from ddtrace.internal import atexit
 from ddtrace.internal import forksafe
-from ddtrace.internal import process_tags
 from ddtrace.internal import service
 from ddtrace.internal import uwsgi
 from ddtrace.internal.datadog.profiling import ddup
@@ -169,9 +168,6 @@ def _build_default_exporters(self):
         profiler_config = config_str(profiling_config)
         self.tags.update({"profiler_config": profiler_config})
 
-        if p_tags := process_tags.process_tags:
-            self.tags.update({"process_tags": p_tags})
-
         endpoint_call_counter_span_processor = self.tracer._endpoint_call_counter_span_processor
         if self.endpoint_collection_enabled:
             endpoint_call_counter_span_processor.enable()

tests/profiling/test_profiler.py

@@ -144,20 +144,3 @@ def test_profiler_serverless(monkeypatch):
     p = profiler.Profiler()
     assert isinstance(p._scheduler, scheduler.ServerlessScheduler)
     assert p.tags["functionname"] == "foobar"
-
-
-def test_process_tags_deactivated():
-    p = profiler.Profiler()
-    assert "process_tags" not in p.tags
-
-
-@pytest.mark.subprocess(
-    env=dict(
-        DD_EXPERIMENTAL_PROPAGATE_PROCESS_TAGS_ENABLED="true",
-    )
-)
-def test_process_tags_activated():
-    from ddtrace.profiling import profiler
-
-    p = profiler.Profiler()
-    assert "process_tags" in p.tags

tests/profiling_v2/exporter/test_ddup.py

@@ -62,47 +62,3 @@ def test_tags_propagated():
     # Profiler could add tags, so check that tags is a superset of config.tags
     for k, v in config.tags.items():
         assert tags[k] == v
-
-
-@pytest.mark.subprocess()
-def test_process_tags_deactivated():
-    import sys
-    from unittest import mock
-
-    sys.modules["ddtrace.internal.datadog.profiling.ddup"] = mock.Mock()
-
-    from ddtrace.profiling.profiler import Profiler  # noqa: I001
-    from ddtrace.internal.datadog.profiling import ddup
-
-    # When Profiler is instantiated and libdd is enabled, it should call ddup.config
-    Profiler()
-
-    ddup.config.assert_called()
-
-    tags = ddup.config.call_args.kwargs["tags"]
-
-    assert "process_tags" not in tags
-
-
-@pytest.mark.subprocess(
-    env=dict(
-        DD_EXPERIMENTAL_PROPAGATE_PROCESS_TAGS_ENABLED="true",
-    )
-)
-def test_process_tags_activated():
-    import sys
-    from unittest import mock
-
-    sys.modules["ddtrace.internal.datadog.profiling.ddup"] = mock.Mock()
-
-    from ddtrace.profiling.profiler import Profiler  # noqa: I001
-    from ddtrace.internal.datadog.profiling import ddup
-
-    # When Profiler is instantiated and libdd is enabled, it should call ddup.config
-    Profiler()
-
-    ddup.config.assert_called()
-
-    tags = ddup.config.call_args.kwargs["tags"]
-
-    assert "process_tags" in tags