You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
requirements in discovered workflows is opt-in - #260
Remove requirements gathering by default as it takes time and also the requirements in the locally cached workflows could get out of sync with the worker.
Claude produced this server startup profiling (worker: bes, ewoksmx, ewoksid31, blissoda):
Category
Fresh + requirements
Fresh + no requirements
Cached + requirements
Cached + no requirements
Import ewoksserver
4.9 s
4.7 s
4.8 s
4.7 s
Spawn worker and import in it (first job)
4.0 s
4.1 s
3.9 s
4.3 s
Task discovery on the worker
3.8 s
3.7 s
3.7 s
3.8 s
Workflow discovery on the worker
<0.1 s
<0.1 s
<0.1 s
<0.1 s
Convert external workflows on the worker
180.3 s (242)
33.3 s (242)
0.9 s (8)
0.8 s (8)
Write task and workflow JSON files
4.8 s (~600)
4.8 s (~600)
0.1 s (~366)
0.1 s (~366)
Copy defaults, create app, index
~0.2 s
~0.2 s
~0.2 s
~0.2 s
Total until ready
198.1 s
50.8 s
13.6 s
13.7 s
Shutdown
0.9 s
0.9 s
0.9 s
0.9 s
Fresh means it runs for the first time.
Cost of requirements: gathering them adds about 0.6 s per workflow (one pip freeze each), which is about 147 s on a fresh start.
Why cached is the same either way: on a cached start, the only workflows still converted are the 8 broken bes ones. They fail while loading, before pip freeze would run.
242: workflows found by discovery. bes 149, blissoda 73, ewoksmx 16, ewoksid31 4. On a fresh start every one of them is converted.
8: the broken bes workflows: 5 Massif1*Twitter, ExecuteMXPressOrig, ExecuteMXPressTest and MXPressR_dehydration. They fail to convert, so they're never cached and are tried again on every startup.
366: tasks found by discovery. bes 292, ewoksmx 32, ewoksxrpd 31, ewokscore 8, ewoksid31 3. The server writes every task JSON on every startup, cached or not.
600: JSON files written on a fresh start: the 366 tasks plus 234 cached workflows (242 minus the 8 that fail).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR summary
Remove requirements gathering by default as it takes time and also the requirements in the locally cached workflows could get out of sync with the worker.
AI Disclosure