Make the analysis of experiments more flexible
- Allow arbitrary prometheus range queres (update prometheusResponse)
- Allow usage of more than one slo checker
- Update the json which send to the python slo checker
- refactoring
Update CRDs in order to set the query string as an dedicated parameter for a slo checker-
Update exisiting python slo checker in order to use the new JSON format(handled in !185 (merged))
Closes #249 (closed)
Merge request reports
Activity
- Resolved by Benedikt Wetzel
For offline analysis we store the result of Prometheus query in a
.csvfile. This file has the columns"labels", "timestamp", "value"(before this MR the names of these columns were"group", "timestamp", "value"). However, I am not sure if we should store additionalmetadatafor evaluation, like "warmup"`, the "prometheus query" or similar (in the future, when we have more different SLO checkers, there could be more different metadata). But I am not sure, what do you think? @sheApart from this question, this MR should be finished.
- Last reply by Benedikt Wetzel
added 5 commits
-
3da8dc8e...eb63b0c8 - 4 commits from branch
she:master - 33d7d120 - Merge branch 'master' of git.se.informatik.uni-kiel.de:she/theodolite into...
-
3da8dc8e...eb63b0c8 - 4 commits from branch
added 1 commit
- f2e26bd0 - Load paramter via quarkus configuration mechanism
added 1 commit
- 07125382 - minor code changes in evaluation related parts
added 8 commits
-
54239255...11fcf2b8 - 7 commits from branch
she:master - 90e8de9f - Merge branch 'master' of git.se.informatik.uni-kiel.de:she/theodolite into...
-
54239255...11fcf2b8 - 7 commits from branch
requested review from @she
changed milestone to %0.6.0
added 27 commits
-
f632d24b...edb188b1 - 26 commits from branch
she:master - ae0130e6 - resolve merge conflicts (changing from lag trend percentage to ratio, support both versions)
-
f632d24b...edb188b1 - 26 commits from branch
added 5 commits
-
ae0130e6...f8211093 - 4 commits from branch
she:master - ec4d5bf1 - Resolve merge conflicts, update from lag trend percentages to ratio
-
ae0130e6...f8211093 - 4 commits from branch
added modul/theodolite label
- Resolved by Benedikt Wetzel
@wetzel As you recently updated our analysis notebooks, do these have to be adjusted as well as due this MR?
- Last reply by Benedikt Wetzel
added 20 commits
-
ec4d5bf1...ec6acfa9 - 19 commits from branch
she:master - 98edf33f - Merge branch 'master' of git.se.informatik.uni-kiel.de:she/theodolite into...
-
ec4d5bf1...ec6acfa9 - 19 commits from branch
