Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Remove update_event as a single event loop for the system #160

Merged
merged 42 commits into from
Oct 17, 2020
Merged

Remove update_event as a single event loop for the system #160

merged 42 commits into from
Oct 17, 2020

Conversation

bmc-msft
Copy link
Contributor

@bmc-msft bmc-msft commented Oct 15, 2020

Builds on #159

TODO:

  • Move major components into their own timer loops (task/job in one, etc)
  • Update Task timer loop to pull from the task heartbeat queue (requires Heartbeat updates)
  • Update Node timer loop to pull from the node heartbeat queue (requires Heartbeat updates)
  • Merge pool resize & scaleset timer loops into a single loop
  • Ensure tasks stop after their duration
  • Ensure jobs stop after their duration
  • Ensure repro stop after their duration

@bmc-msft bmc-msft marked this pull request as ready for review October 16, 2020 18:49
@bmc-msft
Copy link
Contributor Author

The remaining todo items will be done in a separate PR once #164 is finished.

@bmc-msft bmc-msft changed the title WIP - Remove update_event as a single event loop for the system Remove update_event as a single event loop for the system Oct 16, 2020
src/api-service/__app__/onefuzzlib/orm.py Show resolved Hide resolved
src/api-service/__app__/onefuzzlib/orm.py Outdated Show resolved Hide resolved
src/api-service/__app__/onefuzzlib/pools.py Outdated Show resolved Hide resolved
src/api-service/__app__/timer_tasks/__init__.py Outdated Show resolved Hide resolved
src/api-service/__app__/timer_workers/__init__.py Outdated Show resolved Hide resolved
@bmc-msft bmc-msft merged commit 75f29b9 into microsoft:main Oct 17, 2020
@bmc-msft bmc-msft deleted the remove-queues-by-default branch October 17, 2020 01:42
anshuman-goel added a commit to anshuman-goel/onefuzz that referenced this pull request Oct 19, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants