Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

DaanHoogland commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2164431267 Yes @BryanMLima but our lab is reserved for 4.19 work this week. main will have to wait. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

BryanMLima commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2163646878 @DaanHoogland, can we run the CI one last time here? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

blueorangutan commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2161154722 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9870 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

blueorangutan commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2160986963 @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

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

winterhazel commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2160984589 @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

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

winterhazel commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2160825431 > @winterhazel , do you think my comment is worth your effort now? If not I'll add a suggestion for a comment for later work. I'll move it to the DAO layer in this PR

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

DaanHoogland commented on code in PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#discussion_r1634287418 ## engine/storage/src/main/java/org/apache/cloudstack/storage/image/BaseImageStoreDriverImpl.java: ## @@ -536,7 +536,7 @@ private Integer

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

DaanHoogland commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2159918549 @winterhazel , do you think my comment is worth your effort now? If not I'll add a suggestion for a comment for later work. -- This is an automated message from the Apache Git

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

blueorangutan commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2136043310 [SF] Trillian test result (tid-10297) Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7 Total time taken: 50862 seconds Marvin logs:

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

blueorangutan commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2134391661 @DaanHoogland a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests -- This is an automated message from the Apache Git Service. To

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

DaanHoogland commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2134391485 @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 NPE when sending copy command to least busy SSVM [cloudstack]

blueorangutan commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2129625481 Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9700 -- This is an automated message from the Apache Git Service. To respond to the message, please log

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

codecov[bot] commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2129469047 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/9125?dropdown=coverage=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=apache) Report

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

blueorangutan commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2129461752 @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

Re: [PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

winterhazel commented on PR #9125: URL: https://github.com/apache/cloudstack/pull/9125#issuecomment-2129458456 @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

[PR] Fix NPE when sending copy command to least busy SSVM [cloudstack]

winterhazel opened a new pull request, #9125: URL: https://github.com/apache/cloudstack/pull/9125 ### Description In order to copy an object from a secondary storage to another, the MS sends a copy command to the least busy SSVM; however, removed SSVMs are considered when listing