Alternative: Add task events to the scheduler #2214
Closed
Conversation
This file contains 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
* main: Add less verbose logging and introduce structured logging in the scheduler (#2143) Remove default setting of id to None (#2166) Web and system report integration testing (#2203) Download report JSON and ReportData OOI (#2205) Fixes sidemenu animation and behaviour (#2192) Feat/clearance level trail (#2202) Add scheduler stats to boefje/normaliser task pages in rocky. (#2204) Fix/ooi detail improvements (#2201) Translations update from Hosted Weblate (#2148) Add a request model for push endpoint in scheduler (#2169) fix rpki delete_on_close, can be just delete (#2170) Increases gap between items within dl (#2193)
* main: Updated deploy-pages action (#2251) Bump msgpackr from 1.6.1 to 1.10.1 in /rocky (#2219) Add note about Debian packages to 1.14 release notes (#2234) Upgrade GitHub actions (#2235) Remove RPKI data files (#2228) Add 1.14 release notes (#2227) remove feature flag for reports. (#2226) Add multi organization report (#2209) Preselected OOIs for Reporting (#2208) Text changes OOI into Object (#2224) Adds the option to add chapter letters (#2225) Fix typos in aggregate report (#2168) Alert icons (#2221) Fix health endpoint scheduler (#2218) Table captions (#2217) Initial version of aggregate report appendix (#2212) add html entities escaping to normalizer results (#2211) Pdf and print styling (#2206) Adding recommendations to the aggregate report (#2210)
6 tasks
|
Closing in favor of #2451 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Changes
This PR add task events functionality to the scheduler.
queued(how long was the task on the queue),runtime(how long did the taskrunner take),duration(how long total time of processing of the task from creation to finalization). These new fields are exposed by the api. Since these fields are@computed_fieldwe can also opt for adding these to their own jsonb field if these values regularly need to be queried.eventsfield added toTask, and is also exposed by the api, such that events (status change events for now) can be inspected through the api.Changes to initial PR (#2043)
post_push,post_pop,patch_task)from_status,to_status