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

Revert "Trigger tasks for limiting assets and results/logs hourly" due to never finishing cleanup #2613

Closed
wants to merge 1 commit into from

Conversation

okurz
Copy link
Member

@okurz okurz commented Dec 17, 2019

Reverts #2491 as there were problems reported.
It seems that the cleanup blocked the /admin/assets view for way longer
than usual, the cleanup job was triggered but never finished traversing
directories.

See https://progress.opensuse.org/issues/55241#note-11 for details.

@codecov
Copy link

codecov bot commented Dec 17, 2019

Codecov Report

Merging #2613 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2613      +/-   ##
==========================================
+ Coverage   91.69%   91.71%   +0.02%     
==========================================
  Files         182      182              
  Lines       11216    11219       +3     
==========================================
+ Hits        10284    10289       +5     
+ Misses        932      930       -2
Impacted Files Coverage Δ
lib/OpenQA/Schema/Result/ScheduledProducts.pm 97.32% <100%> (+0.02%) ⬆️
lib/OpenQA/WebAPI/Controller/API/V1/Job.pm 87.73% <100%> (+0.03%) ⬆️
lib/OpenQA/Worker/Job.pm 71.45% <0%> (+0.33%) ⬆️

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 f092e4b...9704e5e. Read the comment docs.

@okurz okurz closed this Dec 18, 2019
@Martchus Martchus deleted the revert-2491-trigger-cleanup-via-systemd-timer branch December 18, 2019 16:15
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.

None yet

1 participant