You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by da...@apache.org on 2014/05/06 22:30:07 UTC
[1/2] git commit: updated refs/heads/4.4-forward to 4c30432
Repository: cloudstack
Updated Branches:
refs/heads/4.4-forward 64ccf4b48 -> 4c304324a
cidr field removed
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/0c0a0c25
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/0c0a0c25
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/0c0a0c25
Branch: refs/heads/4.4-forward
Commit: 0c0a0c25622301816c93c0eaf0edb4a50728c79f
Parents: 64ccf4b
Author: Daan Hoogland <da...@onecht.net>
Authored: Tue May 6 17:44:54 2014 +0200
Committer: Daan Hoogland <da...@onecht.net>
Committed: Tue May 6 22:29:13 2014 +0200
----------------------------------------------------------------------
.../com/cloud/network/vpc/NetworkACLItemVO.java | 42 ++++++++------------
1 file changed, 16 insertions(+), 26 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/0c0a0c25/engine/schema/src/com/cloud/network/vpc/NetworkACLItemVO.java
----------------------------------------------------------------------
diff --git a/engine/schema/src/com/cloud/network/vpc/NetworkACLItemVO.java b/engine/schema/src/com/cloud/network/vpc/NetworkACLItemVO.java
index 0d2897d..6eb9cb0 100644
--- a/engine/schema/src/com/cloud/network/vpc/NetworkACLItemVO.java
+++ b/engine/schema/src/com/cloud/network/vpc/NetworkACLItemVO.java
@@ -16,9 +16,10 @@
// under the License.
package com.cloud.network.vpc;
-import java.util.ArrayList;
import java.util.Date;
+import java.util.LinkedList;
import java.util.List;
+import java.util.StringTokenizer;
import java.util.UUID;
import javax.persistence.Column;
@@ -29,6 +30,7 @@ import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.Table;
+import javax.persistence.Transient;
import com.cloud.utils.db.GenericDao;
import com.cloud.utils.net.NetUtils;
@@ -76,8 +78,11 @@ public class NetworkACLItemVO implements NetworkACLItem {
@Enumerated(value = EnumType.STRING)
TrafficType trafficType;
- @Column(name = "cidr", length = 2048)
- String sourceCidrs;
+ // This is a delayed load value. If the value is null,
+ // then this field has not been loaded yet.
+ // Call the NetworkACLItem dao to load it.
+ @Transient
+ List<String> sourceCidrs;
@Column(name = "uuid")
String uuid;
@@ -113,32 +118,12 @@ public class NetworkACLItemVO implements NetworkACLItem {
}
public void setSourceCidrList(List<String> sourceCidrs) {
- if (sourceCidrs == null) {
- this.sourceCidrs = null;
- } else {
- StringBuilder sb = new StringBuilder();
- for (String cidr : sourceCidrs) {
- if (sb.length() != 0) {
- sb.append(",");
- }
- sb.append(cidr);
- }
- this.sourceCidrs = sb.toString();
- }
+ this.sourceCidrs = sourceCidrs;
}
@Override
public List<String> getSourceCidrList() {
- if (sourceCidrs == null || sourceCidrs.isEmpty()) {
- return null;
- } else {
- List<String> cidrList = new ArrayList<String>();
- String[] cidrs = sourceCidrs.split(",");
- for (String cidr : cidrs) {
- cidrList.add(cidr);
- }
- return cidrList;
- }
+ return sourceCidrs;
}
@Override
@@ -239,7 +224,12 @@ public class NetworkACLItemVO implements NetworkACLItem {
}
public void setSourceCidrs(String sourceCidrs) {
- this.sourceCidrs = sourceCidrs;
+ List<String> srcCidrs = new LinkedList<String>();
+ StringTokenizer st = new StringTokenizer(sourceCidrs,",;");
+ while(st.hasMoreTokens()) {
+ srcCidrs.add(st.nextToken());
+ }
+ this.sourceCidrs = srcCidrs;
}
public void setNumber(int number) {
[2/2] git commit: updated refs/heads/4.4-forward to 4c30432
Posted by da...@apache.org.
fixed network acl cidr issues
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/4c304324
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/4c304324
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/4c304324
Branch: refs/heads/4.4-forward
Commit: 4c304324a36ad25d7756da1ca9f2426a96ad8837
Parents: 0c0a0c2
Author: Jayapal <ja...@apache.org>
Authored: Tue May 6 17:59:53 2014 +0200
Committer: Daan Hoogland <da...@onecht.net>
Committed: Tue May 6 22:29:42 2014 +0200
----------------------------------------------------------------------
.../src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java | 6 ++++--
setup/db/db/schema-430to440.sql | 2 +-
2 files changed, 5 insertions(+), 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/4c304324/engine/schema/src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java
----------------------------------------------------------------------
diff --git a/engine/schema/src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java b/engine/schema/src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java
index c940247..abd8bae 100644
--- a/engine/schema/src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java
+++ b/engine/schema/src/com/cloud/network/vpc/dao/NetworkACLItemDaoImpl.java
@@ -115,7 +115,7 @@ public class NetworkACLItemDaoImpl extends GenericDaoBase<NetworkACLItemVO, Long
for(NetworkACLItemVO item :list) {
loadCidrs(item);
}
- return listBy(sc);
+ return list;
}
@Override
@@ -132,7 +132,9 @@ public class NetworkACLItemDaoImpl extends GenericDaoBase<NetworkACLItemVO, Long
sc.setParameters("aclId", aclId);
sc.setParameters("number", number);
NetworkACLItemVO vo = findOneBy(sc);
- loadCidrs(vo);
+ if(vo != null) {
+ loadCidrs(vo);
+ }
return vo;
}
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/4c304324/setup/db/db/schema-430to440.sql
----------------------------------------------------------------------
diff --git a/setup/db/db/schema-430to440.sql b/setup/db/db/schema-430to440.sql
index d49f4d4..d0dd073 100644
--- a/setup/db/db/schema-430to440.sql
+++ b/setup/db/db/schema-430to440.sql
@@ -1654,7 +1654,7 @@ CREATE TABLE `cloud`.`op_vpc_distributed_router_sequence_no` (
INSERT INTO `cloud`.`configuration`(category, instance, component, name, value, description, default_value) VALUES ('Advanced', 'DEFAULT', 'NetworkOrchestrationService', 'router.aggregation.command.each.timeout', '3', 'timeout in seconds for each Virtual Router command being aggregated. The final aggregation command timeout would be determined by this timeout * commands counts ', '3') ON DUPLICATE KEY UPDATE category='Advanced';
-CREATE TABLE `cloud`.`network_acl_item_cidr` (
+CREATE TABLE `cloud`.`network_acl_item_cidrs` (
`id` bigint unsigned UNIQUE NOT NULL auto_increment,
`network_acl_item_id` bigint unsigned NOT NULL COMMENT 'Network ACL Item id',
`cidr` varchar(255) NOT NULL,