sync-single-account.py #968
Conversation
| sync_engine_class: ClassVar[type["FolderSyncEngine"]] | ||
|
|
There was a problem hiding this comment.
I've moved this out of an initializer so the thread class used to sync a single folder can be accessed without constructing a monitor. A monitor is a parent thread used to sync all the folders.
| @@ -55,174 +55,6 @@ | |||
| MAX_DOWNLOAD_COUNT = 1 | |||
|
|
|||
|
|
|||
| class GmailSyncMonitor(ImapSyncMonitor): | |||
There was a problem hiding this comment.
I just switched the order of classes here so I can access GmailFolderSyncEngine from GmailSyncMonitor
| @cache | ||
| def get_monitor_classes() -> dict[str, Type[BaseMailSyncMonitor]]: | ||
| """ | ||
| Return a dictionary mapping provider names to their respective monitor | ||
| """ | ||
| monitor_classes = { | ||
| module.PROVIDER: getattr(module, module.SYNC_MONITOR_CLS) | ||
| for module in module_registry.values() | ||
| if hasattr(module, "SYNC_MONITOR_CLS") | ||
| } | ||
|
|
||
| for provider_name, _ in providers.items(): | ||
| if provider_name not in monitor_classes: | ||
| monitor_classes[provider_name] = monitor_classes["generic"] | ||
|
|
||
| return monitor_classes |
There was a problem hiding this comment.
This was extracted from SyncService intializer so I can use this from the script. This code is itself terribly dynamic but sync-engine was built to be extended by installing external modules like plugins, something that we don't do ourselves but I don't want to change it right now.
| class GmailSyncMonitor(ImapSyncMonitor): | ||
| sync_engine_class: ClassVar[type[FolderSyncEngine]] = GmailFolderSyncEngine |
There was a problem hiding this comment.
And this code was moved below with
sync_engine_class: ClassVar[type[FolderSyncEngine]] = GmailFolderSyncEngine
moved from the initializer to class body.
squeaky-pl
force-pushed
the
sync-single-account
branch
from
9cec5bf
to
43adde2
Compare
This is something that I did over and over using sync-engine console to debug escalations and otherwise troubleshoot syncing of a single account. Since it is not obvious how to do it I wrapped it in a script so everybody can use it.