Re: Job Queue Question

2014-07-14 Thread Kelven Yang
, "Mike Tutkowski" wrote: >I see...so, in case, for example, this API call is being executed in >multiple places at the same time, we are making sure we send the commands >to the VM in a serial fashion. > > >On Mon, Jul 14, 2014 at 4:09 PM, Kelven Yang >wrote: >

Re: Job Queue Question

2014-07-14 Thread Kelven Yang
Mike, This is related to serializing activities to the VM. When VM has multiple disks and volume-semantic API could create situations that there exist multiple volume operations happening at the same time on the same VM. Kelven On 7/14/14, 2:23 PM, "Mike Tutkowski" wrote: >Hi, > >I have a ques

[ACS44] Cherry-pick for CLOUDSTACK-6970

2014-06-20 Thread Kelven Yang
Daan, Please cherry-pick following from 4.4-forward to 4.4 commit 06d8d3a03f7a4efc41c45d9f67fd5b7586a78454 Author: Kelven Yang Date: Thu May 29 09:35:03 2014 -0700 CLOUDSTACK-6970: Protect event interpretion from causing NPE exception Thanks Kelven

Re: unused private methods in VirtualMachinManagerImpl

2014-06-12 Thread Kelven Yang
> > >On Wed, Jun 11, 2014 at 10:52 AM, Kelven Yang >wrote: > >> I guess we can create one. Currently available annotations that are to >>be >> applied to java code are as following >> >> * @Override >> * @Deprecated >> * @SuppressWarnings >

Re: unused private methods in VirtualMachinManagerImpl

2014-06-11 Thread Kelven Yang
; >Thanks! > > >On Wed, Jun 11, 2014 at 2:06 AM, Daan Hoogland >wrote: > >> is there one availible or are we implementing @usedPrivate ? >> >> On Wed, Jun 11, 2014 at 2:54 AM, Mike Tutkowski >> wrote: >> > Good idea >> > >> >

Re: unused private methods in VirtualMachinManagerImpl

2014-06-10 Thread Kelven Yang
provide sufficient comments in >each >of these cases so no one removes the fields thinking that they are old and >no longer in use. > > >On Tue, Jun 10, 2014 at 5:13 PM, Kelven Yang >wrote: > >> The usage of these private methods are through reflection, making it >

Re: unused private methods in VirtualMachinManagerImpl

2014-06-10 Thread Kelven Yang
work. Check >>handleVmWorkJob() method in the same java file. >> >> >> - Koushik Das >> >> >> On June 10, 2014, 4:06 a.m., Rajani Karuturi wrote: >>> >>> --- >>> This is an automatically generated e-mail. To reply, visit: >>>

[ACS4.4] Cherry-pick e3bb8b98bd9055115004da2ec67b29271a3974c5

2014-05-21 Thread Kelven Yang
There is an typo on the bug id, it should be the fix for CLOUDSTACK-6743 commit e3bb8b98bd9055115004da2ec67b29271a3974c5 Author: Kelven Yang Date: Wed May 21 16:14:14 2014 -0700 CLOUDSTACK-5485: Use edge-triggering in MessageDetector to handle bogus wakeup gracefully. Level triggering

Re: @ActionEvent generation doesn't work when method is called within the same manager context

2014-05-12 Thread Kelven Yang
ask developers always to publish events programmatically, unless we have a better solution for this. Kelven From: Darren Shepherd mailto:darren.sheph...@citrix.com>> Date: Wednesday, May 7, 2014 at 2:30 PM To: Alena Prokharchyk mailto:alena.prokharc...@citrix.com>> Cc:

[ACS44] Cherry pick for CLOUDSTACK-6647

2014-05-12 Thread Kelven Yang
Hi Daan, Please cherry pick the fix for CLOUDSTACK-6647 to 4.4. commit 7e6390d3c845d9a0c918aa525b5cbfb2aa7fb3c5 Author: Kelven Yang Date: Thu May 1 15:40:04 2014 -0700 CLOUDSTACK-6647: appending instance name with custom supplied info that contains - character can break vmsync. Thanks

[ACS44] cherry pick

2014-05-06 Thread Kelven Yang
Hi Daan, Could you cherry-pick following fix in 4.4-forward to 4.4 branch. commit be2b5918e6b09a488d75966c49d34bdeb803fdd8 Author: Kelven Yang Date: Mon May 5 15:21:59 2014 -0700 CLOUDSTACK-6586: Move EventBus hookup on job framework to ApiServer to decouple job framework away from

Re: [QUESTION] Debugging CS VMware logic in Eclipse

2014-04-25 Thread Kelven Yang
Mike, Sorry for replying late, I¹ve been on a vacation. Re-routing storage commands to SSVM has been there from very beginning. Having the logic running on management server is mainly for two reasons 1) to bootstrap system VMs 2) To rescue when SSVM is not online. VMware behave under CloudSta

Re: AsyncJobEvents missing from Event Bus in 4.3

2014-04-14 Thread Kelven Yang
I was the original author for AsyncJob management and I wasn¹t aware of that there is a hook-up to EventBus? Kelven On 4/14/14, 1:37 PM, "David Grizzanti" wrote: >Hi All, > >I came across an issue this week that I wanted to see if anyone is aware >of. Until recently we've been running on 4.2 an

Re: OpenSSL vunerability (bleedheart)

2014-04-09 Thread Kelven Yang
What is the process name of that daemon in CPVM? I remember that we only have SSH and HTTPS port open in console proxy, and the later one is running Java based SSL engine. Kelven On 4/9/14, 1:38 PM, "John Kinsella" wrote: >CPVM runs a monit daemon which is at least linked to libssl. I haven¹t >

Re: [QUESTION] Debugging CS VMware logic in Eclipse

2014-04-08 Thread Kelven Yang
akpoints inside VmwareResource >and VmwareStorageProcessor and was able to debug that way. > >As you say, that code was running inside the CS MS. > >When would this code run inside of SSVM instead of CS MS? I'm unclear >about >that. > >Thanks for clarifying! >Mike > &

Re: 4.3.0 vmware

2014-04-08 Thread Kelven Yang
Yes, if you are using mvn to launch management server. Kelven On 4/8/14, 10:19 AM, "Michael Phillips" wrote: >So your referring to actually launching the cloudstack-management >services? > >> From: kelven.y...@citrix.com >> To: dev@cloudstack.apache.org >> Subject: Re: 4.3.0 vmware >> Date: Tue

Re: 4.3.0 vmware

2014-04-08 Thread Kelven Yang
mvn install command only runs command to build and install the artifacts into maven repo. If you launch management server using maven, you need to -Dnoredist option as you do in build. Kelven On 4/8/14, 9:46 AM, "Michael Phillips" wrote: >I did just "mvn install -Dnoredist" >That should work c

Re: Remove Mgmt Server

2014-04-08 Thread Kelven Yang
Just by stopping the running management server would do it already. no further action is mandatory, however, if you prefer a complete clean, login into DB and remove the corresponding record in mshost table Kelven On 4/5/14, 10:32 AM, "Michael Phillips" wrote: >Hey dev guys, what's the proper w

Re: 4.3.0 vmware

2014-04-08 Thread Kelven Yang
If you start from maven command line. you need to define noredist as well, as following example mvn -Dnoredist -pl client jetty:run Kelven On 4/8/14, 9:30 AM, "Michael Phillips" wrote: >All, has anything changed with the procedures to build the noredist code >for 4.3.0? >The reason I ask is be

Re: [QUESTION] Debugging CS VMware logic in Eclipse

2014-04-07 Thread Kelven Yang
If it is only for debugging purpose, you can temporarily make code changes to re-route command to the VMware resource that¹s running inside management server. That will be more convenient than debugging inside SSVM Kelven On 4/6/14, 11:19 PM, "Mike Tutkowski" wrote: >Sorry - perhaps I should ha

Re: [QUESTION] VMware ServerResource

2014-03-31 Thread Kelven Yang
owski" wrote: >Interesting...I can look into that. Do you know off hand if we already >have >such a call to perform an unmount? > >Thanks, Kelven! > > >On Mon, Mar 31, 2014 at 3:28 PM, Kelven Yang >wrote: > >> >> On 3/31/14, 1:54 PM, "Mike Tutkowski"

Re: role of HA-Worker

2014-03-31 Thread Kelven Yang
HA-Workers are pool of threads to execute "HA" related tasks, I quoted "HA" is that not all these tasks are purely issued from HA, since some operations like putting a host into maintenance mode also use the facility. ha.workers configures the size of the thread pool. If you set it to 0, you will

Re: [QUESTION] Unsupported operation on VMware?

