Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

Ensure deadline errors are legit #80

Merged
merged 1 commit into from
Apr 5, 2023
Merged

Conversation

aarshkshah1992
Copy link
Contributor

No description provided.

Copy link
Contributor

@willscott willscott left a comment

Choose a reason for hiding this comment

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

this is for a hypothesis that we're getting deadline exceeded errors when we shouldn't be.

Happy to run with this for a while, but we should remind ourselves to revert in a week rather than leaving the complexity in the case that we don't find something here

@aarshkshah1992
Copy link
Contributor Author

aarshkshah1992 commented Apr 5, 2023

@willscott Agreed. I think the whole metrics collection code is going to need a refactor(it's getting messy) but resisting the temptation to do so right now while we are focused on bigger fish.

@aarshkshah1992 aarshkshah1992 merged commit de0c7e8 into main Apr 5, 2023
@aarshkshah1992 aarshkshah1992 deleted the feat/time-timeouts branch April 5, 2023 14:08
lidel added a commit to ipfs-inactive/bifrost-gateway that referenced this pull request Apr 5, 2023
@willscott
Copy link
Contributor

do we see a count of errors from this fetchIncorrectDeadlineErrorTotalMetric?
if not, do we want to revert this to reduce the complexity?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants