Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reinstate AA test ElasticSearch 6.7 #3135

Merged
merged 1 commit into from
Aug 17, 2023
Merged

Conversation

Rosa-Fox
Copy link
Contributor

@Rosa-Fox Rosa-Fox commented Aug 16, 2023

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 CDN config PR

⚠️ This repo is Continuously Deployed: make sure you follow the guidance ⚠️

Follow these steps if you are doing a Rails upgrade.

@govuk-ci govuk-ci temporarily deployed to finder-frontend-pr-3135 August 16, 2023 14:40 Inactive
@Rosa-Fox Rosa-Fox force-pushed the aa-test-es-six-seven-user-scope branch from 2b1a364 to dde5947 Compare August 16, 2023 14:44
@govuk-ci govuk-ci temporarily deployed to finder-frontend-pr-3135 August 16, 2023 14:44 Inactive
Copy link
Contributor

@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 🥇

app/controllers/finders_controller.rb Outdated Show resolved Hide resolved
We recently ran an AA test for the Search team (#3053).

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.
@Rosa-Fox Rosa-Fox force-pushed the aa-test-es-six-seven-user-scope branch from dde5947 to e2c29a9 Compare August 16, 2023 16:14
@govuk-ci govuk-ci temporarily deployed to finder-frontend-pr-3135 August 16, 2023 16:14 Inactive
@Rosa-Fox Rosa-Fox merged commit 3ff4c8b into main Aug 17, 2023
7 checks passed
@Rosa-Fox Rosa-Fox deleted the aa-test-es-six-seven-user-scope branch August 17, 2023 08:51
Rosa-Fox added a commit that referenced this pull request Sep 4, 2023
There is currently an AA test running with a 50/50 split [1].
We now want to run a throttled version with a 0.5% split.
The C variant has therefore been added to the corresponding
govuk-cdn-config PR to soak up the traffic not in A or B.
This PR allows the C variant. Z will remain as the control variant.
[1]#3135
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.

3 participants