2014-03-31 Thread Kelven Yang
On 3/25/14, 1:58 PM, "Mike Tutkowski" wrote: >Hi, > >I've noticed the following exception on VMware on 4.4 (when trying to >create a VM from a template and when trying to create a template from a >volume): > >Unsupported command >issued:org.apache.cloudstack.storage.command.CopyCommand. Are you

Re: [QUESTION] VMware ServerResource

2014-03-31 Thread Kelven Yang
make sure the datastore is unmounted from all hosts. > >Thanks! >Mike > > >On Mon, Mar 31, 2014 at 2:45 PM, Kelven Yang >wrote: > >> >> >> On 3/29/14, 7:31 PM, "Sateesh Chodapuneedi" >> wrote: >> >> >> -

Re: [QUESTION] VMware ServerResource

2014-03-31 Thread Kelven Yang
On 3/29/14, 7:31 PM, "Sateesh Chodapuneedi" wrote: >> -Original Message- >> From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com] >> Sent: 30 March 2014 00:06 >> To: dev@cloudstack.apache.org >> Subject: [QUESTION] VMware ServerResource >> >> Hi, >> >> Quick question: >> >> For

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Kelven Yang
nc that then master, since it >> has been cut already. >> >> >> On Fri, Mar 14, 2014 at 4:00 PM, Kelven Yang >>wrote: >>> Marcus, >>> >>> I¹ve pushed the fix to master already. You probably need to sync your >>> local branch w

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Kelven Yang
Marcus, I¹ve pushed the fix to master already. You probably need to sync your local branch with master Kelven On 3/14/14, 11:08 AM, "Marcus" wrote: >It's in branch resize-root > >On Fri, Mar 14, 2014 at 10:47 AM, Min Chen wrote: >> Marcus, >> >> What is the latest commit you have pick

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Kelven Yang
That¹s my bad. I cherry-picked a fix after IAM¹s merge and this has broken it. The problem didn¹t show up in my local run. I¹m working on a fix of it. Kelven On 3/14/14, 9:47 AM, "Min Chen" wrote: >Marcus, > > What is the latest commit you have picked up on your local setup from >master?

Re: Master is broken?

2014-03-13 Thread Kelven Yang
otentially affected? > >Thanks, >Alena. > >On 3/13/14, 4:10 PM, "Kelven Yang" wrote: > >>Anyone encounter this problem on master? >> >>WARN [c.c.a.d.ParamGenericValidationWorker] >>(163545706@qtp-538020828-6:ctx-afe77737 ctx-06c20804) Received unk

Master is broken?

2014-03-13 Thread Kelven Yang
Anyone encounter this problem on master? WARN [c.c.a.d.ParamGenericValidationWorker] (163545706@qtp-538020828-6:ctx-afe77737 ctx-06c20804) Received unknown parameters for command listRegions. Unknown parameters : listall WARN [c.c.a.d.ParamGenericValidationWorker] (1770041501@qtp-538020828-3:

Re: [DISCUSS] realhostip.com going away

2014-03-06 Thread Kelven Yang
On 3/2/14, 8:15 AM, "Paul Angus" wrote: >There are a few issues with the current console proxy setup, not least of >which is the need to have internet access to resolve realhostip.com in >the first place - so console proxy can't work if you don't have internet >access on your client. I have co

Re: Exception adding XenServer host to CS

2014-02-28 Thread Kelven Yang
I was also trying master this morning and ran into the same problem Kelven On 2/28/14, 10:19 AM, "Mike Tutkowski" wrote: >Just curious if anyone else has been able to successfully add a XenServer >host to a newly created cloud on master recently? > > >On Thu, Feb 27, 2014 at 10:58 PM, Mike Tutk

Re: Anti-patterns

2014-02-25 Thread Kelven Yang
Having equals() and hashCode() at the same time and in consistent is important when objects are used as keys in collections. I remember that we ran into issues in CloudStack a long time ago when some of these basic rules were broken Kelven On 2/25/14, 3:13 PM, "Mike Tutkowski" wrote: >This is a

Re: [PROPOSAL] Windowsfication Of ACS

2014-02-25 Thread Kelven Yang
Current way of mounting NFS into management server is the legacy of the rushing old days when VMware support was originally built. To get rid of NFS mount in management server, we should file it as a feature request along with the Windowsfication effort Kelven On 2/25/14, 1:54 PM, "Alex Huang" w

Re: Anti-patterns

2014-02-25 Thread Kelven Yang
http://passanyinterview.com/top-101-core-java-interview-questions/ Kelven On 2/25/14, 2:07 PM, "Marcus" wrote: >Also keep in mind that with an Object or custom objects that extend it, >doing > >if (obj.equals(thing)) > >is usually the same as: > >if (obj == thing) > >Since the Object.e

Re: VmWare SDK to vijava

2014-02-18 Thread Kelven Yang
is not true any more. The conclusion is that we can use wsimport tool from JDK to directly generate vim25.jar from WSDL and use it. Kelven > >On 2/18/14 2:42 PM, "Kelven Yang" wrote: > >>The reason why it ended up at noredist build is that the binary jars are >&

Re: VmWare SDK to vijava

2014-02-18 Thread Kelven Yang
sources like >the vijava project has done? > >On 2/18/14 2:42 PM, "Kelven Yang" wrote: > >>The reason why it ended up at noredist build is that the binary jars are >>copied from VMware SDK, we are not sure about the license implication and >>we don’t

Re: VmWare SDK to vijava

2014-02-18 Thread Kelven Yang
to WSDL >>>>>>> 3. ? >>>>>>> >>>>>>> -- >>>>>>> Chiradeep >>>>>>> >>>>>>> On 2/18/14 7:14 AM, "Hugo Trippaers" wrote: >>>>>>> >>>>

Re: Callback pattern

2014-01-29 Thread Kelven Yang
; >> I changed it to this in my local repo: >> >> public class AsyncCallbackDispatcher >>implementsAsyncCompletionCallback >> >> That makes sense to me. Do you see any issue with that change? >> >> Thanks again! >> >> >> On Tue,

Re: Callback pattern

2014-01-28 Thread Kelven Yang
a bit of over-engineering going on here? If >they are always going to be done synchronously, we could just call the >"callback" method after calling the "async" method. > >Thoughts on this? > >Thanks :) > > >On Tue, Jan 28, 2014 at 3:42 PM, Kelven

Re: Callback pattern

2014-01-28 Thread Kelven Yang
hacking tricks can go away Kelven On 1/28/14, 2:29 PM, "Mike Tutkowski" wrote: >I see...just for chaining purposes. In some places I notice we chain this >and in others we don't. > >Thanks for the clarification! > > >On Tue, Jan 28, 2014 at 3:13 PM, Kelven Y

Re: Callback pattern

2014-01-28 Thread Kelven Yang
;this"; > >Wouldn't caller.getTarget().createVolumeFromBaseImageCallBack(null, null); >by itself work exactly the same way? > > >On Tue, Jan 28, 2014 at 11:57 AM, Kelven Yang >wrote: > >> Mike, >> >> This is a very dirty hack that I personally h

Re: Callback pattern

2014-01-28 Thread Kelven Yang
Mike, This is a very dirty hack that I personally hate it. This is the hack to utilize Eclipse¹s (or other smart IDE) to do auto-completion for you to find the right callback method. if you write caller.getTarget().createVolumeFromBaseImageCallBack(null, null); Java will interprete it as a

Re: [ISSUES] Master VMware is broken

2014-01-23 Thread Kelven Yang
(methodName, (Class[])null); and Method getMorMethod = dpCls.getDeclaredMethod(methodName, null); will be translated by compiler as Method getMorMethod = dpCls.getDeclaredMethod(methodName, null); Kelven On 1/23/14, 11:54 AM, "Kelven Yang" wrote: >Hugo, > >Th

Re: [ISSUES] Master VMware is broken

2014-01-23 Thread Kelven Yang
d you encounter the error? I wonder why i didn¹t catch it with my >>tests against vmware. >> >>Cheers, >> >>Hugo >> >>On 23 jan. 2014, at 03:01, Min Chen wrote: >> >>> FYI, I checked in a fix 206c35c620a8e7a707f371e5a9e5dfd795912f5b for >&

[ISSUES] Master VMware is broken

2014-01-22 Thread Kelven Yang
Master VMware was broken by this commit author Hugo Trippaers

Re: VmWare SDK to vijava

2014-01-21 Thread Kelven Yang
Q. Can I redistribute the VI SDK libraries and sample code? A. You can redistribute only those parts of the SDK package that have been designated as ³distributable code². In VI SDK 2.5, the following components can be redistributed: vim.jar, vim25.jar. To note developers typically generate web serv

Re: VmWare SDK to vijava

2014-01-21 Thread Kelven Yang
Hugo, I think we seems to come to a census that to use WSDL to generate the java stub which is pretty much compatible to what we have right now. Kelven On 1/21/14, 12:40 AM, "Hugo Trippaers" wrote: >Heya, > >Does anyone know about the current status the legal discussions around >including the

Re: [Proposal] Switch to Java 7

2014-01-07 Thread Kelven Yang
14.04 (due in April) will all have >Java 7 available. > >I don't see a problem in switching to Java 7 with CloudStack 4.4 or 4.5 > >Wido > >On 01/07/2014 12:18 AM, Kelven Yang wrote: >> Java 7 has been around for some time now. I strongly suggest CloudStack >>

Re: VMware snapshot question

2014-01-07 Thread Kelven Yang
;>> The value contains the folder the VMDK file is in. Since the path >>>>>>> column for VMware volumes in the DB doesn't contain the folder the >>>>>>>VMDK >>>>>>> file is in, I think this may be incorrect, as well. >>

Re: VMware snapshot question

2014-01-06 Thread Kelven Yang
On 1/6/14, 5:33 PM, "Mike Tutkowski" wrote: >Actually, the more I look at this code, the more I think perhaps VMware >snapshots are broken because the newPath field should probably not be >assigned null after creating a new VMware snapshot >(I'm thinking the intend >is to replace the other pa

Re: [Proposal] Switch to Java 7

2014-01-06 Thread Kelven Yang
es, there was another discussion here: >http://markmail.org/thread/uf6bxab6u4z4fmrp > > > >On 1/6/14 3:18 PM, "Kelven Yang" wrote: > >>Java 7 has been around for some time now. I strongly suggest CloudStack >>to adopt Java 7 as early as possible, the reason

[Proposal] Switch to Java 7

2014-01-06 Thread Kelven Yang
Java 7 has been around for some time now. I strongly suggest CloudStack to adopt Java 7 as early as possible, the reason I feel like to raise the issue is from the some of practicing with the new DB transaction pattern, as following example shows. The new Transaction pattern uses anonymous clas

[RESOLVED] ACS Git repo problem

2013-12-31 Thread Kelven Yang
The problem is resolved. ACS Git is now fine with push Kelven On 12/31/13, 11:20 AM, "Kelven Yang" wrote: >I can¹t push changes to ACS repo, does anyone else have similar problem? > >Kelven

Re: VMware Question

2013-12-31 Thread Kelven Yang
We need a more intelligent way to detect real ³local storage² on the host. The logic was intended for auto-discovery of attached local storage(s) on the ESXi host. Kelven On 12/27/13, 5:23 PM, "Mike Tutkowski" wrote: >Investigating this a bit more today, I came to the conclusion that the NFS >

[ISSUE] ACS Git repo problem

2013-12-31 Thread Kelven Yang
I can’t push changes to ACS repo, does anyone else have similar problem? Kelven

Re: 4.3 vmware automation blocker

2013-12-09 Thread Kelven Yang
Oops, not the same problem. 5408 is out of system memory, different than the one I mentioned about Min¹s fix which is about JVM heap Kelven On 12/9/13, 2:53 PM, "Kelven Yang" wrote: >This issue is caused by trying to solve a mysterious JVM incompatibility >issue in a wrong

Re: 4.3 vmware automation blocker

2013-12-09 Thread Kelven Yang
This issue is caused by trying to solve a mysterious JVM incompatibility issue in a wrong trying in commit ed0fbcc81c1928062054190ffcfab8bb59969cc2. Min has already reverted the commit and this should have fixed the memory issue. Kelven On 12/7/13, 5:07 PM, "Sateesh Chodapuneedi" wrote: >>

Re: persistence layer

2013-12-03 Thread Kelven Yang
On 12/2/13, 5:00 PM, "Frank Zhang" wrote: >It's really long > >I don't know if it's impossible to move us to JPA but I do agree that >whatever >path we choose it's lot of work to do. > >Given you have tried JOOQ and Lukas's explanation, my only concern is if >we can leverage Spring TX with

Re: persistence layer

2013-12-02 Thread Kelven Yang
I generally in agree with what Darren has pointed. Within the context of ACS, declarative transaction support is not as urgent as solving the problem as we are having in writing queries. Solving basic CRUD and query problem alone in a consistent pattern can move CloudStack persistence layer a big s

Re: Bug? Should we allow detaching volumes when VMs have snapshots

2013-11-06 Thread Kelven Yang
We should dis-allow detaching volume from VM when it has VM snapshots. VMware keeps VM snapshot meta information at VM basis, disk membership change behind its back will cause problem and it does not provide official API to manipulate at volume level for a VM snapshot. This is a bug Kelven On 11

Re: Not able to view the console of the VM. Getting Timeout error

2013-11-05 Thread Kelven Yang
This is the flow, Local browser -> Management Server -> token to Console Proxy VM -> client redirection with token -> Console Proxy VM -> Management Server for authentication -> Console Proxy to create session -> Browser To trouble shooting the problem 1) check assigned public IP address of con

Re: LXC and SSVM/CPVM on the host

2013-10-29 Thread Kelven Yang
che.tomcat.util.net.NioEndpoint$SocketProcessor.run(NioEndpoint.jav >a:2268) > at >java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java: >1146) > at >java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java >:615) > at java.

Re: ifconfig and MacAddress

2013-10-29 Thread Kelven Yang
t 8:43 AM To: Laszlo Hornyak mailto:laszlo.horn...@gmail.com>>, "dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>" mailto:dev@cloudstack.apache.org>> Cc: Kelven Yang mailto:kelven.y...@citrix.com>> Subject: RE: ifconfig and MacAddress I can confirm t

Re: LXC and SSVM/CPVM on the host

2013-10-28 Thread Kelven Yang
As long as the container VNC console is reachable from console proxy agent, I don't see a technical issue here. Need some log details to understand the situation. For example, what information does LXC hypervisor resource agent returns on GetVncPort command? Kelven On 10/28/13 10:59 AM, "Franco

Re: ifconfig and MacAddress

2013-10-28 Thread Kelven Yang
In a setup with multiple interfaces or Multiple IPs on one interfaces, it sounds more simple to just get the decision resolved through configuration. Looking for interfaces that has IP configured as what "cluster.node.IP" says so, can get us a deterministic mac address. I'm not sure about a KVM ca

Re: [VOTE] Accept the donation of RDP client code into Apache CloudStack

2013-10-22 Thread Kelven Yang
+1 On 10/21/13 11:11 AM, "Donal Lafferty" wrote: >As stated in a previous thread [1], Citrix is proposing the donation of >source for an RDP client. After donation, the client would be integrated >with the console system VM in order to provide access to Hyper-V based >VMs. > >The client's sourc

Re: [DISCUSS] Checked Exceptions

2013-10-14 Thread Kelven Yang
On 10/14/13 10:29 AM, "Darren Shepherd" wrote: >I'm going through and updating all the code to use what I had proposed >with Transaction management. This has uncovered more fun issues >around exception handling. Well, not an issue per say, but just >pointed out the ridiculous amount of checke

Re: [DISCUSS] Transaction Hell

2013-10-09 Thread Kelven Yang
+1 Original Transaction class also has many tightly-coupled assumptions about the underlying data source, lock master. Developers are usually lost on when and where they should use @DB, for nested transactions, it does not really work as expected. Kelven On 10/9/13 10:38 AM, "Chiradeep Vittal"

Re: [MERGE] spring-modularization to master - Spring Modularization

2013-10-09 Thread Kelven Yang
+1 Kelven On 10/9/13 9:18 AM, "Chip Childers" wrote: >+1 from me too. > > >On Wed, Oct 9, 2013 at 12:13 PM, Prasanna Santhanam >wrote: > >> Here's the BVT result from spring-modularization branch: Failures that >> are listed exist on master as well. So the branch doesn't break >> anything add

Re: Latest Master DB issue

2013-10-08 Thread Kelven Yang
The problem seems to me is whether or not a background job that touches with database respects the bootstrap initialization order. As of VmwareContextPool itself, its background job does something fully within its own territory (no database, no reference outside). and vmware-base package was origin

Re: [DISCUSS] Leaky abstractions [was review requests 13238, 13896, 14320]

2013-10-03 Thread Kelven Yang
*_details table is currently used as the generic way of passing key/value pairs that are transparent to API layer and core CloudStack orchestration flow. When I say transparent, the scope is only limited to API layer and core generic orchestration flow, specific plugin elements or Gurus still have

Re: [MERGE] spring-modularization to master - Spring Modularization

2013-10-02 Thread Kelven Yang
correctness over speed right >>now. >> >> Cglib is no longer used for AOP. The only use of cglib now is in >>GenericDaoBase when it creates the VOs and also in the async framework. >> >> I should point out all of these changes apply only to the mgmt server. &g

Re: HA is broken on master

