> @@ -186,6 +202,16 @@ private void 
> getOrCreateFirewalls(GoogleComputeEngineTemplateOptions templateOpt
>           operationDone.apply(operation);
>           checkState(operation.get().httpErrorStatusCode() == null, "Could 
> not insert firewall, operation failed %s",
>                 operation);
> +
> +         // Add tags for firewalls
> +         tags.add(name);
> +      }
> +      templateOptions.tags(tags);
> +   }
> +
> +   public static void validateFirewall(Firewall firewall, URI networkUri) {

Does it make more sense for the signature to be validateFirewall(Firewall 
firewall, Network network)?

---
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/985/files/e8bd802d681d4292d924e9b71898e591e3968289#r73166600

Reply via email to