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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix deployment race condition #1194

Merged
merged 1 commit into from
May 23, 2023
Merged

Fix deployment race condition #1194

merged 1 commit into from
May 23, 2023

Conversation

glahaye
Copy link
Contributor

@glahaye glahaye commented May 23, 2023

Motivation and Context

The deployment templates often fail because of race conditions leading to the app service to not be in a good state when we want to apply settings, executable or extensions to it.

Description

Adjust "dependsOn" statements in bicep files

Contribution Checklist

@glahaye glahaye added the PR: ready for review All feedback addressed, ready for reviews label May 23, 2023
Copy link
Member

@Vivihung Vivihung left a comment

Choose a reason for hiding this comment

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

馃憤

@adrianwyatt adrianwyatt merged commit 292bdc6 into microsoft:main May 23, 2023
19 checks passed
shawncal pushed a commit to shawncal/semantic-kernel that referenced this pull request Jul 6, 2023
### Motivation and Context
The deployment templates often fail because of race conditions leading
to the app service to not be in a good state when we want to apply
settings, executable or extensions to it.

### Description
Adjust "dependsOn" statements in bicep files
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
### Motivation and Context
The deployment templates often fail because of race conditions leading
to the app service to not be in a good state when we want to apply
settings, executable or extensions to it.

### Description
Adjust "dependsOn" statements in bicep files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready for review All feedback addressed, ready for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants