Skip to content

Roll variant headlines out to 25% of users - #16673

Merged
abeddow91 merged 3 commits into
mainfrom
fp/headline-test-rollout-stage-one
Sep 22, 2026
Merged

abeddow91 merged 3 commits into
mainfrom
fp/headline-test-rollout-stage-one

Conversation

@Fweddi

@Fweddi Fweddi commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Begin rolling out variant headlines rendered through the platform test fronts-and-curation-editorial-test: #16554.

The end goal is a 100% 50/50 test, we are reaching this threshold in stages to avoid overloading the cache.

@Fweddi
Fweddi requested a review from a team as a code owner September 3, 2026 12:56
@Fweddi Fweddi added the feature Departmental tracking: work on a new feature label Sep 3, 2026
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

Hello 👋! When you're ready to run Chromatic, please apply the run_chromatic label to this PR.

You will need to reapply the label each time you want to run Chromatic.

Click here to see the Chromatic project.

@Fweddi Fweddi linked an issue Sep 3, 2026 that may be closed by this pull request
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown

@Fweddi

Fweddi commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

We should report to Ophan that the test is renamed to fronts-and-curation-editorial-test when this is merged.

@Fweddi

Fweddi commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

We should report to Ophan that the test is renamed to fronts-and-curation-editorial-test when this is merged.

Possibly we could do this in a separate PR so we don't release the test at the same time as renaming it.

@Fweddi
Fweddi changed the base branch from main to fp/expand-name-of-platform-ab-test September 4, 2026 09:11
@Fweddi
Fweddi force-pushed the fp/headline-test-rollout-stage-one branch from 727ac9d to 658935b Compare September 4, 2026 09:11
@Fweddi

Fweddi commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

We should report to Ophan that the test is renamed to fronts-and-curation-editorial-test when this is merged.

Possibly we could do this in a separate PR so we don't release the test at the same time as renaming it.

I've now done that in #16684.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

🚀 Image pushed to AWS ECR

Image digest: sha256:43e98ac1836b64cb3d27e3594db6089393699859fec27d956780fa7b58c91710

🐛 Run the image locally

The following can be used to run the image locally:

# Refer to image using the immutable digest. Find alternatives below.
IMAGE_IDENTIFIER="@sha256:43e98ac1836b64cb3d27e3594db6089393699859fec27d956780fa7b58c91710"

# Refer to image using branch tag
# IMAGE_IDENTIFIER=":branch-fp-headline-test-rollout-stage-one"

# Refer to image using build tag
# IMAGE_IDENTIFIER=":build-30894"

# Refer to image via the GitHub commit SHA tag
# IMAGE_IDENTIFIER=":sha-35394833bd64cdcab4ceda9986680e12885a9c05"

# Set environment variables for the AWS CLI
AWS_PROFILE="<A_PROFILE_FROM_JANUS>"
AWS_DEFAULT_REGION="eu-west-1"

IMAGE_ACCOUNT_ID=$(aws ssm get-parameter --name /organisation/accounts/deployTools --query "Parameter.Value" --output text)
REGISTRY="${IMAGE_ACCOUNT_ID}.dkr.ecr.${AWS_DEFAULT_REGION}.amazonaws.com"
IMAGE="${REGISTRY}/guardian/dotcom-rendering${IMAGE_IDENTIFIER}"

# Login to AWS ECR https://docs.aws.amazon.com/AmazonECR/latest/userguide/registry_auth.html
aws ecr get-login-password | docker login --username AWS --password-stdin $REGISTRY

# Pull the image
docker pull $IMAGE

# Run the image. You'll likely need to set additional flags. See https://docs.docker.com/reference/cli/docker/container/run.
docker run $IMAGE

From guardian/actions-publish-image.

@Fweddi
Fweddi force-pushed the fp/expand-name-of-platform-ab-test branch from 2d7aedb to f1d57ea Compare September 7, 2026 13:36
Base automatically changed from fp/expand-name-of-platform-ab-test to main September 7, 2026 14:28
The end goal is a 50/50 test, we are reaching this threshold in stages to avoid overloading the cache
@Fweddi
Fweddi force-pushed the fp/headline-test-rollout-stage-one branch from 658935b to 3539483 Compare September 8, 2026 10:45
@abeddow91
abeddow91 added this pull request to the merge queue Sep 22, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Sep 22, 2026
@abeddow91
abeddow91 added this pull request to the merge queue Sep 22, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Sep 22, 2026
@abeddow91
abeddow91 added this pull request to the merge queue Sep 22, 2026
Merged via the queue into main with commit 6339015 Sep 22, 2026
33 checks passed
@abeddow91
abeddow91 deleted the fp/headline-test-rollout-stage-one branch September 22, 2026 11:35
@gu-prout

gu-prout Bot commented Sep 22, 2026

Copy link
Copy Markdown

Seen on PROD (created by @Fweddi and merged by @abeddow91 9 minutes and 11 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Do not merge ⚠️ feature Departmental tracking: work on a new feature Seen-on-PROD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Set Platform AB Test to 50/50

3 participants