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

force solution to build for amd64 architecture as arm64 does not supp… #506

Merged
merged 5 commits into from
Mar 24, 2024

Conversation

RenSilvaAU
Copy link
Contributor

…ort azure functions

Purpose

  • It makes it possible for this repo to run on Macs (Apple Silicon) with Rosetta.
  • It forces the devcontainer to build under amd64 architecture - this can run successfully in Macs with Apple Silicon and Rosetta.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

  • Get the code
Once you see the message **Reopen in Container**, do click that button

What to Check

Verify that the following are valid

  • Container builds successfully in both PCs and Macs with Apple Silicon

Other Information

closes #486

Copy link

github-actions bot commented Mar 21, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
code
   app.py1234166%29, 110, 149–150, 154, 157–158, 172–174, 176, 184, 188–192, 194, 201, 205–206, 212, 218–219, 221–222, 225, 239–240, 257, 259–260, 264, 273–277, 294, 296, 344
code/backend
   Admin.py16160%1–5, 7, 9, 11, 16, 23, 30, 33–35, 37, 39
code/backend/batch
   AddURLEmbeddings.py28280%1–4, 6–7, 9, 11, 14–16, 18–23, 25, 27–31, 34–36, 40, 45
   BatchPushResults.py24240%1–5, 7–9, 11, 13, 16–18, 26, 29–30, 34–35, 37, 39, 41–42, 49–50
   BatchStartProcessing.py21210%1–7, 9–11, 14–16, 18, 20, 22, 27, 29, 35–36, 38
   GetConversationResponse.py29290%1–6, 8, 10–11, 14–16, 18, 20–24, 29–32, 38, 40, 47, 55, 57–59
   function_app.py10100%1–5, 7, 10–13
code/backend/batch/utilities/common
   Answer.py24483%26, 39, 44–45
   SourceDocument.py58493%31, 35, 39, 123
code/backend/batch/utilities/document_chunking
   DocumentChunkingBase.py10280%10, 16
   FixedSizeOverlap.py190100% 
   Layout.py190100% 
   Page.py170100% 
   Paragraph.py990%1–4, 7–9, 12, 15
   Strategies.py25484%24–25, 27, 29
   __init__.py70100% 
code/backend/batch/utilities/document_loading
   DocumentLoadingBase.py9188%13
   Layout.py12120%1–4, 7–9, 11–13, 16, 25
   Read.py12120%1–4, 7–9, 11–13, 16, 25
   Strategies.py20860%13, 15, 17, 19, 24–25, 27, 29
   Web.py19194%23
   WordDocument.py25250%1–6, 9–12, 21–24, 26–27, 29–30, 32–37, 45
   __init__.py110100% 
code/backend/batch/utilities/helpers
   AzureBlobStorageHelper.py655023%21, 23–25, 28, 33–35, 38, 41, 43, 46, 49–51, 56, 64–65, 67, 71, 75, 79, 85, 100, 103, 115, 118, 122, 125, 127, 135–139, 162, 166–170, 172, 175–176, 180, 184, 186, 188, 192, 203
   AzureFormRecognizerHelper.py81810%1–6, 9–11, 13, 16–17, 25, 27, 35, 43–45, 52–55, 60–68, 70, 73–75, 77–78, 81, 84–86, 88–90, 93, 97–98, 105–109, 111–114, 117–131, 133, 135–137, 139–140, 143, 145–147
   AzureSearchHelper.py17947%14, 17–19, 66, 80–82, 142
   ConfigHelper.py613444%17–20, 28–29, 32, 37, 40, 43, 46, 52–56, 61, 66–67, 73–74, 76–78, 81–84, 86, 90–91, 97–98, 216
   DocumentChunkingHelper.py12191%21
   DocumentLoadingHelper.py12191%14
   DocumentProcessorHelper.py311938%15–17, 22, 25–35, 38–41
   EnvHelper.py961287%85–87, 160, 165–166, 169–171, 178–180
   LLMHelper.py332039%11–13, 15–16, 22, 28–29, 34, 37–38, 47, 58–59, 71, 83–84, 91, 101, 109
   OrchestratorHelper.py12466%20–22, 25
code/backend/batch/utilities/loggers
   ConversationLogger.py362822%8, 11–12, 15–24, 27–30, 33–42, 46
   TokenLogger.py15846%8–11, 16, 19, 23–24
