As title. Depends on D5745.
the existing test suite won't exercise any of this new code and i don't see any new tests, I assume you did some manual testing at least?
these are StringSource in dagster config space right? so should allow
secretKey: env: MY_ENV_KEY
which i am guessing this won't do
Hm, I've changed the ones that are string source to the correct schema. However for this azure compute log manager, it's typed as string which looks like a bug. https://sourcegraph.com/github.com/dagster-io/dagster/-/blob/python_modules/libraries/dagster-azure/dagster_azure/blob/compute_log_manager.py#L88-96
|1–16 ↗||(On Diff #28985)|
This is the first place in the helm chart where a StringSource field is exposed in values.yaml - all other instances of env: XXX are defined by us in the dagster.yaml using environment variables we have defined in a configmap.
not sure how else to coerce StringSource correctly in the template