[DO NOT MERGE] Experiment with WDS App Insights metrics reduction #4493
Conversation
| @@ -386,6 +386,7 @@ azure { | |||
| wds-app-config { | |||
| environment = "dev" | |||
| environment-base = "live" | |||
| image: "us-central1-docker.pkg.dev/dsp-artifact-registry/mf-dataplane-metrics-wip/wds:insights" | |||
There was a problem hiding this comment.
Often to avoid the whole Leo dance I just kubectl edit deploy on a live WDS/CBAS/etc deployment to change the image and iterate.
There was a problem hiding this comment.
Thanks! yeah I know that's faster. wanted to test something end to end :) With a fresh landing zone and clean deploy of wds. I don't actually need to iterate on WDS itself that much. The change I'm making is a one liner. But it's going to be hard to tell if it did the right thing in pre-existing landing zone with a bunch of other WDSes in it.
There was a problem hiding this comment.
Also apparently I don't have kubectl access to landing zone aks clusters and I'm not sure how give that access to myself. I'm being blocked by deny assignments. I am in the terra-workspace-dev AD group
|
Whenever you return to this PR, note that while you were out we shipped https://github.com/broadinstitute/terra-helmfile/pull/5519, which reduces WDS frequency to 5 minutes |
This is the same thing that I did in #4164. I have no intention of merging this. I'm just trying to test a custom build of wds in my bee and don't want to have to jump through the hoops of publishing a new helm chart etc.. so I'm doing this instead. I will close this once I'm done testing.