Page MenuHomeSoftware Heritage
Differential D4841

Make test_content_add_race fail for the right reason.
ClosedPublic
Actions

Authored by vlorentz on Jan 12 2021, 12:39 PM.

Details

Reviewers
ardumont
Group Reviewers
Reviewers
Commits
rDSTOda55308f3df5: Make test_content_add_race fail for the right reason.
Summary

Since 209de5dbaa127dacd114fbbd084f22632982eb77 , it was failing because of:
TypeError("content_add() got an unexpected keyword argument 'db'")

Diff Detail

Event Timeline

vlorentz created this revision.Jan 12 2021, 12:39 PM
Herald added a reviewer: Reviewers. · View Herald TranscriptJan 12 2021, 12:39 PM
vlorentz edited the summary of this revision. (Show Details)Jan 12 2021, 12:42 PM
swh-public-ci added a comment.Jan 12 2021, 12:43 PM

Build has FAILED

Patch application report for D4841 (id=17140)

Rebasing onto 2b35198d30...

First, rewinding head to replay your work on top of it...
Applying: Make test_content_add_race fail for the right reason.
Changes applied before test
commit 3cd80317b170e7d55fd1780944506febdb7ecc72
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 12:38:33 2021 +0100

    Make test_content_add_race fail for the right reason.
    
    Since 209de5dbaa127dacd114fbbd084f22632982eb77, it was failing because of:
    TypeError("content_add() got an unexpected keyword argument 'db'")

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

Harbormaster returned this revision to the author for changes because remote builds failed.Jan 12 2021, 12:43 PM
Harbormaster failed remote builds in B18284: Diff 17140!
swh-public-ci added a comment.Jan 12 2021, 12:57 PM

Build has FAILED

Patch application report for D4841 (id=17140)

Rebasing onto 2b35198d30...

First, rewinding head to replay your work on top of it...
Applying: Make test_content_add_race fail for the right reason.
Changes applied before test
commit 7279c62c2e708929cfd3245ff32cbee7b7d9334a
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 12:38:33 2021 +0100

    Make test_content_add_race fail for the right reason.
    
    Since 209de5dbaa127dacd114fbbd084f22632982eb77, it was failing because of:
    TypeError("content_add() got an unexpected keyword argument 'db'")

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

vlorentz updated this revision to Diff 17142.Jan 12 2021, 1:26 PM

rebase

vlorentz updated this revision to Diff 17144.Jan 12 2021, 1:34 PM

rebase

vlorentz added a parent revision: D4842: Fix compatibility with hypothesis >= 5.49.0..Jan 12 2021, 1:34 PM
swh-public-ci added a comment.Jan 12 2021, 1:35 PM

Build has FAILED

Patch application report for D4841 (id=17142)

Could not rebase; Attempt merge onto 2b35198d30...

Merge made by the 'recursive' strategy.
 swh/storage/postgresql/storage.py    | 29 ++++++++++++++++-------------
 swh/storage/tests/storage_tests.py   | 13 +++++++++++--
 swh/storage/tests/test_postgresql.py |  4 ++--
 3 files changed, 29 insertions(+), 17 deletions(-)
Changes applied before test
commit 688335193282907daf7f558a8b21996e881d49a8
Merge: 2b35198d b52f30cb
Author: Jenkins user <jenkins@localhost>
Date:   Tue Jan 12 12:30:53 2021 +0000

    Merge branch 'diff-target' into HEAD

commit b52f30cb852ba3db98aaa35785befe97287d6be2
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 12:38:33 2021 +0100

    Make test_content_add_race fail for the right reason.
    
    Since 209de5dbaa127dacd114fbbd084f22632982eb77, it was failing because of:
    TypeError("content_add() got an unexpected keyword argument 'db'")

commit c0d035ae04edfa302be808482a5e37ac67a7940b
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 13:25:04 2021 +0100

    Fix compatibility with hypothesis >= 5.49.0.
    
    It errored with:
    
        the 'swh_storage' fixture, which is reset between function calls but
        not between test cases generated by `@given(...)`.

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

Harbormaster failed remote builds in B18286: Diff 17142!Jan 12 2021, 1:35 PM
swh-public-ci added a comment.Jan 12 2021, 1:45 PM

Build is green

Patch application report for D4841 (id=17144)

Could not rebase; Attempt merge onto 2b35198d30...

Merge made by the 'recursive' strategy.
 swh/storage/postgresql/storage.py        | 29 ++++++++++++++++-------------
 swh/storage/tests/algos/test_snapshot.py |  4 +++-
 swh/storage/tests/storage_tests.py       | 13 +++++++++++--
 swh/storage/tests/test_postgresql.py     |  4 ++--
 4 files changed, 32 insertions(+), 18 deletions(-)
Changes applied before test
commit ea5310be3780a9263fa4881fdd405c019eee70d0
Merge: 2b35198d 41b364f2
Author: Jenkins user <jenkins@localhost>
Date:   Tue Jan 12 12:40:22 2021 +0000

    Merge branch 'diff-target' into HEAD

commit 41b364f287dd2669444ce739164284d6df427da7
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 12:38:33 2021 +0100

    Make test_content_add_race fail for the right reason.
    
    Since 209de5dbaa127dacd114fbbd084f22632982eb77, it was failing because of:
    TypeError("content_add() got an unexpected keyword argument 'db'")

commit 25429b8590438e8ca2827c1ceb1f61a70b3833f3
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 13:25:04 2021 +0100

    Fix compatibility with hypothesis >= 5.49.0.
    
    It errored with:
    
        the 'swh_storage' fixture, which is reset between function calls but
        not between test cases generated by `@given(...)`.

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

Harbormaster completed remote builds in B18288: Diff 17144.Jan 12 2021, 1:45 PM
vlorentz requested review of this revision.Jan 12 2021, 1:45 PM
olasd added a subscriber: olasd.Jan 12 2021, 6:11 PM

Rather than introducing a new function, we could test race conditions on _content_add_metadata directly?

olasd added a comment.Jan 13 2021, 5:19 PM
In D4841#120971, @olasd wrote:

Rather than introducing a new function, we could test race conditions on _content_add_metadata directly?

(ping @vlorentz)

vlorentz planned changes to this revision.Jan 14 2021, 2:20 PM
vlorentz updated this revision to Diff 17226.Jan 14 2021, 2:27 PM

apply comment

swh-public-ci added a comment.Jan 14 2021, 2:32 PM

Build is green

Patch application report for D4841 (id=17226)

Rebasing onto 0b44b37254...

First, rewinding head to replay your work on top of it...
Applying: Make test_content_add_race fail for the right reason.
Changes applied before test
commit ac996c22e46d5acea7bd45f4001a988fd456c1c7
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 12:38:33 2021 +0100

    Make test_content_add_race fail for the right reason.
    
    Since 209de5dbaa127dacd114fbbd084f22632982eb77, it was failing because of:
    TypeError("content_add() got an unexpected keyword argument 'db'")

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

Harbormaster completed remote builds in B18370: Diff 17226.Jan 14 2021, 2:33 PM
ardumont accepted this revision.Jan 14 2021, 7:16 PM
This revision is now accepted and ready to land.Jan 14 2021, 7:16 PM
vlorentz updated this revision to Diff 17247.Jan 15 2021, 11:31 AM

rebase

This revision was landed with ongoing or failed builds.Jan 15 2021, 11:31 AM
Closed by commit rDSTOda55308f3df5: Make test_content_add_race fail for the right reason. (authored by vlorentz). · Explain Why
This revision was automatically updated to reflect the committed changes.
vlorentz added a commit: rDSTOda55308f3df5: Make test_content_add_race fail for the right reason..
swh-public-ci added a comment.Jan 15 2021, 11:36 AM

Build is green

Patch application report for D4841 (id=17247)

Rebasing onto 0b44b37254...

Current branch diff-target is up to date.
Changes applied before test
commit da55308f3df5cbd31bfbcd8920c12d874ceb4733
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Tue Jan 12 12:38:33 2021 +0100

    Make test_content_add_race fail for the right reason.
    
    Since 209de5dbaa127dacd114fbbd084f22632982eb77, it was failing because of:
    TypeError("content_add() got an unexpected keyword argument 'db'")

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

Harbormaster completed remote builds in B18389: Diff 17247.Jan 15 2021, 11:36 AM

Revision Contents
Changeset List

PathSizeCoverage (All)Coverage (Touched)
swh/
storage/
postgresql/
29 lines95%
Loading...
tests/
4 lines100%
Loading...

Diff 17144

View Options

swh/storage/postgresql/storage.py

Loading...
View Options

swh/storage/tests/test_postgresql.py

Loading...
About · Developer information · Legal