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

Make parameters for splitting screenshot cleanup configurable #2821

Merged

Conversation

Martchus
Copy link
Contributor

On different production instances the runtime of the splitted cleanup tasks differs a lot so it makes sense to make the parameters for splitting configurable.


See https://progress.opensuse.org/issues/55922#note-18

@Martchus Martchus force-pushed the make-job-splitting-params-configurable branch from 144399d to 2683b61 Compare March 11, 2020 10:43
@codecov
Copy link

codecov bot commented Mar 11, 2020

Codecov Report

Merging #2821 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2821      +/-   ##
==========================================
+ Coverage   92.42%   92.48%   +0.05%     
==========================================
  Files         190      190              
  Lines       11889    11889              
==========================================
+ Hits        10989    10996       +7     
+ Misses        900      893       -7
Impacted Files Coverage Δ
lib/OpenQA/Setup.pm 95.45% <100%> (+0.03%) ⬆️
lib/OpenQA/Task/Job/Limit.pm 95.16% <100%> (-0.08%) ⬇️
lib/OpenQA/Scheduler/Model/Jobs.pm 91.49% <0%> (+2.83%) ⬆️

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 d6b2bf4...ae884a9. Read the comment docs.

On different production instances the runtime of the splitted
cleanup tasks differs a lot so it makes sense to make the parameters
for splitting configurable.

See https://progress.opensuse.org/issues/55922#note-18
@Martchus Martchus force-pushed the make-job-splitting-params-configurable branch from 2683b61 to ae884a9 Compare March 11, 2020 15:39
@okurz okurz merged commit 207d945 into os-autoinst:master Mar 11, 2020
@Martchus Martchus deleted the make-job-splitting-params-configurable branch March 13, 2020 09:13
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

2 participants