diff --git a/osidb/sync_manager.py b/osidb/sync_manager.py index c7b0f66f4..baa150b00 100644 --- a/osidb/sync_manager.py +++ b/osidb/sync_manager.py @@ -547,20 +547,9 @@ def schedule(cls, sync_id, *args, **kwargs): """ Schedule BZSyncManager's sync_task to Celery queue. - TODO: - - This is a workaround. SyncManager.schedule should not - decide to skip or postpone tasks, but we need it to: - - Reduce noise in flower logs that would be generated - by duplicate tasks failing. - - Reduce amount of requests sent needlessly to Bugzilla. - - Avoid sending potentially incomplete data immediately. - - Move the deduplication logic to Flaw.bzsync: - - No BZSyncManager-related action upon client request. - - A periodic task to query recently changed tasks once per - X minutes and call BZSyncManager.schedule for them. - - BZSyncManager.schedule to be left not implemented, - inherited SyncManager.schedule used. - - OSIDB-3205 + This implementation uses custom de-duplication logic + and a 20 seconds delay to mitigate "outdated model" conflicts on bugzilla. + See OSIDB-3205 for more details. :param sync_id: Unique ID for synchronized data object. """