Page MenuHomeSoftware Heritage

Add test_directory_add_raw_manifest__different_entries
ClosedPublic

Authored by vlorentz on Aug 4 2022, 11:56 AM.

Details

Summary

This reproduces what I think is the issue found in
https://jenkins.softwareheritage.org/job/debian/job/packages/job/DSTO/job/gbp-buildpackage/423/consoleFull

This does not fix the issue as it is a consequence of the design,
but documents this problematic behavior.

Diff Detail

Repository
rDSTO Storage manager
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 30602
Build 47849: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 47848: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D8180 (id=29535)

Rebasing onto fbe3803820...

Current branch diff-target is up to date.
Changes applied before test
commit 2205fa6e1fa9b1946b4f6fabfadec6e42488792e
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Aug 4 11:55:28 2022 +0200

    Add test_directory_add_raw_manifest__different_entries
    
    This reproduces what I think is the issue found in
    https://jenkins.softwareheritage.org/job/debian/job/packages/job/DSTO/job/gbp-buildpackage/423/consoleFull
    
    This does not fix the issue as it is a consequence of the design,
    but documents this problematic behavior.

See https://jenkins.softwareheritage.org/job/DSTO/job/tests-on-diff/1637/ for more details.

ardumont added inline comments.
swh/storage/tests/storage_tests.py
833

maybe mentions it's about different implementations (storage pg: 0, storage cass: 1 if i'm not wrong).

This revision is now accepted and ready to land.Aug 4 2022, 12:33 PM