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.

Sign up for GitHub

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

Jump to bottom

Frozen APY chart #5

Closed
micahalcorn opened this issue Nov 30, 2022 · 6 comments
Closed

Frozen APY chart #5

micahalcorn opened this issue Nov 30, 2022 · 6 comments
Assignees
@sparrowDom
Labels
bug Something isn't working P1 Large amount of significant user impact, a meaningful feature is broken unplanned Added to the sprint due to urgency
Milestone
Sprint 40

Comments

@micahalcorn
Copy link
Member

On seemingly random occasions, the graph doesn't update when the moving average selection is changed. Last time I experienced this, I noticed some CORS-related issues coming from the analytics API.

Screen Shot 2022-11-30 at 2 29 47 PM
@micahalcorn micahalcorn added bug Something isn't working P1 Large amount of significant user impact, a meaningful feature is broken unplanned Added to the sprint due to urgency labels Nov 30, 2022
@micahalcorn micahalcorn added this to the Sprint 40 milestone Nov 30, 2022
@franckc
Copy link
Collaborator

franckc commented Dec 1, 2022 •
edited

This is intermittent. So I don't understand why CORS would block the requests some times but not others?

But regardless, looking at Google App Engine I see a clear pattern of 500 errors on the analytics stack:
Screen Shot 2022-11-30 at 7 11 04 PM

Also note that the requests are cached by Cloudflare for 120sec (that's the value of the Max-Age response header set by the server). If there wouldn't be caching, the error rate might be even higher.

@franckc
Copy link
Collaborator

franckc commented Dec 1, 2022

@shahthepro or @sparrowDom Could you please take a look?

@franckc
Copy link
Collaborator

franckc commented Dec 1, 2022

This is most likely related to OriginProtocol/ousd-analytics#239

@sparrowDom
Copy link
Member

In the last 6 hours all of those errors were 503 so related to this issue:
OriginProtocol/ousd-analytics#254

@sparrowDom
Copy link
Member

This PR addresses it: OriginProtocol/ousd-analytics#256

@sparrowDom
Copy link
Member

Fixed via OriginProtocol/ousd-analytics#256 & OriginProtocol/ousd-analytics#257

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

@sparrowDom sparrowDom

Labels
bug Something isn't working P1 Large amount of significant user impact, a meaningful feature is broken unplanned Added to the sprint due to urgency
Projects
Origin DeFi (OUSD + OETH)
Status: Done
Milestone
Sprint 40
Development

No branches or pull requests
3 participants
@micahalcorn @sparrowDom @franckc