Differential D4941

cooker: Add retry behavior on storage instance
ClosedPublic
Actions

Authored by ardumont on Jan 25 2021, 3:40 PM.

Details

Reviewers

tenma

Maniphest Tasks

T2862: vault: retry RPC calls on transient network errors

Commits

rSPSITEd17fc97d35fd: cooker: Add retry behavior on storage instance

Summary

Related to T2862

Test Plan

octocatalog-diff worker1.internal.staging.swh.network:

*******************************************
  File[/etc/softwareheritage/vault_cooker.yml] =>
   parameters =>
     content =>
      @@ -1,7 +1,9 @@
       ---
       storage:
      -  cls: remote
      -  args:
      -    url: http://storage1.internal.staging.swh.network:5002/
      +  cls: retry
      +  storage:
      +    cls: remote
      +    args:
      +      url: http://storage1.internal.staging.swh.network:5002/
       vault:
         cls: remote
*******************************************
*** End octocatalog-diff on worker1.internal.staging.swh.network

or on azure node (prod):

*******************************************
  File[/etc/softwareheritage/vault_cooker.yml] =>
   parameters =>
     content =>
      @@ -1,7 +1,9 @@
       ---
       storage:
      -  cls: remote
      -  args:
      -    url: http://storage01.euwest.azure.internal.softwareheritage.org:5002/
      +  cls: retry
      +  storage:
      +    cls: remote
      +    args:
      +      url: http://storage01.euwest.azure.internal.softwareheritage.org:5002/
       vault:
         cls: remote
*******************************************
*** End octocatalog-diff on worker11.euwest.azure.internal.softwareheritage.org