Commit c13dd39
Create Istio analysis datagatherer (#211)
* Initial implementation
Signed-off-by: wwwil <[email protected]>
* Allow list of resources to be specified in data gatherer config, otherwise use the default list
Signed-off-by: wwwil <[email protected]>
* Only ignore dynamic data gatherer errors if they are for a missing resource
Signed-off-by: wwwil <[email protected]>
* Implement full test of Istio data gatherer fetch
Signed-off-by: wwwil <[email protected]>
* go mod tidy
Signed-off-by: Charlie Egan <[email protected]>
* WIP on fixing fake dynamic client
Signed-off-by: Charlie Egan <[email protected]>
* Fix package name
Signed-off-by: wwwil <[email protected]>
* Improve variable names
Signed-off-by: wwwil <[email protected]>
* Use strings.Join to build YAML string
Signed-off-by: wwwil <[email protected]>
* Remove redundant TODO and commented-out test
Signed-off-by: wwwil <[email protected]>
* Fix issues with fake client in test for Kubernetes dynamic data gatherer
Signed-off-by: wwwil <[email protected]>
* Add missing YAML tag
Signed-off-by: wwwil <[email protected]>
* Add explanation for why httptest LocalServer is used
Signed-off-by: wwwil <[email protected]>
* Fix typo in variable name
Signed-off-by: Charlie Egan <[email protected]>
Co-authored-by: Charlie Egan <[email protected]>1 parent 7f4cb90 commit c13dd39
File tree
6 files changed
+1316
-126
lines changed- pkg/datagatherer
- istio
- k8s
6 files changed
+1316
-126
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
13 | | - | |
| 12 | + | |
| 13 | + | |
14 | 14 | | |
15 | | - | |
16 | 15 | | |
17 | | - | |
18 | | - | |
19 | | - | |
20 | | - | |
21 | | - | |
| 16 | + | |
22 | 17 | | |
23 | 18 | | |
24 | 19 | | |
| |||
27 | 22 | | |
28 | 23 | | |
29 | 24 | | |
30 | | - | |
| 25 | + | |
31 | 26 | | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
38 | | - | |
| 27 | + | |
| 28 | + | |
39 | 29 | | |
40 | 30 | | |
41 | 31 | | |
42 | 32 | | |
43 | | - | |
44 | | - | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
45 | 36 | | |
46 | | - | |
47 | | - | |
| 37 | + | |
48 | 38 | | |
49 | 39 | | |
50 | | - | |
| 40 | + | |
0 commit comments