vault: Open the beginning of the tarball to check it makes sense.
c12f5202f0be
Actions

Description

vault: Open the beginning of the tarball to check it makes sense.

This can catch potential errors like file corruption, wrong encoding,
crashes after response headers were sent, etc.

Details

Provenance

vlorentz	Authored on Mar 4 2022, 3:50 PM
vlorentz	Pushed on Mar 4 2022, 5:21 PM

Differential Revision

D7298: vault: Open the beginning of the tarball to check it makes sense.

Parents

rDICPc0de368aa7af: vault: Check content-type

Branches

Unknown

Tags

Unknown

References

tag: v0.4.0

Tasks

T3983: Add monitoring of vault bundle downloads

Build Status

Buildable 27307
Build 42716: test-and-build	Jenkins console · Jenkins

Event Timeline

vlorentz committed rDICPc12f5202f0be: vault: Open the beginning of the tarball to check it makes sense. (authored by vlorentz).Mar 4 2022, 5:16 PM

vlorentz added a task: T3983: Add monitoring of vault bundle downloads.Mar 4 2022, 5:21 PM

vlorentz added an edge: D7298: vault: Open the beginning of the tarball to check it makes sense..

Harbormaster completed building B27307: rDICPc12f5202f0be: vault: Open the beginning of the tarball to check it makes sense..Mar 4 2022, 5:23 PM

swh-public-ci mentioned this in D7301: Remove useless HTTP header from tests.Mar 4 2022, 5:51 PM

swh-public-ci mentioned this in D7302: vault: Open tarballs in stream mode + add test for empty tar files.

swh-public-ci mentioned this in D7303: deposit: Fetch only metadata created after the check started.Mar 4 2022, 6:07 PM

Changes (2)

Path

Size

swh/

icinga_plugins/

tests/

test_vault.py

vault.py

rDICPc12f5202f0be

View Options

swh/icinga_plugins/tests/test_vault.py