Skip to content

Conversation

@vbudhram
Copy link
Contributor

Because

  • We were getting timeout errors when calling nimbus experiments

This pull request

  • Updates timeout to 400ms (it should take on avg 50ms)
  • Only captures sentry event if its an exception
  • Emits a statsd metric instead to track timeouts

Issue that this pull request solves

Closes: https://mozilla-hub.atlassian.net/browse/FXA-12711

Checklist

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

@vbudhram vbudhram requested a review from a team as a code owner December 10, 2025 21:18
@vbudhram vbudhram changed the title fix(nimbus): Update nimbus timeout to 500ms fix(nimbus): Update nimbus timeout to 400ms Dec 10, 2025
Copy link
Contributor

@vpomerleau vpomerleau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also consider caching the successful responses to reduce the number of fetches?

@vbudhram vbudhram merged commit 827afdd into main Dec 11, 2025
20 checks passed
@vbudhram vbudhram deleted the fxa-12711 branch December 11, 2025 16:30
@vbudhram
Copy link
Contributor Author

Should we also consider caching the successful responses to reduce the number of fetches?

We could, but I think its ok for now. These failures are very small % of requests.

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