Browse Source

feat(feedback): add option for ingesting feedback+attachments in same envelope (#69351)

Needed for https://github.com/getsentry/relay/pull/3403

For relay, using an option is way simpler than a FF. This option will
also be used to test separating the logic for producing feedbacks
(`user_report_v2`'s in `store.rs`)
Andrew Liu 10 months ago
parent
commit
03c4839d20
2 changed files with 7 additions and 0 deletions
  1. 6 0
      src/sentry/options/defaults.py
  2. 1 0
      src/sentry/relay/globalconfig.py

+ 6 - 0
src/sentry/options/defaults.py

@@ -470,6 +470,12 @@ register(
     default=0.0,
     default=0.0,
     flags=FLAG_AUTOMATOR_MODIFIABLE,
     flags=FLAG_AUTOMATOR_MODIFIABLE,
 )
 )
+# Separate the logic for producing feedbacks from generic events, and handle attachments in the same envelope
+register(
+    "feedback.ingest-inline-attachments",
+    default=False,
+    flags=FLAG_AUTOMATOR_MODIFIABLE,
+)
 
 
 # Extract spans only from a random fraction of transactions.
 # Extract spans only from a random fraction of transactions.
 #
 #

+ 1 - 0
src/sentry/relay/globalconfig.py

@@ -18,6 +18,7 @@ RELAY_OPTIONS: list[str] = [
     "relay.metric-bucket-distribution-encodings",
     "relay.metric-bucket-distribution-encodings",
     "relay.metric-stats.rollout-rate",
     "relay.metric-stats.rollout-rate",
     "feedback.ingest-topic.rollout-rate",
     "feedback.ingest-topic.rollout-rate",
+    "feedback.ingest-inline-attachments",
     "relay.span-extraction.sample-rate",
     "relay.span-extraction.sample-rate",
 ]
 ]