Repository: cloudstack Updated Branches: refs/heads/4.4-forward 38c99ffda -> bc3a8378d
CLOUDSTACK-6458: IAM - When a domain is deleted , the group created for this domian is not removed. Changes: - When domain is deleted or is removed during account cleanup, send message to IAM to remove the domain group Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/bc3a8378 Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/bc3a8378 Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/bc3a8378 Branch: refs/heads/4.4-forward Commit: bc3a8378d0e58ad53f862b6102c0dfa39d97a06a Parents: 38c99ff Author: Prachi Damle <[email protected]> Authored: Tue Apr 22 18:34:01 2014 -0700 Committer: Prachi Damle <[email protected]> Committed: Thu Apr 24 11:39:26 2014 -0700 ---------------------------------------------------------------------- server/src/com/cloud/user/DomainManagerImpl.java | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cloudstack/blob/bc3a8378/server/src/com/cloud/user/DomainManagerImpl.java ---------------------------------------------------------------------- diff --git a/server/src/com/cloud/user/DomainManagerImpl.java b/server/src/com/cloud/user/DomainManagerImpl.java index fbbe0c2..fe6e530 100644 --- a/server/src/com/cloud/user/DomainManagerImpl.java +++ b/server/src/com/cloud/user/DomainManagerImpl.java @@ -227,7 +227,11 @@ public class DomainManagerImpl extends ManagerBase implements DomainManager, Dom @Override public boolean removeDomain(long domainId) { - return _domainDao.remove(domainId); + boolean removed = _domainDao.remove(domainId); + if (removed) { + _messageBus.publish(_name, MESSAGE_REMOVE_DOMAIN_EVENT, PublishScope.LOCAL, domainId); + } + return removed; } @Override @@ -324,6 +328,9 @@ public class DomainManagerImpl extends ManagerBase implements DomainManager, Dom " because it can't be removed due to resources referencing to it"); domain.setState(Domain.State.Active); _domainDao.update(domain.getId(), domain); + } else { + // succeeded in removing the domain + _messageBus.publish(_name, MESSAGE_REMOVE_DOMAIN_EVENT, PublishScope.LOCAL, domain.getId()); } } }