code/backend/batch/utilities/orchestrator
   LangChainAgent.py684533%20–24, 56, 59–60, 65–66, 71, 74–76, 81, 88, 91–95, 97–99, 103–106, 110–115, 117–120, 126–127, 132–133, 136, 139, 144
   OpenAIFunctions.py55550%1–2, 4–11, 14–18, 53, 56, 59–60, 65–66, 71, 74, 76, 82–85, 87, 90, 97–99, 103–104, 106, 112–115, 119–120, 123, 126–127, 130, 135–136, 139–140, 145–146, 149, 152, 157
   OrchestratorBase.py301840%12–18, 21–23, 29, 38–40, 47–49, 59
   Strategies.py12741%10–11, 13–15, 17, 19
   __init__.py11190%10
code/backend/batch/utilities/parser
   OutputParserTool.py380100% 
   ParserBase.py9277%9, 19
   __init__.py7271%7, 11
code/backend/batch/utilities/tools
   AnswerProcessingBase.py8275%8, 12
   AnsweringToolBase.py9277%9, 15
   ContentSafetyChecker.py412734%13, 15–16, 21, 27–29, 32–33, 39–40, 46–56, 58, 62–64, 66
   PostPromptTool.py221340%11, 14–15, 17–18, 22, 29, 36–37, 45, 51–52, 60
   QuestionAnswerTool.py321940%17–19, 22–23, 28, 31, 36, 39, 43–44, 46–47, 50–52, 61, 63, 70
   TextProcessingTool.py16943%9, 12–15, 19, 21, 28, 35
code/backend/pages
   01_Ingest_Data.py1031030%1–11, 17–20, 22–23, 25, 28, 34, 41, 44, 48–49, 51, 56, 59–60, 63–72, 76–78, 81–89, 92–94, 96, 99, 101–104, 109–114, 117, 120–121, 124, 129, 142–145, 148–149, 153, 157, 163, 177–180, 183, 188–189, 191–192, 194–196, 200, 203–204, 210–213, 220–221, 226, 228–229
   02_Explore_Data.py30300%1–9, 11, 13, 15, 18, 24, 31, 34, 42, 45–47, 49–52, 54, 58, 62–63, 66–67
   03_Delete_Data.py53530%1–7, 9, 11, 13, 16, 22, 29, 32, 40, 43–44, 47–51, 53, 55–59, 61–62, 64, 67–69, 71–74, 76–78, 80, 82, 85–87, 89–90, 92–94, 96–97
   04_Configuration.py71710%1–7, 9, 11, 13, 16, 23, 30, 32, 37–42, 45–46, 49–50, 52–55, 57–58, 68–72, 75–76, 80–82, 85–86, 89–90, 93–97, 104–106, 108, 110, 118, 125–126, 133, 135, 147–148, 165–166, 170, 172–173, 189, 211–212, 214–215
TOTAL163397740% 

Tests Skipped Failures Errors Time
40 0 💤 0 ❌ 0 🔥 9.792s ⏱️

Copy link
Collaborator

@ross-p-smith ross-p-smith left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Many thanks for fixing this on the Mac

@ross-p-smith ross-p-smith added this pull request to the merge queue Mar 24, 2024
Merged via the queue into Azure-Samples:main with commit fa206f8 Mar 24, 2024
2 checks passed
superhindupur pushed a commit that referenced this pull request Mar 26, 2024
#506)

* force solution to build for amd64 architecture as arm64 does not support azure functions

* installing git in container

* Update .devcontainer/Dockerfile

---------

Co-authored-by: Ross Smith <[email protected]>
github-merge-queue bot pushed a commit that referenced this pull request Mar 26, 2024
* Add basic ui tests for admin and user websites.

* Move ui tests to tests folder and create uitest target on makefile.

* Update UI tests README to use latest env variable names.

* Delete package-lock.json file added in error.

* Update wrong command in ui tests README.md.

* Remove unnecessary fixture file.

