[KVM] Reorder migration logs to prevent populating agent logs on migrations#12883
[KVM] Reorder migration logs to prevent populating agent logs on migrations#12883nvazquez wants to merge 1 commit intoapache:4.22from
Conversation
|
@blueorangutan package |
|
@nvazquez 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. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 4.22 #12883 +/- ##
============================================
+ Coverage 3.70% 17.60% +13.89%
- Complexity 0 15664 +15664
============================================
Files 448 5917 +5469
Lines 38041 531524 +493483
Branches 7038 64981 +57943
============================================
+ Hits 1409 93568 +92159
- Misses 36445 427402 +390957
- Partials 187 10554 +10367
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 17234 |
|
@blueorangutan test |
|
@nvazquez a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
| boolean isMigrateDowntimeSet = false; | ||
|
|
||
| final int migrateWait = libvirtComputingResource.getMigrateWait(); | ||
| logger.info("vm.migrate.wait value set to: {}for VM: {}", migrateWait, vmName); |
There was a problem hiding this comment.
| logger.info("vm.migrate.wait value set to: {}for VM: {}", migrateWait, vmName); | |
| logger.info("vm.migrate.wait value set to: {} secs for VM: {}", migrateWait, vmName); |
| logger.info("vm.migrate.wait value set to: {}for VM: {}", migrateWait, vmName); | ||
|
|
||
| final int migratePauseAfter = libvirtComputingResource.getMigratePauseAfter(); | ||
| logger.info("vm.migrate.pauseafter value set to: {} for VM: {}", migratePauseAfter, vmName); |
There was a problem hiding this comment.
| logger.info("vm.migrate.pauseafter value set to: {} for VM: {}", migratePauseAfter, vmName); | |
| logger.info("vm.migrate.pauseafter value set to: {} ms for VM: {}", migratePauseAfter, vmName); |
|
[SF] Trillian test result (tid-15726)
|
Description
This PR improves the KVM migration logging, preventing to populate the agent logs on migrations after the improvements added on #12332
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
Before the fix:
After the fix:
How did you try to break this feature and the system with this change?