andreaturli commented on this pull request.
> } catch (Exception ex) { logger.warn(ex, "Error cleaning up resources for node %s", deadNode); } + + List<SecurityGroup> securityGroups = cleanupResources.findOrphanedSecurityGroups(regionAndId.regionId(), deadNode.getGroup()); + for (SecurityGroup securityGroup : securityGroups) { + logger.debug(">> destroying security group %s ...", securityGroup.id()); + if (cleanupResources.cleanupSecurityGroupIfOrphaned(regionAndId.regionId(), securityGroup.id())) { + logger.debug(">> security group: (%s) has been deleted.", securityGroup.id()); + } else { + logger.warn(">> security group: (%s) has not been deleted.", securityGroup.id()); + } + } + + // FIXME not sure it is correct to always delete vSwitch and VPC @nacx wdyt? I'm not completely sure about this default behaviour -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/443#pullrequestreview-144485695