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

Update AA test config for ElasticSearch 6.7 #3149

Closed
wants to merge 1 commit into from

Conversation

Rosa-Fox
Copy link
Contributor

@Rosa-Fox Rosa-Fox commented 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 that is not in A or B. This PR allows the C variant. Z will remain as the control variant.

[1]#3135

Manual testing:

GOVUK-ABTest-EsSixPointSeven header not set

no-header

Header value set to A

a-header

Header value set to B

b-header

Header value set to C

c-header

Header value set to a value that isn't ABC

non-abc-header

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

Follow these steps if you are doing a Rails upgrade.

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
@govuk-ci govuk-ci temporarily deployed to finder-frontend-pr-3149 September 4, 2023 16:37 Inactive
Rosa-Fox added a commit to alphagov/govuk-fastly that referenced this pull request Sep 7, 2023
The AA test is currently deployed at a 50/50 variant split. This PR updates the AA test to throttle the split at 0.5%.
The rendering app code has been updated in this PR to handle the C variant[1].

The C variant has been added to make the difference between the CDN traffic allocation and anything that falls into the Z variant (handled by the rendering app) more explicit.

This PR was originally opened on the govuk-cdn-config repo which has been archived. Therefore it has been re-added here and I will close the other PR.
[1]alphagov/finder-frontend#3149
@Rosa-Fox
Copy link
Contributor Author

Closing this as the throttled test is no longer required.

@Rosa-Fox Rosa-Fox closed this Sep 21, 2023
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.

2 participants