-
Notifications
You must be signed in to change notification settings - Fork 154
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Container App secret settings #964
Comments
Those are difficult to get working and an improvement will be welcome. What would the breaking change be? |
@isaacabraham any news on a possible PR for this? Happy to take a look |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
At the moment, container apps has a multitude of ways to apply settings. The thing that you can't currently (easily) do in Farmer is have a secret setting and have it applied into the container app - it doesn't get set properly. @ninjarobot I remember we spoke about this - I'd like to submit a PR that fixes this but it might be a breaking change in terms of the API (but not in terms of ARM generation as such). The currently codebase is IIRC (I need to dig up a prototype fix I made) broken.
The text was updated successfully, but these errors were encountered: