dheeraj12347 commented on PR #14189: URL: https://github.com/apache/cloudstack/pull/14189#issuecomment-6826830142
Hi @bhouse-nexthop , @DaanHoogland I investigated the separate storage pool refcount leak mentioned in your PR description and have submitted a follow-up PR to address it: https://github.com/apache/cloudstack/pull/14694. The fix ensures that the reference count is decremented if storage pool creation fails after the increment, including failures that throw `CloudRuntimeException`. I've also added a regression test covering the `getStoragePool()` failure case. This change is intentionally scoped to the refcount leak and does not modify the race or unmount fixes in your PR. Thanks for highlighting this issue! -- 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]