2013-10-02 Thread Kelven Yang
I'm now rebasing the VMsync work to master, will send a merge request once I'm done Kelven On 10/2/13 4:31 PM, "Chiradeep Vittal" wrote: >My bad. I thought this was merged into master, but it isn't. > >On 10/2/13 4:24 PM, "David Nalley" wrote: > >>Why is the work happening in master? >> >> >>O

Re: [MERGE] spring-modularization to master - Spring Modularization

2013-10-02 Thread Kelven Yang
Darren, This looks really nice. A few questions on Spring AOP replacement. 1) Spring AOP is proxy-based, the reason we ended up of using customized AOP is mainly due to that inside existing CloudStack codebase, we have many places that are doing run-time type-casting, the code in these places ass

Re: Review Request 14190: Switch to setter injection for extensibility

2013-09-30 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14190/#review26521 --- Ship it! Ship It! - Kelven Yang On Sept. 18, 2013, 3:49 p.m

Re: Review Request 14218: Add spring lifecycle framework to handle modularization

2013-09-30 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14218/#review26518 --- Ship it! Ship It! - Kelven Yang On Sept. 19, 2013, 9:41 p.m

Re: Review Request 14187: Framework to discover and load spring contexts in a hierarchy

2013-09-30 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14187/#review26516 --- Ship it! Ship It! - Kelven Yang On Sept. 18, 2013, 5:51 p.m

Re: Review Request 14190: Switch to setter injection for extensibility

2013-09-29 Thread Kelven Yang
break; 68 } 69 } 70 } 71 72 return result; 59 } 73 -Kelven - Kelven Yang On Sept. 18, 2013, 3:49 p.m., Darren Shepherd wrote

Re: VmWare SDK to vijava

2013-09-24 Thread Kelven Yang
;> wrapper and we open a discussion thread when both branches are ready to >>> see which we want to merge in master. Anybody who wants to pick that >>>up? >>> >>> I'm stubbornly going to continue with converting to vijava, I put some >>> effort i

Re: VmWare SDK to vijava

2013-09-24 Thread Kelven Yang
On 9/24/13 4:54 PM, "Alex Huang" wrote: > > >> -Original Message- >> From: Alex Huang [mailto:alex.hu...@citrix.com] >> Sent: Tuesday, September 24, 2013 4:23 PM >> To: dev@cloudstack.apache.org >> Subject: RE: VmWare SDK to vijava >> >> So this discussion took a big turn that I did not

Re: VmWare SDK to vijava

2013-09-24 Thread Kelven Yang
ish between the >two. +1, although I don't want to disappoint developer's motivations. The wsdl approach makes more business sense to me. -kelven > > >Please, let's discuss on this and drive a consensus together rather than >try to accommodate all. > >--Alex > >

Re: VmWare SDK to vijava

2013-09-24 Thread Kelven Yang
about current customers. > > >> -Original Message- >> From: Kelven Yang [mailto:kelven.y...@citrix.com] >> Sent: Tuesday, September 24, 2013 2:06 PM >> To: dev@cloudstack.apache.org >> Subject: Re: VmWare SDK to vijava >> >> It is about the interf

Re: VmWare SDK to vijava

2013-09-24 Thread Kelven Yang
; >> -----Original Message- >> From: Kelven Yang [mailto:kelven.y...@citrix.com] >> Sent: Tuesday, September 24, 2013 11:59 AM >> To: dev@cloudstack.apache.org >> Subject: Re: VmWare SDK to vijava >> >> We have commercial releases on top of existing code

Re: VmWare SDK to vijava

2013-09-24 Thread Kelven Yang
va, I put some >effort into it and i want at least to see it running once ;-) And the >more i work with it the more i'm seeing to benefits of the library so i >might be able to be more convincing in the end :-) > >Cheers, > >Hugo > > >On Sep 24, 2013, at 2:18 AM,

Re: [PROPOSAL] Modularize Spring

2013-09-23 Thread Kelven Yang
getRunLevel() order for the beans in the >context. > >Darren > > >On Mon, Sep 23, 2013 at 11:34 AM, Kelven Yang >wrote: >> Darren, >> >> Due to internal release work, I haven't got a chance to review it but >>I'm >> planning to do so lat

Re: [PROPOSAL] Modularize Spring

2013-09-23 Thread Kelven Yang
Darren, Due to internal release work, I haven't got a chance to review it but I'm planning to do so later today and tomorrow. Before that, I have one question about hierarchy-orginzated context structure, could you elaborate an example to the ML on how two sibling plugins to declare their runtime