* Update all UI dependencies (#502)

* Update minor/patch depedencies

* @vitejs/plugin-react major update

* Update vite

* Bump follow-redirects from 1.15.5 to 1.15.6 in /extensions/teams (#442)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump nodemon from 2.0.22 to 3.1.0 in /extensions/teams (#493)

Bumps [nodemon](https://github.com/remy/nodemon) from 2.0.22 to 3.1.0.
- [Release notes](https://github.com/remy/nodemon/releases)
- [Commits](remy/nodemon@v2.0.22...v3.1.0)

---
updated-dependencies:
- dependency-name: nodemon
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update tasks.json for func start (#511)

* Bump ts-node from 10.9.1 to 10.9.2 in /extensions/teams (#496)

Bumps [ts-node](https://github.com/TypeStrong/ts-node) from 10.9.1 to 10.9.2.
- [Release notes](https://github.com/TypeStrong/ts-node/releases)
- [Changelog](https://github.com/TypeStrong/ts-node/blob/main/development-docs/release-template.md)
- [Commits](TypeStrong/ts-node@v10.9.1...v10.9.2)

---
updated-dependencies:
- dependency-name: ts-node
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump botbuilder from 4.21.4 to 4.22.1 in /extensions/teams (#491)

Bumps [botbuilder](https://github.com/Microsoft/botbuilder-js) from 4.21.4 to 4.22.1.
- [Release notes](https://github.com/Microsoft/botbuilder-js/releases)
- [Commits](microsoft/botbuilder-js@4.21.4...4.22.1)

---
updated-dependencies:
- dependency-name: botbuilder
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump typescript from 4.9.5 to 5.4.3 in /extensions/teams (#509)

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.9.5 to 5.4.3.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v4.9.5...v5.4.3)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump restify from 10.0.0 to 11.1.0 in /extensions/teams (#489)

Bumps [restify](https://github.com/restify/node-restify) from 10.0.0 to 11.1.0.
- [Release notes](https://github.com/restify/node-restify/releases)
- [Changelog](https://github.com/restify/node-restify/blob/master/CHANGELOG.md)
- [Commits](restify/node-restify@v10.0.0...v11.1.0)

---
updated-dependencies:
- dependency-name: restify
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add functional test for /conversation/custom endpoint (#504)

* Add functional test for /conversation/custom endpoint

- Allow the disabling of app insights and loading
config from blob storage to make testing easier
- Switch to `Thread` from `Process` for running the app
  - This is to stop pytest calling the session scoped fixure multiple times
  - It also fixes an issue where we were not seeing app logs
- Setup the mock server as https due to the azure libraries enforcing it
  - This introduced additional fixtures and the `trustme` dependency
- Commit the encoding file for tiktoken
  - This is to stop it making a http call over the internet to retrive it

Required by #420

* Parameterize tests with pytest.mark.parametrize

* Improve test logging

* Extract boolean env var behaviour

* Add note about tiktoken issue

* Allow tests to be run from anywhere

* Update @types/node to v18 (#513)

* force solution to build for amd64 architecture as arm64 does not supp… (#506)

* force solution to build for amd64 architecture as arm64 does not support azure functions

* installing git in container

* Update .devcontainer/Dockerfile

---------

Co-authored-by: Ross Smith <[email protected]>

* Add test for failing downstream (#520)

Required by #420

Co-authored-by: Ross Smith <[email protected]>

* Add destroy to CI workflow (#521)

* Bump @types/restify from 8.5.11 to 8.5.12 in /extensions/teams (#533)

Bumps [@types/restify](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/restify) from 8.5.11 to 8.5.12.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/restify)

---
updated-dependencies:
- dependency-name: "@types/restify"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Verify the downstream calls being made by /conversations/custom (#519)

* Verify the downstream calls being made by /conversations/custom

- To ensure we are making the correct calls, with the correct api keys,
  index information and request bodies
- Implemented using a generic request matcher for now
  - This should be added to the mock server library by csernazs/pytest-httpserver#292

Required by #420

* Improve error message

* Add fixture comments for the https mock server

* Fix test comments

---------

Co-authored-by: Ross Smith <[email protected]>

* Bump @fluentui/react-icons from 2.0.232 to 2.0.233 in /code/frontend (#527)

Bumps [@fluentui/react-icons](https://github.com/microsoft/fluentui-system-icons) from 2.0.232 to 2.0.233.
- [Commits](https://github.com/microsoft/fluentui-system-icons/commits)

---
updated-dependencies:
- dependency-name: "@fluentui/react-icons"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump vite from 5.2.2 to 5.2.6 in /code/frontend (#528)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.2.2 to 5.2.6.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.2.6/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump postcss from 8.4.37 to 8.4.38 in /code/frontend (#531)

Bumps [postcss](https://github.com/postcss/postcss) from 8.4.37 to 8.4.38.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.4.37...8.4.38)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @types/react from 18.2.67 to 18.2.70 in /code/frontend (#532)

Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.2.67 to 18.2.70.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react)

---
updated-dependencies:
- dependency-name: "@types/react"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @fluentui/react from 8.116.0 to 8.117.0 in /code/frontend (#530)

Bumps [@fluentui/react](https://github.com/microsoft/fluentui) from 8.116.0 to 8.117.0.
- [Release notes](https://github.com/microsoft/fluentui/releases)
- [Changelog](https://github.com/microsoft/fluentui/blob/master/azure-pipelines.release-fluentui.yml)
- [Commits](https://github.com/microsoft/fluentui/compare/@fluentui/react_v8.116.0...@fluentui/react_v8.117.0)

---
updated-dependencies:
- dependency-name: "@fluentui/react"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump typescript from 5.4.2 to 5.4.3 in /code/frontend (#529)

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.4.2 to 5.4.3.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v5.4.2...v5.4.3)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* AZD can use containers instead of code for deployment (#505)

* Add adminweb docker image

* Build off my branch

* Create Requirements.

* Seperate process

* Dont traverse paths

* Use poetry in docker build

* dont install the root app

* Add frontend image

* Revert docker changes

* Install the root again

* Use requirements rather than toml

* Use Requirements for images again

* Rename docker images

* Frontend use app directory

* Add python path

* Copy the whole code folder

* Frontend runs locally

* Frontend and Admin working

* Use containers as default hosting model

* Dont build from my branch anymore

* Only build docker images - after test

* Change trigger workflow

* Only run when tests succeed

* Trigger builds only if tests pass

* Only changed App Service names

* Reverting

* Bump pytest-cov from 4.1.0 to 5.0.0 (#526)

Bumps [pytest-cov](https://github.com/pytest-dev/pytest-cov) from 4.1.0 to 5.0.0.
- [Changelog](https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst)
- [Commits](pytest-dev/pytest-cov@v4.1.0...v5.0.0)

---
updated-dependencies:
- dependency-name: pytest-cov
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump pre-commit from 3.6.2 to 3.7.0 (#524)

Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 3.6.2 to 3.7.0.
- [Release notes](https://github.com/pre-commit/pre-commit/releases)
- [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md)
- [Commits](pre-commit/pre-commit@v3.6.2...v3.7.0)

---
updated-dependencies:
- dependency-name: pre-commit
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Set AZURE_APP_SERVICE_HOSTING_MODEL for CI (#536)

* Create env before deploying (#537)

* Add basic ui tests for admin and user websites.

* Move ui tests to tests folder and create uitest target on makefile.

* Commit updated main.json.

* Move apt-get install for libraries required by cypress to dockerfile.

* Remove newline at the end of postCreate.sh.

* Conditional website address and install npm

* Move UI tests out of code folder and revert changes to azure.yaml

* Update ui tests location.

* Update dependabot.yml with new location of ui tests package.json.

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Chinedum Echeta <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Adam Dougal <[email protected]>
Co-authored-by: Ren Silva <[email protected]>
Co-authored-by: Ross Smith <[email protected]>
Co-authored-by: ross-p-smith <[email protected]>
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this pull request Apr 30, 2024
Azure-Samples#506)

* force solution to build for amd64 architecture as arm64 does not support azure functions

* installing git in container

* Update .devcontainer/Dockerfile

---------

Co-authored-by: Ross Smith <[email protected]>
eduardogch pushed a commit to devopsdale/chat-with-your-data-solution-accelerator that referenced this pull request Apr 30, 2024
* Add basic ui tests for admin and user websites.

* Move ui tests to tests folder and create uitest target on makefile.

* Update UI tests README to use latest env variable names.

* Delete package-lock.json file added in error.

* Update wrong command in ui tests README.md.

* Remove unnecessary fixture file.

* Update all UI dependencies (Azure-Samples#502)

* Update minor/patch depedencies

* @vitejs/plugin-react major update

* Update vite

* Bump follow-redirects from 1.15.5 to 1.15.6 in /extensions/teams (Azure-Samples#442)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump nodemon from 2.0.22 to 3.1.0 in /extensions/teams (Azure-Samples#493)

Bumps [nodemon](https://github.com/remy/nodemon) from 2.0.22 to 3.1.0.
- [Release notes](https://github.com/remy/nodemon/releases)
- [Commits](remy/nodemon@v2.0.22...v3.1.0)

---
updated-dependencies:
- dependency-name: nodemon
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Update tasks.json for func start (Azure-Samples#511)

* Bump ts-node from 10.9.1 to 10.9.2 in /extensions/teams (Azure-Samples#496)

Bumps [ts-node](https://github.com/TypeStrong/ts-node) from 10.9.1 to 10.9.2.
- [Release notes](https://github.com/TypeStrong/ts-node/releases)
- [Changelog](https://github.com/TypeStrong/ts-node/blob/main/development-docs/release-template.md)
- [Commits](TypeStrong/ts-node@v10.9.1...v10.9.2)

---
updated-dependencies:
- dependency-name: ts-node
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump botbuilder from 4.21.4 to 4.22.1 in /extensions/teams (Azure-Samples#491)

Bumps [botbuilder](https://github.com/Microsoft/botbuilder-js) from 4.21.4 to 4.22.1.
- [Release notes](https://github.com/Microsoft/botbuilder-js/releases)
- [Commits](microsoft/botbuilder-js@4.21.4...4.22.1)

---
updated-dependencies:
- dependency-name: botbuilder
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump typescript from 4.9.5 to 5.4.3 in /extensions/teams (Azure-Samples#509)

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.9.5 to 5.4.3.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v4.9.5...v5.4.3)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump restify from 10.0.0 to 11.1.0 in /extensions/teams (Azure-Samples#489)

Bumps [restify](https://github.com/restify/node-restify) from 10.0.0 to 11.1.0.
- [Release notes](https://github.com/restify/node-restify/releases)
- [Changelog](https://github.com/restify/node-restify/blob/master/CHANGELOG.md)
- [Commits](restify/node-restify@v10.0.0...v11.1.0)

---
updated-dependencies:
- dependency-name: restify
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Add functional test for /conversation/custom endpoint (Azure-Samples#504)

* Add functional test for /conversation/custom endpoint

- Allow the disabling of app insights and loading
config from blob storage to make testing easier
- Switch to `Thread` from `Process` for running the app
  - This is to stop pytest calling the session scoped fixure multiple times
  - It also fixes an issue where we were not seeing app logs
- Setup the mock server as https due to the azure libraries enforcing it
  - This introduced additional fixtures and the `trustme` dependency
- Commit the encoding file for tiktoken
  - This is to stop it making a http call over the internet to retrive it

Required by Azure-Samples#420

* Parameterize tests with pytest.mark.parametrize

* Improve test logging

* Extract boolean env var behaviour

* Add note about tiktoken issue

* Allow tests to be run from anywhere

* Update @types/node to v18 (Azure-Samples#513)

* force solution to build for amd64 architecture as arm64 does not supp… (Azure-Samples#506)

* force solution to build for amd64 architecture as arm64 does not support azure functions

* installing git in container

* Update .devcontainer/Dockerfile

---------

Co-authored-by: Ross Smith <[email protected]>

* Add test for failing downstream (Azure-Samples#520)

Required by Azure-Samples#420

Co-authored-by: Ross Smith <[email protected]>

* Add destroy to CI workflow (Azure-Samples#521)

* Bump @types/restify from 8.5.11 to 8.5.12 in /extensions/teams (Azure-Samples#533)

Bumps [@types/restify](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/restify) from 8.5.11 to 8.5.12.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/restify)

---
updated-dependencies:
- dependency-name: "@types/restify"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Verify the downstream calls being made by /conversations/custom (Azure-Samples#519)

* Verify the downstream calls being made by /conversations/custom

- To ensure we are making the correct calls, with the correct api keys,
  index information and request bodies
- Implemented using a generic request matcher for now
  - This should be added to the mock server library by csernazs/pytest-httpserver#292

Required by Azure-Samples#420

* Improve error message

* Add fixture comments for the https mock server

* Fix test comments

---------

Co-authored-by: Ross Smith <[email protected]>

* Bump @fluentui/react-icons from 2.0.232 to 2.0.233 in /code/frontend (Azure-Samples#527)

Bumps [@fluentui/react-icons](https://github.com/microsoft/fluentui-system-icons) from 2.0.232 to 2.0.233.
- [Commits](https://github.com/microsoft/fluentui-system-icons/commits)

---
updated-dependencies:
- dependency-name: "@fluentui/react-icons"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump vite from 5.2.2 to 5.2.6 in /code/frontend (Azure-Samples#528)

Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 5.2.2 to 5.2.6.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/main/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.2.6/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump postcss from 8.4.37 to 8.4.38 in /code/frontend (Azure-Samples#531)

Bumps [postcss](https://github.com/postcss/postcss) from 8.4.37 to 8.4.38.
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.4.37...8.4.38)

---
updated-dependencies:
- dependency-name: postcss
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @types/react from 18.2.67 to 18.2.70 in /code/frontend (Azure-Samples#532)

Bumps [@types/react](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/react) from 18.2.67 to 18.2.70.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/react)

---
updated-dependencies:
- dependency-name: "@types/react"
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump @fluentui/react from 8.116.0 to 8.117.0 in /code/frontend (Azure-Samples#530)

Bumps [@fluentui/react](https://github.com/microsoft/fluentui) from 8.116.0 to 8.117.0.
- [Release notes](https://github.com/microsoft/fluentui/releases)
- [Changelog](https://github.com/microsoft/fluentui/blob/master/azure-pipelines.release-fluentui.yml)
- [Commits](https://github.com/microsoft/fluentui/compare/@fluentui/react_v8.116.0...@fluentui/react_v8.117.0)

---
updated-dependencies:
- dependency-name: "@fluentui/react"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump typescript from 5.4.2 to 5.4.3 in /code/frontend (Azure-Samples#529)

Bumps [typescript](https://github.com/Microsoft/TypeScript) from 5.4.2 to 5.4.3.
- [Release notes](https://github.com/Microsoft/TypeScript/releases)
- [Changelog](https://github.com/microsoft/TypeScript/blob/main/azure-pipelines.release.yml)
- [Commits](microsoft/TypeScript@v5.4.2...v5.4.3)

---
updated-dependencies:
- dependency-name: typescript
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* AZD can use containers instead of code for deployment (Azure-Samples#505)

* Add adminweb docker image

* Build off my branch

* Create Requirements.

* Seperate process

* Dont traverse paths

* Use poetry in docker build

* dont install the root app

* Add frontend image

* Revert docker changes

* Install the root again

* Use requirements rather than toml

* Use Requirements for images again

* Rename docker images

* Frontend use app directory

* Add python path

* Copy the whole code folder

* Frontend runs locally

* Frontend and Admin working

* Use containers as default hosting model

* Dont build from my branch anymore

* Only build docker images - after test

* Change trigger workflow

* Only run when tests succeed

* Trigger builds only if tests pass

* Only changed App Service names

* Reverting

* Bump pytest-cov from 4.1.0 to 5.0.0 (Azure-Samples#526)

Bumps [pytest-cov](https://github.com/pytest-dev/pytest-cov) from 4.1.0 to 5.0.0.
- [Changelog](https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst)
- [Commits](pytest-dev/pytest-cov@v4.1.0...v5.0.0)

---
updated-dependencies:
- dependency-name: pytest-cov
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Bump pre-commit from 3.6.2 to 3.7.0 (Azure-Samples#524)

Bumps [pre-commit](https://github.com/pre-commit/pre-commit) from 3.6.2 to 3.7.0.
- [Release notes](https://github.com/pre-commit/pre-commit/releases)
- [Changelog](https://github.com/pre-commit/pre-commit/blob/main/CHANGELOG.md)
- [Commits](pre-commit/pre-commit@v3.6.2...v3.7.0)

---
updated-dependencies:
- dependency-name: pre-commit
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Set AZURE_APP_SERVICE_HOSTING_MODEL for CI (Azure-Samples#536)

* Create env before deploying (Azure-Samples#537)

* Add basic ui tests for admin and user websites.

* Move ui tests to tests folder and create uitest target on makefile.

* Commit updated main.json.

* Move apt-get install for libraries required by cypress to dockerfile.

* Remove newline at the end of postCreate.sh.

* Conditional website address and install npm

* Move UI tests out of code folder and revert changes to azure.yaml

* Update ui tests location.

* Update dependabot.yml with new location of ui tests package.json.

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Chinedum Echeta <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Adam Dougal <[email protected]>
Co-authored-by: Ren Silva <[email protected]>
Co-authored-by: Ross Smith <[email protected]>
Co-authored-by: ross-p-smith <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants