-
Notifications
You must be signed in to change notification settings - Fork 197
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
[.NET] <module>.parameters.json would be nice to have back #2548
Comments
Was also chatting with @bradygaster today about some of these scenarios. @rajeshkamal5050 would be good to prioritize smoothing this out next iteration. |
Thanks @savannahostrowski - @rajeshkamal5050 we essentially have some plans we're trying to shore up for November, and one of the things we want to invest in is a tighter integration option between .NET and AZD, and if it's possible that we could get these JSON files back in, or, be updated on how individual modules can be parameterized via JSON, it'd be super-helpful as we're creating click-throughs and mock-ups. |
I do wonder how much of this you could build outside of
|
I specifically tried using |
IIRC, We only ever supported using a What I envisioned is doing something more in the template itself, as you could do with any bicep template. Azure-Samples/todo-nodejs-mongo@main...ellismg:todo-nodejs-mongo:ellismg/use-json-file-with-modules Has a little sketch of what I was thinking you could do today, with the released version of This file can only contain fixed strings, not replacements like |
Let's get @wbreza input as he implemented the module.parameters.json initially and then recently removed it. He'll have more insight. |
That's unfortunate and would mitigate me wanting to try that route. I'd be giving up the ability to read environment variables generated by |
Ahh - this makes me better understand what the request was here. This support was only for container apps and was tied to how we did Azure Container App deployments when we first added the feature. In that world, Since this was a top level deployment, we were able to use Initially I thought this was about using a parameters file to control the parameters for any arbitrary module used from the root deployment. I see now that this is perhaps specific to ACA. We removed the Somewhat related to this was the work we were thinking about with container apps to use a yaml based file to explain the deployment instead of doing all this stuff in bicep. I had sketched out what that might mean here. In this world, instead of authoring bicep for each container app you want to deploy, you'd instead write a yaml file that describes your app, similar to a k8s manifest and then |
@bradygaster closing this one. Given the integrations happening on the .NET side which is tracked here - https://github.com/Azure/azure-dev-pr/issues/1579 |
A few of the investments we're thinking of making on the .NET side would lend themselves very-very well to having support for individual module parameter JSON files added back in. Take a complex app with 5 container apps in it, for example, but with only 2-3 variations per app in terms of their settings or environment variables. If customers could have the JSON approach to editing these settings and having single-project capabilities for editing variables, it could be a lot less daunting for developers who don't-yet-know Bicep but want to take advantage of it.
CC @jongio, with whom I met today on this, as he'll have more context for the team.
The text was updated successfully, but these errors were encountered: