Page MenuHomeSoftware Heritage

cassandra: Fix flakiness of test_directory_add_atomic[concurrent]
ClosedPublic

Authored by vlorentz on Fri, Aug 5, 12:12 PM.

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 has FAILED

Patch application report for D8198 (id=29579)

Could not rebase; Attempt merge onto 4825f40a4f...

Updating 4825f40a..a3c3228f
Fast-forward
 swh/storage/cassandra/storage.py    |  2 +-
 swh/storage/tests/storage_tests.py  | 30 +++++++++++++++++++++++++-----
 swh/storage/tests/test_cassandra.py | 13 +++++++++++--
 3 files changed, 37 insertions(+), 8 deletions(-)
Changes applied before test
commit a3c3228f9a13b153646e7e3300560fad87832545
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 12:10:15 2022 +0200

    cassandra: Fix flakiness of test_directory_add_atomic[concurrent]

commit 68b93a608537478cc20a2f05f0f43a6d4b0d46e9
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 11:27:08 2022 +0200

    Fix crash of test_*_arbitrary when given objects with the same id

commit 9b9eb282e83eb7ef8c11b7e6ca8d0164bf0dd141
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 11:20:33 2022 +0200

    cassandra: Make origin_visit_status_get_random's interval consistent with postgresql
    
    The postgresql implementation uses '3 months', which is closer to 13 weeks
    than to 12 weeks.

commit 56f69e56b81c5abc28cb6e217e298ab95fb501cb
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 11:18:46 2022 +0200

    Fix flakiness of test_origin_visit_status_get_random_nothing_found
    
    start is increased from 13 to 14, because 13 weeks is 91 days,
    ie. 30+31+30; so it is sometimes smaller than 3 months.
    
    This was only hit rarely because the number of visits was small,
    so this commit also increases the number of visits to make the
    test more likely to fail if it should actually fail.

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

Build is green

Patch application report for D8198 (id=29581)

Could not rebase; Attempt merge onto 4825f40a4f...

Updating 4825f40a..280ecc38
Fast-forward
 swh/storage/cassandra/storage.py    |  2 +-
 swh/storage/tests/storage_tests.py  | 30 +++++++++++++++++++++++++-----
 swh/storage/tests/test_cassandra.py | 13 +++++++++++--
 3 files changed, 37 insertions(+), 8 deletions(-)
Changes applied before test
commit 280ecc38742361c0c0d7acda2b2bbc6db361ed74
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 12:10:15 2022 +0200

    cassandra: Fix flakiness of test_directory_add_atomic[concurrent]

commit 68b93a608537478cc20a2f05f0f43a6d4b0d46e9
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 11:27:08 2022 +0200

    Fix crash of test_*_arbitrary when given objects with the same id

commit 9b9eb282e83eb7ef8c11b7e6ca8d0164bf0dd141
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 11:20:33 2022 +0200

    cassandra: Make origin_visit_status_get_random's interval consistent with postgresql
    
    The postgresql implementation uses '3 months', which is closer to 13 weeks
    than to 12 weeks.

commit 56f69e56b81c5abc28cb6e217e298ab95fb501cb
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Aug 5 11:18:46 2022 +0200

    Fix flakiness of test_origin_visit_status_get_random_nothing_found
    
    start is increased from 13 to 14, because 13 weeks is 91 days,
    ie. 30+31+30; so it is sometimes smaller than 3 months.
    
    This was only hit rarely because the number of visits was small,
    so this commit also increases the number of visits to make the
    test more likely to fail if it should actually fail.

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

This revision is now accepted and ready to land.Fri, Aug 5, 1:44 PM