Page MenuHomeSoftware Heritage

Fix InMemoryJournalWriter: do not anonymize objects by default
ClosedPublic

Authored by douardda on Wed, Jun 8, 12:19 PM.

Details

Summary

908f0154 introduced and unexpected change of behavior of the
InMemoryJournalWriter, making it always anonymize objects. This broke
tests for some dependencies (swh-storage).

This fix adds an 'anonymize' (bool) argument to the class constructor
(similar to the KafkaJournalWriter).

Diff Detail

Repository
rDJNL Journal infrastructure
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

Build is green

Patch application report for D7970 (id=28714)

Rebasing onto a208f882a7...

Current branch diff-target is up to date.
Changes applied before test
commit 1b12d85c4c2ec876b2eb25457518278cef74257a
Author: David Douard <david.douard@sdfa3.org>
Date:   Wed Jun 8 12:15:11 2022 +0200

    Fix InMemoryJournalWriter: do not anonymize objects by default
    
    908f0154 introduced and unexpected change of behavior of the
    InMemoryJournalWriter, making it always anonymize objects. This broke
    tests for some dependencies (swh-storage).
    
    This fix adds an 'anonymize' (bool) argument to the class constructor
    (similar to the KafkaJournalWriter).

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

anlambert added a subscriber: anlambert.

Looks good, thanks !

This revision is now accepted and ready to land.Wed, Jun 8, 12:27 PM
This revision was landed with ongoing or failed builds.Wed, Jun 8, 12:30 PM
This revision was automatically updated to reflect the committed changes.

Build is green

Patch application report for D7970 (id=28716)

Rebasing onto a208f882a7...

Current branch diff-target is up to date.
Changes applied before test
commit 6b05a6c949b4f278832904965b02ed04bae8dce3
Author: David Douard <david.douard@sdfa3.org>
Date:   Wed Jun 8 12:15:11 2022 +0200

    Fix InMemoryJournalWriter: do not anonymize objects by default
    
    908f0154 introduced an unexpected change of behavior of the
    InMemoryJournalWriter, making it always anonymize objects. This broke
    tests for some dependencies (esp. swh-storage).
    
    This fix adds an 'anonymize' (bool) argument to the class constructor
    (similar to the KafkaJournalWriter).

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