Re: Review Request 13047: VMSnapshot chainSize check

2013-07-29 Thread mice xia
single snapshot volume size, so no matter current snapshot is memoryDisk or Disk, all memory image associated with this vm should be considered - mice xia On July 30, 2013, 12:38 a.m., Amogh Vasekar wrote: > > --- > T

Re: Review Request 12326: CLOUDSTACK-3094: Adding a shared network to already deployed VM is failing.

2013-07-11 Thread mice xia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12326/#review23062 --- Ship it! Ship It! - mice xia On July 8, 2013, 2:30 p.m

答复: VM snapshots on KVM

2013-06-18 Thread Mice Xia
codes for vm snapshot for KVM has not been merged due to it depends on a new version libvirt-java. can we skip the tests for KVM right now? -Mice -Original Message- From: Prasanna Santhanam [mailto:t...@apache.org] Sent: 2013-6-18 (星期二) 16:12 To: CloudStack Dev Subject: VM snapshots on

RE: [MERGE] Merge VMSync improvement branch into master

2013-06-17 Thread Mice Xia
Kelven, After the refactoring, will CS still restart HA enabled VM when it is power off externally (e.g. using xencenter) or internally (user initiated shutdown or crash)? Is seems the method HighAvailabilityManagerImpl .scheduleRestart() is not called when VM's actual state is stopped while e

RE: [VOTE] Pushback 4.2.0 Feature Freeze

2013-06-03 Thread Mice Xia
+1 -Mice -Original Message- From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com] Sent: Tuesday, June 04, 2013 12:04 PM To: dev@cloudstack.apache.org Subject: RE: [VOTE] Pushback 4.2.0 Feature Freeze +1 [Binding] Regards, Sateesh > -Original Message- > From: Abhi

Re: Review Request: CLOUDSTACK-2130:UpdateDefaultNicForVirtualMachine api should also create usage events for updating new default network

2013-05-30 Thread mice xia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11511/#review21184 --- Ship it! Ship It! - mice xia On May 29, 2013, 12:56 p.m

Re: [ACS][VMWARE][BUG?] job execution does not stop while zone, pod and cluster are disabled

2013-05-29 Thread Mice Xia
if remember correctly, by design disabling a cluster will stop user VM deploy in this cluster but does not exclude it from systemvm's destination; and vmware based cluster is always chosen as the destination when vmware systemvm template is ready. this should be a bug, primary storage's status shou

Re: Review Request: CLOUDSTACK-2130: UpdateDefaultNicForVirtualMachine api should also create usage events for updating new default network

2013-05-15 Thread mice xia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11156/#review20636 --- Ship it! Ship It! - mice xia On May 14, 2013, 3:53 p.m

Re: Fix for CLOUDSTACK-2277

2013-05-09 Thread Mice Xia
o look into the difference in behavior. > But for now, I guess at-least the add api must be allowed to pass. > Your thoughts? > > Thanks, > Saksham > > On Thursday 09 May 2013 02:37 PM, Mice Xia wrote: > > Saksham, > > > > Really sorry I didn’t notice it'

RE: Fix for CLOUDSTACK-2277

2013-05-09 Thread Mice Xia
your env? Regards Mice -Original Message- From: Saksham Srivastava [mailto:saksham.srivast...@citrix.com] Sent: Thursday, May 09, 2013 4:37 PM To: Mice Xia; dev@cloudstack.apache.org Subject: Fix for CLOUDSTACK-2277 Hi Mice , I just saw your commit for https://issues.apache.org/jira/b

RE: "_Enhancer" Question

2013-05-08 Thread Mice Xia
Here is what I know about enhancer: Enhancer is a term in cglib, an enhancer is a subclass of original class which 'enhances' its functionality In Cloudstack, enhancer (and interceptor) is used to implement aspect-oriented features, e.g, transaction, events framework.. Classes that implement Com

[ACS42] 4.2 Build Failed

