-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Fix volume allocation logs #10475
base: main
Are you sure you want to change the base?
Fix volume allocation logs #10475
Conversation
Applies sugested changes
...random/src/main/java/org/apache/cloudstack/storage/allocator/RandomStoragePoolAllocator.java
Outdated
Show resolved
Hide resolved
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10475 +/- ##
============================================
+ Coverage 4.00% 16.15% +12.15%
- Complexity 0 13274 +13274
============================================
Files 396 5666 +5270
Lines 32530 498078 +465548
Branches 5766 60267 +54501
============================================
+ Hits 1302 80480 +79178
- Misses 31078 408584 +377506
- Partials 150 9014 +8864
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@blueorangutan package |
@julien-vaz a [SL] 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]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 12667 |
Description
The logs in storage allocation algorithms were displaying incorrect information about the allocation pool. Therefore, the logs were rewritten.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
How Has This Been Tested?
When trying to migrate a volume, the expected log messages were displayed correctly.