Merge branch 'master' into DominikB2014/add-cache-item-size-to-metrics

Author: DominikB2014

.github/actions/collect-test-data/action.yml

@@ -0,0 +1,35 @@
+name: 'Collect Test Data'
+description: 'Collects test data and uploads it to our internal test data collection service.'
+
+inputs:
+  artifact_path:
+    description: 'Path to test data output artifact to upload to our internal test data collection service'
+    required: true
+  gcs_bucket:
+    description: 'The GCS bucket to upload the test data to'
+    required: true
+  gcp_project_id:
+    description: 'The GCP project ID to use for uploading the test data'
+    required: true
+  workload_identity_provider:
+    description: 'The workload identity provider to use for uploading the test data'
+    required: true
+  service_account_email:
+    description: 'The service account email to use for uploading the test data'
+    required: true
+  matrix_instance_number:
+    description: 'The instance number of the matrix job'
+    required: false
+
+runs:
+  using: 'composite'
+  steps:
+    - name: Collect test data
+      uses: getsentry/action-collect-test-data@c0a2fd3f79d3867e2c8b486ccb1c4bf925477742 # v0.3.0
+      with:
+        path: ${{ inputs.artifact_path }}
+        gcs_path: ${{ inputs.gcs_bucket }}/pytest/${{ github.repository }}/${{ github.workflow }}/${{ github.job }}${{ inputs.matrix_instance_number != '' && '/instance' || '' }}${{ inputs.matrix_instance_number != '' && inputs.matrix_instance_number || '' }}/${{ github.repository_id }}/${{ github.run_id }}/${{ github.run_attempt }}
+        gcp_project_id: ${{ inputs.gcp_project_id }}
+        workload_identity_provider: ${{ inputs.workload_identity_provider }}
+        service_account_email: ${{ inputs.service_account_email }}
+      continue-on-error: true

.github/workflows/acceptance.yml

@@ -46,6 +46,9 @@ jobs:
     name: acceptance
     runs-on: ubuntu-22.04
     timeout-minutes: 30
+    permissions:
+      contents: read
+      id-token: write
     strategy:
       # This helps not having to run multiple jobs because one fails, thus, reducing resource usage
       # and reducing the risk that one of many runs would turn red again (read: intermittent tests)
@@ -113,6 +116,17 @@ jobs:
       - name: Run acceptance tests (#${{ steps.setup.outputs.matrix-instance-number }} of ${{ steps.setup.outputs.matrix-instance-total }})
         run: make run-acceptance
 
+      - name: Collect test data
+        uses: ./.github/actions/collect-test-data
+        if: ${{ !cancelled() && needs.files-changed.outputs.backend_all == 'true' }}
+        with:
+          artifact_path: .artifacts/pytest.acceptance.json
+          gcs_bucket: ${{ secrets.COLLECT_TEST_DATA_GCS_BUCKET }}
+          gcp_project_id: ${{ secrets.COLLECT_TEST_DATA_GCP_PROJECT_ID }}
+          workload_identity_provider: ${{ secrets.SENTRY_GCP_DEV_WORKLOAD_IDENTITY_POOL }}
+          service_account_email: ${{ secrets.COLLECT_TEST_DATA_SERVICE_ACCOUNT_EMAIL }}
+          matrix_instance_number: ${{ steps.setup.outputs.matrix-instance-number }}
+
       # This job runs when FE or BE changes happen, however, we only upload coverage data for
       # BE changes since it conflicts with codecov's carry forward functionality
       # Upload coverage data even if running the tests step fails since

.github/workflows/backend.yml

@@ -67,6 +67,9 @@ jobs:
     name: backend test
     runs-on: ubuntu-22.04
     timeout-minutes: 60
+    permissions:
+      contents: read
+      id-token: write
     strategy:
       # This helps not having to run multiple jobs because one fails, thus, reducing resource usage
       # and reducing the risk that one of many runs would turn red again (read: intermittent tests)
@@ -106,6 +109,17 @@ jobs:
         run: |
           make test-python-ci
 
+      - name: Collect test data
+        uses: ./.github/actions/collect-test-data
+        if: ${{ !cancelled() }}
+        with:
+          artifact_path: .artifacts/pytest.json
+          gcs_bucket: ${{ secrets.COLLECT_TEST_DATA_GCS_BUCKET }}
+          gcp_project_id: ${{ secrets.COLLECT_TEST_DATA_GCP_PROJECT_ID }}
+          workload_identity_provider: ${{ secrets.SENTRY_GCP_DEV_WORKLOAD_IDENTITY_POOL }}
+          service_account_email: ${{ secrets.COLLECT_TEST_DATA_SERVICE_ACCOUNT_EMAIL }}
+          matrix_instance_number: ${{ steps.setup.outputs.matrix-instance-number }}
+
       # Upload coverage data even if running the tests step fails since
       # it reduces large coverage fluctuations
       - name: Handle artifacts
@@ -245,6 +259,9 @@ jobs:
     name: monolith-dbs test
     runs-on: ubuntu-22.04
     timeout-minutes: 20
+    permissions:
+      contents: read
+      id-token: write
     steps:
       - uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
         with:
@@ -260,6 +277,16 @@ jobs:
         run: |
           make test-monolith-dbs
 
+      - name: Collect test data
+        uses: ./.github/actions/collect-test-data
+        if: ${{ !cancelled() }}
+        with:
+          artifact_path: .artifacts/pytest.monolith-dbs.json
+          gcs_bucket: ${{ secrets.COLLECT_TEST_DATA_GCS_BUCKET }}
+          gcp_project_id: ${{ secrets.COLLECT_TEST_DATA_GCP_PROJECT_ID }}
+          workload_identity_provider: ${{ secrets.SENTRY_GCP_DEV_WORKLOAD_IDENTITY_POOL }}
+          service_account_email: ${{ secrets.COLLECT_TEST_DATA_SERVICE_ACCOUNT_EMAIL }}
+
       # Upload coverage data even if running the tests step fails since
       # it reduces large coverage fluctuations
       - name: Handle artifacts

CHANGES

@@ -1,3 +1,35 @@
+24.4.0
+------
+
+### Various fixes & improvements
+
+- Revert "ref(stories): Collapse `app` root (#68939)" (13dcc607) by @getsentry-bot
+- feat(ui): Add `actor` to delegating Avatar component (#68933) by @evanpurkhiser
+- ref(ui): Consolidate TeamBadge (#68927) by @evanpurkhiser
+- ref(stories): Improve types on TreeMapping (#68940) by @evanpurkhiser
+- ref(stories): Collapse `app` root (#68939) by @evanpurkhiser
+- feat(perf): Use `@sentry/status-page-list` for domain status link (#68899) by @gggritso
+- deps(ui): Update react-router to latest v3, create-react-class (#68926) by @scttcper
+- fix(trace): fix drawer bg color (#68934) by @JonasBa
+- feat(issue-details): Revise Context UI behind feature flag (#68081) by @leeandher
+- Revert "feat(replay): Persist has-viewed state to the server when replays are seen (#68743)" (ad312261) by @getsentry-bot
+- feat(perf): Bump "Requests" module from "Alpha" to "Beta" (#68930) by @gggritso
+- feat(perf): Adjust HTTP sample table column widths (#68901) by @gggritso
+- chore(self-hosted): 24.3.0 release tests (#67091) by @hubertdeng123
+- deps(ui): Remove react-test-renderer (#68929) by @scttcper
+- fix(trace): fix text overlaps (#68928) by @JonasBa
+- fix(replay): Fix canvas replays when seeking without actively playing (#68646) by @billyvg
+- fix(perf): Link Response module domains to a specific project (#68905) by @gggritso
+-  chore(issues): Remove legacy browser flag from BE  (#67717) by @leeandher
+- ref(uI): Cleanup unused ts-ignore after update (#68924) by @scttcper
+- fix(replay): Fixes type error if same param is in url (#68851) by @c298lee
+- fix(bug): logging bug (#68919) by @ykamo001
+- fix(crons): Update slug in quotas when deleting monitors (#68917) by @evanpurkhiser
+- chore(relocation): Add server-side Sentry capture logging (#68906) by @azaslavsky
+- feat(replays): Delete videos on replay delete request (#68463) by @cmanallen
+
+_Plus 1311 more_
+
 24.3.0
 ------
 

Makefile

@@ -91,7 +91,7 @@ fetch-release-registry:
 
 run-acceptance:
 	@echo "--> Running acceptance tests"
-	pytest tests/acceptance --cov . --cov-report="xml:.artifacts/acceptance.coverage.xml" --json-report --json-report-file=".artifact/pytest.acceptance.json" --json-report-omit=log
+	pytest tests/acceptance --cov . --cov-report="xml:.artifacts/acceptance.coverage.xml" --json-report --json-report-file=".artifacts/pytest.acceptance.json" --json-report-omit=log
 	@echo ""
 
 test-cli: create-db

fixtures/backup/model_dependencies/detailed.json

@@ -1748,11 +1748,6 @@
         "kind": "FlexibleForeignKey",
         "model": "sentry.docintegration",
         "nullable": false
-      },
-      "file_id": {
-        "kind": "ImplicitForeignKey",
-        "model": "sentry.file",
-        "nullable": true
       }
     },
     "model": "sentry.docintegrationavatar",
@@ -1766,9 +1761,6 @@
       [
         "control_file_id"
       ],
-      [
-        "file_id"
-      ],
       [
         "ident"
       ]
@@ -5538,11 +5530,6 @@
         "model": "sentry.controlfile",
         "nullable": true
       },
-      "file_id": {
-        "kind": "ImplicitForeignKey",
-        "model": "sentry.file",
-        "nullable": true
-      },
       "sentry_app": {
         "kind": "FlexibleForeignKey",
         "model": "sentry.sentryapp",
@@ -5560,9 +5547,6 @@
       [
         "control_file_id"
       ],
-      [
-        "file_id"
-      ],
       [
         "ident"
       ]
@@ -5986,11 +5970,6 @@
         "model": "sentry.controlfile",
         "nullable": true
       },
-      "file_id": {
-        "kind": "ImplicitForeignKey",
-        "model": "sentry.file",
-        "nullable": true
-      },
       "user": {
         "kind": "FlexibleForeignKey",
         "model": "sentry.user",
@@ -6008,9 +5987,6 @@
       [
         "control_file_id"
       ],
-      [
-        "file_id"
-      ],
       [
         "ident"
       ],

fixtures/backup/model_dependencies/flat.json

@@ -240,8 +240,7 @@
   "sentry.docintegration": [],
   "sentry.docintegrationavatar": [
     "sentry.controlfile",
-    "sentry.docintegration",
-    "sentry.file"
+    "sentry.docintegration"
   ],
   "sentry.email": [],
   "sentry.environment": [
@@ -763,7 +762,6 @@
   ],
   "sentry.sentryappavatar": [
     "sentry.controlfile",
-    "sentry.file",
     "sentry.sentryapp"
   ],
   "sentry.sentryappcomponent": [
@@ -823,7 +821,6 @@
   "sentry.user": [],
   "sentry.useravatar": [
     "sentry.controlfile",
-    "sentry.file",
     "sentry.user"
   ],
   "sentry.useremail": [

fixtures/backup/model_dependencies/sorted.json

@@ -14,6 +14,7 @@
   "sentry.controltombstone",
   "sentry.deletedorganization",
   "sentry.docintegration",
+  "sentry.docintegrationavatar",
   "sentry.fileblob",
   "sentry.identityprovider",
   "sentry.integration",
@@ -51,6 +52,7 @@
   "sentry.team",
   "sentry.teamreplica",
   "sentry.user",
+  "sentry.useravatar",
   "sentry.userip",
   "sentry.useroption",
   "sentry.userpermission",
@@ -92,7 +94,6 @@
   "sentry.eventattachment",
   "sentry.environment",
   "sentry.email",
-  "sentry.docintegrationavatar",
   "sentry.distribution",
   "sentry.discoversavedquery",
   "sentry.deploy",
@@ -124,7 +125,6 @@
   "feedback.feedback",
   "sentry.userreport",
   "sentry.useremail",
-  "sentry.useravatar",
   "sentry.teamkeytransaction",
   "sentry.snubaquery",
   "sentry.sentryapp",

fixtures/backup/model_dependencies/truncate.json

@@ -14,6 +14,7 @@
   "sentry_controltombstone",
   "sentry_deletedorganization",
   "sentry_docintegration",
+  "sentry_docintegrationavatar",
   "sentry_fileblob",
   "sentry_identityprovider",
   "sentry_integration",
@@ -51,6 +52,7 @@
   "sentry_team",
   "sentry_teamreplica",
   "auth_user",
+  "sentry_useravatar",
   "sentry_userip",
   "sentry_useroption",
   "sentry_userpermission",
@@ -92,7 +94,6 @@
   "sentry_eventattachment",
   "sentry_environment",
   "sentry_email",
-  "sentry_docintegrationavatar",
   "sentry_distribution",
   "sentry_discoversavedquery",
   "sentry_deploy",
@@ -124,7 +125,6 @@
   "feedback_feedback",
   "sentry_userreport",
   "sentry_useremail",
-  "sentry_useravatar",
   "sentry_performanceteamkeytransaction",
   "sentry_snubaquery",
   "sentry_sentryapp",

fixtures/js-stubs/monitor.tsx

@@ -5,8 +5,8 @@ import {
   ScheduleType,
 } from 'sentry/views/monitors/types';
 
+import {ActorFixture} from './actor';
 import {ProjectFixture} from './project';
-import {UserFixture} from './user';
 
 export function MonitorFixture(params: Partial<Monitor> = {}): Monitor {
   return {
@@ -16,7 +16,7 @@ export function MonitorFixture(params: Partial<Monitor> = {}): Monitor {
     project: ProjectFixture(),
     slug: 'my-monitor',
     status: 'active',
-    owner: `user:${UserFixture().id}`,
+    owner: ActorFixture(),
     type: MonitorType.CRON_JOB,
     config: {
       checkin_margin: 5,

migrations_lockfile.txt

@@ -9,5 +9,5 @@ feedback: 0004_index_together
 hybridcloud: 0016_add_control_cacheversion
 nodestore: 0002_nodestore_no_dictfield
 replays: 0004_index_together
-sentry: 0696_remove_monitor_owner_actor_id
+sentry: 0698_remove_file_id_from_control_avatars
 social_auth: 0002_default_auto_field

package.json

@@ -62,6 +62,7 @@
     "@sentry/node": "^7.109.0",
     "@sentry/react": "^7.109.0",
     "@sentry/release-parser": "^1.3.1",
+    "@sentry/status-page-list": "^0.0.1",
     "@sentry/types": "^7.109.0",
     "@sentry/utils": "^7.109.0",
     "@spotlightjs/spotlight": "^1.2.13",
@@ -86,7 +87,7 @@
     "@types/react-lazyload": "3.2.3",
     "@types/react-grid-layout": "^1.3.2",
     "@types/react-mentions": "4.1.6",
-    "@types/react-router": "^3.0.22",
+    "@types/react-router": "^3.0.28",
     "@types/react-select": "4.0.18",
     "@types/react-sparklines": "^1.7.2",
     "@types/react-virtualized": "^9.21.22",
@@ -157,7 +158,7 @@
     "react-lazyload": "^3.2.1",
     "react-mentions": "4.4.2",
     "react-popper": "^2.3.0",
-    "react-router": "3.2.0",
+    "react-router": "3.2.6",
     "react-select": "4.3.1",
     "react-sparklines": "1.7.0",
     "react-virtualized": "^9.22.5",
@@ -203,7 +204,6 @@
     "jest-sentry-environment": "3.0.0",
     "postcss-styled-syntax": "0.6.4",
     "react-refresh": "0.14.0",
-    "react-test-renderer": "18.2.0",
     "stylelint": "16.2.1",
     "stylelint-config-recommended": "^14.0.0",
     "terser": "5.27.0",

pyproject.toml

@@ -589,6 +589,7 @@ disable_error_code = [
 [[tool.mypy.overrides]]
 module = [
     "sentry.api.endpoints.issues.*",
+    "sentry.api.helpers.source_map_helper",
     "sentry.buffer.base",
     "sentry.buffer.redis",
     "sentry.eventstore.reprocessing.redis",