boring-cyborg[bot] commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3067921814
Awesome work, congrats on your first merged pull request!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
DaanHoogland merged PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109
--
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: commits-unsubscr...@
DaanHoogland commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3067921482
low impact and I see no-one testing this beyond regression, merging.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Gi
blueorangutan commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3059410539
[SF] Trillian test result (tid-13748)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 51709 seconds
Marvin logs:
https://github.co
blueorangutan commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3056096461
@DaanHoogland 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
DaanHoogland commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3056089782
@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
blueorangutan commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3054762680
[SF] Trillian test result (tid-13732)
Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8
Total time taken: 59864 seconds
Marvin logs:
https://github.co
DaanHoogland commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3051507448
> @DaanHoogland, @sureshanaparti, @slavkap Is there anything further
required from our side, or can this be considered for merging?
@shrikantjoshi-hpe , at this moment the
blueorangutan commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3051504135
@DaanHoogland 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
DaanHoogland commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3051500481
@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
shrikantjoshi-hpe commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3051095943
@DaanHoogland, @sureshanaparti, @slavkap Is there anything further required
from our side, or can this be considered for merging?
--
This is an automated message from the
shrikantjoshi-hpe commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3043486994
@DaanHoogland, code change has been reviewed by a second reviewer as well.
Can you now consider it for merging?
--
This is an automated message from the Apache Git Servi
blueorangutan commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3036886908
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 14040
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
blueorangutan commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3036124755
@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 prog
sureshanaparti commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3036118351
@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 spec
Copilot commented on code in PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#discussion_r2185242492
##
plugins/storage/volume/primera/src/main/java/org/apache/cloudstack/storage/datastore/adapter/primera/PrimeraAdapter.java:
##
@@ -311,6 +311,7 @@ public Provider
DaanHoogland commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3031763266
@shrikantjoshi-hpe , great. Try to find a second reviewer. Preferably
someone still active that has changed the code that you touched. Couldn’t hurt
to look at other storage ven
shrikantjoshi-hpe commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3031497991
@DaanHoogland, Thank you for the details. With this change there is no
proprietary information involved. Regarding testing, we performed multiple
tests related to these cod
DaanHoogland commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3028638711
> @DaanHoogland Is anything expected from us regarding the
status:needs-testing label? I haven’t found any test coverage for the Primera
plugin in the repository. Please advise.
shrikantjoshi-hpe commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3027429920
@DaanHoogland Is anything expected from us regarding the
status:needs-testing label? I haven’t found any test coverage for the Primera
plugin in the repository. Please advi
codecov[bot] commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3023016192
##
[Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/11109?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comme
shrikantjoshi-hpe commented on code in PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#discussion_r2177099712
##
tools/marvin/setup.py:
##
@@ -27,7 +27,7 @@
raise RuntimeError("python setuptools is required to build Marvin")
-VERSION = "4.21.0.0-SNAP
DaanHoogland commented on code in PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#discussion_r2177004297
##
tools/marvin/setup.py:
##
@@ -27,7 +27,7 @@
raise RuntimeError("python setuptools is required to build Marvin")
-VERSION = "4.21.0.0-SNAPSHOT"
boring-cyborg[bot] commented on PR #11109:
URL: https://github.com/apache/cloudstack/pull/11109#issuecomment-3019079221
Congratulations on your first Pull Request and welcome to the Apache
CloudStack community! If you have any issues or are unsure about any anything
please check our Contrib
shrikantjoshi-hpe opened a new pull request, #11109:
URL: https://github.com/apache/cloudstack/pull/11109
### Description
This PR adds support to set the priority to high for volume copy operations
using the Primera plugin.
This enhancement allows volume copy tasks handled by the
25 matches
Mail list logo