connection often times out on `test_download` and `test_get_download_normalization_process_handler`
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- python
- Ambito
- networking, testing-qa
Direzione di ricerca
Inizia dal timeout alla riga 71 di labcontrol/gui/testing.py e riproduci i fallimenti in test_download e test_get_download_normalization_process_handler utilizzando la build Travis indicata. Confronta se aumentare il timeout o aggiungere retry all’interno dei test risolve il fallimento intermittente della rete; il lavoro è completato quando entrambi i test di download superano l’esecuzione in modo affidabile mantenendo le asserzioni sulle risposte esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Something potentially gnarly, but likely out of the scope of this PR... I ran ea536f3's build twice (this one), and it passed the first time but failed on the two "download" tests (
test_downloadandtest_get_download_normalization_process_handler) when run again.Since these failures were both caused by
AssertionError: Async operation timed out after 10 seconds, it seems like this is an intermittent error due to network problems (???). Not sure how best to handle this, but as a brute-force solution usingtravis_retrywith the python tests might work.
You're right, this has been a recurrent problem I've noticed in the past couple weeks. I think there are a couple things we can do to try to reduce this.
- Increase the
timeoutlimit in the line below:
https://github.com/biocore/LabControl/blob/1427ad163682ff11b1086e0e3b1010ba1a2166ee/labcontrol/gui/testing.py#L71 - add retry functionality within the test, e.g. something like this:
class TestDownloadLibraryPrepShotgunProcessHandler(TestHandlerBase):
def test_download(self):
retries = 0
response = None
while retries < 5:
try:
response = self.get(
'/process/library_prep_shotgun/%d/echo_pick_list' % 1)
except AssertionError:
print("Error downloading on try {}.".format(retries))
else:
break
if response is None:
raise AssertionError("Async operation timed out after maximum number of retries.")
self.assertNotEqual(response.body, '')
self.assertTrue(response.body.startswith(
b'Sample ID\tSource Plate Name\t'))
self.assertEqual(response.headers['Content-Disposition'],
"attachment; filename=2017-10-25_"
"Test_compressed_gDNA_plates_1-4_indices.txt")
Originally posted by @gwarmstrong in https://github.com/biocore/LabControl/pull/585#issuecomment-529705330
- Lingua principale
- Python
- Stelle
- 2
- Fork
- 15
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di biocore/LabControl
-
bug front-end question
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
biocore/LabControl#594 ·
-
front-end question
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
biocore/LabControl#593 ·
-
front-end
biocore/LabControl#592 · 1 assegnatario ·
-
code refactor front-end
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
biocore/LabControl#591 ·
-
priority:low
Difficoltà 3/5 1-2 giorni Idoneità per principianti 20/100
biocore/LabControl#590 ·
Tutte le issue di biocore/LabControl
Issue simili
-
agent-ready documentation needs-triage
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
-
documentation
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 92/100
-
instance instance add
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 72/100
searxng/searx-instances#939 · 1 commento ·
-
area-deployment area-integrations triage:bot-seen
Difficoltà 2/5 Mezza giornata Idoneità per principianti 86/100