>           final Map<String, String> tags = ImmutableMap.of("description", 
> "jClouds managed VMs");
> -         resourceGroupName = resourceGroupApi.create(group, location, 
> tags).name();
> -      } else {
> -         resourceGroupName = resourceGroup.name();
> +         resourceGroupApi.create(azureGroupName, location, tags).name();
> +      }
> +
> +      this.vnetName = this.azureGroupName + "virtualnetwork";
> +      this.subnetName = this.azureGroupName + "subnet";
> +
> +      if (options.getVirtualNetworkName() != null) {
> +         this.vnetName = options.getVirtualNetworkName();

Don't use member variables (shared mutable state!) for things that depend on 
particular method execution parameters.

---
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/291/files/8a39037237f5cccc7d100bc58a9a82bdb89924b1#r67884602

Reply via email to