fix: move vroom-seaweed migration upwards

[aldy505]

Closes #4263

[github-actions]

Changelog Preview

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).

---

New Features ✨

• Run SeaweedFS admin and worker instance by aldy505 in #4259
• Support custom CA certificates for all containers by aldy505 in #4216
• Remove 'vroom-cleanup' container by aldy505 in #4217

Bug Fixes 🐛

• (install) Gracefully handle missing containers in minimize-downtime by shameemkpofficial-git in #4246

• Move vroom-seaweed migration upwards by aldy505 in #4268

• Use default buildx builder to resolve local images during build by maiqigh in #4250

Internal Changes 🔧

Deps

• Bump j178/prek-action from 2.0.0 to 2.0.1 by dependabot in #4264
• Bump BYK/docker-volume-cache-action from be89365902126f508dcae387a32ec3712df6b1cd to 0efa5cf5178c9906cb46ed8d1a357df8fd6b1a06 by dependabot in #4253
• Bump astral-sh/setup-uv from 7.6.0 to 8.0.0 by dependabot in #4254
• Bump getsentry/craft from 2.23.2 to 2.24.1 by dependabot in #4221
• Bump astral-sh/setup-uv from 7.2.1 to 7.5.0 by dependabot in #4220

Other

• Restore unpinned actions by aldy505 in #4243
• Swap pre-commit with prek by aldy505 in #4235

Other

• Bump postgres 14.22-bookworm by aminvakil in #4249

---

_{🤖 This preview updates automatically when you update the PR.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Fresh installs get incomplete config file from reordering
  • This is a real bug, and the bootstrap script now only appends profiles config when sentry/config.yml already exists, preventing accidental creation of an incomplete file on fresh installs.

Or push these changes by commenting:

@cursor push 1fd751e7f1

Preview (1fd751e7f1)

diff --git a/install/bootstrap-s3-profiles.sh b/install/bootstrap-s3-profiles.sh
--- a/install/bootstrap-s3-profiles.sh
+++ b/install/bootstrap-s3-profiles.sh
@@ -22,7 +22,7 @@
   if ! echo "$bucket_list" | grep -q "s3://profiles"; then
     apply_config_changes_profiles=0
     # Only touch if no existing profiles config is found
-    if ! grep -q "filestore.profiles-backend" $SENTRY_CONFIG_YML; then
+    if [[ -f "$SENTRY_CONFIG_YML" ]] && ! grep -q "filestore.profiles-backend" "$SENTRY_CONFIG_YML"; then
       if [[ -z "${APPLY_AUTOMATIC_CONFIG_UPDATES:-}" ]]; then
         echo
         echo "We are migrating the Profiles data directory from the 'sentry-vroom' volume to SeaweedFS."
@@ -66,7 +66,7 @@
 
       if [[ "$APPLY_AUTOMATIC_CONFIG_UPDATES" == 1 || "$apply_config_changes_profiles" == 1 ]]; then
         profiles_config=$(sed -n '/filestore.profiles-backend/,/s3v4"/{p}' sentry/config.example.yml)
-        echo "$profiles_config" >>$SENTRY_CONFIG_YML
+        echo "$profiles_config" >>"$SENTRY_CONFIG_YML"
       fi
     fi

_{This Bugbot Autofix run was free. To enable autofix for future PRs, go to the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 6a246fa. Configure here.}

[aminvakil]

Maybe PR name should be changed to something like "Turn things off after vroom-seaweed migration" ?

The goal would be clearer.

Other than this nitpicking, I think this can fix the issue, but maybe other improvements can be added later on?

Maybe make it optional opt-out for users who do not care about their profiling data? So their data can be removed and make this faster?
Or add this to docs, something like docker compose down -v && docker volume rm sentry-self-hosted_sentry-vroom?

I don't like either of these two solutions I've written :)