-
Notifications
You must be signed in to change notification settings - Fork 558
feat: Added EXP Changes, Model updates & End to End tests #1828
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
Merged
Merged
Changes from 90 commits
Commits
Show all changes
92 commits
Select commit
Hold shift + click to select a range
2dbb0b1
ci: Updated workflow to handle main, dev and demo branch | Dependabot…
Roopan-Microsoft 97ced9f
fix: SFI Fixes & scope reverted to subscription (#1513)
Roopan-Microsoft 903c259
ci: workflow updated for build docker (#1514)
Roopan-Microsoft 4cf2972
ci: build docker updated for dev (#1522)
Roopan-Microsoft 9158819
ci: workflow branch code updated (#1525)
Roopan-Microsoft 4545a48
updated comment
AjitPadhi-Microsoft 84a33b5
updated workflow
AjitPadhi-Microsoft f8c1c8f
updated workflow
AjitPadhi-Microsoft e31a661
updated workflow
AjitPadhi-Microsoft 847ce7e
updated bicep for registry
AjitPadhi-Microsoft 8f8cc98
fixed bicep
AjitPadhi-Microsoft 15b948c
fix: bicep updated (#1527)
AjitPadhi-Microsoft 1645e43
Merge branch 'main' into dev
Roopan-Microsoft 8df387a
Merge branch 'dev' of https://github.com/Azure-Samples/chat-with-your…
Roopan-Microsoft e4dd5f2
fix: Container issue fix for multiple branch (#1539)
AjitPadhi-Microsoft 88d06dc
fix: Updated workflow (#1540)
AjitPadhi-Microsoft e5cf4dd
ci: Psl container fix for checkout code from head branch (#1541)
Roopan-Microsoft 02d0056
fix: multiple container tag issue fix (#1552)
AjitPadhi-Microsoft 90e1040
fix: Workflow issue fix on docker image (#1554)
AjitPadhi-Microsoft 8a5a1cb
fix: Downmerge dev (#1566)
Pavan-Microsoft 131c323
fix: CWYD Citation Links to Documents Break After Specific Timeframe …
Harmanpreet-Microsoft 93b84ed
fix: Commit changes bug (#1568)
UtkarshMishra-Microsoft 0875b92
fix: Post-Deployment Script for Managing Bicep Outputs in .env File …
Pavan-Microsoft 76190b3
fix: import error in env_helper.py (#1571)
Pavan-Microsoft c65bf01
fix: add conversation flow in environment variable for adminweb app (…
Priyanka-Microsoft 1f326d9
fix: Update Hosting Model Configuration in ARM/Bicep Template (#1570)
Prasanjeet-Microsoft fc688c2
fix: Update main.json for changes conversation flow changes in bicep …
Priyanka-Microsoft 56adb59
Merge branch 'main' into dev
Roopan-Microsoft afaabdc
build: Dependabotchanges merge to Dev branch (#1602)
Roopan-Microsoft 03f52b3
feat: Configurable System Prompts for Flexibility and Maintenance - C…
Pavan-Microsoft e92eba1
fix: Add Missing Logs (#1609)
Pavan-Microsoft b0860ee
commented the prompt flow in azure yaml file
Roopan-Microsoft 983b1f7
Merge branches 'dev' and 'dev' of https://github.com/Azure-Samples/ch…
Roopan-Microsoft dae4043
Add execute permissions parse_env.sh in postprovision hook
Pavan-Microsoft 08ad207
fix: Build issue with docker
Prajwal-Microsoft c49ddfd
Update build-docker.yml
Prajwal-Microsoft 5994f99
fix: Docker file failure issue
Prajwal-Microsoft dc7b87c
fix: Response getting ']' brackets, it's inconsistent (#1611)
AjitPadhi-Microsoft 2d1af79
fix: Fix Duplication of Flattened JSON Keys in .env File During Multi…
Pavan-Microsoft c553312
fix: Role Duplication Error in azd up Command for PostgreSQL (#1621)
Pavan-Microsoft 8213f0a
Merge branch 'main' into dev
Roopan-Microsoft 1f22786
feat: Remove AI search service for PostgreSQL configuration to optimi…
Prasanjeet-Microsoft 7c2391c
feat: Implement Consistent Versioning and Tagging for Accelerator Ima…
Pavan-Microsoft 8393614
Merge branch 'main' into dev
Roopan-Microsoft 3789d45
fix: Warning message displayed twice in chat history update (#1630)
Pavan-Microsoft 699dff8
test: unit test cases changes (#1353)
Kiran-Siluveru-Microsoft 461779f
Update default.json
Roopan-Microsoft e0be943
Update default_contract_assistant_prompt.txt
Roopan-Microsoft f28ddc6
Update open_ai_functions.py
Roopan-Microsoft 63ec389
Update build-docker.yml
Roopan-Microsoft e1d37f1
Update create_app.py
Roopan-Microsoft ec42447
Update test_conversation.py
Roopan-Microsoft 86dd916
Update test_iv_question_answer_tool.py
Roopan-Microsoft b1f2615
Update test_response_with_search_documents_tool.py
Roopan-Microsoft 3431205
Update test_conversation_flow.py
Roopan-Microsoft 7025c6e
ci: update docker registry logic for main, dev, and demo branches (#1…
Priyanka-Microsoft 8fca481
fix: Updates to change the Azure OpenAI model from `gpt-35-turbo` to …
Roopan-Microsoft 18e1dca
docs: updated numerical queries section in best practices readme (#1670)
Harsh-Microsoft c13b4e3
Merge branch 'main' into dev
Roopan-Microsoft 1ed3f43
fix: Admin configuration issue fix (#1676)
AjitPadhi-Microsoft f083af8
Merge branch 'main' into dev
Roopan-Microsoft 32ebef3
build: Merge Dependabot Changes into Dev Branch (#1679)
Prasanjeet-Microsoft 2803e48
Merge branch 'main' into dev
Roopan-Microsoft b5f35ac
Package lock updated
Roopan-Microsoft 0b01abb
Merge remote-tracking branch 'origin/main' into dev
Prajwal-Microsoft 21b57da
docs: Added migration readme file (#1668)
Priyanka-Microsoft 6f30b3f
refactor: enforcing authentication based on environment variable "ENF…
Harsh-Microsoft 9de26fd
build: Merge Dependabot Changes into Dev Branch (#1749)
Prasanjeet-Microsoft f7a2ce9
chore: Update Azure OpenAI Vision Model configuration to turbo-2024-0…
Harmanpreet-Microsoft dc763c2
feat: json document processor (#1661)
cristofima 9664143
fix: Resolve chunking issue during deployment when enabling advanced …
Priyanka-Microsoft e8382d3
Merge branch 'main' into dev
Roopan-Microsoft 4f09b12
fix: bug(#1653) added tooltip for chat history items. (#1785)
Bangarraju-Microsoft c7f3e8d
build: Add .dockerignore to optimize Docker build context (#1790)
Prasanjeet-Microsoft cea820d
fix: Updated reset configuration to use dialog instead of popover whi…
Vamshi-Microsoft 5d79d5d
refactor: centralize Azure resource naming with abbreviations.json an…
Harmanpreet-Microsoft a9d8511
fix: Changed the button label from "Reset Now" to "Reset" (#1796)
Vamshi-Microsoft 88caa69
Merge branch 'main' into dev
Roopan-Microsoft 5383461
main json updated
Roopan-Microsoft 86ebab4
fix: hardcoded abbreviations in bicep (#1805)
Harsh-Microsoft 03606b7
fix: Enhance Bicep Outputs for Environment Variable Configuration in …
Prasanjeet-Microsoft 7ae4b79
fix: Add Cosmos DB Account Key Output for Environment Configuration (…
Prasanjeet-Microsoft 69669bb
fix: error in 'Process and Ingest Web Pages' under 'Add URLs to the K…
Priyanka-Microsoft 464139c
test: Migrate test automation scripts, llm evaluator and pipeline for…
Harsh-Microsoft 27211e9
ci: configuration to include optional arguments for running tests dur…
Harsh-Microsoft 4fec5b7
Merge branch 'main' into dev
Prasanjeet-Microsoft b3fcae0
test: add citation link interaction support to e2e tests (#1821)
Harmanpreet-Microsoft f075a9b
chore: update default OpenAI model to gpt-4.1 across codebase and inf…
Harmanpreet-Microsoft edb03ab
feat: Down merge from dev & resolved conflicts (#1826)
Prajwal-Microsoft 7f4a427
feat: EXP Changes to reuse existing resource group and Log analytics …
Prajwal-Microsoft 74e1d6c
feat: rebuilt main.json
Prajwal-Microsoft eccd758
Merge branch 'main' into dev
Roopan-Microsoft File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,106 @@ | ||
name: Test Automation CWYD | ||
on: | ||
push: | ||
branches: | ||
- main | ||
- dev | ||
paths: | ||
- 'tests/e2e-test/**' | ||
schedule: | ||
- cron: '0 13 * * 3' # Runs at 1 PM UTC once a week on Wednesday | ||
workflow_dispatch: | ||
|
||
env: | ||
web_url: ${{ vars.CWYD_WEB_URL }} | ||
admin_url: ${{ vars.CWYD_ADMIN_URL }} | ||
accelerator_name: "Chat with your Data" | ||
|
||
jobs: | ||
test: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- name: Checkout repository | ||
uses: actions/checkout@v4 | ||
|
||
- name: Set up Python | ||
uses: actions/setup-python@v4 | ||
with: | ||
python-version: '3.13' | ||
|
||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install -r tests/e2e-test/requirements.txt | ||
|
||
- name: Ensure browsers are installed | ||
run: python -m playwright install --with-deps chromium | ||
|
||
- name: Run tests(1) | ||
id: test1 | ||
run: | | ||
xvfb-run pytest --headed --html=report/report.html --self-contained-html | ||
working-directory: tests/e2e-test | ||
continue-on-error: true | ||
|
||
- name: Sleep for 30 seconds | ||
if: ${{ steps.test1.outcome == 'failure' }} | ||
run: sleep 30s | ||
shell: bash | ||
|
||
- name: Run tests(2) | ||
id: test2 | ||
if: ${{ steps.test1.outcome == 'failure' }} | ||
run: | | ||
xvfb-run pytest --headed --html=report/report.html --self-contained-html | ||
working-directory: tests/e2e-test | ||
continue-on-error: true | ||
|
||
- name: Sleep for 60 seconds | ||
if: ${{ steps.test2.outcome == 'failure' }} | ||
run: sleep 60s | ||
shell: bash | ||
|
||
- name: Run tests(3) | ||
id: test3 | ||
if: ${{ steps.test2.outcome == 'failure' }} | ||
run: | | ||
xvfb-run pytest --headed --html=report/report.html --self-contained-html | ||
working-directory: tests/e2e-test | ||
|
||
- name: Upload test report | ||
id: upload_report | ||
uses: actions/upload-artifact@v4 | ||
if: ${{ !cancelled() }} | ||
with: | ||
name: cwyd-test-report | ||
path: tests/e2e-test/report/* | ||
|
||
- name: Send Notification | ||
if: always() | ||
run: | | ||
RUN_URL="https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}" | ||
REPORT_URL=${{ steps.upload_report.outputs.artifact-url }} | ||
IS_SUCCESS=${{ steps.test1.outcome == 'success' || steps.test2.outcome == 'success' || steps.test3.outcome == 'success' }} | ||
# Construct the email body | ||
if [ "$IS_SUCCESS" = "true" ]; then | ||
EMAIL_BODY=$(cat <<EOF | ||
{ | ||
"body": "<p>Dear Team,</p><p>We would like to inform you that the ${{ env.accelerator_name }} Test Automation process has completed successfully.</p><p><strong>Run URL:</strong> <a href=\"${RUN_URL}\">${RUN_URL}</a><br></p><p><strong>Test Report:</strong> <a href=\"${REPORT_URL}\">${REPORT_URL}</a></p><p>Best regards,<br>Your Automation Team</p>", | ||
"subject": "${{ env.accelerator_name }} Test Automation - Success" | ||
} | ||
EOF | ||
) | ||
else | ||
EMAIL_BODY=$(cat <<EOF | ||
{ | ||
"body": "<p>Dear Team,</p><p>We would like to inform you that the ${{ env.accelerator_name }} Test Automation process has encountered an issue and has failed to complete successfully.</p><p><strong>Run URL:</strong> <a href=\"${RUN_URL}\">${RUN_URL}</a><br> ${OUTPUT}</p><p><strong>Test Report:</strong> <a href=\"${REPORT_URL}\">${REPORT_URL}</a></p><p>Please investigate the matter at your earliest convenience.</p><p>Best regards,<br>Your Automation Team</p>", | ||
"subject": "${{ env.accelerator_name }} Test Automation - Failure" | ||
} | ||
EOF | ||
) | ||
fi | ||
|
||
# Send the notification | ||
curl -X POST "${{ secrets.EMAILNOTIFICATION_LOGICAPP_URL_TA }}" \ | ||
-H "Content-Type: application/json" \ | ||
-d "$EMAIL_BODY" || echo "Failed to send notification" | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,58 +1,33 @@ | ||
# Set Up Authentication in Azure App Service | ||
|
||
This document provides step-by-step instructions to configure Azure App Registrations for a front-end application. | ||
|
||
## Prerequisites | ||
|
||
- Access to **Microsoft Entra ID** | ||
- Necessary permissions to create and manage **App Registrations** | ||
|
||
## Step 1: Add Authentication in Azure App Service configuration | ||
|
||
1. Click on `Authentication` from left menu. | ||
|
||
 | ||
|
||
2. Click on `+ Add Provider` to see a list of identity providers. | ||
2. Click on `+ Add identity provider` to see a list of identity providers. | ||
|
||
 | ||
|
||
3. Click on `+ Add Provider` to see a list of identity providers. | ||
3. Click on `Identity Provider` dropdown to see a list of identity providers. | ||
|
||
 | ||
|
||
4. Select the first option `Microsoft Entra Id` from the drop-down list. If `Create new app registration` is disabled, go to [Step 1a](#step-1a-creating-a-new-app-registration). | ||
4. Select the first option `Microsoft Entra Id` from the drop-down list and select `client secret expiration` under App registration. | ||
> NOTE: If `Create new app registration` is disabled, then go to [Create new app registration](/docs/create_new_app_registration.md) and come back to this step to complete the app authentication. | ||
|
||
 | ||
|
||
5. Accept the default values and click on `Add` button to go back to the previous page with the identify provider added. | ||
|
||
 | ||
|
||
### Step 1a: Creating a new App Registration | ||
|
||
1. Click on `Home` and select `Microsoft Entra ID`. | ||
|
||
 | ||
|
||
2. Click on `App registrations`. | ||
|
||
 | ||
|
||
3. Click on `+ New registration`. | ||
|
||
 | ||
|
||
4. Provide the `Name`, select supported account types as `Accounts in this organizational directory only(Contoso only - Single tenant)`, select platform as `Web`, enter/select the `URL` and register. | ||
|
||
 | ||
|
||
5. After application is created sucessfully, then click on `Add a Redirect URL`. | ||
|
||
 | ||
|
||
6. Click on `+ Add a platform`. | ||
|
||
 | ||
|
||
7. Click on `Web`. | ||
|
||
 | ||
|
||
8. Enter the `web app URL` (Provide the app service name in place of XXXX) and Save. Then go back to [Step 1](#step-1-add-authentication-in-azure-app-service-configuration) and follow from _Point 4_ choose `Pick an existing app registration in this directory` from the Add an Identity Provider page and provide the newly registered App Name. | ||
E.g. https://appservicename.azurewebsites.net/.auth/login/aad/callback | ||
|
||
 | ||
6. You have successfully added app authentication, and now required to log in to access the application. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.