Skip to content

Conversation

@Jakeii
Copy link
Member

@Jakeii Jakeii commented Jan 15, 2026

What does this change?

Instead of updating both dictionaries in parallel, update them sequentially with a small delay.

Why?

There's been a few reports now of opt in and out links 404ing, I've traced the issue to an issue with edge dictionary updates reflecting in fastly's VCL.

The edge dictionaries are updated successfully according to the API and the GUI. But it isn't reflected in the VCL.

I suspect this could be a bug in Fastly's API with updating multiple dictionaries simultaneously. Fastly reccomend avoiding concurrent requests! See below

To confirm/mitigate, I suspect giving a few milliseconds between the API calls will help.

There's also a note on this docs page about avoiding concurrent updates https://www.fastly.com/documentation/reference/api/#rate-limiting

IMPORTANT: Overlapping write requests to the API may result in lost updates. We strongly recommend avoiding concurrent requests that modify configuration within the same service.

Screenshots

The Dictionary GUI is always updated with changes as expected
Screenshot 2026-01-15 at 11 55 55

But this is not always reflecting in the "Complete VCL"

table gu_ab_test_groups {
    "commercial-prebid-v10:control": "exp=1773100800,type=client",
    "commercial-prebid-v10:variant": "exp=1773100800,type=client",
    "commercial-user-module-liveramp:control": "exp=1769731200,type=client",
    "commercial-user-module-liveramp:variant": "exp=1769731200,type=client",
    "commercial-user-module-uid2:control": "exp=1768435200,type=client",
    "commercial-user-module-uid2:variant": "exp=1768435200,type=client",
}

@Jakeii Jakeii requested a review from a team as a code owner January 15, 2026 11:59
@github-actions
Copy link

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.

@github-actions
Copy link

github-actions bot commented Jan 15, 2026

@github-actions
Copy link

github-actions bot commented Jan 15, 2026

@Jakeii Jakeii added the fix Departmental tracking: fix label Jan 15, 2026
@Jakeii Jakeii added the run_chromatic Runs chromatic when label is applied label Jan 15, 2026
@github-actions github-actions bot removed the run_chromatic Runs chromatic when label is applied label Jan 15, 2026
@Jakeii Jakeii merged commit ca3d050 into main Jan 15, 2026
32 checks passed
@Jakeii Jakeii deleted the jlk/ab-testing/call-fastly-sequentially branch January 15, 2026 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Departmental tracking: fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants