Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Updating deploy.py for dotnet deployments #1752 #1830

Conversation

AdamL-Microsoft
Copy link
Contributor

@AdamL-Microsoft AdamL-Microsoft commented Apr 22, 2022

Summary of the Pull Request

Adding flag and function for '--dotnet_deploy' to deploy.py to specify deploying the dotnet version of the services along with the python ones, with the default (e.g. no --dotnet_deploy flag) still being to only deploy the python versions.

Also something recently changed on the backend of Azure creating a possible race condition for our function app deployments when using bicep templates. In order for the deployment to work and pass all of the application settings to both the python and dotnet function apps at deployment time, we had to reconfigure the .bicep templates to order the app settings to be setup after both of the function apps have already been deployed.

PR Checklist

Info on Pull Request

Updated:
src/deployment/deploy.py
src/deployment/azuredeploy.bicep
src/deployment/bicep-templates/function.bicep

Added:
src/deployment/bicep-templates/function-settings.bicep

@AdamL-Microsoft AdamL-Microsoft requested a review from chkeita April 22, 2022 17:00
src/deployment/deploy.py Outdated Show resolved Hide resolved
@microsoft microsoft deleted a comment Apr 30, 2022
@AdamL-Microsoft AdamL-Microsoft self-assigned this May 2, 2022
@AdamL-Microsoft AdamL-Microsoft enabled auto-merge (squash) May 3, 2022 14:50
@AdamL-Microsoft AdamL-Microsoft merged commit fd55126 into microsoft:main May 3, 2022
@AdamL-Microsoft AdamL-Microsoft deleted the updating-deploy.py-for-dotnet-deployments branch May 3, 2022 15:23
@ghost ghost locked as resolved and limited conversation to collaborators Jun 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants