Details
-
Suggestion
-
Resolution: Done
-
P3: Somewhat important
-
None
-
None
-
None
Description
Coin initial design was based on micro-services. The design didn't survived (IPC costs) and slowly, on python side we were moving to event loop based approach: asyncio. The shift was case by case, whenever we thought it is possible or easy to use async code. Sadly that incremental development caused some fragmentation, we use many threads to call blocking functions and in addition testing async code was considered harder. To be able to maintain the code this needs to be cleanup, otherwise debugging things like QTQAINFRA-1779 is just hard.
Attachments
Issue Links
- is duplicated by
-
COIN-140 Do not use python for multiprocessing
- Closed
For Gerrit Dashboard: COIN-137 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
202020,7 | Speedup archives creation by factor of ~25, at least for qt5 | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222187,10 | Move repositorymanager away from staticmethods | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222188,6 | Detect event loop in asyncDirectoryLock automatically | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222189,6 | Fix utils.schedule_coroutine_from_thread_and_block_until_finished | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222190,6 | Fix test storage cleaning in test_storage | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222192,13 | Long live ash | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222221,12 | Re-introduce lru_cache in repositorymanager | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222222,16 | Introduce static_cache parameter for async_lru_cache | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222224,6 | Make ModuleFactory.modules static | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222225,8 | Port modules.py to asyncio | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222381,12 | Simplify schedule_coroutine_from_thread_and_block_until_finished | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222382,9 | Add a way to override default service name | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222383,9 | Fix test_storage | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222384,12 | Add a way to call staticmethods when WrapAsyncClass is used | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222385,12 | Port storage to asyncio | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222386,12 | Simplify TestAgent.test_check_reconnections | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222387,11 | Simplify TestAgent.test_core_file_upload | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222388,12 | Update asyncDirectoryLock | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222420,9 | Move asyncio_test decorator to test_helpers | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222421,9 | Port test_dependency_resolver to RepositoryManagerAsync | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222422,9 | Port test_utils to RepostoryManagerAsync | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222423,9 | Port test_scheduler to RepositoryManagerAsync and LocalStorageAsync | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222424,6 | Remove unused variables from debug_integrator | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222474,8 | Port debug_integrator to asyncio | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222475,8 | Cleanup debug_integrator a bit | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222476,8 | Use nonblocking process creation in test_roundtrip | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222477,8 | Port test_scheduler_deps to LocalStorageAsync | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222478,10 | Retry git fetch if network connection is broken | master | qtqa/tqtc-coin-ci | Status: ABANDONED | -1 | 0 |
222508,1 | Improve usability of schedule_coroutine_from_thread | master | qtqa/tqtc-coin-ci | Status: ABANDONED | -1 | 0 |
222510,10 | Port platform_configurations to asyncio | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222511,10 | Port test_workitemfactory to RepostoryManagerAsync | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
222970,7 | Port WorkItemFactory to asyncio | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |
223030,5 | Use RepositoryManagerAsync in test_configurations | master | qtqa/tqtc-coin-ci | Status: MERGED | +2 | 0 |