Re: [PR] Improve logs in primary storage removal process [cloudstack]

JoaoJandre merged PR #8649: URL: https://github.com/apache/cloudstack/pull/8649 -- 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...@clou

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2314916708 @JoaoJandre at your discretion. -- 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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2314819106 [SF] Trillian test result (tid-11205) Environment: kvm-ol8 (x2), Advanced Networking with Mgmt server ol8 Total time taken: 49462 seconds Marvin logs: https://github.com/

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2313313874 @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 co

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2310176085 @DaanHoogland a [LL] 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 r

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2310174656 @blueorangutan LLtest -- 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] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2309631298 @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] Improve logs in primary storage removal process [cloudstack]

GutoVeronezi commented on code in PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#discussion_r1729249659 ## server/src/main/java/com/cloud/storage/StorageManagerImpl.java: ## @@ -1514,6 +1515,25 @@ private boolean deleteDataStoreInternal(StoragePoolVO sPool, boole

Re: [PR] Improve logs in primary storage removal process [cloudstack]

hsato03 commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2246360621 > @hsato03 seeing `2024-07-23T18:43:54,232 INFO [c.c.a.ApiServlet] (qtp253011924-22:[ctx-35b7f332, ctx-24c715d9]) (logid:0c5ca90f) (userId=2 accountId=2 sessionId=node048rydzt ycoujeea

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2246004483 @hsato03 seeing `2024-07-23T18:43:54,232 INFO [c.c.a.ApiServlet] (qtp253011924-22:[ctx-35b7f332, ctx-24c715d9]) (logid:0c5ca90f) (userId=2 accountId=2 sessionId=node048rydzt y

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2242386357 @GutoVeronezi , all satisfied with the current state? -- 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] Improve logs in primary storage removal process [cloudstack]

blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2240873419 [LL]Trillian test result (tid-6961) Environment: kvm-alma8 (x2), Advanced Networking with Mgmt server a8 Total time taken: 47354 seconds Marvin logs: https://github.com/b

Re: [PR] Improve logs in primary storage removal process [cloudstack]

blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2238531837 [LL] Trillian Build Failed (tid-6957) -- 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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2238586438 [LL] Trillian Build Failed (tid-6959) -- 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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2238529483 @DaanHoogland a [LL] 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 r

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2238528645 @blueorangutan LLtest -- 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] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2236475354 @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] Improve logs in primary storage removal process [cloudstack]

blueorangutan commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2227076406 [SF] Trillian Build Failed (tid-10822) -- 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 g

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2227066800 @blueorangutan test alma9 kvm-alma9 -- 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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2225012899 @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 co

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-860242 @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] Improve logs in primary storage removal process [cloudstack]

github-actions[bot] commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2197856101 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache Gi

[PR] Improve logs in primary storage removal process [cloudstack]

hsato03 opened a new pull request, #8649: URL: https://github.com/apache/cloudstack/pull/8649 ### Description While listing volumes from a primary storage, volumes from system VMs are not listed, making the primary storage removal process difficult as an exception is thrown if the st

Re: [PR] Improve logs in primary storage removal process [cloudstack]

BryanMLima closed pull request #8649: Improve logs in primary storage removal process URL: https://github.com/apache/cloudstack/pull/8649 -- 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] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

BryanMLima commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2075005065 @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 c

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2046992709 @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] Improve logs in primary storage removal process [cloudstack]

hsato03 opened a new pull request, #8649: URL: https://github.com/apache/cloudstack/pull/8649 ### Description While listing volumes from a primary storage, volumes from system VMs are not listed, making the primary storage removal process difficult as an exception is thrown if the st

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland closed pull request #8649: Improve logs in primary storage removal process URL: https://github.com/apache/cloudstack/pull/8649 -- 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 specif

Re: [PR] Improve logs in primary storage removal process [cloudstack]

codecov-commenter commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2045876329 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8649?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+com

Re: [PR] Improve logs in primary storage removal process [cloudstack]

github-actions[bot] commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-2037167710 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache Gi

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1961292904 @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 co

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1960919770 @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] Improve logs in primary storage removal process [cloudstack]

github-actions[bot] commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1952018519 This pull request has merge conflicts. Dear author, please fix the conflicts and sync your branch with the base branch. -- This is an automated message from the Apache Gi

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

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

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1948122125 @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] Improve logs in primary storage removal process [cloudstack]

GutoVeronezi commented on code in PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#discussion_r1491402149 ## server/src/main/java/com/cloud/storage/StorageManagerImpl.java: ## @@ -1254,7 +1254,8 @@ private boolean deleteDataStoreInternal(StoragePoolVO sPool, boolea

Re: [PR] Improve logs in primary storage removal process [cloudstack]

codecov[bot] commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1943431988 ## [Codecov](https://app.codecov.io/gh/apache/cloudstack/pull/8649?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)

Re: [PR] Improve logs in primary storage removal process [cloudstack]

DaanHoogland commented on PR #8649: URL: https://github.com/apache/cloudstack/pull/8649#issuecomment-1943420111 not sure if we have any automated way of testing this (yet) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[PR] Improve logs in primary storage removal process [cloudstack]

hsato03 opened a new pull request, #8649: URL: https://github.com/apache/cloudstack/pull/8649 ### Description While listing volumes from a primary storage, volumes from system VMs are not listed, making the primary storage removal process difficult as an exception is thrown if the st