Re: VmWare SDK to vijava

2013-09-23 Thread Kelven Yang
Prior to 5.1, VMware provides java binding in its SDK and this is where CloudStack VMware integration began with. Starting from 5.1, VMware no longer provides the java binding in binary form and recommends its customers to generate directly from its WS WSDL. Since we are not sure if we can distrib

Re: [PROPOSAL] Storage Subsystem API Interface Additions

2013-09-19 Thread Kelven Yang
nefit of doing so may not sound as valuable as it is. -Kelven > >-- >Chris Suich >chris.su...@netapp.com >NetApp Software Engineer >Data Center Platforms ­ Cloud Solutions >Citrix, Cisco & Red Hat > >On Sep 19, 2013, at 1:00 AM, Kelven Yang wrote: > >

Re: [PROPOSAL] Storage Subsystem API Interface Additions

2013-09-18 Thread Kelven Yang
On 9/18/13 9:10 AM, "Darren Shepherd" wrote: >Here's my general concern about multiple volume snapshots at once. Giving >such a feature leads the user to believe that snapshotting multiple >volumes >at once will give them consistency across the volumes in the snapshot. >This is not true, and d

Re: Review Request 14189: UserVmDomRInvestigator and ManagementIPSystemVMInvestigator mask adapter name

2013-09-18 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14189/#review26250 --- Ship it! Ship It! - Kelven Yang On Sept. 18, 2013, midnight

Re: Review Request 14186: Create parent pom to allow maven standard paths in project

2013-09-18 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14186/#review26249 --- Ship it! Ship It! - Kelven Yang On Sept. 17, 2013, 11:08 p.m

Re: Master nonoss is broken?

2013-09-18 Thread Kelven Yang
>commit 2614b00c513734ce6b1c29e572fbd7a37d4059fc >Author: Daan Hoogland >mailto:dhoogl...@schubergphilis.com>> >Date: Thu Aug 1 16:25:27 2013 +0200 > >sdn hosted vpc gateways (using lswitch) > > > >On Wed, Sep 18, 2013 at 4:19 PM, Alena Prokharchyk < >a

Re: conflicting dependencies between CloudStack and Whirr

2013-09-18 Thread Kelven Yang
I almost faced that there is no choice but to go for it for VMware recently, we found that VMware vSphere 5.1 SDK we are currently using has some backwards compatibility issue with VMware vSphere 4.x systems. Therefore it makes a valid business case that CloudStack may have to support different ver

Master nonoss is broken?

2013-09-18 Thread Kelven Yang
In JuniperSrxResource.java, someone to look at it? Kelven

Re: Error in start MS in master

2013-09-16 Thread Kelven Yang
You need to use command line like mvn -Dnonoss -pl client jetty:run Kelven On 9/13/13 1:56 PM, "Min Chen" wrote: >Hi there, > >Did anybody experience any issues in starting MS in latest master? I ran >into the following issue: > >org.springframework.beans.factory.BeanCreationException: Could n

Re: Review Request 14084: Updates to @ActionEvent to be compatible with Spring AOP

2013-09-13 Thread Kelven Yang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14084/#review26086 --- Ship it! - Kelven Yang On Sept. 11, 2013, 5:07 p.m., Darren

Re: Review Request 14084: Updates to @ActionEvent to be compatible with Spring AOP

2013-09-13 Thread Kelven Yang
annotated in various places, is there any test done for the business logic validation? - Kelven Yang On Sept. 11, 2013, 5:07 p.m., Darren Shepherd wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: [Responsiveness report] users 2013w36

2013-09-09 Thread Kelven Yang
4.1.1 is actually using SSL tunneling port 443 opened at XS host to connect to VNC terminal. Changing VNC configuration in XS host does not help to resolve the console connectivity issue. Kelven On 9/9/13 3:14 AM, "Daan Hoogland" wrote: >http://markmail.org/message/5pfd3por7uvhntgh Change vncte

Re: what do you do if your mshost mac changes?

2013-09-09 Thread Kelven Yang
On 9/9/13 12:07 PM, "Darren Shepherd" wrote: >I just ran into this. For whatever reason my box listed eth2 before >eth0 for "ifconfig -a" so it looked like management server mac changed. > Then nothing worked... It is supposed to work if you are sure a previous running MS java process is kil

  1   2   3   >