fix(hc): Move OrganizationApiKeyDetailsEndpoint to control silo (#52776)

The ApiKey model lives in the control silo.

Author: mrduncan

src/sentry/api/endpoints/organization_api_key_details.py

@@ -3,8 +3,11 @@
 from rest_framework.response import Response
 
 from sentry import audit_log
-from sentry.api.base import region_silo_endpoint
-from sentry.api.bases.organization import OrganizationAdminPermission, OrganizationEndpoint
+from sentry.api.base import control_silo_endpoint
+from sentry.api.bases.organization import (
+    ControlSiloOrganizationEndpoint,
+    OrganizationAdminPermission,
+)
 from sentry.api.exceptions import ResourceDoesNotExist
 from sentry.api.serializers import serialize
 from sentry.models import ApiKey
@@ -16,11 +19,11 @@ class Meta:
         fields = ("label", "scope_list", "allowed_origins")
 
 
-@region_silo_endpoint
-class OrganizationApiKeyDetailsEndpoint(OrganizationEndpoint):
+@control_silo_endpoint
+class OrganizationApiKeyDetailsEndpoint(ControlSiloOrganizationEndpoint):
     permission_classes = (OrganizationAdminPermission,)
 
-    def get(self, request: Request, organization, api_key_id) -> Response:
+    def get(self, request: Request, organization_context, organization, api_key_id) -> Response:
         """
         Retrieves API Key details
         `````````````````````````
@@ -37,7 +40,7 @@ def get(self, request: Request, organization, api_key_id) -> Response:
 
         return Response(serialize(api_key, request.user))
 
-    def put(self, request: Request, organization, api_key_id) -> Response:
+    def put(self, request: Request, organization_context, organization, api_key_id) -> Response:
         """
         Update an API Key
         `````````````````
@@ -73,7 +76,7 @@ def put(self, request: Request, organization, api_key_id) -> Response:
 
         return Response(serializer.errors, status=status.HTTP_400_BAD_REQUEST)
 
-    def delete(self, request: Request, organization, api_key_id) -> Response:
+    def delete(self, request: Request, organization_context, organization, api_key_id) -> Response:
         """
         Deletes an API Key
         ``````````````````

tests/sentry/api/endpoints/test_organization_api_key_details.py

@@ -1,6 +1,6 @@
 from sentry.models import ApiKey
 from sentry.testutils import APITestCase
-from sentry.testutils.silo import region_silo_test
+from sentry.testutils.silo import control_silo_test
 
 DEFAULT_SCOPES = ["project:read", "event:read", "team:read", "org:read", "member:read"]
 
@@ -16,7 +16,7 @@ def setUp(self):
         )
 
 
-@region_silo_test
+@control_silo_test(stable=True)
 class OrganizationApiKeyDetails(OrganizationApiKeyDetailsBase):
     def test_api_key_no_exist(self):
         self.get_error_response(self.organization.slug, 123456, status_code=404)
@@ -26,7 +26,7 @@ def test_get_api_details(self):
         assert response.data.get("id") == str(self.api_key.id)
 
 
-@region_silo_test
+@control_silo_test(stable=True)
 class OrganizationApiKeyDetailsPut(OrganizationApiKeyDetailsBase):
     method = "put"
 
@@ -40,7 +40,7 @@ def test_update_api_key_details(self):
         assert api_key.allowed_origins == "sentry.io"
 
 
-@region_silo_test
+@control_silo_test(stable=True)
 class OrganizationApiKeyDetailsDelete(OrganizationApiKeyDetailsBase):
     method = "delete"