HomecareHomebase / azure-bake

An Open Source Deployment Management Framework for DevOps and Developers
MIT License
9 stars 14 forks source link

Update docs regarding BAKE_VARIABLES config #242

Open nmarinel opened 2 years ago

nmarinel commented 2 years ago

BAKE_VARIABLES is actually meant to be a path to a YAML file, not a string of JSON.

Here is where help describes the setting as a path and here is where the parsing happens.

nmarinel commented 2 years ago

btw this came up because not having this set causes an error in the Kubernetes ingredient here. This probably qualifies as an issue - that _ctx.Environment.variables is completely undefined when the variables file isn't specified, but when an empty file is specified then _ctx.Environment.variables is populated with recipe data from the bake.yaml file.