azure-search-openai-demo-csharp icon indicating copy to clipboard operation
azure-search-openai-demo-csharp copied to clipboard

azd up requires openAIApiKey infra param?

Open diberry opened this issue 1 year ago • 6 comments

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

  1. azd auth login
  2. azd up

@achandmsft

Any log messages given by the failure

@diberry ➜ /workspaces/azure-search-openai-demo-csharp (main) $ azd up
? Enter a new environment name: diberry-dotnet-chat
  (✓) Done: Downloading Bicep
? Select an Azure Subscription to use: 23. Azure Dev Platform Services Content Team (2123cb08-23fa-4a80-afec-4ec9e015433f)
? Select an Azure location to use: 11. (US) East US 2 (eastus2)
? Enter a value for the 'openAIApiKey' infrastructure parameter: [? for help] 

Expected/desired behavior

AZD up shouldn't ask for anything other than sub and locations.

OS and Version?

Windows 7, 8 or 10. Linux (which distribution). macOS (Yosemite? El Capitan? Sierra?) Codespaces

Versions

Mention any other details that might be useful


Thanks! We'll be in touch soon.

diberry avatar May 10 '24 21:05 diberry

I do agree. it ask for it, while it deploys a new OpenAI endpoint

lgmorand avatar May 18 '24 08:05 lgmorand

Where do we get the openAIApiKey? It's not part of the demo instructions.

KristinBethea avatar May 21 '24 19:05 KristinBethea

you can ignore it. the openakey is when you provide your own existing instance. but the azd up will pop up a 3.5gpt and a dedicated openai instance by default, even with the key

furthermore, it store the openAIkey in .azure but without the endpoint, nor the deployment name, to me, that's useless

lgmorand avatar May 21 '24 20:05 lgmorand

Related: #295

jaliyaudagedara avatar Jun 07 '24 17:06 jaliyaudagedara

It will use an existing OpenAI service with this key if provided. If let empty the provisioning will create an Azure openAI service and use that one.

fboucher avatar Jun 07 '24 18:06 fboucher

I ran into the same thing and found out I can just skip the parameter thanks to @FBoucher 🙂.

It would be nice if hint actually says so, so I created a PR to add that to the param description in bicep: #340

EdwinOtten avatar Jul 26 '24 14:07 EdwinOtten