Skip to content

[DNM] test(symbolicator): Test compression + range requests for DIFs - #121182

Closed
lcian wants to merge 2 commits into
masterfrom
test/compressed-debugfile-range-failure
Closed

[DNM] test(symbolicator): Test compression + range requests for DIFs#121182
lcian wants to merge 2 commits into
masterfrom
test/compressed-debugfile-range-failure

Conversation

@lcian

@lcian lcian commented Aug 4, 2026

Copy link
Copy Markdown
Member

This makes Objectstore debug-file uploads zstd-compressed and adds an integration test with a post-compression payload larger than 384 MiB. The upload deliberately uses session.put rather than multipart, and Symbolicator reads it directly from Objectstore.
The test asserts that the stored Objectstore object is zstd-compressed and still exceeds the multi-range threshold post-compression.

The test reproduces the failure: Symbolicator issues its initial 128 MiB range request plus three additional ranges, then reports Unknown frame descriptor (this can only be seen in the symbolicator container logs, sadly).
The expected successful symbolication assertion consequently fails with candidate.download.status == "notfound".

Ref FS-390

Upload large debug files through Objectstore with zstd and exercise Symbolicator's multi-range retrieval path.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Aug 4, 2026
@lcian lcian changed the title DO NOT MERGE: test(symbolicator): Reproduce zstd range download failure [DNM] test(symbolicator): Test compression + range requests for DIFs Aug 4, 2026
@linear-code

linear-code Bot commented Aug 5, 2026

Copy link
Copy Markdown

FS-390

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Backend Test Failures

Failures on 0ff0c6f in this run:

tests/symbolicator/test_minidump_full.py::SymbolicatorLargeDebugFileIntegrationTest::test_symbolicates_large_compressed_debug_filelog
[gw0] linux -- Python 3.13.1 /home/runner/work/sentry/sentry/.venv/bin/python3
tests/symbolicator/test_minidump_full.py:292: in test_symbolicates_large_compressed_debug_file
    assert candidate["download"]["status"] == "ok"
E   AssertionError: assert 'error' == 'ok'
E     
E     �[0m�[91m- ok�[39;49;00m�[90m�[39;49;00m
E     �[92m+ error�[39;49;00m�[90m�[39;49;00m

@lcian lcian closed this Aug 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant