Skip to content

Conversation

symstu-tempesta
Copy link
Contributor

No description provided.

@symstu-tempesta symstu-tempesta marked this pull request as draft July 10, 2025 01:05
@symstu-tempesta symstu-tempesta marked this pull request as ready for review July 13, 2025 00:38
@krizhanovsky krizhanovsky self-requested a review July 14, 2025 13:58
Copy link
Contributor

@RomanBelozerov RomanBelozerov left a comment

Choose a reason for hiding this comment

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

I think you should clean up the commit history, or you'll make a squish merge?

Copy link
Contributor

@krizhanovsky krizhanovsky left a comment

Choose a reason for hiding this comment

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

I still have questions, let's discuss them on a call

### Blog or Online Shop
These types of applications typically don’t have a large number of concurrent users and often operate within a traffic range of 0 to 50 active users.
However, it's important to account for the fact that static files (like CSS, JS, and images) are also requested.
On initial page load, a single user might generate up to 200 or more HTTP requests. All farther requests should receive some
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
On initial page load, a single user might generate up to 200 or more HTTP requests. All farther requests should receive some
On initial page load, a single user might generate up to 200 or more HTTP requests. All further requests should receive some

requests_threshold=average_stats.requests,
time_threshold=average_stats.time,
errors_threshold=average_stats.errors,
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why don't we use standard deviation? It could be hard to configure the multipliers, basically, one need to do some data analysis to reveal what the multipliers are.

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.

4 participants