Page MenuHomeSoftware Heritage

in_memory storage: Adapt internal implementations to use origin visit status model representation
ClosedPublic

Authored by ardumont on Apr 1 2020, 12:09 PM.

Details

Summary

Introduce internally the origin visit status model change.
Without changing the client facing api.

Related to T2310
Related to D2879

Test Plan

tox

Diff Detail

Repository
rDSTO Storage manager
Branch
make-origin-visit-immutable
Lint
Lint Skipped
Unit
Unit Tests Skipped
Build Status
Buildable 12090
Build 18344: Phabricator diff pipeline on jenkinsJenkins console · Jenkins
Build 18343: arc lint + arc unit

Event Timeline

Build is green

Patch application report for D2937 (id=10448)

Rebasing onto 377e6a8072...

Current branch diff-target is up to date.
Changes applied before test
commit b29327fd7cb7316ae6ba3ad27f53971d696d324f
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

Amend commit with specific in-memory code (committed in the wrong one)

Build is green

Patch application report for D2937 (id=10454)

Rebasing onto 377e6a8072...

Current branch diff-target is up to date.
Changes applied before test
commit c0b05980ca3a19afa30fba6e75d067aaeac2cc96
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

This revision is now accepted and ready to land.Apr 1 2020, 3:21 PM

Build is green

Patch application report for D2937 (id=10459)

Rebasing onto df3207a6b7...

Current branch diff-target is up to date.
Changes applied before test
commit c7ca0d04ff2d74150a1acde201b278d03d5a94d1
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

Rebase on latest master and blackify

Build is green

Patch application report for D2937 (id=10641)

Rebasing onto b0b0313c96...

Current branch diff-target is up to date.
Changes applied before test
commit 65a4044601d1c8259585a127c28a7b5c23a3d1c6
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

  • rebase to latest master
  • update to use the latest OriginVisitStatus model object

Build is green

Patch application report for D2937 (id=10692)

Rebasing onto ddac3d27e3...

Current branch diff-target is up to date.
Changes applied before test
commit 88fa1b0e22dd1996afeee99ffcbccd0c8c3be110
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

ardumont retitled this revision from in_memory storage: Adapt internal implementations to use origin visit update model representation to in_memory storage: Adapt internal implementations to use origin visit status model representation.Apr 10 2020, 5:57 PM

Build is green

Patch application report for D2937 (id=10819)

Rebasing onto bca643acab...

Current branch diff-target is up to date.
Changes applied before test
commit 13fc814a0c0bef9487321138cf472b0fdbe48a28
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

Build is green

Patch application report for D2937 (id=10932)

Rebasing onto 49109d173f...

Current branch diff-target is up to date.
Changes applied before test
commit 9cc80f8f92f37ecdd948179b6bf954b3109382d5
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

  • rename visit "update" into visit "status" where it makes sense
  • fix one method to add assertion instead of being an optional result

Build is green

Patch application report for D2937 (id=10936)

Rebasing onto 49109d173f...

Current branch diff-target is up to date.
Changes applied before test
commit d3e97b9d9cfa636b7e4a079ee3dcf0851ee34dba
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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

Build is green

Patch application report for D2937 (id=10946)

Rebasing onto 49109d173f...

Current branch diff-target is up to date.
Changes applied before test
commit ead80889fb44d7c0e929b913f3e852ab851c0a3d
Author: Antoine R. Dumont (@ardumont) <antoine.romain.dumont@gmail.com>
Date:   Wed Mar 25 17:53:48 2020 +0100

    in_memory: Adapt internal implementations to use origin visit update
    
    (pairing with @vlorentz)
    
    Related to T2310

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