From b35dff1266182b74e2765c3a79b21b087b323186 Mon Sep 17 00:00:00 2001 From: "Antoine R. Dumont (@ardumont)" Date: Thu, 29 Oct 2020 16:54:10 +0100 Subject: [PATCH] Create listing task with a default of 3 if unspecified This will allow to retry task if they do not specify it. --- swh/lister/core/lister_base.py | 1 + swh/lister/launchpad/tests/test_lister.py | 3 +-- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/swh/lister/core/lister_base.py b/swh/lister/core/lister_base.py index d8d06eb..ca7d65c 100644 --- a/swh/lister/core/lister_base.py +++ b/swh/lister/core/lister_base.py @@ -456,6 +456,7 @@ class ListerBase(abc.ABC): if "priority" in self.config: m["priority"] = self.config["priority"] task_dict = self.task_dict(**m) + task_dict.setdefault("retries_left", 3) tasks[_task_key(task_dict)] = (ir, m, task_dict) gen_tasks = (task_dicts for (_, _, task_dicts) in tasks.values()) diff --git a/swh/lister/launchpad/tests/test_lister.py b/swh/lister/launchpad/tests/test_lister.py index 963a0dd..0b305d4 100644 --- a/swh/lister/launchpad/tests/test_lister.py +++ b/swh/lister/launchpad/tests/test_lister.py @@ -5,7 +5,6 @@ def test_launchpad_lister(lister_launchpad, datadir): - lister_launchpad.run() assert ( @@ -30,4 +29,4 @@ def test_launchpad_lister(lister_launchpad, datadir): assert row["policy"] == "recurring" assert row["priority"] is None - assert row["retries_left"] == 0 + assert row["retries_left"] == 3