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

Reduce backend calls (pdp, AA, ...) timeout #1777

Open
baszoetekouw opened this issue Dec 11, 2024 · 2 comments
Open

Reduce backend calls (pdp, AA, ...) timeout #1777

baszoetekouw opened this issue Dec 11, 2024 · 2 comments
Assignees
@pablothedude
Labels
easy-pickings

Comments

@baszoetekouw
Copy link
Member

The current PDP timeout is rather long (30s), so the loadbalancer gives up sooner than EB. Let's look in the logs what the median reponse time and determine a more reasionable timeout based on that.

Ideally, this timeout should be confgurable in the EB config.
@thijskh
Copy link
Member

thijskh commented Dec 11, 2024

Maybe make it more generic. Every backend API call from EB (to PDP, to AA) should not take more than $seconds where $seconds is most definitely < 10.

@baszoetekouw
Copy link
Member Author

Agreed.

@baszoetekouw baszoetekouw moved this from New to Backlog in PHP development Jan 22, 2025
@phavekes phavekes changed the title Reduce PDP timeout Reduce backend calls (pdp, AA, ...) timeout Jan 29, 2025
@pablothedude pablothedude moved this from Backlog to In Progress in PHP development Jan 29, 2025
@pablothedude pablothedude self-assigned this Jan 29, 2025
pablothedude added a commit that referenced this issue Jan 29, 2025
@pablothedude
Add configurable client timeout
e824cf7 
This is done for AA and PDP to be able to handle
timeouts before infrastructure limits are met.

#1777
@pablothedude pablothedude mentioned this issue Jan 29, 2025
Merged
@pablothedude pablothedude moved this from In Progress to Delivered in PHP development Feb 4, 2025
tvdijen pushed a commit to tvdijen/OpenConext-engineblock that referenced this issue Feb 12, 2025
@pablothedude @tvdijen
Add configurable client timeout
0446a60 
This is done for AA and PDP to be able to handle
timeouts before infrastructure limits are met.

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

@pablothedude pablothedude

Labels
easy-pickings
Projects
PHP development
Status: Delivered
Milestone
No milestone
Development

No branches or pull requests
3 participants
@baszoetekouw @thijskh @pablothedude