Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove support for task monitor in cwl-WES #250

Open
kushagra189 opened this issue Nov 15, 2022 · 1 comment
Open

Remove support for task monitor in cwl-WES #250

kushagra189 opened this issue Nov 15, 2022 · 1 comment
Assignees
@kushagra189
Labels
dependencies Pull requests that update a dependency file status: in progress Currently being worked on type flag: breaking change Proposed change breaks current API type: maintenance Related to general repository maintenance type: security Related to security workload: days Likely takes days to resolve

Comments

@kushagra189
Copy link
Contributor

Currently, the executed task runs are monitored using the evening send and receive methods provided by Celery. This implementation needs to be changed as it might pose a possible vulnerability for scaling in the future. We send event logs over receivers which if exceed the required size, cannot be transferred over events. Hence, to combat the same, we will remove the task monitor setup and implement these actions as background tasks.
@kushagra189 kushagra189 added status: in progress Currently being worked on type flag: breaking change Proposed change breaks current API type: maintenance Related to general repository maintenance type: security Related to security workload: days Likely takes days to resolve dependencies Pull requests that update a dependency file labels Nov 15, 2022
@kushagra189 kushagra189 self-assigned this Nov 15, 2022
@kushagra189 kushagra189 mentioned this issue Nov 15, 2022
Merged
@kushagra189 kushagra189 mentioned this issue Nov 26, 2022
Merged
@uniqueg
Copy link
Member

uniqueg commented Nov 26, 2022

Duplicate of #194

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kushagra189 kushagra189

Labels
dependencies Pull requests that update a dependency file status: in progress Currently being worked on type flag: breaking change Proposed change breaks current API type: maintenance Related to general repository maintenance type: security Related to security workload: days Likely takes days to resolve
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@uniqueg @kushagra189