Skip to content

Increase default timeout to 60 seconds for transforms. #7299

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

Open
wants to merge 3 commits into
base: stable
Choose a base branch
from
+3 −2
Open

Increase default timeout to 60 seconds for transforms. #7299

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3beeaff
Increase default timeout to 60 seconds for transforms.
FragmentedPacket Oct 1, 2025
8021b60
Increase default timeout for checks as well.
FragmentedPacket Oct 1, 2025
84dd02d
Add news fragment.
FragmentedPacket Oct 2, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion backend/infrahub/core/schema/definitions/core/transform.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@
Attr(name="name", kind="Text", unique=True),
Attr(name="label", kind="Text", optional=True),
Attr(name="description", kind="Text", optional=True),
Attr(name="timeout", kind="Number", default_value=10),
Attr(name="timeout", kind="Number", default_value=60),
Copy link
Contributor

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion | 🟠 Major

🧩 Analysis chain

LGTM! Verify test expectations and documentation.

The timeout increase from 10 to 60 seconds aligns with the PR objective to provide customers more time for artifact transforms.

Run the following script to verify that tests and documentation don't hardcode the old timeout value:

Additionally, consider:

  1. Whether existing transform instances with the old default (10 seconds) will automatically pick up the new default (60 seconds) or if a migration is needed
  2. If release notes or upgrade documentation should mention this change for existing deployments

🏁 Script executed:

#!/bin/bash
# Description: Check for hardcoded references to the old 10-second timeout that may need updating.

# Search for test files that might assert or expect a 10-second timeout for transforms
echo "=== Checking test files for transform timeout references ==="
rg -n -C3 --type=py -g '*test*.py' 'timeout.*10|10.*timeout' backend/

# Search for documentation files mentioning the 10-second timeout
echo -e "\n=== Checking documentation for timeout references ==="
fd -e md -e rst -e txt . | xargs rg -n -C2 'transform.*timeout|timeout.*10'

# Search for other schema or config files that might reference transform timeout
echo -e "\n=== Checking for other transform timeout references ==="
rg -n -C3 --type=py 'transform.*timeout|timeout.*transform' backend/ | grep -v 'definitions/core/transform.py'

Length of output: 18775

Update tests to reflect new default timeout

Unit tests in backend/tests/unit/git/test_git_transform.py (lines 29, 54, 80, 102) and the default in backend/tests/unit/git/conftest.py:1248 still reference a 10-second timeout; update these to 60 seconds.

🤖 Prompt for AI Agents 
In backend/infrahub/core/schema/definitions/core/transform.py around line 32 the
default timeout was changed to 60s; update the unit tests that still expect 10s:
change the timeout assertions/expected values at
backend/tests/unit/git/test_git_transform.py lines 29, 54, 80, and 102 from 10
to 60, and update the default timeout value in
backend/tests/unit/git/conftest.py at line 1248 from 10 to 60 (and any related
fixture/comment values) so tests match the new default.

],
relationships=[
Rel(
Expand Down
Loading