Skip to content

src, e2e-tests: remove www server fallback logic - #1067

Merged
flakey5 merged 1 commit into
mainfrom
flakey5/1051
Oct 25, 2026
Merged

flakey5 merged 1 commit into
mainfrom
flakey5/1051

Conversation

@flakey5

@flakey5 flakey5 commented Oct 21, 2026

Copy link
Copy Markdown
Member

Closes #1051

Closes #1051

Signed-off-by: flakey5 <73616808+flakey5@users.noreply.github.com>
@flakey5
flakey5 requested a review from a team as a code owner October 21, 2026 23:21
@MattIPv4

Copy link
Copy Markdown
Member

Do we have metrics on how frequently we're falling back currently?

@flakey5

flakey5 commented Oct 22, 2026

Copy link
Copy Markdown
Member Author

Do we have metrics on how frequently we're falling back currently?

Nothing's explicitly tracking this though can get a rough idea based off of the logs on the www server I think cc @ryanaslett. The worker's user agent should be release-cloudflare-worker if it's possible to filter out by that

@MattIPv4

Copy link
Copy Markdown
Member

🤔 Do we not get sub-request analytics from the worker in the Cf dash? If not, www logs might be good enough! Just want to validate we're not falling through frequently before we rip this out.

@flakey5

flakey5 commented Oct 24, 2026 •

Copy link
Copy Markdown
Member Author

🤔 Do we not get sub-request analytics from the worker in the Cf dash? If not, www logs might be good enough! Just want to validate we're not falling through frequently before we rip this out.

Just looked and we do apparently, total of 5 requests in the past 24 hours (out of 1.1m total that the worker saw) https://dash.cloudflare.com/07be8d2fbc940503ca1be344714cb0d1/workers/services/view/dist-worker-prod/production/metrics

@MattIPv4

MattIPv4 commented Oct 24, 2026 •

Copy link
Copy Markdown
Member

Looking even over the last 30 days, it is a mere 490 requests (940 if we include 4xx/5xx). I think we're fine to remove this 👍

@flakey5
flakey5 merged commit ad443f7 into main Oct 25, 2026
8 checks passed
@flakey5
flakey5 deleted the flakey5/1051 branch October 25, 2026 02:25
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.

Remove www fallback

2 participants