Azure-Samples / azure-search-openai-demo

A sample app for the Retrieval-Augmented Generation pattern running in Azure, using Azure AI Search for retrieval and Azure OpenAI large language models to power ChatGPT-style and Q&A experiences.
https://azure.microsoft.com/products/search
MIT License
5.91k stars 4.05k forks source link

main.parameters.json missing Key Vault and Computer Vision #1077

Open Space2Move opened 9 months ago

Space2Move commented 9 months ago

Please provide us with the following information:

This issue is for a: (mark with an x)

- [x] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Try to follow the steps to deploy using only existing Azure resources. Discover that there is no way to specify the Key Vault and Computer Vison resources that you have pre-created according to your organization's naming and tagging policies. Run "azd up" and have it fail as it tries to create those resources in violation of your organization's policies. (Or at least I think that is what's causing the policy violations I'm seeing. I'm new to this...)

Any log messages given by the failure

ERROR: deployment failed: failing invoking action 'provision', error deploying infrastructure: starting deployment to subscription: (the rest is specific to my organization)

Expected/desired behavior

Make it VERY clear for beginners how to use ONLY existing Azure resources for all the services required to run this.

OS and Version?

Running via the GitHub Codespace associated with this repo.

azd version?

azd version 1.5.0 (commit 012ae734904e0c376ce5074605a6d0d3f05789ee)

Versions

Mention any other details that might be useful


Thanks! We'll be in touch soon.

pamelafox commented 9 months ago

Thanks for filing! Until we've improved main.parameters.json and the README, you should hopefully be able to specify your existing resources by overriding parameters in main.bicep. Let us know if that doesn't work.