[Usage] Fix wrong usage_type

[SadiJr]

Description

IO write events have been saved in the database using the usage_type of the VM_DISK_BYTES_WRITE type, which causes an inconsistency in the database. This behavior has been fixed by this PR. Plus, the old data saved in the database has been normalized.

Types of changes

• Breaking change (fix or feature that would cause existing functionality to change)
• New feature (non-breaking change which adds functionality)
• Bug fix (non-breaking change which fixes an issue)
• Enhancement (improves an existing feature and functionality)
• Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Bug Severity

• BLOCKER
• Critical
• Major
• Minor
• Trivial

Screenshots (if appropriate):

How Has This Been Tested?

It was tested in a local lab:

1. I waited for the job of the Usage plugin to run;
2. Before, ACS saved the IO write event with the wrong usage_type in the database;
3. Now, ACS saves these events with the right usage_type, and the old events are normalized.

[codecov]

Codecov Report

Merging #7229 (42e8800) into main (eef63d9) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #7229      +/-   ##
============================================
- Coverage     12.67%   12.67%   -0.01%     
  Complexity     8641     8641              
============================================
  Files          2716     2716              
  Lines        256112   256111       -1     
  Branches      39926    39926              
============================================
- Hits          32461    32460       -1     
  Misses       219522   219522              
  Partials       4129     4129              

Impacted Files	Coverage Δ
...ervisor/kvm/resource/LibvirtComputingResource.java	18.30% <ø> (-0.04%)	⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[github-actions]

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

[github-actions]

This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[stephankruggg]

CLGTM, not manually tested

[DaanHoogland]

clgtm

[weizhouapache]

code lgtm

[BryanMLima]

LGTM, tested this in a local lab. I upgraded from a version with this error and the old events were fixed; furthermore, the new usage events were with the correct usage type in the DB.

[BryanMLima]

As the milestone changed to version 4.18.1.0 this needs to be changed to the correct schema file.

[weizhouapache]

@BryanMLima
I think this can be merged into 4.18.0.0