Skip to content
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.

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

split heavy and low maintenance process #1006

Merged
merged 1 commit into from
Apr 14, 2022

Conversation

ArtificialOwl
Copy link
Member

fix #276

Circles already had a kind of level about maintenance, heavy process were already run during night.

This PR will create a new BackgroundJob for heavy maintenance processes (level 4 and 5) with TIME_INSENSITIVE

while the low maintenance processes will keep using TIME_SENSITIVE

@ArtificialOwl ArtificialOwl force-pushed the enh/276/split-low-and-heavy-maintenance branch from e3fe72d to 9c3e247 Compare April 14, 2022 15:16
Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
@ArtificialOwl ArtificialOwl force-pushed the enh/276/split-low-and-heavy-maintenance branch from 9c3e247 to d75da94 Compare April 14, 2022 15:51
@codecov-commenter
Copy link

codecov-commenter commented Apr 14, 2022

Codecov Report

Merging #1006 (6a7e94b) into master (bc0484b) will increase coverage by 0.00%.
The diff coverage is 0.00%.

❗ Current head 6a7e94b differs from pull request most recent head d75da94. Consider uploading reports for the commit d75da94 to get more accurate results

@@           Coverage Diff            @@
##             master   #1006   +/-   ##
========================================
  Coverage      0.69%   0.69%           
+ Complexity     5744    5739    -5     
========================================
  Files           290     291    +1     
  Lines         19862   19859    -3     
========================================
  Hits            139     139           
+ Misses        19723   19720    -3     
Impacted Files Coverage Δ
lib/Cron/Maintenance.php 0.00% <0.00%> (ø)
lib/Cron/MaintenanceHeavy.php 0.00% <0.00%> (ø)
lib/Service/MaintenanceService.php 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update bc0484b...d75da94. Read the comment docs.

Copy link
Member

@CarlSchwan CarlSchwan left a comment

Choose a reason for hiding this comment

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

Looks good to me :)

@ArtificialOwl ArtificialOwl merged commit 4f2edc8 into master Apr 14, 2022
@delete-merged-branch delete-merged-branch bot deleted the enh/276/split-low-and-heavy-maintenance branch April 14, 2022 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants