Return-Path: X-Original-To: apmail-cloudstack-dev-archive@www.apache.org Delivered-To: apmail-cloudstack-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id BFA101043D for ; Sun, 16 Mar 2014 05:46:36 +0000 (UTC) Received: (qmail 93138 invoked by uid 500); 16 Mar 2014 05:20:33 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 91477 invoked by uid 500); 16 Mar 2014 05:18:02 -0000 Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list dev@cloudstack.apache.org Received: (qmail 85625 invoked by uid 99); 16 Mar 2014 05:14:43 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 16 Mar 2014 05:14:43 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id D0F2C1D4B89; Sun, 16 Mar 2014 05:14:40 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============4284620518211871367==" MIME-Version: 1.0 Subject: Re: Review Request 19258: AclService changes to clone Acl for NetworkId From: "Antonio Fornie" To: "daan Hoogland" , "Hugo Trippaers" Cc: "Antonio Fornie" , "cloudstack" Date: Sun, 16 Mar 2014 05:14:40 -0000 Message-ID: <20140316051440.30484.33889@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Antonio Fornie" X-ReviewGroup: cloudstack X-ReviewRequest-URL: https://reviews.apache.org/r/19258/ X-Sender: "Antonio Fornie" References: <20140316031450.10551.64244@reviews.apache.org> In-Reply-To: <20140316031450.10551.64244@reviews.apache.org> Reply-To: "Antonio Fornie" X-ReviewRequest-Repository: cloudstack-git --===============4284620518211871367== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19258/ ----------------------------------------------------------- (Updated March 16, 2014, 5:14 a.m.) Review request for cloudstack, daan Hoogland and Hugo Trippaers. Changes ------- Fixed conflicts with last changes in master Repository: cloudstack-git Description ------- When creating a new ACL Rule, and ONLY if provided Network Id instead of VPC Id, we should make sure we don't change an ACL that is used by other networks. So if other networks use it, we clone it, assign the new one to the given network and add the new ACL Rule. This functionality should dissapear for next release, for that we will not accept the Network Id as a parameter for that command, but only the Vpc ID (required instead of optional) On the other hand, having the functionality of cloning ACLs is useful. Diffs (updated) ----- api/src/com/cloud/network/vpc/NetworkACLService.java 7cd1d3b build/replace.properties 265f335 engine/schema/src/com/cloud/network/vpc/NetworkACLItemDao.java e278485 engine/schema/src/com/cloud/network/vpc/NetworkACLVO.java b6ed5cb engine/schema/src/com/cloud/network/vpc/dao/NetworkACLDao.java 5e2a6f5 engine/schema/src/com/cloud/network/vpc/dao/NetworkACLDaoImpl.java bc178d3 engine/schema/src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java 31ff6f2 server/src/com/cloud/network/vpc/NetworkACLServiceImpl.java e024fbe server/test/com/cloud/network/vpc/NetworkACLServiceTest.java PRE-CREATION server/test/com/cloud/vpc/NetworkACLServiceTest.java 5361639 Diff: https://reviews.apache.org/r/19258/diff/ Testing ------- Full maven build plus execution of ACL and ACL rule functionalities Thanks, Antonio Fornie --===============4284620518211871367==--