Skip to content
This repository was archived by the owner on Jun 22, 2026. It is now read-only.

Increase timeouts and enhance logging - #26

Merged
eldonm merged 1 commit into
mainfrom
0.1.11
Sep 23, 2025
Merged

Increase timeouts and enhance logging#26
eldonm merged 1 commit into
mainfrom
0.1.11

Conversation

@Tharickv75

Copy link
Copy Markdown
Contributor

Type of Change

What type of change does this PR introduce? Mark all that apply:

  • 🐛 Bug Fix
  • 🚀 Feature Request
  • 🔄 Refactor
  • 📖 Documentation Update
  • 🔧 Other (Please specify):

Summary

What does this PR address?

  • Increases the job timeout duration.
  • Adds additional logging for better traceability.

Description

Bug Fixes:

  • Bug: Timeout was too short, causing premature failures in some cases.
  • Root Cause: Insufficient timeout configuration.
  • Resolution: Increased timeout value and introduced more logs to aid in debugging and monitoring.

Changes Made

High-Level Summary:

  1. Increased job timeout duration.
  2. Added detailed logs for better debugging and tracking.

Checklist

Mark all that apply:

  • Code follows the project’s coding guidelines.
  • Tests have been added or updated for new functionality.
  • Documentation has been updated (if applicable).
  • Existing tests pass locally with these changes.
  • Any dependencies introduced are justified and documented.

Steps to Test

  1. Run a job that previously timed out quickly.
  2. Confirm the job now runs within the increased timeout window.
  3. Check logs to ensure the new logging statements appear as expected.

Additional Context

  • This change improves reliability by reducing premature timeouts.
  • Enhanced logging makes debugging easier in case of failures.

Questions or Concerns

  • Should the timeout value be configurable via environment variables for more flexibility?

@Tharickv75
Tharickv75 requested a review from eldonm September 20, 2025 16:27
@Tharickv75 Tharickv75 self-assigned this Sep 20, 2025
@eldonm
eldonm merged commit 32d167d into main Sep 23, 2025
2 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants