-
Notifications
You must be signed in to change notification settings - Fork 1.3k
cleanup processed files in VR #7632
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
Conversation
Codecov Report
@@ Coverage Diff @@
## 4.18 #7632 +/- ##
============================================
- Coverage 13.05% 13.02% -0.04%
+ Complexity 9083 9032 -51
============================================
Files 2720 2720
Lines 257370 257067 -303
Branches 40124 40087 -37
============================================
- Hits 33612 33476 -136
+ Misses 219537 219387 -150
+ Partials 4221 4204 -17
... and 29 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
|
||
| import com.cloud.agent.api.routing.NetworkElementCommand; | ||
|
|
||
| public class CleanProcessedCacheCommand extends NetworkElementCommand { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nts: does this mean the answer should also inherit (from NetworkElementAnswer for instance)
4792a6c to
1563e28
Compare
1563e28 to
28db7f8
Compare
28db7f8 to
38089c7
Compare
|
@blueorangutan package |
|
@DaanHoogland a [SF] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 6690 |
|
@blueorangutan test keepEnv |
|
@DaanHoogland a [SF] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
[SF] Trillian test result (tid-7316)
|
|
vpc creation failes after these changes: needs work |
38089c7 to
38c6f03
Compare
|
@DaanHoogland |
let's push this forward then (marking 4.18.2 for now and we can discuss if it is still applicable) |
38c6f03 to
555e054
Compare
|
@blueorangutan LLpackage |
|
@DaanHoogland a [LL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
|
Packaging result [LL]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 6191 |
|
@blueorangutan LLtest matrix |
|
@DaanHoogland a [LL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
|
[LL] Trillian Build Failed (tid-6763) |
|
no interest |
Description
When a PR is long running, the directory /var/cache/cloud/processed filles up and might glock the filesystem, rendring the VR unusable.
This PR implements a mech for cleaning that directory on demand.
(to be triggered yet, by API/schedule/configurable timer???)
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?