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

Decrease select_with_timeout to fix 520 error #287

Merged
merged 1 commit into from
Dec 23, 2023
Merged

Decrease select_with_timeout to fix 520 error #287

merged 1 commit into from
Dec 23, 2023

Conversation

NathanFlurry
Copy link
Member

Changes

@NathanFlurry Graphite App
Copy link
Member Author

NathanFlurry commented Dec 21, 2023 •
edited
Loading

Current dependencies on/for this PR:

This stack of pull requests is managed by Graphite.

@NathanFlurry NathanFlurry mentioned this pull request Dec 21, 2023
Merged
@NathanFlurry NathanFlurry force-pushed the 12-21-Decrease_select_with_timeout_to_fix_520_error branch from 7bc88b7 to 6a18bb7 Compare December 21, 2023 03:12
AngelOnFira
AngelOnFira requested changes Dec 21, 2023
View reviewed changes
Copy link
Member

@AngelOnFira AngelOnFira left a comment

Choose a reason for hiding this comment

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

Some clarity requests

@NathanFlurry NathanFlurry force-pushed the 12-21-Decrease_select_with_timeout_to_fix_520_error branch from 6a18bb7 to f4d985a Compare December 21, 2023 08:42
@NathanFlurry NathanFlurry force-pushed the 12-21-Decrease_select_with_timeout_to_fix_520_error branch from f4d985a to 757fc34 Compare December 21, 2023 08:48
This was referenced Dec 21, 2023
Merged
Merged
MasterPtato
MasterPtato reviewed Dec 22, 2023
View reviewed changes
Copy link
Contributor

@MasterPtato MasterPtato left a comment

Choose a reason for hiding this comment

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

What exactly caused the 520 and why was decreasing the timeout to 40s a fix? Was it traefik's 60 second timeout? Shouldnt we be able to configure that? I don't understand from the docs

@NathanFlurry Graphite App
Copy link
Member Author

What exactly caused the 520 and why was decreasing the timeout to 40s a fix? Was it traefik's 60 second timeout? Shouldnt we be able to configure that? I don't understand from the docs

This didn't fix the 520, but it's still good to clean it up. I'm fairly certain it's something between CF & AWS.

@NathanFlurry NathanFlurry merged commit 888ddfd into main Dec 23, 2023
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MasterPtato MasterPtato MasterPtato left review comments

@AngelOnFira AngelOnFira AngelOnFira approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@NathanFlurry @AngelOnFira @MasterPtato