RE: what's the status of Bare Metal component?

2015-01-13 Thread Frank Zhang
; Lucian > > -- > Sent from the Delta quadrant using Borg technology! > > Nux! > www.nux.ro > > - Original Message - > > From: "Frank Zhang" > > To: "" > > Sent: Tuesday, 13 January, 2015 19:29:56 > > Subject: RE: what's

RE: what's the status of Bare Metal component?

2015-01-13 Thread Frank Zhang
CloudStack offers baremetal for basic zone (see https://cwiki.apache.org/confluence/display/CLOUDSTACK/Baremetal+KickStart) and advanced zone (see https://cwiki.apache.org/confluence/display/CLOUDSTACK/Baremetal+Advanced+Networking+Support). Basic zone support has been there for years, advanced

RE: Removes the use local storage for systemvms

2014-12-18 Thread Frank Zhang
Alright, so it won't affect customer From: Rohit Yadav [mailto:bhais...@apache.org] Sent: Thursday, December 18, 2014 12:17 AM To: Frank Zhang Cc: dev@cloudstack.apache.org Subject: Re: Removes the use local storage for systemvms Hi Frank, On Thu, Dec 18, 2014 at 4:02 AM, Frank

Removes the use local storage for systemvms

2014-12-17 Thread Frank Zhang
Hi Rohit: I see your commit: commit d276c0faab0ac4d200714d8eb2081ed1c47f13aa Author: Rohit Yadav Date: Mon Dec 8 21:54:31 2014 +0530 developer: increase disable thresholds global settings for developers - Increases the disable thresholds for developers - R

RE: certificate tests

2014-11-13 Thread Frank Zhang
Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Thursday, November 13, 2014 6:49 AM > To: Frank Zhang > Cc: dev > Subject: certificate tests > > Frank > > 380ecf2253dd03905df25f30422b67ec1b17ff19 > > fix expried certificate in

Re: Review Request 26361: CLOUDSTACK-7660: Enhance system vm template to support baremetal

2014-10-29 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26361/#review59058 --- Ship it! Ship It! - Frank Zhang On Oct. 21, 2014, 1:02 p.m

RE: [HELP] cloudstack 4.4.0 src build failed

2014-10-24 Thread Frank Zhang
As you are building from source tarball, the simplest way to fix is to delete CertServiceTest.java > -Original Message- > From: Ian Duffy [mailto:i...@ianduffy.ie] > Sent: Friday, October 24, 2014 9:54 AM > To: CloudStack Dev > Subject: Re: [HELP] cloudstack 4.4.0 src build failed > > Th

RE: Certificate expired

2014-10-21 Thread Frank Zhang
tutkow...@solidfire.com] > Sent: Tuesday, October 21, 2014 3:39 PM > To: dev@cloudstack.apache.org > Subject: Re: Certificate expired > > I see - thanks. > > On Tue, Oct 21, 2014 at 4:36 PM, Frank Zhang > wrote: > > > I made a new certificate, but there are lots of u

RE: Certificate expired

2014-10-21 Thread Frank Zhang
vn -P developer,systemvm clean install > > But it failed on the same test. > > On Tue, Oct 21, 2014 at 4:17 PM, Frank Zhang > wrote: > > > What did you update? You replace the file with you manually built > > certificate? > > > > > -Original Messa

RE: Certificate expired

2014-10-21 Thread Frank Zhang
gt; > Interesting...I just updated master and tried to re-build, but that test > failed > again. > > On Tue, Oct 21, 2014 at 4:12 PM, Frank Zhang > wrote: > > > I met the error too. > > It's caused by file ./server/test/resources/certs/rsa_self_signed.crt >

RE: Certificate expired

2014-10-21 Thread Frank Zhang
I met the error too. It's caused by file ./server/test/resources/certs/rsa_self_signed.crt which is expired today. I thought we need to replace it with a new one commit 0076307863e9155273d9e4c14282de429388c9e9 Author: Syed Date: Wed Nov 6 15:08:42 2013 -0500 Squashed merge of Ssl Terminat

RE: master broken

2014-10-16 Thread Frank Zhang
Do you know what part of baremetal causes the issue? I built master on Wed and didn't see any issue. I will give it a try right now. > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent: Thursday, October 16, 2014 2:28 PM > To: dev; Frank Zh

Re: Review Request 25266: Simulator build support need to extends for RPM build

2014-10-08 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25266/#review55820 --- Ship it! Ship It! - Frank Zhang On Sept. 19, 2014, 7:17 p.m

RE: baremetal in master and 4.4

2014-09-23 Thread Frank Zhang
Thanks Daan. When I wrote this schema, schema-430to440.sql was the latest one I could find. I have moved it to schema-441to450.sql From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Tuesday, September 23, 2014 2:00 AM To: Frank Zhang Cc: dev Subject: baremetal in master and 4.4 Frank

RE: Jenkins build is back to normal : build-master-noredist #3562

2014-09-19 Thread Frank Zhang
Thanks Hugo > -Original Message- > From: jenk...@cloudstack.org [mailto:jenk...@cloudstack.org] > Sent: Thursday, September 18, 2014 3:30 AM > To: dev@cloudstack.apache.org; bhais...@apache.org > Subject: Jenkins build is back to normal : build-master-noredist #3562 > > See

Re: Review Request 25797: CLOUDSTACK-7522 : RPM build need to exit with return code 1, if there are failure in build

2014-09-18 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25797/#review53888 --- Ship it! Ship It! - Frank Zhang On Sept. 18, 2014, 10:22 p.m

Re: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-04 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25289/#review52323 --- Ship it! Ship It! - Frank Zhang On Sept. 3, 2014, 6:25 a.m

Re: Review Request 25289: CLOUDSTACK-7474-Failed-to-start-MS-with-java7-version

2014-09-03 Thread Frank Zhang
---- > > (Updated Sept. 3, 2014, 6:25 a.m.) > > > Review request for cloudstack, Frank Zhang and Hugo Trippaers. > > > Bugs: CLOUDSTACK-7474 > https://issues.apache.org/jira/browse/CLOUDSTACK-7474 > > > Repository: cl

RE: Build failed in Jenkins: cloudstack-rat-master #4744

2014-04-23 Thread Frank Zhang
mailto:jenk...@builds.apache.org] > Sent: Wednesday, April 23, 2014 10:54 AM > To: cloudstack-comm...@incubator.apache.org; mike.tutkow...@solidfire.com; > gir...@clogeny.com; Frank Zhang; Alena Prokharchyk; run...@gmail.com; > tall...@apache.org; Min Chen > Subject: Build failed in Jenk

Re: Review Request 20445: Add new Command line options to setup/bindir/cloud-setup-databases.in and remove OS specific commands

2014-04-23 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20445/#review41164 --- Ship it! Ship It! - Frank Zhang On April 17, 2014, 9:41 a.m

Re: Review Request 20445: Add new Command line options to setup/bindir/cloud-setup-databases.in and remove OS specific commands

2014-04-18 Thread Frank Zhang
they don't, by default the script should automatically find ip of nic for default route. Yes the old code doesn't handle this, as you are improving it we should take this chance to make it further. 2. I see there are some new options, but it seems they are not used in code? - Frank

Re: Review Request 20257: CLOUDSTACK-6390 : System vm failed to build with openssl installation error,

2014-04-11 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20257/#review40172 --- Ship it! Ship It! - Frank Zhang On April 11, 2014, 2:39 p.m

Re: Review Request 20136: mysql-connector should be added class path, while starting usage server

2014-04-08 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20136/#review39834 --- Ship it! Ship It! - Frank Zhang On April 8, 2014, 10:12 p.m

Re: Review Request 20132: Usage server should start with java 1.7 in 4.4

2014-04-08 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20132/#review39835 --- Ship it! Ship It! - Frank Zhang On April 8, 2014, 10:05 p.m

RE: Maven style checker issue

2014-04-08 Thread Frank Zhang
; > I am using it and haven't seen any issues so far. > > > ~Rajani > > > > On 08-Apr-2014, at 5:12 am, Frank Zhang > mailto:frank.zh...@citrix.com>> wrote: > > I struggled two days to fix my several tens of files for trailing space, tab, > u

FW: Maven style checker issue

2014-04-07 Thread Frank Zhang
-Original Message- From: Frank Zhang [mailto:frank.zh...@citrix.com] Sent: Monday, April 07, 2014 4:43 PM To: cloudstack-...@incubator.apache.org Subject: Maven style checker issue I struggled two days to fix my several tens of files for trailing space, tab, unused imports... manually

Maven style checker issue

2014-04-07 Thread Frank Zhang
I struggled two days to fix my several tens of files for trailing space, tab, unused imports... manually(vim + sed). Yes, manually because I am using Intellj the ApacheCloudStack formatter profile won't work for me, even though Intellj 13 claims supporting Eclipse formatter profile, it doesn't

which Juniper plugin is for L2 switch programming?

2014-03-17 Thread Frank Zhang
Hi, I am reading functional spec of "Cloudstack network-element plugin to orchestrate Juniper's switches (for L2 services) " (https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=36274178). It says on branch 4.4. However, I only see juniper-contrail/juniper-srx which both seem

[Merge]Please cherry pick below commits from 4.3 to 4.3-forward

2014-02-25 Thread Frank Zhang
commit 20efa0c8a6b83c6590eab48fefac81b72b2082a9 Author: Frank.Zhang Date: Mon Feb 24 13:30:24 2014 -0800 CLOUDSTACK-6155 Baremetal agent RPM missing in spec file commit c1806bd873fe661756aeee94b25339c25f9724a5 Author: Frank.Zhang Date: Fri Feb 21 13:22:55 2014 -0800 CLOUDSTACK-

RE: Build failed in Jenkins: build-master-noredist #2088

2014-01-10 Thread Frank Zhang
Hi Hugo: I notice you upgrade Jenkin recently, now below error happens very often. It looks like either Jenkin's git plugin has some problem or our git repo has instable network. Do you have any idea? > -Original Message- > From: jenk...@cloudstack.org [mailto:jen

Re: Review Request 16774: Fix for defect CLOUDSTACK-5053; KVM QEMU need to be present in agent hosts

2014-01-10 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16774/#review31546 --- Ship it! ship in master/4.3 - Frank Zhang On Jan. 9, 2014, 11

RE: 4.3 : Developer Profile, tools module commented

2014-01-07 Thread Frank Zhang
> 3. There was a reference to speed up build time @ > http://markmail.org/message/3ch4lgaq4yjaop7e?q=darren+shepherd+mvn > > Regards, > Santhosh > > From: Frank Zhang [frank.zh...@citrix.com] > Sent: Monday, January 06, 2014 6:17 PM &g

RE: Committer license for IntelliJ set to expire

2014-01-07 Thread Frank Zhang
I am > -Original Message- > From: Chip Childers [mailto:chipchild...@apache.org] > Sent: Tuesday, January 07, 2014 7:18 AM > To: dev@cloudstack.apache.org > Subject: Committer license for IntelliJ set to expire > > Hey folks, > > The free license we have for IntelliJ for committers is s

RE: 4.3 : Developer Profile, tools module commented

2014-01-06 Thread Frank Zhang
Sorry this was by a mistake. The maven build costs too long so I comment tools out as it's rarely used by developer build. later on I found there is a profile called 'impatient' which does exactly the same thing. before bring it back, I wonder if it's right to put 'tools' in developer profile? I

RE: [DISCUSS][PROPOSAL] Move docs to .rst format and readthedocs

2013-12-09 Thread Frank Zhang
+1. And suggest using Sphinx with the plugin http://bronto.github.io/javasphinx/ > -Original Message- > From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers > Sent: Monday, December 09, 2013 2:19 PM > To: dev@cloudstack.apache.org > Cc: Radhika Puthiyetath; msweet@gmai

RE: persistence layer

2013-12-05 Thread Frank Zhang
this discussion, I can see that many concerns have > been resolved. So I'll merely answer the open points Frank had. > > 2013/12/3 Frank Zhang > > > Hi Lukas: > > Thank for detailed reply, see my comments inline > > > > > > > > Hi F

why do we have a maven repo on https://repository.apache.org/content/groups/snapshots/org/apache/cloudstack/

2013-12-04 Thread Frank Zhang
[INFO] Total time: 1:11.962s [INFO] Finished at: Wed Dec 04 14:16:26 PST 2013 [INFO] Final Memory: 43M/111M [INFO] [ERROR] Failed to execute goal on project cloud-framework-jobs: Could not resolve dependencies for project or

RE: persistence layer

2013-12-02 Thread Frank Zhang
his fluent API usage: > > > > > create.selectFrom(BOOK) > > > .where(BOOK.PUBLISHED_IN.eq(2011)) > > > .orderBy(BOOK.TITLE) > > > > With this SelectQuery usage: > > > > SelectQuery select = create.selectQuery(BOOK); > > select.addC

RE: persistence layer

2013-12-02 Thread Frank Zhang
/ > > Record mapping in order to provide custom mapping algorithms from jOOQ > records to your domain model: > http://www.jooq.org/doc/3.2/manual/sql-execution/fetching/pojos-with- > recordmapper-provider/ > > A query transformation listener that is still being worked on, allowing

RE: persistence layer

2013-11-27 Thread Frank Zhang
/9234021/521799 > > If you want cross-database support, these things show that you should really > consider moving away from using plain JDBC and use an abstraction that will > take care of these incompatibilities for you. You might be choosing between > Hibernate and jOOQ (

RE: persistence layer

2013-11-22 Thread Frank Zhang
I recommend QueryDSL. Having a quick look at JOOQ, it's very similar to QueryDSL. QueryDSL has been integrated into Spring for 2 years, and JOOQ haven't had official doc for Spring. Besides Sql, QueryDSL also gets fair credit on manipulating NoSql which is an additional plus. > -Original Me

RE: [LXC]How to start System VM

2013-11-18 Thread Frank Zhang
t: Re: [LXC]How to start System VM > > See this discussion: > http://goo.gl/Kj0rNr > > On 11/18/13 2:03 PM, "Frank Zhang" wrote: > > >I have successfully added a LXC host in ACS4.2, however, the system vm > >didn't come up. Is there any specific instruction to make SSVM up? > >Thanks

[LXC]How to start System VM

2013-11-18 Thread Frank Zhang
I have successfully added a LXC host in ACS4.2, however, the system vm didn't come up. Is there any specific instruction to make SSVM up? Thanks

Re: Review Request 14843: CLOUDSTACK-1049 : git commit by using command cloudstack-sccs from installed setup

2013-10-22 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14843/#review27339 --- Ship it! ship to master - Frank Zhang On Oct. 22, 2013, 7:21

RE: [ANNOUNCE] New PMC member: Animesh Chaturvedi

2013-10-21 Thread Frank Zhang
Congrats Animesh! You deserve it! > -Original Message- > From: Prachi Damle [mailto:prachi.da...@citrix.com] > Sent: Monday, October 21, 2013 2:24 PM > To: dev@cloudstack.apache.org > Subject: RE: [ANNOUNCE] New PMC member: Animesh Chaturvedi > > Congrats Animesh! > > -Original Mess

Re: Review Request 14729: Fix UCS HTTPS redirect

2013-10-18 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14729/#review27204 --- Ship it! Ship It! - Frank Zhang On Oct. 17, 2013, 11:53 p.m

RE: Wiki access

2013-10-15 Thread Frank Zhang
Me too. Username: frank.zhang > -Original Message- > From: Parth Jagirdar [mailto:parth.jagir...@citrix.com] > Sent: Tuesday, October 15, 2013 11:58 AM > To: dev@cloudstack.apache.org > Subject: RE: Wiki access > > I need edit access too; > > Username: parth > > Thanks, > .. Parth > >

RE: [ANNOUNCE] A GCE interface to CloudStack

2013-09-26 Thread Frank Zhang
;> I got this running earlier today and was really impressed with it, good > >> job! > >> > >> Our current AWS is just SOAP right? How about when we do the query > >> api we do it as a separate python project just like this. > >> > >> Darren &

RE: [ANNOUNCE] A GCE interface to CloudStack

2013-09-24 Thread Frank Zhang
; > I got this running earlier today and was really impressed with it, good job! > > Our current AWS is just SOAP right? How about when we do the query api we > do it as a separate python project just like this. > > Darren > > > On Sep 24, 2013, at 11:40 AM, Fran

RE: VmWare SDK to vijava

2013-09-24 Thread Frank Zhang
de. I'm > leaning towards for us to to have some control in generating the API stub for > this(the direct WSDL way), if it is to vijava, it depends on whether or not > it has > built-in side-by-side VMware API support. and we will have more things to > worry and test about i

RE: VmWare SDK to vijava

2013-09-24 Thread Frank Zhang
Agree. Given current CloudStack code base changing something to another thing is really not a good way. As Darren is working on modular spring, why not construct a new VMWare plugin separately using vijava? Then we can reduce the risk of surprising existing customer and switch to new module when

RE: [ANNOUNCE] A GCE interface to CloudStack

2013-09-24 Thread Frank Zhang
Take a look at source. Clean and lightweight implementation. Awesome. I hope our AWS someday can go this way someday, a separate web server in separate repo and using some dynamically language which makes smaller code > -Original Message- > From: Musayev, Ilya [mailto:imusa...@webmd.net]

RE: [PROPOSAL] Modularize Spring

2013-09-18 Thread Frank Zhang
le of the code. For example, today > in > ACS the *ManagerImpl classes are a huge mess. They implement too many > interfaces and the code crosses to many architectural boundaries. Its about > time we start splitting things up to be more maintainable. > > If you have som

RE: [PROPOSAL] Modularize Spring

2013-09-18 Thread Frank Zhang
ng as I want today, and then tomorrow we say this > is > way too much overhead, moving to a flat context is simple. Don't think we are > on some slippery slope to classloaders and dependency hell. > > Darren > > [1] > https://cwiki.apache.org/confluence/display/CLOUD

RE: [PROPOSAL] Modularize Spring

2013-09-18 Thread Frank Zhang
What's the point in using separate spring context per plugin? Separate class loader is the thing I hate most in OSGI, I am afraid we are on the same way. Frankly speaking, I never see benefits of this *separate* model, our project(or most projects) is not like Chrome which has to create sandbox

RE: plain text authenticator

2013-09-12 Thread Frank Zhang
Are all authentication plugins loaded by default and working in an authentication chain? Otherwise why should we keep the hash type in DB? > -Original Message- > From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] > Sent: Thursday, September 12, 2013 9:56 AM > To: dev@cloudstack.ap

Re: Review Request 13890: CLOUDSTACK-4509 - /var/cache/cloudstack need to deleted during installation; else it will effect 4.1 to 4.2 upgrade

2013-08-28 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13890/#review25672 --- Ship it! Ship It! - Frank Zhang On Aug. 28, 2013, 6:09 p.m

RE: Introduction

2013-08-21 Thread Frank Zhang
Welcome back! > -Original Message- > From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] > Sent: Wednesday, August 21, 2013 12:17 PM > To: dev@cloudstack.apache.org > Subject: Introduction > > All, > > I want to introduce myself to the Apache CloudStack community. I've have had

Re: Review Request 13664: Fix for rhel 5 upgrade failure defect CLOUDSTACK-4370

2013-08-19 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13664/#review25320 --- Ship it! Ship It! - Frank Zhang On Aug. 19, 2013, 9:19 p.m

RE: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Frank Zhang
I reverted it from 4.2. now is adding the Apache header to master > -Original Message- > From: Animesh Chaturvedi > Sent: Monday, August 12, 2013 5:56 PM > To: dev@cloudstack.apache.org; Frank Zhang; John Burwell > Cc: Laszlo Hornyak > Subject: RE: Review Request

RE: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Frank Zhang
Emm. Actually it's not a blocker or even critical for 4.2. Though I thought it's not harm, if needs I can revert it from 4.2 but still keep it in master. > -Original Message- > From: Animesh Chaturvedi > Sent: Monday, August 12, 2013 5:37 PM > To: dev@cloudstack.a

Re: Review Request 13459: cloudstack-3759: deployVM with state set to be stopped, attach volume to it. The volume should not be created.

2013-08-12 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13459/#review25039 --- please update your test result - Frank Zhang On Aug. 10, 2013

make cloud-daemonize obsolete using cloudstack-agent instead of cloudstack-management

2013-08-12 Thread Frank Zhang
Hi Hugo: I have an issue needing to double confirm with you. https://issues.apache.org/jira/browse/CLOUDSTACK-4149 In short, we used to make old cloud-daemonize obsolete using cloudstack-management that makes it unnecessarily installed during upgrading KVM agent(cloud-agent) on h

Re: Review Request 11942: ProcessUtil cleanup

2013-08-12 Thread Frank Zhang
is known as unnecessary complex, we should take advantage of Apache common-io as much as possible. shipped in 4.2 and master - Frank Zhang On July 24, 2013, 4:42 a.m., Laszlo Hornyak wrote: > > --- > This is an automatically ge

Re: Review Request 13453: Fix for CLOUDSTACK-3858, 2.2.14 does not contions symbolic link to create sever.xml, we need to check both file or symbolic link exist or not in saved folder

2013-08-09 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13453/#review24956 --- Ship it! Ship It! - Frank Zhang On Aug. 9, 2013, 11:09 p.m

RE: Inclusion of commons-httpclient-contrib jar

2013-07-24 Thread Frank Zhang
/24/13 1:22 PM, "Chip Childers" wrote: > > > > >I don't think you need to wait. I actually need the files in the > > >repo to pick up on them anyway for the docs. > > > > > > > > >On Wed, Jul 24, 2013 at 08:13:59PM +, Frank Zha

Re: Review Request 12877: Support HTTP to HTTPS redirect for UCS

2013-07-24 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12877/#review23782 --- Ship it! shipped in both 4.2/master - Frank Zhang On July 23

RE: Inclusion of commons-httpclient-contrib jar

2013-07-24 Thread Frank Zhang
just need to include it in the legal docs. It's compatible, but all > external > code and dependencies need to be accounted for. > > I see that this is from the ASF actually, so adding it is easy. I'll take a > look at > doing it shortly. > > > On Wed,

RE: Inclusion of commons-httpclient-contrib jar

2013-07-24 Thread Frank Zhang
f commons-httpclient-contrib jar > > Nobody has bothered to deal with the LICENSE file for the file described > below. > > Anyone want to take this on, or at least give me the information required? > > > On Wed, Jul 24, 2013 at 05:59:59PM +, Frank Zhang wrote: > >

RE: Inclusion of commons-httpclient-contrib jar

2013-07-24 Thread Frank Zhang
; >> > From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] > > >> > Sent: Tuesday, July 23, 2013 9:41 PM > > >> > To: dev@cloudstack.apache.org > > >> > Subject: Re: Inclusion of commons-httpclient-contrib jar > > >> > > >

RE: Inclusion of commons-httpclient-contrib jar

2013-07-24 Thread Frank Zhang
mplementation, but a copy. > > > > On 7/24/13 2:14 AM, "Amogh Vasekar" wrote: > > > > >In the absence of an officially published jar on maven for > > >contrib-ssl, the current idea is to put this package in cloud-utils > > >(it is a dependency fo

RE: [DESIGN] Why is enum a class...

2013-07-23 Thread Frank Zhang
as no way to guarantee compile time enumeration of all of the values like Enum did so it's possible for errors to be introduced but certainly we can follow a convention in this regard.  Another way would be to use Spring to gather them and inject them into a holding class. --Alex -Original Message

RE: [DESIGN] Why is enum a class...

2013-07-23 Thread Frank Zhang
t; > The first thing I noticed about Frank's comment is that he started it with > "Frankly speaking." (Frank/Frankly) Probably not intended as a pun, but funny > nonetheless. :) > > > On Mon, Jul 22, 2013 at 6:45 PM, Frank Zhang > wrote: > > > F

RE: [DESIGN] Why is enum a class...

2013-07-22 Thread Frank Zhang
Frankly speaking, if we are going to change enum, I would suggest not using enmu anymore, instead, defining our own class like: public class VmType { private static Map types = Collections.synchronizedMap(new HashMap()); private final String typeName; public VmType (Stri

RE: Inclusion of commons-httpclient-contrib jar

2013-07-22 Thread Frank Zhang
I prefer to just use httpclient-contribs.jar as it has been proven by lots of users. Reimplementing such code brings no gain but maintenance effort. > -Original Message- > From: Amogh Vasekar [mailto:amogh.vase...@citrix.com] > Sent: Monday, July 22, 2013 5:17 PM > To: dev@cloudstack.apa

Re: Review Request 12782: Add encryption for Ucs password

2013-07-22 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12782/#review23661 --- Ship it! shipped in both master/4.2 - Frank Zhang On July 19

Re: Review Request 12782: Add encryption for Ucs password

2013-07-22 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12782/#review23660 --- Ship it! Ship It! - Frank Zhang On July 19, 2013, 11:28 p.m

RE: code formatting for enums

2013-07-19 Thread Frank Zhang
Yes. we do need to fix such kind of typo > -Original Message- > From: Donal Lafferty [mailto:donal.laffe...@citrix.com] > Sent: Friday, July 19, 2013 12:28 AM > To: dev@cloudstack.apache.org > Subject: RE: code formatting for enums > > > -Original Message---

RE: code formatting for enums

2013-07-19 Thread Frank Zhang
we need to should be keying > > off of a value that is unlikely to change (e.g. an artificial ID). I > > would submit that Java enumeration value names, as code symbols, are > > subject to an unacceptable level of change to be used for persistence. > > > > Thanks, > >

RE: code formatting for enums

2013-07-18 Thread Frank Zhang
tificial keys to code > tables in schema. This conversation has expanded a bit, and it seems > appropriate to open a new [DISCUSS] thread to delve into it further. Do y'all > agree? > > Thanks, > -John > > On Jul 18, 2013, at 7:19 PM, Frank Zhang wrote: > > >

Re: Review Request 12611: Patch to fix stale cookie and refresh issue in UCS plugin

2013-07-18 Thread Frank Zhang
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12611/#review23455 --- Ship it! Ship It! - Frank Zhang On July 17, 2013, 12:11 a.m

RE: code formatting for enums

2013-07-18 Thread Frank Zhang
Those enums cannot be simply considered as internal data structures where code convention applies to, they should be considered in API level. Most CloudStack API responses retrieve its fields from xxxVO classes which represent database table. In our example, enum State directly maps to UserVmRes

RE: [Docs] Questions on baremetal feature (CLOUDSTACK-774)

2013-04-30 Thread Frank Zhang
From: Jessica Tomechak [mailto:jessica.tomec...@gmail.com] Sent: Monday, April 29, 2013 10:02 PM To: dev@cloudstack.apache.org; Frank Zhang Subject: [Docs] Questions on baremetal feature (CLOUDSTACK-774) I was finishing up a doc draft for this feature today, and generated the following

RE: CS dnsmasq DNS services - filterwin2k

2013-04-29 Thread Frank Zhang
> > On 4/26/13 3:27 PM, "Dennis Lawler" wrote: > > >The dnsmasq "filterwin2k" option disables SRV record requests, > >affecting Kerberos, SIP, Windows KMS licensing, XMPP, and Google Talk. > > > >This was disabled by default in dnsmasq config, but was re-added via > >commit 3b75abb, which does

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

2013-04-22 Thread Frank Zhang
Before we have most of tests automated, 4 months release cycle seems too tight to me > -Original Message- > From: Animesh Chaturvedi [mailto:animesh.chaturv...@citrix.com] > Sent: Monday, April 22, 2013 4:08 PM > To: dev@cloudstack.apache.org; cloudstack-...@incubator.apache.org > Subject