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

Update scan failed

At least one of the branches involved have failed to scan. You can manually schedule a rescan if required.

Preview Diff

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

Subscribers

People subscribed via source and target branches