Merge ~r00ta/maas:lp-2056781-images-sync-max-attempts into maas:master

Proposed by Jacopo Rota
Status: Merged
Approved by: Alexsander de Souza
Approved revision: 29fce3c24a3880b03c6fcec9e7c8ca65e6bf8bfe
Merge reported by: MAAS Lander
Merged at revision: not available
Proposed branch: ~r00ta/maas:lp-2056781-images-sync-max-attempts
Merge into: maas:master
Diff against target: 13 lines (+1/-1)
1 file modified
src/maasserver/workflow/bootresource.py (+1/-1)
Reviewer Review Type Date Requested Status
Alexsander de Souza Approve
MAAS Lander Approve
Review via email: mp+462226@code.launchpad.net

Commit message

fix: lp-2056781. no maximum attempts for the download-bootresourcefile activity in the sync-bootresources workflow

Description of the change

The image sync process should not be stopped in case of exceptions. We should keep retrying in any case, otherwise the user would not know if the import of the images has completed successfully.

To post a comment you must log in.
Revision history for this message
MAAS Lander (maas-lander) wrote :

UNIT TESTS
-b lp-2056781-images-sync-max-attempts lp:~r00ta/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: SUCCESS
COMMIT: 29fce3c24a3880b03c6fcec9e7c8ca65e6bf8bfe

review: Approve
Revision history for this message
Alexsander de Souza (alexsander-souza) wrote :

+1

review: Approve
Revision history for this message
MAAS Lander (maas-lander) wrote :

LANDING
-b lp-2056781-images-sync-max-attempts lp:~r00ta/maas/+git/maas into -b master lp:~maas-committers/maas

STATUS: FAILED BUILD
LOG: http://maas-ci.internal:8080/job/maas-tester/4935/console

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/maasserver/workflow/bootresource.py b/src/maasserver/workflow/bootresource.py
2index 9d9875c..5c6528e 100644
3--- a/src/maasserver/workflow/bootresource.py
4+++ b/src/maasserver/workflow/bootresource.py
5@@ -225,7 +225,7 @@ class DownloadBootResourceWorkflow:
6 heartbeat_timeout=HEARTBEAT_TIMEOUT,
7 cancellation_type=ActivityCancellationType.WAIT_CANCELLATION_COMPLETED,
8 retry_policy=RetryPolicy(
9- maximum_attempts=max(2, len(input.source_list)),
10+ maximum_attempts=0, # No maximum attempts
11 maximum_interval=timedelta(seconds=60),
12 ),
13 )

Subscribers

People subscribed via source and target branches