Damans227 commented on PR #13630: URL: https://github.com/apache/cloudstack/pull/13630#issuecomment-5092869791
I investigated this. Looks like you're right there's an issue, but `account.cleanup.interval` isn't it. That setting only retries failed cleanup steps later, it doesn't control when the account gets marked removed. That happens immediately for any account deletion, not just domain cleanup. The real reason the domain delete fails: `DomainManagerImpl` won't delete a domain if any account under it still needs cleanup, and any error during that account's cleanup sets that flag. Templates are one way to trigger it. I found another: a domain admin deleting a sub-account with a remote access VPN user or a GSLB rule hits the same problem in a different spot (`DomainChecker`). Fixed both, same idea: look up the account including removed ones instead of excluding them. -- 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]
