fix(Upwind): fix Function App deployment and repackage v3.0.2#14158
Merged
sreedharande merged 5 commits intoAzure:masterfrom May 1, 2026
Merged
fix(Upwind): fix Function App deployment and repackage v3.0.2#14158sreedharande merged 5 commits intoAzure:masterfrom
sreedharande merged 5 commits intoAzure:masterfrom
Conversation
…fork URLs - Rebuild UpwindLogsLoader.zip with files at root (no deployment/ prefix) - Remove macOS-compiled .python_packages; rely on Oryx pip install at deploy time - Update packageUri default to raw GitHub fork URL - Update Deploy to Azure button link to raw fork ARM template URL
…nd-fix-zipdeploy branch
Remove explicit Microsoft.Web/serverfarms (Y1/Dynamic/linux) resource to avoid SubscriptionIsOverQuotaForSku failures in high-demand regions. Azure now assigns Consumption infrastructure implicitly. - Remove Microsoft.Web/serverfarms resource block - Remove serverFarmId from Microsoft.Web/sites dependsOn and properties - Add alwaysOn: true and reserved: true to sites properties - packageUri already points to correct branch (upwind-fix-zipdeploy)
Apply Robert's deployment fixes (flat zip layout, remove separate App Service Plan) and replace fork URLs with official aka.ms short links. Repackage solution as v3.0.2. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Contributor
|
Hi @yael-nisanov, don't modify existing zip 3.0.1 uncommit that change and keep only latest zip which created with version 3.0.2. Thanks! |
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
sreedharande
approved these changes
May 1, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
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.
Summary
UpwindLogsLoader.zipto flat layout (removeddeployment/prefix and pre-compiled macOS packages)Microsoft.Web/serverfarmsApp Service Plan from ARM template; addedalwaysOnandreservedproperties to Function Appaka.msshort links in place of fork URLs (4 locations across 3 files)createSolutionV3.ps1Test plan
aka.ms/sentinel-Upwind-functionappresolves to the updatedUpwindLogsLoader.zipaka.ms/sentinel-Upwind-azuredeployresolves to the ARM template🤖 Generated with Claude Code