Skip to content
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

azd up requires openAIApiKey infra param? #318

Closed
diberry opened this issue May 10, 2024 · 6 comments
Closed

azd up requires openAIApiKey infra param? #318

diberry opened this issue May 10, 2024 · 6 comments

Comments

@diberry
Copy link

diberry commented May 10, 2024

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.

@lgmorand
Copy link

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

@KristinBethea
Copy link

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

@lgmorand
Copy link

lgmorand commented May 21, 2024

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

@jaliyaudagedara
Copy link

jaliyaudagedara commented Jun 7, 2024

Related: #295

@FBoucher
Copy link
Contributor

FBoucher commented Jun 7, 2024

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.

EdwinOtten added a commit to EdwinOtten/azure-search-openai-demo-csharp that referenced this issue Jul 26, 2024
Improve the experience when running `azd up` by providing more context on how to use the openAIApiKey parameter.
@EdwinOtten
Copy link
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants