> @@ -56,7 +57,7 @@ public FirewallOptions addAllowedRule(Firewall.Rule
> allowedRule) {
> /**
> * @see org.jclouds.googlecomputeengine.domain.Firewall#getAllowed()
> */
> - public FirewallOptions allowedRules(Set<Firewall.Rule> allowedRules) {
> + public FirewallOptions allowedRules(List<Firewall.Rule> allowedRules) {
This was an attempt, that I forgot to remove, sorry @demobox
I agree `Set` is better here, but there is a unit test
testCreateNodeWhenNetworkNorFirewallExistDoesNotExist which fails randomly
because of the order of the rules when I use Set. Any suggestions?
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/20/files#r9470867
