Better telemetry

Author: emyller

api/integrations/gitlab/views.py

@@ -16,8 +16,8 @@ def _log_for(
         self, instance: GitLabConfiguration
     ) -> structlog.typing.FilteringBoundLogger:
         return logger.bind(  # type: ignore[no-any-return]
-            project_id=instance.project.id,
-            organisation_id=instance.project.organisation_id,
+            project__id=instance.project.id,
+            organisation__id=instance.project.organisation_id,
         )
 
     def perform_create(self, serializer: GitLabConfigurationSerializer) -> None:  # type: ignore[override]

api/tests/unit/integrations/gitlab/test_views.py

@@ -44,8 +44,8 @@ def test_create_configuration__valid_data__persists_and_masks_token(
             "event": "gitlab-configuration-created",
             "level": "info",
             "gitlab_instance_url": "https://gitlab.example.com",
-            "project_id": project.id,
-            "organisation_id": project.organisation_id,
+            "project__id": project.id,
+            "organisation__id": project.organisation_id,
         },
     ]
 
@@ -98,8 +98,8 @@ def test_update_configuration__valid_data__persists_and_masks_token(
             "event": "gitlab-configuration-updated",
             "level": "info",
             "gitlab_instance_url": "https://gitlab.updated.com",
-            "project_id": project.id,
-            "organisation_id": project.organisation_id,
+            "project__id": project.id,
+            "organisation__id": project.organisation_id,
         },
     ]
 
@@ -123,8 +123,8 @@ def test_delete_configuration__existing__soft_deletes(
         {
             "event": "gitlab-configuration-deleted",
             "level": "info",
-            "project_id": project.id,
-            "organisation_id": project.organisation_id,
+            "project__id": project.id,
+            "organisation__id": project.organisation_id,
         },
     ]