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

fix: Added missing env variables in main.bicep for function app #1147

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

Prasanjeet-Microsoft
Copy link
Contributor

Purpose

Added azureOpenAISystemMessage and azureSearchTopK param in main.bicep files for function app to set/update the value of AZURE_SEARCH_TOP_K and AZURE_OPENAI_SYSTEM_MESSAGE env variable in azure function app using azd env set and azd up command

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Run below commands from VS code powershell terminal:

azd env refresh
azd set AZURE_OPENAI_SYSTEM_MESSAGE "You help people find all the information."
azd up

SSH to the function app:
echo $AZURE_OPENAI_SYSTEM_MESSAGE

Check whether you are able to see the expected SYSTEM_MESSAGE.

Please find below screenshot for your reference:

image image

Copy link

Coverage

Coverage Report •
FileStmtsMissCoverMissing
TOTAL264558377% 
report-only-changed-files is enabled. No files were changed during this commit :)

Tests Skipped Failures Errors Time
298 0 💤 0 ❌ 0 🔥 39.320s ⏱️

@adamdougal adamdougal added this pull request to the merge queue Jul 18, 2024
Merged via the queue into main with commit 4a88aff Jul 18, 2024
11 checks passed
@adamdougal adamdougal deleted the PSL-BUG-4943 branch July 18, 2024 08:11
Copy link

🎉 This PR is included in version 1.9.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

2 participants