cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Antonio Fornie" <>
Subject Re: Review Request 19258: AclService changes to clone Acl for NetworkId
Date Sun, 16 Mar 2014 05:14:40 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated March 16, 2014, 5:14 a.m.)

Review request for cloudstack, daan Hoogland and Hugo Trippaers.


Fixed conflicts with last changes in master

Repository: cloudstack-git


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/ 7cd1d3b 
  build/ 265f335 
  engine/schema/src/com/cloud/network/vpc/ e278485 
  engine/schema/src/com/cloud/network/vpc/ b6ed5cb 
  engine/schema/src/com/cloud/network/vpc/dao/ 5e2a6f5 
  engine/schema/src/com/cloud/network/vpc/dao/ bc178d3 
  engine/schema/src/com/cloud/network/vpc/dao/ 31ff6f2 
  server/src/com/cloud/network/vpc/ e024fbe 
  server/test/com/cloud/network/vpc/ PRE-CREATION 
  server/test/com/cloud/vpc/ 5361639 



Full maven build plus execution of ACL and ACL rule functionalities


Antonio Fornie

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message