2013-05-07 Thread Mice Xia
After master commit e0d8f01ecd92a1a7b74558d02a28be3b2f20a10d, which set default test source directory for all sub-projects, unit tests in cloud-vmware-base have compile errors: [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.5.1:testCompile (default-testCompi

RE: [PROPOSAL] VMSync improvement to better co-operate with external managers for features like HA/DRS and FT

2013-05-05 Thread Mice Xia
+1 for this, currently VMsync/HA/external state change have conflicts occasionally, serializing VM operations can solve concurrency related issues. So fullsync and deltasync will be deprecated and a new unified sync framework will be introduced? -Mice -Original Message- From: Kelven Yan

RE: New Committer: Isaac Chiang

2013-05-05 Thread Mice Xia
Congrats, Isaac! -Mice -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Saturday, May 04, 2013 8:54 PM To: dev@cloudstack.apache.org Subject: New Committer: Isaac Chiang The Project Management Committee (PMC) for Apache CloudStack has asked Isaac Chiang to be

Re: Review Request: fix bug CLOUDSTACK-2160: add a huge size guest network will cause out of memory

2013-05-01 Thread mice xia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10750/#review20048 --- Ship it! Ship It! - mice xia On April 24, 2013, 4:20 a.m

Re: [DISCUSS] Scaling up CPU and RAM for running VMs

2013-04-24 Thread Mice Xia
Nitin & Kelven, I haven't updated the codes with syncAsyncJobExecution, and the conclusion is making all vm api synchronized? I'm ok with this change, currently, for a single vm instance, few apis can work on it concurrently. -Mice On Fri, Mar 15, 2013 at 12:20 PM, Nitin Mehta wrote: > Thanks

RE: [DISCUSS] ACS Release 4 month v/s 6 month

2013-04-22 Thread Mice Xia
+1 for 6 months, leave buffer for testing and bug fixing -Mice -Original Message- From: Radhika Puthiyetath [mailto:radhika.puthiyet...@citrix.com] Sent: Tuesday, April 23, 2013 12:36 PM To: dev@cloudstack.apache.org; cloudstack-...@incubator.apache.org Subject: RE: [DISCUSS] ACS Release

Re: [ACS41] RC Blocker issue. Help Needed!

2013-04-21 Thread Mice Xia
chip, in tools/apidoc/pom.xml - ../../client/target/cloud-client-ui-4.1.0-SNAPSHOT/WEB-INF/ + ../../client/target/cloud-client-ui-4.1.1-SNAPSHOT/WEB-INF/ -mice 2013/4/22 Chip Childers > I was about to release the RC for voting, when I hit an error building > from the 4.1 branch via the follow

RE: New Committer: Go Chiba

2013-04-18 Thread Mice Xia
Congrats! -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Thursday, April 18, 2013 2:56 PM To: dev@cloudstack.apache.org Subject: New Committer: Go Chiba The Project Management Committee (PMC) for Apache CloudStack has asked Go Chiba to become a committer and

Re: [ACS41][Patch Request]

2013-04-17 Thread Mice Xia
There's another one in reviewboard: https://reviews.apache.org/r/10536/ Summary: CLOUDSTACK-1894 User is not able to deploy VMs in a project[permission denied to access the guest network) 2013/4/17 Chip Childers > On Wed, Apr 17, 2013 at 09:36:20PM +0800, Mice Xia wrote: &g

[ACS41][Patch Request]

2013-04-17 Thread Mice Xia
commit 4d0a460bad05bfc89a9f2d93724221c7c517626e Author: Mice Xia Date: Wed Apr 17 21:24:33 2013 +0800 fix CLOUDSTACK 1991 CreateTagsCmd.java, DeleteTagsCmd.java, and ListTagsCmd.java all contain invalid version --- https

RE: [DISCUSS] Granular Global Parameters

2013-04-16 Thread Mice Xia
Not meant to stray from the topic, I raised this question because I think this is a good opportunity to refactor global configs. Yes, they are two separate issues, but need think it through before work in parallel otherwise there will be many code conflicts.. +1 for Nitin's proposal: Firstly in

RE: [DOCS] Please review: VM Snapshots section (CLOUDSTACK-1561)

2013-04-16 Thread Mice Xia
eview: VM Snapshots section (CLOUDSTACK-1561) I have posted a review draft at the URL below [1]. Please take a look. A few more replies to Mice's input inline: > > > *From:* Mice Xia [mailto:mice_...@tcloudcomputing.com] > *Sent:* Tuesday, April 09, 2013 10:25 PM > *To:* Jes

RE: [DISCUSS] Getting rid of premium

2013-04-15 Thread Mice Xia
+1 -Original Message- From: Sateesh Chodapuneedi [mailto:sateesh.chodapune...@citrix.com] Sent: Tuesday, April 16, 2013 12:16 PM To: dev@cloudstack.apache.org Subject: RE: [DISCUSS] Getting rid of premium > -Original Message- > From: Kelven Yang [mailto:kelven.y...@citrix.com] >

Review Request: CLOUDSTACK-1894 User is not able to deploy VMs in a project[permission denied to access the guest network)

2013-04-15 Thread mice xia
merged from cbcb83b086f4a87aa0703e18a03938b808ed044f in master This addresses bug CS-1894. Diffs - server/src/com/cloud/network/NetworkModelImpl.java d39214a Diff: https://reviews.apache.org/r/10536/diff/ Testing --- Thanks, mice xia

RE: [DISCUSS] Granular Global Parameters

2013-04-10 Thread Mice Xia
If we make config parameters granular to zone/cluster/account.. level, do we have to restart mgmt server to take it effect? And it seems this involves a lot of change in codes, is it possible to take this chance and improve global configs so that we can change global config at runtime ( no need

RE: [MERGE] Support VM Snapshot

2013-04-10 Thread Mice Xia
, March 22, 2013 5:09 AM To: Mice Xia; cloudstack-...@incubator.apache.org Cc: Chandan Purushothama; Anthony Xu; Hari Kannan Subject: RE: [MERGE] Support VM Snapshot That's good, thanks for your response. > -Original Message----- > From: Mice Xia [mailto:mice_...@tcloudcomputing.c

Re: Review Request: Remove VM snapshot table creation from schema-40to410.sql

2013-04-09 Thread mice xia
developer -pl developer -Ddeploydb succeed Thanks, mice xia

Review Request: Remove VM snapshot table creation from schema-40to410.sql

2013-04-09 Thread mice xia
, related schema creation script should not be in schema-40to410.sql Diffs - setup/db/db/schema-40to410.sql f327ce9 Diff: https://reviews.apache.org/r/10390/diff/ Testing --- mvn -P developer -pl developer -Ddeploydb succeed Thanks, mice xia

Re: Review Request: Remove db id in exception message.

2013-04-09 Thread mice xia
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10159/#review18920 --- Ship it! Ship It! - mice xia On March 27, 2013, 11:26 p.m

RE: [MERGE] Support VM Snapshot

2013-03-20 Thread Mice Xia
ble/disable this feature on a per hypervisor > basis but if that's complicated adding a top level flag to > enable/disable the functionality is reasonable. > > > -Original Message- > > From: Mice Xia [mailto:mice_...@tcloudcomputing.com] > > Sent: Monday, March 1