Page MenuHomeSoftware Heritage

postgresql: Increase timeouts that often fail
ClosedPublic

Authored by vlorentz on Mar 11 2022, 1:20 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 is green

Patch application report for D7337 (id=26529)

Rebasing onto 4e78014b07...

Current branch diff-target is up to date.
Changes applied before test
commit b0cdab58b71b25e07d40dc022a5f0b0c77298c3f
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Fri Mar 11 11:36:01 2022 +0100

    postgresql: Increase timeouts that often fail
    
    According to Sentry, in the last 30 days:
    
    * directory_entry_get_by_path: 958 events, https://sentry.softwareheritage.org/share/issue/c4c2124953a145b2bd325f6f6b7df5a6/
    * revision_get: 841 events, https://sentry.softwareheritage.org/share/issue/55fbe01c6f4d4c9bbf684c7608a62ad9/
    * release_get: 14 events, https://sentry.softwareheritage.org/share/issue/37c53354541b4c4eaa1faf4e20a68418/
    * origin_visit_find_by_date: 114 events, https://sentry.softwareheritage.org/share/issue/a674c12049a941968a717661a0226559/
    * origin_get: 79 events, https://sentry.softwareheritage.org/share/issue/bf21d6bc7b24442eb18643d80d936d27/ ; 67 events, https://sentry.softwareheritage.org/share/issue/010a4b1e085a4e2089ba4897c6de6038/

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

ardumont added a reviewer: olasd. ardumont added 1 blocking reviewer(s): Reviewers.
ardumont added a subscriber: ardumont.

lgtm

It's only a workaround but short of time for analysing and trying to make those queries perform better...
i'm fine with this for now.

Others might have better ideas though.

This revision is now accepted and ready to land.Mar 11 2022, 3:08 PM
vlorentz added a task: Restricted Maniphest Task.Mar 16 2022, 2:26 PM