Skip to content

Enable analytics logs during load testing#1180

Merged
monfresh merged 1 commit intomasterfrom
mb-allow-analytics-when-load-testing
Mar 7, 2017
Merged

Enable analytics logs during load testing#1180
monfresh merged 1 commit intomasterfrom
mb-allow-analytics-when-load-testing

Conversation

@monfresh
Copy link
Copy Markdown
Contributor

@monfresh monfresh commented Mar 6, 2017

Why: Ahoy excludes bots by default, and it detects Locust as a bot.
Luckily, Ahoy makes it easy to customize what gets excluded by
overriding the exclude? method. In this case, we're telling Ahoy to
allow everything during load testing. Otherwise, use the default
settings.

This allows us to better simulate server load.

**Why**: Ahoy excludes bots by default, and it detects Locust as a bot.
Luckily, Ahoy makes it easy to customize what gets excluded by
overriding the `exclude?` method. In this case, we're telling Ahoy to
allow everything during load testing. Otherwise, use the default
settings.

This allows us to better simulate server load.
Copy link
Copy Markdown
Contributor

@jessieay jessieay left a comment

Choose a reason for hiding this comment

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

🐱

@monfresh monfresh merged commit 449bf15 into master Mar 7, 2017
@monfresh monfresh deleted the mb-allow-analytics-when-load-testing branch March 7, 2017 15:31
amoose pushed a commit that referenced this pull request Mar 7, 2017
**Why**: Ahoy excludes bots by default, and it detects Locust as a bot.
Luckily, Ahoy makes it easy to customize what gets excluded by
overriding the `exclude?` method. In this case, we're telling Ahoy to
allow everything during load testing. Otherwise, use the default
settings.

This allows us to better simulate server load.
amoose pushed a commit that referenced this pull request Mar 8, 2017
**Why**: Ahoy excludes bots by default, and it detects Locust as a bot.
Luckily, Ahoy makes it easy to customize what gets excluded by
overriding the `exclude?` method. In this case, we're telling Ahoy to
allow everything during load testing. Otherwise, use the default
settings.

This allows us to better simulate server load.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants