geomacy commented on this pull request.
> @@ -220,10 +223,16 @@ void deleteSecurityGroup(String region, String group) {
checkNotNull(emptyToNull(group), "group must be defined");
String groupName = namingConvention.create().sharedNameForGroup(group);
- if
(!client.getSecurityGroupApi().get().describeSecurityGroupsInRegion(region,
groupName).isEmpty()) {
+ Multimap<String, String> securityGroupFilterByName =
ImmutableMultimap.of("group-name", groupName);
+ Set<SecurityGroup> securityGroupsToDelete =
client.getSecurityGroupApi().get()
+ .describeSecurityGroupsInRegionWithFilter(region,
securityGroupFilterByName);
I guess this is already including the region value so maybe this is fine as-is?
--
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/1091#discussion_r114362734
