Skip to content

ref(ci): test_distributed only runs clickhouse_db tests #6812

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
+15 −19
Draft

ref(ci): test_distributed only runs clickhouse_db tests #6812

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
d733691
ref(ci): test_distributed only runs clickhouse_db tests
untitaker Jan 24, 2025
6b816fd
test_distributed_migrations does not have that marker
untitaker Jan 24, 2025
b301b33
fix snuba_settings
untitaker Jan 24, 2025
665f193
remove impossible case
untitaker Jan 24, 2025
94ff484
fix test location
untitaker Jan 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 15 additions & 19 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -207,13 +207,19 @@ jobs:
timeout-minutes: 30
strategy:
matrix:
snuba_settings:
[
"test",
"test_rust",
"test_distributed",
"test_distributed_migrations",
]
include:
- snuba_settings: test
docker_target: snuba-test
- snuba_settings: test
docker_target: snuba-test-rust
- snuba_settings: test_distributed
docker_target: snuba-test
pytest_addopts: '-m clickhouse_db'
- snuba_settings: test_distributed_migrations
docker_target: snuba-test
- snuba_settings: test
docker_target: snuba-test
test_location: test_initialization
steps:
- name: Checkout code
uses: actions/checkout@v4
Expand All @@ -233,27 +239,17 @@ jobs:
run: |
docker network create --attachable cloudbuild

- name: Docker Snuba Rust tests
run: |
SNUBA_IMAGE=ghcr.io/getsentry/snuba-ci:${{ github.sha }} SNUBA_SETTINGS=test docker compose -f docker-compose.gcb.yml run --rm snuba-test-rust
if: ${{ matrix.snuba_settings == 'test_rust' }}

- name: Docker Snuba tests
run: |
SNUBA_IMAGE=ghcr.io/getsentry/snuba-ci:${{ github.sha }} SNUBA_SETTINGS=${{ matrix.snuba_settings }} docker compose -f docker-compose.gcb.yml run --rm snuba-test
if: ${{ matrix.snuba_settings == 'test' || matrix.snuba_settings == 'test_distributed' }}
SNUBA_IMAGE=ghcr.io/getsentry/snuba-ci:${{ github.sha }} SNUBA_SETTINGS=${{ matrix.snuba_settings }} TEST_LOCATION=${{ matrix.test_location }} docker compose -f docker-compose.gcb.yml run --rm -e 'PYTEST_ADDOPTS=${{ matrix.pytest_addopts }}' ${{ matrix.docker_target }}
if: ${{ matrix.snuba_settings == 'test' || matrix.snuba_settings == 'test_distributed' }}

- name: Docker Snuba Multi-Node Tests
run: |
SNUBA_IMAGE=ghcr.io/getsentry/snuba-ci:${{ github.sha }} SNUBA_SETTINGS=test_distributed_migrations docker compose --profile multi_node -f docker-compose.gcb.yml up -d
SNUBA_IMAGE=ghcr.io/getsentry/snuba-ci:${{ github.sha }} SNUBA_SETTINGS=test_distributed_migrations TEST_LOCATION=test_distributed_migrations docker compose --profile multi_node -f docker-compose.gcb.yml run --rm snuba-test
if: ${{ matrix.snuba_settings == 'test_distributed_migrations' }}

- name: Docker Snuba Init Tests
run: |
SNUBA_IMAGE=ghcr.io/getsentry/snuba-ci:${{ github.sha }} SNUBA_SETTINGS=test_initialization TEST_LOCATION=test_initialization docker compose -f docker-compose.gcb.yml run --rm snuba-test
if: ${{ matrix.snuba_settings == 'test' }}

- name: Upload test results to Codecov
if: ${{ !cancelled() }}
uses: codecov/test-results-action@v1
Expand Down
Loading