Skip to content

Commit

Permalink
chore: Remove upgrade test (#3541)
Browse files Browse the repository at this point in the history
As discussed, removing the upgrade test since it doesn't provide that much utility as we are already testing upgrades from restoring docker volumes from cache
  • Loading branch information
hubertdeng123 authored Jan 17, 2025
1 parent 0ce2cd7 commit 9ceb00f
Showing 1 changed file with 0 additions and 71 deletions.
71 changes: 0 additions & 71 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -33,77 +33,6 @@ jobs:
- name: Unit Tests
run: ./unit-test.sh

upgrade-test:
if: github.repository_owner == 'getsentry'
runs-on: ubuntu-22.04
name: "Sentry upgrade test"
env:
REPORT_SELF_HOSTED_ISSUES: 0
steps:
- name: Get latest self-hosted release version
run: |
LATEST_TAG=$(curl -s https://api.github.com/repos/getsentry/self-hosted/releases/latest | jq -r '.tag_name')
echo "LATEST_TAG=$LATEST_TAG" >> $GITHUB_ENV
- name: Get Compose
uses: getsentry/self-hosted/get-compose-action@master

- name: Checkout latest release
uses: actions/checkout@v4
with:
ref: ${{ env.LATEST_TAG }}

- name: Restore DB Volumes Cache
id: restore_cache
uses: BYK/docker-volume-cache-action/restore@be89365902126f508dcae387a32ec3712df6b1cd
with:
key: db-volumes-v6-${{ env.LATEST_TAG }}
restore-keys: |
db-volumes-v6-${{ env.LATEST_TAG }}
db-volumes-v6-
volumes: |
sentry-postgres
sentry-clickhouse
sentry-kafka
- name: Install ${{ env.LATEST_TAG }}
env:
# Remove SKIP_DB_MIGRATIONS after releasing 25.1.1 or 25.2.0
SKIP_DB_MIGRATIONS: ${{ steps.restore_cache.outputs.cache-hit == 'true' && '1' || '' }}
SKIP_SENTRY_MIGRATIONS: ${{ steps.restore_cache.outputs.cache-hit == 'true' && '1' || '' }}
SKIP_SNUBA_MIGRATIONS: ${{ steps.restore_cache.outputs.cache-hit == 'true' && '1' || '' }}
run: |
# This is to compensate for a bug in upgrade-clickhouse where
# if we have sentry-clickhouse volume without the rest, it fails
# We may get sentry-clickhouse from the cache step above
source install/create-docker-volumes.sh
./install.sh
- name: Save DB Volumes Cache
if: steps.restore_cache.outputs.cache-hit != 'true'
uses: BYK/docker-volume-cache-action/save@be89365902126f508dcae387a32ec3712df6b1cd
with:
key: ${{ steps.restore_cache.outputs.cache-primary-key }}
volumes: |
sentry-postgres
sentry-clickhouse
sentry-kafka
- name: Checkout current ref
uses: actions/checkout@v4

- name: Install current ref
run: |
# This is for the cache restore on Kafka to work in older releases
docker run --rm -v "sentry-kafka:/data" busybox chown -R 1000:1000 /data
./install.sh
- name: Inspect failure
if: failure()
run: |
docker compose ps
docker compose logs
integration-test:
if: github.repository_owner == 'getsentry'
runs-on: ubuntu-22.04
Expand Down

0 comments on commit 9ceb00f

Please sign in to comment.