Added support for storpool_qos service #8755
Conversation
boring-cyborg
Bot
added
component:integration-test
component:marvin
component:primary-storage
component:storage
Python
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8755 +/- ##
============================================
- Coverage 30.98% 22.97% -8.01%
+ Complexity 33526 23449 -10077
============================================
Files 5361 5262 -99
Lines 376250 357229 -19021
Branches 54933 51350 -3583
============================================
- Hits 116564 82062 -34502
- Misses 244259 263288 +19029
+ Partials 15427 11879 -3548
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
@blueorangutan package |
|
@sureshanaparti 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]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8897 |
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
slavkap
force-pushed
the
sp-support-tiers
branch
from
93618c7 to
662e636
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8755 +/- ##
============================================
- Coverage 15.57% 15.57% -0.01%
- Complexity 12035 12041 +6
============================================
Files 5501 5505 +4
Lines 482219 482543 +324
Branches 58938 60302 +1364
============================================
+ Hits 75119 75148 +29
- Misses 398798 399091 +293
- Partials 8302 8304 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
slavkap
force-pushed
the
sp-support-tiers
branch
from
97aa23a to
f83d77b
Compare
|
This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. |
|
@blueorangutan package |
|
@DaanHoogland 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]: ✔️ el7 ✔️ el8 ✔️ el9 ✖️ debian ✔️ suse15. SL-JID 9630 |
slavkap
force-pushed
the
sp-support-tiers
branch
from
996e9b0 to
e80728c
Compare
| @classmethod | ||
| def tearDownClass(cls): | ||
| cls.cleanUpCloudStack() | ||
|
|
||
| @classmethod | ||
| def cleanUpCloudStack(cls): | ||
| super(TestStorPoolTiers, cls).tearDownClass() |
There was a problem hiding this comment.
why the double redirection and not just call super() directly?
There was a problem hiding this comment.
thanks, I've removed the unnecessary method
|
not sure if this requires more testing than smoke tests (for other storage solutions cc @harikrishna-patnala @rp- @rg9975 ) ??? |
|
@blueorangutan package |
|
@DaanHoogland 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 10841 |
|
@blueorangutan test |
|
@DaanHoogland a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
I don't think this can in anyway break the Linstor driver |
|
[SF] Trillian test result (tid-11218)
|
7 participants
Description
This PR provides support to use the storpool_qos service (to change tiers) via the Disk offering's details.
StorPool provides the ‘storpool_qos’ service that tracks and configures the storage tier for all volumes based on a specifically provided
qctag specifying the storage tier for each volume.Types of changes
Feature/Enhancement Scale
How Has This Been Tested?
manual and smoke tests