Commits
Click on a commit to change the comparison rangefix(tasks): Enhance retry logic for GitHub app rate limits across tasks
- Updated retry handling in BaseCodecovTask, BundleAnalysisProcessorTask, and NotifyTask to utilize actual retry times from GitHub API when available.
- Implemented fallback to minimum delay of 60 seconds if no retry time is provided.
- Added unit tests to ensure correct behavior of retry logic under rate-limited conditions.
- Improved clarity and robustness of error handling related to GitHub app availability.18 days ago
by drazisil-codecov test(tasks): Add comprehensive unit tests for BundleAnalysisNotifyTask lock retry handling
- Implemented tests to verify behavior during token acquisition and notification lock scenarios, including handling of LockRetry and LockRetryLimitExceededError.
- Ensured graceful failure and correct retry behavior when max retries are exceeded.
- Added tests for stale fencing token handling and scenarios where no notifications are configured, enhancing overall test coverage for the notification task.18 days ago
by drazisil-codecov test(github_apps): Add unit tests for rate limit handling in GitHub app functionality
- Introduced tests for `_get_rate_limited_apps` and `_get_earliest_rate_limit_ttl` functions to validate behavior with various scenarios, including empty lists, all rate-limited apps, and mixed cases.
- Enhanced error handling tests to ensure Redis exceptions are managed correctly and do not disrupt processing.
- Verified that the `get_github_app_info_for_owner` function correctly sets retry parameters based on rate limits and handles exceptions gracefully.18 days ago
by drazisil-codecov fix(tasks): Refine retry handling in NotifyTask for lock acquisition failures
- Improved error handling by adding explicit retry exception handling in NotifyTask to ensure task retries are scheduled correctly when lock acquisition fails.
- Enhanced logging and flow control during lock retry scenarios, ensuring that notifications are processed appropriately even when locks are unobtainable.
- This change increases the robustness of the notification task's error management and retry logic.18 days ago
by drazisil-codecov Merge remote-tracking branch 'origin/main' into seer/fix/bundle-analysis-app-rate-limit-retry18 days ago
by drazisil-codecov test(tasks): Enhance unit tests for TestAnalyticsNotifierTask
- Added comprehensive tests for debounce retry logic, ensuring all required parameters are included during retries to prevent TypeErrors.
- Implemented tests for handling max retries exceeded during debounce, stale fencing tokens, and successful notification flows.
- Improved coverage for scenarios involving lock retry limits and graceful failure handling, enhancing the robustness of the notification task's error management.14 days ago
by drazisil-codecov fix(tasks): Enhance logging for notification lock errors in TestAnalyticsNotifierTask14 days ago
by drazisil-codecov fix(tasks): Update retry logic in NotificationDebouncer and TestAnalyticsNotifierTask14 days ago
by drazisil-codecov feat(github_apps): Add _partition_apps_by_rate_limit_status function to improve rate limit handling14 days ago
by drazisil-codecov