Browse Source

Cleaned up code and api owners for owners-native (#72090)

Co-authored-by: jernejstrasner <jernejstrasner@users.noreply.github.com>
Jernej Strasner 9 months ago
parent
commit
ec9c0a9f37

+ 19 - 23
.github/CODEOWNERS

@@ -398,24 +398,6 @@ static/app/components/events/eventStatisticalDetector/                    @getse
 ## End of Enterprise
 
 
-## Native
-/src/sentry/api/endpoints/chunk.py                                   @getsentry/owners-native
-/src/sentry/api/endpoints/project_app_store_connect_credentials.py   @getsentry/owners-native
-/src/sentry/lang/native/                                             @getsentry/owners-native
-/src/sentry/processing/realtime_metrics/                             @getsentry/owners-native
-/src/sentry/tasks/app_store_connect.py                               @getsentry/owners-native
-/src/sentry/tasks/assemble.py                                        @getsentry/owners-native
-/src/sentry/tasks/low_priority_symbolication.py                      @getsentry/owners-native
-/src/sentry/tasks/symbolication.py                                   @getsentry/owners-native
-/src/sentry/utils/appleconnect/                                      @getsentry/owners-native
-/tests/sentry/tasks/test_low_priority_symbolication.py               @getsentry/owners-native
-/src/sentry/tasks/reprocessing.py                                    @getsentry/processing
-/src/sentry/tasks/reprocessing2.py                                   @getsentry/processing
-/src/sentry/reprocessing2.py                                         @getsentry/processing
-/src/sentry/tasks/store.py                                           @getsentry/processing
-## End of Native
-
-
 ## APIs
 /src/sentry/apidocs/                  @getsentry/owners-api
 /src/sentry/api/urls.py               @getsentry/owners-api
@@ -538,11 +520,25 @@ static/app/components/events/eventStatisticalDetector/                    @getse
 ## End of ML & AI
 
 ## Processing
-/src/sentry/processing/*                                @getsentry/processing
-/src/sentry/api/endpoints/project_user_reports.py       @getsentry/processing
-/src/sentry/api/endpoints/event_attachments.py          @getsentry/processing
-/src/sentry/api/endpoints/event_reprocessable.py        @getsentry/processing
-/src/sentry/api/endpoints/project_reprocessing.py       @getsentry/processing
+/src/sentry/processing/*                                             @getsentry/processing
+/src/sentry/api/endpoints/project_user_reports.py                    @getsentry/processing
+/src/sentry/api/endpoints/event_attachments.py                       @getsentry/processing
+/src/sentry/api/endpoints/event_reprocessable.py                     @getsentry/processing
+/src/sentry/api/endpoints/project_reprocessing.py                    @getsentry/processing
+/src/sentry/api/endpoints/chunk.py                                   @getsentry/processing
+/src/sentry/api/endpoints/project_app_store_connect_credentials.py   @getsentry/processing
+/src/sentry/lang/native/                                             @getsentry/processing
+/src/sentry/processing/realtime_metrics/                             @getsentry/processing
+/src/sentry/tasks/app_store_connect.py                               @getsentry/processing
+/src/sentry/tasks/assemble.py                                        @getsentry/processing
+/src/sentry/tasks/low_priority_symbolication.py                      @getsentry/processing
+/src/sentry/tasks/symbolication.py                                   @getsentry/processing
+/src/sentry/utils/appleconnect/                                      @getsentry/processing
+/tests/sentry/tasks/test_low_priority_symbolication.py               @getsentry/processing
+/src/sentry/tasks/reprocessing.py                                    @getsentry/processing
+/src/sentry/tasks/reprocessing2.py                                   @getsentry/processing
+/src/sentry/reprocessing2.py                                         @getsentry/processing
+/src/sentry/tasks/store.py                                           @getsentry/processing
 ## End of Processing
 
 ## Ecosystem

+ 0 - 1
src/sentry/api/api_owners.py

@@ -18,7 +18,6 @@ class ApiOwner(Enum):
     ML_AI = "machine-learning-ai"
     OPEN_SOURCE = "open-source"
     OWNERS_INGEST = "ingest"
-    OWNERS_NATIVE = "owners-native"
     OWNERS_SNUBA = "owners-snuba"
     PERFORMANCE = "performance"
     PROCESSING = "processing"

+ 1 - 1
src/sentry/api/endpoints/builtin_symbol_sources.py

@@ -19,7 +19,7 @@ def normalize_symbol_source(key, source):
 
 @region_silo_endpoint
 class BuiltinSymbolSourcesEndpoint(Endpoint):
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     publish_status = {
         "GET": ApiPublishStatus.PRIVATE,
     }

+ 1 - 1
src/sentry/api/endpoints/chunk.py

@@ -53,7 +53,7 @@ class ChunkUploadEndpoint(OrganizationEndpoint):
         "GET": ApiPublishStatus.PRIVATE,
         "POST": ApiPublishStatus.PRIVATE,
     }
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     permission_classes = (OrganizationReleasePermission,)
     rate_limits = RateLimitConfig(group="CLI")
 

+ 4 - 4
src/sentry/api/endpoints/debug_files.py

@@ -178,7 +178,7 @@ class ProguardArtifactReleasesEndpoint(ProjectEndpoint):
 
 @region_silo_endpoint
 class DebugFilesEndpoint(ProjectEndpoint):
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     publish_status = {
         "DELETE": ApiPublishStatus.UNKNOWN,
         "GET": ApiPublishStatus.UNKNOWN,
@@ -358,7 +358,7 @@ class DebugFilesEndpoint(ProjectEndpoint):
 
 @region_silo_endpoint
 class UnknownDebugFilesEndpoint(ProjectEndpoint):
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     publish_status = {
         "GET": ApiPublishStatus.PRIVATE,
     }
@@ -372,7 +372,7 @@ class UnknownDebugFilesEndpoint(ProjectEndpoint):
 
 @region_silo_endpoint
 class AssociateDSymFilesEndpoint(ProjectEndpoint):
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     publish_status = {
         "POST": ApiPublishStatus.PRIVATE,
     }
@@ -385,7 +385,7 @@ class AssociateDSymFilesEndpoint(ProjectEndpoint):
 
 @region_silo_endpoint
 class DifAssembleEndpoint(ProjectEndpoint):
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     publish_status = {
         "POST": ApiPublishStatus.PRIVATE,
     }

+ 1 - 1
src/sentry/api/endpoints/event_apple_crash_report.py

@@ -14,7 +14,7 @@ from sentry.utils.safe import get_path
 
 @region_silo_endpoint
 class EventAppleCrashReportEndpoint(ProjectEndpoint):
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     publish_status = {
         "GET": ApiPublishStatus.PRIVATE,
     }

+ 5 - 5
src/sentry/api/endpoints/project_app_store_connect_credentials.py

@@ -132,7 +132,7 @@ class AppStoreConnectAppsEndpoint(ProjectEndpoint):
     ```
     """
 
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     permission_classes = (StrictProjectPermission,)
 
     def post(self, request: Request, project: Project) -> Response:
@@ -218,7 +218,7 @@ class AppStoreConnectCreateCredentialsEndpoint(ProjectEndpoint):
     they receive the saved configuration.
     """
 
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     permission_classes = (StrictProjectPermission,)
 
     def post(self, request: Request, project: Project) -> Response:
@@ -296,7 +296,7 @@ class AppStoreConnectUpdateCredentialsEndpoint(ProjectEndpoint):
     a sub-set. Useful for API key refreshes.
     """
 
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     permission_classes = (StrictProjectPermission,)
 
     def post(self, request: Request, project: Project, credentials_id: str) -> Response:
@@ -364,7 +364,7 @@ class AppStoreConnectRefreshEndpoint(ProjectEndpoint):
     headers, see the sentry.middleware.ratelimit module.
     """
 
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     permission_classes = (StrictProjectPermission,)
 
     enforce_rate_limit = True
@@ -440,7 +440,7 @@ class AppStoreConnectStatusEndpoint(ProjectEndpoint):
       of whether there were any or no builds in App Store Connect at the time.
     """
 
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     permission_classes = (ProjectPermission,)
 
     def get(self, request: Request, project: Project) -> Response:

+ 1 - 1
src/sentry/api/endpoints/project_symbol_sources.py

@@ -232,7 +232,7 @@ class SourceSerializer(serializers.Serializer):
 @extend_schema(tags=["Projects"])
 @region_silo_endpoint
 class ProjectSymbolSourcesEndpoint(ProjectEndpoint):
-    owner = ApiOwner.OWNERS_NATIVE
+    owner = ApiOwner.PROCESSING
     publish_status = {
         "GET": ApiPublishStatus.PUBLIC,
         "DELETE": ApiPublishStatus.PUBLIC,