ome / scc

OME tools for managing the Git(Hub) workflow
https://pypi.org/project/scc/
GNU General Public License v2.0
0 stars 15 forks source link

Try to mitigate occasional 500 error codes thrown by integration tests #238

Closed sbesson closed 5 years ago

sbesson commented 6 years ago

During the last months, the integration tests have been regularly failing with 500 GitHub exceptions. None of these failures look reproducible. This commit is investigating the addition of 500 to the list of error codes to retry.

sbesson commented 6 years ago

With this PR, https://web-proxy.openmicroscopy.org/east-ci/job/SCC-merge/PYTHON=python,label=testintegration/9/console was green. It might take a couple of iterations to confirm this is sufficient to reduce the flakiness.

About the change itself, my impression is that only the integration tests have been showing the 500 exception. It is possible we are not using all GitHub calls efficiently but one consideration would be to limit the addition of the 500 to the list of codes to retry within the scope of the integration tests only.