nacx approved this pull request.
> for (String securityGroupName : templateOptions.getGroups()) {
-
checkNotNull(novaApi.getSecurityGroupApi(region).get().get(securityGroupName),
"security group %s doesn't exist", securityGroupName);
+ checkState(securityGroupNames.contains(securityGroupName), "Cannot
find security group with name " + securityGroupName + ". \nSecurity groups
available are: \n" + Iterables.toString(securityGroupNames)); // {
}
[minor] Change to `securityGroupNames.containsAll(templateOptions.getGroups())`.
--
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/1128#pullrequestreview-54640480
