chore: fix flaky Windows AAS e2e with a self-contained test app#2376
Merged
ava-silver merged 2 commits intoJun 24, 2026
Conversation
|
ava-silver
marked this pull request as ready for review
June 24, 2026 13:29
ava-silver
marked this pull request as draft
June 24, 2026 14:54
ava-silver
force-pushed
the
ava.silver/chore/fix-windows-aas-e2e-by-shipping-web.config-so-iis-serves-the-node-app
branch
from
June 24, 2026 14:56
0f0bf84 to
d1343ab
Compare
ava-silver
marked this pull request as ready for review
June 24, 2026 15:13
avangelillo
approved these changes
Jun 24, 2026
Drarig29
approved these changes
Jun 24, 2026
ava-silver
deleted the
ava.silver/chore/fix-windows-aas-e2e-by-shipping-web.config-so-iis-serves-the-node-app
branch
June 24, 2026 15:52
Merged
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
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.

TL;DR
Windows e2e was failing because a config issue in the test app -- this just makes it so we use a dedicated test app for this repo and can control the whole config.
What and why?
The serverless e2e job has been red on every
masterrun since #2365, blocking CI for the whole repo. The failing test wasaas (Windows) › telemetry flows, which 403'd when driving traffic at the deployed app:On Windows, App Service serves Node behind IIS via iisnode, which needs a
web.configto hand requests to the app -- without one IIS denies directory browsing and returns 403. The test relied on the prebuiltnode-extension.zipfromserverless-init-self-monitoringplusSCM_DO_BUILD_DURING_DEPLOYMENTto have Kudu generate thatweb.config. Verified directly against Azure, the Kudu build does not generate aweb.configforaz webapp deployon Windows (and a failed build silently rolls the whole deploy back), so the app never served and the test could never pass.How?
Replace the external prebuilt package with a self-contained test app defined in the test:
app.js) -- nonode_modules, no build. Theaas instrumentsite extension injects dd-trace, which auto-instruments the built-inhttpmodule, so requests still produce spans.web.configrouting all requests toapp.js.Both files are deployed straight into
wwwrootwithaz webapp deploy --type static(--type staticdoesn't clean wwwroot, so the two files accumulate). Confirmed end-to-end against Azure: the app returns HTTP 200 and the full instrument/telemetry/uninstrument flow passes.Review checklist