Skip to content

fix(admin-ui): reduce Fastly KV sync page limit from 500 to 100#46

Open
mbradley wants to merge 1 commit intomainfrom
fix/admin-ui-sync-limit
Open

fix(admin-ui): reduce Fastly KV sync page limit from 500 to 100#46
mbradley wants to merge 1 commit intomainfrom
fix/admin-ui-sync-limit

Conversation

@mbradley
Copy link
Copy Markdown
Member

@mbradley mbradley commented May 7, 2026

Summary

  • reduces the admin UI "Sync to Fastly KV" button's page limit from 500 to 100
  • limit=500 causes Worker timeout due to concurrent Fastly API calls
  • limit=100 was validated during the full 131,951-username backfill on 2026-05-07

Test plan

  • tested paginated sync at limit=100 against production (dry run and writes) — completed 131,951 usernames with 0 failures
  • confirmed limit=150+ causes Worker timeout; limit=100 is the safe ceiling

limit=500 exceeds Worker timeout due to concurrent Fastly API calls.
limit=100 was validated during the full 131K backfill on 2026-05-07.
Updates both the callers and the function default.
@mbradley mbradley force-pushed the fix/admin-ui-sync-limit branch from b9380b0 to 4ae6316 Compare May 7, 2026 17:43
@mbradley mbradley marked this pull request as ready for review May 7, 2026 17:44
@mbradley mbradley requested a review from dcadenas May 7, 2026 17:44
@mbradley
Copy link
Copy Markdown
Member Author

mbradley commented May 7, 2026

not urgent -- the other improvements in #44 (hourly cron delta sync, retry queue) should mitigate the need to use this button very often.

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.

1 participant