feat(profiling): Detect chunks without client sdk (#89195)

There were a few sdk versions where the chunks did not contain the
client sdk info. This picks them out and assigns a default to them as we
want to identify them.

Author: Zylphrex

src/sentry/profiles/task.py

@@ -1137,19 +1137,38 @@ def determine_profile_type(profile: Profile) -> EventType:
     raise UnknownProfileTypeException
 
 
-def track_latest_sdk(project: Project, profile: Profile) -> None:
-    event_type = determine_profile_type(profile)
-
+def determine_client_sdk(profile: Profile, event_type: EventType) -> tuple[str, str]:
     client_sdk = profile.get("client_sdk")
 
-    if not client_sdk:
-        raise UnknownClientSDKException
+    if client_sdk:
+        sdk_name = client_sdk.get("name")
+        sdk_version = client_sdk.get("version")
+
+        if sdk_name and sdk_version:
+            return sdk_name, sdk_version
+
+    # some older sdks were sending the profile chunk without the
+    # sdk info, here we hard code a few and assign them a guaranteed
+    # outdated version
+    if event_type == EventType.PROFILE_CHUNK:
+        if profile["platform"] == "python":
+            return "sentry.python", "0.0.0"
+        elif profile["platform"] == "cocoa":
+            return "sentry.cocoa", "0.0.0"
+        elif profile["platform"] == "node":
+            # there are other node platforms but it's not straight forward
+            # to figure out which it is here so collapse them all into just node
+            return "sentry.javascript.node", "0.0.0"
 
-    sdk_name = client_sdk.get("name")
-    sdk_version = client_sdk.get("version")
+        # Other platforms do not have a version released where it sends
+        # a profile chunk without the client sdk info
 
-    if not sdk_name or not sdk_version:
-        raise UnknownClientSDKException
+    raise UnknownClientSDKException
+
+
+def track_latest_sdk(project: Project, profile: Profile) -> None:
+    event_type = determine_profile_type(profile)
+    sdk_name, sdk_version = determine_client_sdk(profile, event_type)
 
     try:
         ProjectSDK.update_with_newest_version_or_create(

tests/sentry/profiles/test_task.py

@@ -977,3 +977,51 @@ def test_track_latest_sdk(
         )
         is not None
     )
+
+
+@patch("sentry.profiles.task._push_profile_to_vroom")
+@patch("sentry.profiles.task._symbolicate_profile")
+@patch("sentry.models.projectsdk.get_sdk_index")
+@pytest.mark.parametrize(
+    ["platform", "sdk_name"],
+    [
+        ("python", "sentry.python"),
+        ("cocoa", "sentry.cocoa"),
+        ("node", "sentry.javascript.node"),
+    ],
+)
+@django_db_all
+def test_unknown_sdk(
+    get_sdk_index,
+    _symbolicate_profile,
+    _push_profile_to_vroom,
+    platform,
+    sdk_name,
+    organization,
+    project,
+    request,
+):
+    _push_profile_to_vroom.return_value = True
+    _symbolicate_profile.return_value = True
+    get_sdk_index.return_value = {
+        sdk_name: {},
+    }
+
+    profile = request.getfixturevalue("sample_v2_profile")
+    profile["organization_id"] = organization.id
+    profile["project_id"] = project.id
+    profile["platform"] = platform
+    del profile["client_sdk"]
+
+    with Feature("organizations:profiling-sdks"):
+        process_profile_task(profile=profile)
+
+    assert (
+        ProjectSDK.objects.get(
+            project=project,
+            event_type=EventType.PROFILE_CHUNK.value,
+            sdk_name=sdk_name,
+            sdk_version="0.0.0",
+        )
+        is not None
+    )