Page MenuHomeSoftware Heritage

storage*: Drop origin_visit_upsert endpoint
ClosedPublic

Authored by ardumont on Jun 15 2020, 2:27 PM.

Details

Summary

This removes the no longer used endpoint (from D3273 onward, we migrated away from it).

Depends on D3276
Related to T2310

Test Plan

tox

Diff Detail

Repository
rDSTO Storage manager
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 D3281 (id=11639)

Could not rebase; Attempt merge onto 0183fec91d...

Updating 0183fec..d6144d2
Fast-forward
 swh/storage/cassandra/storage.py       |  64 ---
 swh/storage/db.py                      |  25 -
 swh/storage/in_memory.py               |  93 ----
 swh/storage/interface.py               |  46 --
 swh/storage/replay.py                  |  15 +-
 swh/storage/storage.py                 |  93 ----
 swh/storage/tests/test_kafka_writer.py |   8 +-
 swh/storage/tests/test_replay.py       |   7 +-
 swh/storage/tests/test_retry.py        |  81 ---
 swh/storage/tests/test_storage.py      | 920 +++++++++++----------------------
 swh/storage/writer.py                  |   3 -
 11 files changed, 304 insertions(+), 1051 deletions(-)
Changes applied before test
commit d6144d27450c487604dc8a5482f951dd0fbad042
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Jun 11 17:26:41 2020 +0200

    storage*: Drop origin_visit_upsert endpoint
    
    Related to T2310

commit c7f3060461a53480f9803c099489f7f27441eee5
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Fri Jun 12 18:51:25 2020 +0200

    storage*: Remove origin-visit-update endpoint
    
    Related to T2310

commit 2bcbc82b708550415ce23a8a91830ad2b75f848c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Thu Jun 11 17:16:29 2020 +0200

    replay: Replay origin-visit and origin-visit-status
    
    This now uses the respective origin-visit-add and origin-visit-status-add
    endpoints.
    
    Related to T2310

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

This revision is now accepted and ready to land.Jun 15 2020, 2:51 PM