Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-13 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-13 Thread via GitHub
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...@

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-13 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-10 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-10 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-10 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-09 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-09 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-09 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-09 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-08 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-06 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-04 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-04 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-04 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-04 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-03 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-03 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-02 Thread via GitHub
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.

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-02 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-01 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-01 Thread via GitHub
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

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-07-01 Thread via GitHub
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"

Re: [PR] fix priority for volume copy operation [cloudstack]

2025-06-30 Thread via GitHub
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

[PR] fix priority for volume copy operation [cloudstack]

2025-06-30 Thread via GitHub
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