Introduce ResourceSets to make loading of resource files more flexible
This MR introduces so called ResourceSets
that allow loading Kubernetes manifests either via ConfigMap or from the file system.
Closes #240 (closed)
Merge request reports
Activity
added modul/theodolite label
removed modul/theodolite label
added In Progress Theodolite Kotlin labels
- Resolved by Benedikt Wetzel
This MR is still WIP, but for some early feedback, are you happy with this declaration of
ResourceSets
? @she Thefiles
field is optional, if it is not set, all available (yaml) files will be loaded.appResourceSets: - FileSystemResourceSet: path: ./config files: - "uc1-kstreams-deployment.yaml" - "aggregation-service.yaml" - "jmx-configmap.yaml" - "uc1-service-monitor.yaml" loadGenResourceSets: - ConfigMapResourceSet: configmap: "test-configmap" files: - uc1-load-generator-service.yaml - uc1-load-generator-deployment.yaml
Edited by Benedikt Wetzel
added 24 commits
-
ec34088f...f7e2b939 - 22 commits from branch
she:master
- 45aef13a - delete incorrect test
- c1ce4104 - merge upstream master
-
ec34088f...f7e2b939 - 22 commits from branch
- Resolved by Benedikt Wetzel
- Resolved by Benedikt Wetzel
mentioned in merge request !172 (merged)
- Resolved by Sören Henning
- Resolved by Sören Henning
- Resolved by Benedikt Wetzel
- Resolved by Benedikt Wetzel
- Resolved by Benedikt Wetzel
- Resolved by Sören Henning
- Resolved by Benedikt Wetzel
added 16 commits
-
7448d5ae...c04b712f - 13 commits from branch
she:master
- 5c39aa6c - merge upstream master
- 92cc56eb - minor changes in crd schemata
- 4dd86699 - switch to correct default values
Toggle commit list-
7448d5ae...c04b712f - 13 commits from branch
I have created new documentation for our crds, both documents are in the same file (until now we had a separate file for each crd). I think this is okay, right? @she
Apart from the open points, the MR is ready from my side.
added 1 commit
- bba6adf8 - add example configmap and update standalone benchmark example in order to use the new resource sets
added 1 commit
- efeb5f17 - use oneOf in order to allow only specifying one resource set per array member
-
Need also to adjust the benchmark files in
theodolite-benchmarks/definitions/
Edited by Benedikt Wetzel-
Need also to adjust the benchmark files in
added 1 commit
- 7c1ee7c3 - update benchmark defintions in order to use resource sets
- Resolved by Sören Henning
I have updated the folder
theodolite-benchmarks/definitions/
. Could you have again a look at this? @she
- Resolved by Sören Henning
Unfortunately, this will still need some time to be merged. I would like to perform a release of the "old" resource-style first to make some experiments reproducible, but for that I have to write some documentation. I hurry up! :)
added 7 commits
-
7c1ee7c3...d7805178 - 6 commits from branch
she:master
- fe71adac - Merge branch 'master' of git.se.informatik.uni-kiel.de:she/theodolite into...
-
7c1ee7c3...d7805178 - 6 commits from branch
changed milestone to %0.6.0
added 42 commits
-
176ecc1c...f8211093 - 41 commits from branch
she:master
- d93f76ee - resolve merge conflicts
-
176ecc1c...f8211093 - 41 commits from branch
assigned to @she
- Resolved by Sören Henning
Can we use the term resources instead of resourceSets to keep it a bit simpler? Can discuss this in person, @wetzel.
mentioned in merge request !176 (merged)
added 26 commits
-
d93f76ee...ffa2212f - 25 commits from branch
she:master
- 171e6a2d - resolve merge conflicts caused by updating the resources of the example benchmarks
-
d93f76ee...ffa2212f - 25 commits from branch
- Resolved by Benedikt Wetzel
- Resolved by Benedikt Wetzel
- Resolved by Benedikt Wetzel
added 1 commit
- e7cb5b94 - Add descriptions for the resourceSets to the benchmark crd
- Resolved by Sören Henning
@she From my side, this MR should be ready.