From 01a47e0717d5db995ce6b7c72325e19a45a6018b Mon Sep 17 00:00:00 2001
From: alena <alena@cloud.com>
Date: Wed, 6 Jul 2011 10:51:20 -0700
Subject: [PATCH] bug 10553: allow networkDomain update for Guest system
 network in Basic zone status 10553: resolved fixed

---
 api/src/com/cloud/api/commands/CreateZoneCmd.java    | 1 -
 api/src/com/cloud/api/commands/UpdateNetworkCmd.java | 1 -
 server/src/com/cloud/network/NetworkManagerImpl.java | 4 ++--
 3 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/api/src/com/cloud/api/commands/CreateZoneCmd.java b/api/src/com/cloud/api/commands/CreateZoneCmd.java
index c150c540dfc..049245dcb5b 100644
--- a/api/src/com/cloud/api/commands/CreateZoneCmd.java
+++ b/api/src/com/cloud/api/commands/CreateZoneCmd.java
@@ -25,7 +25,6 @@ import com.cloud.api.BaseCmd;
 import com.cloud.api.Implementation;
 import com.cloud.api.Parameter;
 import com.cloud.api.ServerApiException;
-import com.cloud.api.BaseCmd.CommandType;
 import com.cloud.api.response.ZoneResponse;
 import com.cloud.dc.DataCenter;
 import com.cloud.user.Account;
diff --git a/api/src/com/cloud/api/commands/UpdateNetworkCmd.java b/api/src/com/cloud/api/commands/UpdateNetworkCmd.java
index 0aa4fd6426f..3d4690cfe94 100644
--- a/api/src/com/cloud/api/commands/UpdateNetworkCmd.java
+++ b/api/src/com/cloud/api/commands/UpdateNetworkCmd.java
@@ -27,7 +27,6 @@ import com.cloud.api.BaseCmd;
 import com.cloud.api.Implementation;
 import com.cloud.api.Parameter;
 import com.cloud.api.ServerApiException;
-import com.cloud.api.BaseCmd.CommandType;
 import com.cloud.api.response.NetworkResponse;
 import com.cloud.exception.ConcurrentOperationException;
 import com.cloud.exception.InsufficientCapacityException;
diff --git a/server/src/com/cloud/network/NetworkManagerImpl.java b/server/src/com/cloud/network/NetworkManagerImpl.java
index c6b1fb8eda4..084fd1f53e8 100755
--- a/server/src/com/cloud/network/NetworkManagerImpl.java
+++ b/server/src/com/cloud/network/NetworkManagerImpl.java
@@ -2917,9 +2917,9 @@ public class NetworkManagerImpl implements NetworkManager, NetworkService, Manag
             throw new InvalidParameterException("Unable to support more than one tag on network yet");
         }
         
-        // Don't allow to update system network
+        // Don't allow to update system network - make an exception for the Guest network in Basic zone
         NetworkOffering offering = _networkOfferingDao.findByIdIncludingRemoved(network.getNetworkOfferingId());
-        if (offering.isSystemOnly()) {
+        if (offering.isSystemOnly() && network.getTrafficType() != TrafficType.Guest) {
             throw new InvalidParameterValueException("Can't update system networks");
         }