You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Sub-minute job timeouts are reported as zero minutes #9527
When a job timeout is configured below 60 seconds, the job runner reports that the job timed out after 0 minutes.
The timeout handlers in packages/syft-job/src/syft_job/job_runner.py format the duration using integer division:
print(f" Job {job_name} timed out after {timeout//60} minutes")
For example, a valid timeout of 30 seconds is displayed as 0 minutes.
How to Reproduce
Set the job timeout to 30 seconds:
export SYFT_DEFAULT_JOB_TIMEOUT_SECONDS=30
Run a job that takes longer than 30 seconds.
Wait for the job to time out.
Observe the following message:
Job <job_name> timed out after 0 minutes
Expected Behavior
Sub-minute timeouts should be reported in seconds, for example:
Job <job_name> timed out after 30 seconds
All job-execution timeout paths should use consistent, human-readable duration formatting.
Screenshots
Not applicable. The problem appears in the job runner’s console output.
System Information
OS: Platform-independent
OS Version: Not applicable
Language Version: Python
Package Manager Version: Not applicable
Browser: Not applicable
Browser Version: Not applicable
Additional Context
The same timeout // 60 formatting is used in multiple timeout-handling paths in job_runner.py. Tests should cover at least one timeout below 60 seconds and one timeout of 60 seconds or more.
I've opened #9560 for this — the fix routes all three timeout // 60 message sites through a shared duration formatter, so a 30-second timeout now reads timed out after 30 seconds while the minutes phrasing is kept for a minute and up. Happy to adjust if you'd rather see a different unit boundary or wording.
Activity
choksi2212 commentedon Oct 12, 2026
I've opened #9560 for this — the fix routes all three
timeout // 60message sites through a shared duration formatter, so a 30-second timeout now readstimed out after 30 secondswhile the minutes phrasing is kept for a minute and up. Happy to adjust if you'd rather see a different unit boundary or wording.pratikverse commentedon Oct 16, 2026
I would love to contribute @MateehUllah, can you assign it to me