> + vm = > client.getVirtualMachineApi().getVirtualMachine(vm.getId()); > + List<Integer> ports = > Ints.asList(templateOptions.getInboundPorts()); > + if (capabilities.getCloudStackVersion().startsWith("2")) { > + logger.debug(">> setting up IP forwarding for > IPAddress(%s) rules(%s)", ip.getId(), ports); > + Set<IPForwardingRule> rules = > setupPortForwardingRulesForIP.apply(ip, ports); > + logger.trace("<< setup %d IP forwarding rules on > IPAddress(%s)", rules.size(), ip.getId()); > + } else { > + logger.debug(">> setting up firewall rules for > IPAddress(%s) rules(%s)", ip.getId(), ports); > + Set<FirewallRule> rules = > setupFirewallRulesForIP.apply(ip, ports); > + logger.trace("<< setup %d firewall rules on > IPAddress(%s)", rules.size(), ip.getId()); > + } > + } > + } > + } catch (RuntimeException re) { > + logger.error("-- exception after node has been created, trying to > destroy the created virtualMachine(%s)", vm.getId()); > + destroyNode(vm.getId());
Ahh. that makes perfect sense :-). I make the change. --- Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds/pull/328/files#r11014695