> +         return group;
> +      }
> +
> +      FirewallOptions fwOptions = new FirewallOptions();
> +      String uniqueFwName = 
> namingConvention.createWithoutPrefix().uniqueNameForGroup(group.getName());
> +      fwOptions.name(uniqueFwName);
> +      fwOptions.network(group.getUri());
> +      if (ipPermission.getGroupIds().size() > 0) {
> +         fwOptions.sourceTags(ipPermission.getGroupIds());
> +      }
> +      if (ipPermission.getCidrBlocks().size() > 0) {
> +         fwOptions.sourceRanges(ipPermission.getCidrBlocks());
> +      }
> +      Firewall.Rule.Builder ruleBuilder = Firewall.Rule.builder();
> +      ruleBuilder.IpProtocol(ipPermission.getIpProtocol());
> +      if (ipPermission.getToPort() > 0) {

In some providers/APIs, you can specify all ports by providing 0 - for GCE, we 
just don't specify a port range to get the same effect, I believe.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/5/files#r7385749

Reply via email to