andreaturli commented on this pull request.


> +
+      String securityGroupId = null;
+      if (!options.getGroups().isEmpty()) {
+         Iterable<String> securityGroupNames = 
api.securityGroupApi().list(location.getId()).concat().transform(new 
Function<SecurityGroup, String>() {
+            @Override
+            public String apply(SecurityGroup input) {
+               return input.name();
+            }
+         });
+         for (String securityGroupName : options.getGroups()) {
+            checkState(Iterables.contains(securityGroupNames, 
securityGroupName), "Cannot find security group with name " + securityGroupName 
+ ". \nSecurity groups available are: \n" + 
Iterables.toString(securityGroupNames)); // {
+         }
+      } else if (options.getInboundPorts().length > 0) {
+         String name = namingConvention.create().sharedNameForGroup(group);
+         SecurityGroupRequest securityGroupRequest = 
api.securityGroupApi().create(location.getId(),
+                 
CreateSecurityGroupOptions.Builder.securityGroupName(name).vpcId(options.getVpcId()));

ok for validating them, not entirely sure about create a vSwitch if not 
available

-- 
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#discussion_r207199172

Reply via email to