> +
> +/**
> + * Nova (or Neutron) network definition
> + * Used to provide support for network, port, and fixed_ip when booting Nova 
> servers.
> + * @author Zack Shoylev
> + */
> +public class Network implements Comparable<Network> {
> +   private String networkUuid;
> +   private String portUuid;
> +   private String fixedIp;
> +
> +   @ConstructorProperties({
> +      "networkUuid", "portUuid", "fixedIp"
> +   })
> +   protected Network(String networkUuid, String portUuid, String fixedIp) {
> +      checkArgument(networkUuid!=null || portUuid!=null, "At least one of 
> networkUuid or portUuid should be specified");

I usually see those checks in the constructor for domain classes. Do you have 
some counter-examples in mind?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/311/files#r10442028

Reply via email to