Skip to content

Conversation

@wvandeun
Copy link
Contributor

Change the default connection timeout of the Python SDK to 60s.

This is mainly to resolve some issue we see with implementations that implement large datasets on:

  • failing to start the Infrahub CI pipeline
  • requests in large batches of concurrent mutations timing out
  • request to retrieve generator group with large amount of members timing out

This should be revisited as soon as we have more through solutions for the above.

@wvandeun wvandeun self-assigned this Oct 11, 2024
@codecov
Copy link

codecov bot commented Oct 11, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
infrahub_sdk/config.py 0.00% 1 Missing ⚠️
Flag Coverage Δ
python-3.10 43.07% <0.00%> (ø)
python-3.11 43.07% <0.00%> (ø)
python-3.12 43.07% <0.00%> (ø)
python-3.9 43.01% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
infrahub_sdk/config.py 20.00% <0.00%> (ø)

... and 3 files with indirect coverage changes

@wvandeun wvandeun merged commit 8f7f780 into develop Oct 11, 2024
10 of 11 checks passed
@wvandeun wvandeun deleted the wvd-20241011-change-default-connection-timeout branch October 11, 2024 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/housekeeping Maintenance task

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants