Skip to content

Conversation

@peterbppb
Copy link
Contributor

Added validation to schedule to prevent jobs running every second as this breaks dkron and the nodes need to be restarted to clean up the hanging jobs.

image

@peterbppb peterbppb requested a review from a team as a code owner March 11, 2025 09:53
@peterbppb peterbppb requested review from ppb007 and tcppb March 11, 2025 09:53
@codecov
Copy link

codecov bot commented Mar 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.33%. Comparing base (21c9383) to head (2099ee3).
Report is 35 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #52      +/-   ##
===========================================
- Coverage    88.49%   81.33%   -7.16%     
===========================================
  Files           14       17       +3     
  Lines          513      627     +114     
===========================================
+ Hits           454      510      +56     
- Misses          59      117      +58     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Bogdan Peter added 7 commits March 11, 2025 13:36
Signed-off-by: Bogdan Peter <bogdan.peter@paddypowerbetfair.com>
Signed-off-by: Bogdan Peter <bogdan.peter@paddypowerbetfair.com>
Signed-off-by: Bogdan Peter <bogdan.peter@paddypowerbetfair.com>
Signed-off-by: Bogdan Peter <bogdan.peter@paddypowerbetfair.com>
Signed-off-by: Bogdan Peter <bogdan.peter@paddypowerbetfair.com>
Signed-off-by: Bogdan Peter <bogdan.peter@paddypowerbetfair.com>
@peterbppb peterbppb closed this Mar 11, 2025
@peterbppb peterbppb deleted the update_admin branch March 11, 2025 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants