Page MenuHomeSoftware Heritage
Differential D6622

SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted
ClosedPublic
Actions

Authored by ardumont on Nov 9 2021, 10:40 AM.

Details

Reviewers
vlorentz
Group Reviewers
Reviewers
Maniphest Tasks
T3719: loader svn fails to clean up properly disk when it crashes
Commits
rDLDSVNe36caff0a4f6: SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted
Summary

This will decrease the disk pressure currently existing when ingesting a svn repository
out of a remote dump. We first fetch a dump, then mount a svn repository out of it, and
at last we ingest the repository and have a growing copy on disk [1]. So we are up to 3
copies which takes lots of disk space.

With the following commit, we take down the 1 unnecessary copy (svn dump) as soon as
possible (when the svn repository is mounted which we ingest).

[1] implementation detail for speed, we iterate over the commit log, apply deltas (and
computing swh hashes) along the way on disk.

This also:

  • improves the existing docstrings
  • add types
  • add tests on missing utils part (currently tested indirectly)

Related to T3719

Test Plan

tox (still happy)

Diff Detail

Event Timeline

ardumont created this revision.Nov 9 2021, 10:40 AM
Herald added a reviewer: Reviewers. · View Herald TranscriptNov 9 2021, 10:40 AM
swh-public-ci added a comment.Nov 9 2021, 10:42 AM

Build is green

Patch application report for D6622 (id=24054)

Rebasing onto 189dfd5300...

Current branch diff-target is up to date.
Changes applied before test
commit 5dec13a169437cea39f02f3325e69151ab7eb22c
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Nov 9 10:35:01 2021 +0100

    SvnLoaderFromRemoteDump: Drop dump as soon as it's no longer required
    
    This will reduce the disk pressure [1] we have when a repository is large.
    
    [1] up to 3 copies of the svn repository: 1 dump, 1 svn repository, and the svn disk
    copy we are creating along the ingestion.
    
    Related to T3719

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

Harbormaster completed remote builds in B24929: Diff 24054.Nov 9 2021, 10:42 AM
ardumont requested review of this revision.Nov 9 2021, 10:42 AM
ardumont edited the summary of this revision. (Show Details)Nov 9 2021, 11:21 AM
ardumont updated this revision to Diff 24056.Nov 9 2021, 11:22 AM

Improve implementations:

  • add tests
  • add types
ardumont edited the summary of this revision. (Show Details)Nov 9 2021, 11:23 AM
swh-public-ci added a comment.Nov 9 2021, 11:24 AM

Build has FAILED

Patch application report for D6622 (id=24056)

Rebasing onto 189dfd5300...

Current branch diff-target is up to date.
Changes applied before test
commit e6ab42d05f2334c895147f78288d48ba745ea0f7
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Nov 9 10:35:01 2021 +0100

    SvnLoaderFromRemoteDump: Drop dump as soon as it's no longer required
    
    This will decrease the disk pressure currently existing when ingesting a svn repository
    out of a remote dump. We first fetch a dump, then mount a svn repository out of it, and
    at last we ingest the repository and have a growing copy on disk [1].
    
    So we are up to 3 copies which takes lots of disk space. With the following, we take
    down the 1 unnecessary copy as soon as possible.
    
    [1] implementation detail for speed.
    
    Related to T3719

Link to build: https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/172/
See console output for more information: https://jenkins.softwareheritage.org/job/DLDSVN/job/tests-on-diff/172/console

Harbormaster failed remote builds in B24931: Diff 24056!Nov 9 2021, 11:24 AM
ardumont updated this revision to Diff 24057.Nov 9 2021, 11:33 AM

Fix tests (i forgot one adaptation in another test)

swh-public-ci added a comment.Nov 9 2021, 11:35 AM

Build is green

Patch application report for D6622 (id=24057)

Rebasing onto 189dfd5300...

Current branch diff-target is up to date.
Changes applied before test
commit 69d8c72e9d675af39e2444d1f3b3cdd0e1eee864
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Nov 9 10:35:01 2021 +0100

    SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted
    
    This will decrease the disk pressure currently existing when ingesting a svn repository
    out of a remote dump. We first fetch a dump, then mount a svn repository out of it, and
    at last we ingest the repository and have a growing copy on disk [1]. So we are up to 3
    copies which takes lots of disk space.
    
    With the following commit, we take down the 1 unnecessary copy (svn dump) as soon as
    possible (when the svn repository is mounted which we ingest).
    
    [1] implementation detail for speed, we iterate over the commit log, apply deltas (and
    computing swh hashes) along the way on disk.
    
    Related to T3719

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

Harbormaster completed remote builds in B24932: Diff 24057.Nov 9 2021, 11:35 AM
ardumont retitled this revision from SvnLoaderFromRemoteDump: Drop dump as soon as it's no longer required to SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted.Nov 9 2021, 11:35 AM
ardumont edited the summary of this revision. (Show Details)
ardumont updated this revision to Diff 24058.Nov 9 2021, 11:36 AM

align docstring

ardumont edited the summary of this revision. (Show Details)Nov 9 2021, 11:36 AM
swh-public-ci added a comment.Nov 9 2021, 11:37 AM

Build is green

Patch application report for D6622 (id=24058)

Rebasing onto 189dfd5300...

Current branch diff-target is up to date.
Changes applied before test
commit 57eeb866a435353a76281e8ed52347fd8dbf06a3
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Nov 9 10:35:01 2021 +0100

    SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted
    
    This will decrease the disk pressure currently existing when ingesting a svn repository
    out of a remote dump. We first fetch a dump, then mount a svn repository out of it, and
    at last we ingest the repository and have a growing copy on disk [1]. So we are up to 3
    copies which takes lots of disk space.
    
    With the following commit, we take down the 1 unnecessary copy (svn dump) as soon as
    possible (when the svn repository is mounted which we ingest).
    
    [1] implementation detail for speed, we iterate over the commit log, apply deltas (and
    computing swh hashes) along the way on disk.
    
    Related to T3719

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

Harbormaster completed remote builds in B24933: Diff 24058.Nov 9 2021, 11:37 AM
ardumont updated this revision to Diff 24060.Nov 9 2021, 11:56 AM

Update

swh-public-ci added a comment.Nov 9 2021, 11:58 AM

Build is green

Patch application report for D6622 (id=24060)

Rebasing onto 189dfd5300...

Current branch diff-target is up to date.
Changes applied before test
commit b73db0485ebd35ea8c4dea8fd0e9e8be16bb9ff7
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Nov 9 10:35:01 2021 +0100

    SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted
    
    This will decrease the disk pressure currently existing when ingesting a svn repository
    out of a remote dump. We first fetch a dump, then mount a svn repository out of it, and
    at last we ingest the repository and have a growing copy on disk [1]. So we are up to 3
    copies which takes lots of disk space.
    
    With the following commit, we take down the 1 unnecessary copy (svn dump) as soon as
    possible (when the svn repository is mounted which we ingest).
    
    [1] implementation detail for speed, we iterate over the commit log, apply deltas (and
    computing swh hashes) along the way on disk.
    
    Related to T3719

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

Harbormaster completed remote builds in B24935: Diff 24060.Nov 9 2021, 11:58 AM
vlorentz accepted this revision.Nov 9 2021, 12:07 PM
vlorentz added a subscriber: vlorentz.
vlorentz added inline comments.
swh/loader/svn/utils.py
149

Can os.remove fail without an error?

This revision is now accepted and ready to land.Nov 9 2021, 12:07 PM
ardumont added inline comments.Nov 9 2021, 12:09 PM
swh/loader/svn/utils.py
149

i'll check

ardumont added inline comments.Nov 9 2021, 12:21 PM
swh/loader/svn/utils.py
149

I guess it could happen say in the scenario where the exception already got raised.
I'll wrap this in a try except. Diff on its way.

ardumont updated this revision to Diff 24062.Nov 9 2021, 12:21 PM

Wrap the remove instruction in case failure to remove happens.

swh/loader/svn/utils.py
149

diff "update" on its way.

ardumont marked an inline comment as done.Nov 9 2021, 12:22 PM
swh-public-ci added a comment.Nov 9 2021, 12:23 PM

Build is green

Patch application report for D6622 (id=24062)

Rebasing onto 189dfd5300...

Current branch diff-target is up to date.
Changes applied before test
commit e36caff0a4f639da835583b49564908392ff87dc
Author: Antoine R. Dumont (@ardumont) <ardumont@softwareheritage.org>
Date:   Tue Nov 9 10:35:01 2021 +0100

    SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted
    
    This will decrease the disk pressure currently existing when ingesting a svn repository
    out of a remote dump. We first fetch a dump, then mount a svn repository out of it, and
    at last we ingest the repository and have a growing copy on disk [1]. So we are up to 3
    copies which takes lots of disk space.
    
    With the following commit, we take down the 1 unnecessary copy (svn dump) as soon as
    possible (when the svn repository is mounted which we ingest).
    
    [1] implementation detail for speed, we iterate over the commit log, apply deltas (and
    computing swh hashes) along the way on disk.
    
    Related to T3719

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

Harbormaster completed remote builds in B24937: Diff 24062.Nov 9 2021, 12:23 PM
Closed by commit rDLDSVNe36caff0a4f6: SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted (authored by ardumont). · Explain WhyNov 9 2021, 12:24 PM
This revision was automatically updated to reflect the committed changes.
ardumont added a commit: rDLDSVNe36caff0a4f6: SvnLoaderFromRemoteDump: Drop dump when svn repository is mounted.

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
swh/
loader/
svn/
1 line92%
Loading...
tests/
9 lines99%
Loading...
65 lines100%
Loading...
79 lines90%
Loading...

Diff 24057

View Options

swh/loader/svn/loader.py

Loading...
View Options

swh/loader/svn/tests/test_loader.py

Loading...
View Options

swh/loader/svn/tests/test_utils.py

Loading...
View Options

swh/loader/svn/utils.py

Loading...
About · Developer information · Legal