demobox commented on this pull request.
> logger.debug(">> the resource group %s is empty. Deleting...",
> group);
- deleted =
resourceDeleted.apply(api.getResourceGroupApi().delete(group));
+ URI uri = api.getResourceGroupApi().delete(group);
+ deleted = uri == null || resourceDeleted.apply(uri);
+ } else {
+ logger.debug(">> the resource group %s contains %s. Not deleting...",
group, attachedResources);
Similar comment as above: should this be `logger.warn`?
--
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/pull/1240#pullrequestreview-156628784
