Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
shwstppr merged PR #10957: URL: https://github.com/apache/cloudstack/pull/10957 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3808061771 [SF] Trillian test result (tid-15292) Environment: kvm-ol8 (x2), zone: Advanced Networking with Mgmt server ol8 Total time taken: 50297 seconds Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr10957-t15292-kvm-ol8.zip Smoke tests completed. 141 look OK, 0 have errors, 0 did not run Only failed and skipped tests results shown below: Test | Result | Time (s) | Test File --- | --- | --- | --- -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
winterhazel commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3805643278 > @winterhazel , let’s wait for regression test results and/before merge? Let's merge after the tests -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
DaanHoogland commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3805208475 @winterhazel , let’s wait for regression test results and/before merge? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
RosiKyu commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3803833548 @blueorangutan test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3803841165 @RosiKyu a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
borisstoyanov commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3803833051 @blueorangutan test -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
weizhouapache commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3801967615 > @RosiKyu I think the issue you are facing is not related to #12515. Instead, it may be happening because the value of `js.interpretation.enabled` is encrypted in the database, but you are setting it to a decrypted value. This way, an exception is thrown when the Management Server attempts to decrypt the already decrypted value. > > Could you check if the following resolves your issue? > > 1. Obtain the key used by the Management Server to encrypt the configurations. > > ```sh > cat /etc/cloudstack/management/key > ``` > > 2. Use the EncryptionCLI package from cloud-utils to encrypt the value `true` using this key. > > ```sh > java -classpath /usr/share/cloudstack-common/lib/cloudstack-utils.jar com.cloud.utils.crypt.EncryptionCLI -p -i true > ``` > > 3. Update the setting to the encrypted value. > > ```sh > mysql -u root -p cloud -e "UPDATE configuration SET value='' WHERE name='js.interpretation.enabled';" > ``` > > 4. Restart the Management Server > or, just copy the value of configurtion "init" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
winterhazel commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3801743998 @RosiKyu I think the issue you are facing is not related to #12515. Instead, it may be happening because the value of `js.interpretation.enabled` is encrypted in the database, but you are setting it to a decrypted value. This way, an exception is thrown when the Management Server attempts to decrypt the already decrypted value. Could you check if the following resolves your issue? 1. Obtain the key used by the Management Server to encrypt the configurations. ```sh cat /etc/cloudstack/management/key ``` 2. Use the EncryptionCLI package from cloud-utils to encrypt the value `true` using this key. ```sh java -classpath /usr/share/cloudstack-common/lib/cloudstack-utils.jar com.cloud.utils.crypt.EncryptionCLI -p -i true ``` 3. Update the setting to the encrypted value. ```sh mysql -u root -p cloud -e "UPDATE configuration SET value='' WHERE name='js.interpretation.enabled';" ``` 4. Restart the Management Server -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
RosiKyu commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3801676752 > Hey @RosiKyu, thanks for your tests! I would just like to point out that the JS interpreter is not working as intended at the current moment (see #12515). Hence, selectors that choose a secondary storage based on information about the volume/account/domain/existing secondary storages will not work as expected. > > You can, however, test this PR by using a simple rule that directs all volumes to a specific secondary storage, for instance: > > ```shell > (admin) 🐱 > create secondarystorageselector name="direct volumes to secondary storage X" description="directs volumes to secondary storage X" zoneid=13b319e9-108c-4925-96aa-ae556d9a11b2 heuristicrule="''" type=VOLUME > ``` > > With this selector, all volumes will pass through secondary storage X during cold migration. Thanks @winterhazel for the clarification! I was hitting exactly that issue - when enabling `js.interpretation.enabled=true` on the current 4.20 branch (required for createSecondaryStorageSelector API), the management server hangs during startup at the module loading phase. Have logged it here, before seeing your comment: https://github.com/apache/cloudstack/issues/12523 Good to know PR #12515 addresses this. I'll proceed with testing PR #10957 using the simple rule workaround you suggested: -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
RosiKyu commented on PR #10957:
URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3801658494
Testing of PR #10957 is **BLOCKED** due to a pre-existing bug in the 4.20
branch introduced by commit `03a4b9f` ("server,utils: improve js interpretation
functionality"). This bug is **not caused by PR #10957**.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
winterhazel commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3801324454 Hey @RosiKyu, thanks for your tests! I would just like to point out that the JS interpreter is not working as intended at the current moment (see #12515). Hence, selectors that choose a secondary storage based on information about the volume/account/domain/existing secondary storages will not work as expected. You can, however, test this PR by using a simple rule that directs all volumes to a specific secondary storage, for instance: ```bash (admin) 🐱 > create secondarystorageselector name="direct volumes to secondary storage X" description="directs volumes to secondary storage X" zoneid=13b319e9-108c-4925-96aa-ae556d9a11b2 heuristicrule="''" type=VOLUME With this selector, all volumes will pass through secondary storage X during cold migration. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3801063246 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 16539 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3800735524 @RosiKyu 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
RosiKyu commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3800724479 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
RosiKyu commented on PR #10957:
URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3800710940
### TC1: Cold Volume Migration Without Secondary Storage Selector
**Objective**
Verify that cold volume migration uses the secondary storage with the most
free capacity when no secondary storage selector (heuristic rule) is configured
for VOLUME type.
**Test Steps**
1. Verified no secondary storage selector exists for VOLUME type in the zone
2. Confirmed three secondary storages exist with equal capacity (~1.08 TB
free each)
3. Started tail on management-server.log to capture secondary storage
selection
4. Executed cold migration of test-vol-1 from primary storage pri2 to pri1
5. Verified migration completed successfully and observed log output
**Expected Result:**
- Migration should complete successfully
- Log should show: "Secondary storage selector did not direct volume
migration to a specific secondary storage; using secondary storage with the
most free capacity."
- System should select sec1 (first storage with most/equal free capacity)
**Actual Result:**
- Migration completed successfully ✓
- Volume test-vol-1 migrated from pri2 to pri1 ✓
- Log confirmed: "Secondary storage selector did not direct volume migration
to a specific secondary storage; using secondary storage with the most free
capacity." ✓
- System used sec1 (id:1, uuid: 27cfbb80-8eda-4403-8ac8-9572c7edd2b7) as
staging storage ✓
**Test Evidence:**
Pre-migration check - No VOLUME selector exists:
```
(localcloud) 🐱 > list secondarystorageselectors
zoneid=0ec45e01-e0b0-4fbf-a6e8-7bb81dd480e2 type=VOLUME
(localcloud) 🐱 >
```
Secondary storage capacity (all equal):
```
(localcloud) 🐱 > list imageStores zoneid=0ec45e01-e0b0-4fbf-a6e8-7bb81dd480e2
{
"count": 3,
"imagestore": [
{
"disksizetotal": 2898029182976,
"disksizeused": 1707413078016,
"id": "27cfbb80-8eda-4403-8ac8-9572c7edd2b7",
"name":
"NFS://10.0.32.4/acs/secondary/ref-trl-10723-k-Mol9-rositsa-kyuchukova/ref-trl-10723-k-Mol9-rositsa-kyuchukova-sec1",
"protocol": "nfs"
},
{
"disksizetotal": 2898029182976,
"disksizeused": 1707413078016,
"id": "6062156f-b9f3-417a-bdd9-202fc5bce258",
"name":
"NFS://10.0.32.4/acs/secondary/ref-trl-10723-k-Mol9-rositsa-kyuchukova/ref-trl-10723-k-Mol9-rositsa-kyuchukova-sec2",
"protocol": "nfs"
},
{
"disksizetotal": 2898029182976,
"disksizeused": 1707413078016,
"id": "910cc03a-0b37-4a4d-a3f6-968cff43c3c2",
"name":
"NFS://10.0.32.4/acs/secondary/ref-trl-10723-k-Mol9-rositsa-kyuchukova/ref-trl-10723-k-Mol9-rositsa-kyuchukova-sec3",
"protocol": "nfs"
}
]
}
```
Migration command and result:
```
(localcloud) 🐱 > migrate volume
volumeid=08caf0fa-ba34-43aa-addd-f415854c854a
storageid=f55f0783-0351-329c-bd4f-8a9b81e4acbd
{
"volume": {
"id": "08caf0fa-ba34-43aa-addd-f415854c854a",
"name": "test-vol-1",
"state": "Ready",
"storage": "ref-trl-10723-k-Mol9-rositsa-kyuchukova-kvm-pri1",
"storageid": "f55f0783-0351-329c-bd4f-8a9b81e4acbd"
}
}
```
Management server log showing secondary storage selection:
```
2026-01-26 16:59:21,409 DEBUG [o.a.c.s.m.AncientDataMotionStrategy]
(Work-Job-Executor-7:[ctx-ae1d605d, job-46/job-47, ctx-cd5b4539])
(logid:99a6f633) copyAsync inspecting src type VOLUME copyAsync inspecting dest
type VOLUME
2026-01-26 16:59:21,409 DEBUG [o.a.c.s.m.AncientDataMotionStrategy]
(Work-Job-Executor-7:[ctx-ae1d605d, job-46/job-47, ctx-cd5b4539])
(logid:99a6f633) About to MIGRATE copy between datasources
2026-01-26 16:59:21,410 DEBUG [o.a.c.s.m.AncientDataMotionStrategy]
(Work-Job-Executor-7:[ctx-ae1d605d, job-46/job-47, ctx-cd5b4539])
(logid:99a6f633) MIGRATE copy using copyVolumeBetweenPools STARTING
2026-01-26 16:59:21,414 DEBUG [o.a.c.s.m.AncientDataMotionStrategy]
(Work-Job-Executor-7:[ctx-ae1d605d, job-46/job-47, ctx-cd5b4539])
(logid:99a6f633) Secondary storage selector did not direct volume migration to
a specific secondary storage; using secondary storage with the most free
capacity.
2026-01-26 16:59:21,417 DEBUG [c.c.s.StatsCollector]
(Work-Job-Executor-7:[ctx-ae1d605d, job-46/job-47, ctx-cd5b4539])
(logid:99a6f633) Verifying image storage [ImageStore
{"id":1,"name":"NFS:\/\/10.0.32.4\/acs\/secondary\/ref-trl-10723-k-Mol9-rositsa-kyuchukova\/ref-trl-10723-k-Mol9-rositsa-kyuchukova-sec1","uuid":"27cfbb80-8eda-4403-8ac8-9572c7edd2b7"}].
Capacity: total=[2.6357 TB], used=[1.5535 TB], threshold=[95.00%].
2026-01-26 16:59:22,732 DEBUG [o.a.c.s.m.AncientDataMotionStrategy]
(Work-Job-Executor-7:[ctx-ae1d605d, job-46/job-47, ctx-cd5b4539])
(logid:99a6f633) MIGRATE copy using copyVolumeBetweenPools DONE: true
```
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3646691421 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 16029 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3646261243 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✖️ debian ✔️ suse15. SL-JID 16022 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
winterhazel commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3646061067 > @winterhazel , is this still relevant for you? (do we need to push through on this?) @DaanHoogland yup, still relevant. Would be nice having this one merged. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
DaanHoogland commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3645880308 @winterhazel , is this still relevant for you? (do we need to push through on this?) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3645875846 @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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
DaanHoogland commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3645873102 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3279875389 @weizhouapache 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3280751055 Packaging result [SF]: ✖️ el8 ✖️ el9 ✔️ debian ✖️ suse15. SL-JID 14956 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
weizhouapache commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3279864550 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
weizhouapache commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3282176368 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3282426993 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 14969 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-3282179672 @weizhouapache 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-2941140752 Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13604 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
winterhazel commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-2940909005 @blueorangutan package -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
codecov[bot] commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-2940916095 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/10957?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report All modified and coverable lines are covered by tests :white_check_mark: > Project coverage is 4.00%. Comparing base [(`823080c`)](https://app.codecov.io/gh/apache/cloudstack/commit/823080cc9c379bfdbb47812bec0db1206c4ef566?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`7071461`)](https://app.codecov.io/gh/apache/cloudstack/commit/7071461e906925680af32f6eb98933e5bcd364b7?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). > :exclamation: There is a different number of reports uploaded between BASE (823080c) and HEAD (7071461). Click for more details. > > HEAD has 1 upload less than BASE > >| Flag | BASE (823080c) | HEAD (7071461) | >|--|--|--| >|unittests|1|0| > Additional details and impacted files ```diff @@ Coverage Diff @@ ## 4.20 #10957 +/- ## = - Coverage 16.14%4.00% -12.15% = Files 5656 401 -5255 Lines49789332598 -465295 Branches 60374 5788-54586 = - Hits 80405 1305-79100 + Misses 40852931144 -377385 + Partials 8959 149 -8810 ``` | [Flag](https://app.codecov.io/gh/apache/cloudstack/pull/10957/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | |---|---|---| | [uitests](https://app.codecov.io/gh/apache/cloudstack/pull/10957/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `4.00% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/apache/cloudstack/pull/10957/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more. [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/cloudstack/pull/10957?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache). :rocket: New features to boost your workflow: - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems. - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] Consider secondary storage selectors during cold volume migration [cloudstack]
blueorangutan commented on PR #10957: URL: https://github.com/apache/cloudstack/pull/10957#issuecomment-2940910207 @winterhazel 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. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
