Skip to content
This repository has been archived by the owner on Sep 5, 2023. It is now read-only.

Reinstate ElasticSearch 6.7 AA test #467

Merged
merged 1 commit into from
Aug 17, 2023
Merged

Conversation

Rosa-Fox
Copy link
Contributor

Trello

We recently ran an AA test for the Search team.

We are going to run it again, hence reinstating it in this PR!

The purpose of the A/A test is to test the % of allocation of traffic/sample population based on the current version of elastic (6.7) against itself. A (ES 6.7) vs A (ES6.7). The A/A is solely to identify how the approach allocates traffic. This will provide benchmark data to aid analysis in future AA tests.

This time around, the code will stay the same. We will use the same custom dimension and the variant split will still be 50/50. The only difference is that the custom dimension has been scoped as ‘user' as opposed to ‘session’ like it was last time. Scoping the custom dimension has already been done by a performance analyst.

We plan to deploy this on Thursday 17 August for 1 week.

Corresponding rendering app PR

⚠️ The changes need to be deployed manually to all relevant environments. Follow the guidance on how to deploy Fastly.

[We recently ran an AA test for the Search team](#463).

We are going to run it again, hence reinstating it in this PR!

The purpose of the A/A test is to test the % of allocation of traffic/sample population based on the current version of elastic (6.7) against itself. A (ES 6.7) vs A (ES6.7). The A/A is solely to identify how the approach allocates traffic. This will provide benchmark data to aid analysis in future AA tests.

This time around, the code will stay the same. We will use the same custom dimension and the variant split will still be 50/50. The only difference is that the custom dimension has been scoped as ‘user' as opposed to ‘session’ like it was last time. Scoping the custom dimension has already been done by a performance analyst.

We plan to deploy this on Thursday 17 August for 1 week.
Copy link

@gclssvglx gclssvglx left a comment

Choose a reason for hiding this comment

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

Looks great 👍

@Rosa-Fox Rosa-Fox merged commit 19cb40a into main Aug 17, 2023
2 checks passed
@Rosa-Fox Rosa-Fox deleted the reinstate-es-six-seven-aa-test branch August 17, 2023 09:11
samsimpson1 added a commit to alphagov/govuk-fastly that referenced this pull request Aug 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants