Re: [DIscuss]Storage image store plugin framework refactoring

2013-04-09 Thread Min Chen
hanks -min On 4/9/13 1:59 AM, "Prasanna Santhanam" wrote: >On Mon, Apr 08, 2013 at 04:45:23PM -0700, Min Chen wrote: >> Hi All, >> >> Currently CloudStack does not offer a flexible pluggable framework >> for users to easily integrate and configure any 3rd-par

Re: [DIscuss]Storage image store plugin framework refactoring

2013-04-09 Thread Min Chen
being deprecated , is there any specific >error message that will be returned? > >-Thanks >Sangeetha > >-Original Message- >From: Min Chen [mailto:min.c...@citrix.com] >Sent: Monday, April 08, 2013 4:45 PM >To: dev@cloudstack.apache.org >Subject: [DIscu

Re: [DIscuss]Storage image store plugin framework refactoring

2013-04-09 Thread Min Chen
/13 10:14 AM, "Chiradeep Vittal" wrote: >We can have alias for an existing API. See the other ML discussion. > >On 4/9/13 9:27 AM, "Min Chen" wrote: > >>Yes, I will include more api change details in FS in next few days. >>According to Chiradeep, it seems

Re: [DIscuss]Storage image store plugin framework refactoring

2013-04-09 Thread Min Chen
them(data copy when cross zone). > > >> How does this change? >> >> On 4/8/13 6:34 PM, "Sangeetha Hariharan" >> >> wrote: >> >> >Min, >> > >> >Could you also include the details of the API changes (new >> >par

Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-11 Thread Min Chen
uration.java <https://reviews.apache.org/r/10294/#comment39552> Where are UserVmDaoTestConfiguration used in your test? You are missing some Spring Junit annotation in your UserVmDaoImplTest. - Min Chen On April 10, 2013, 9:59 p.m., Venkata Siva Vijayendra

Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-11 Thread Min Chen
tested yet. Is this still true? If not, please revise your description to avoid confusing people. - Min Chen On April 10, 2013, 9:59 p.m., Venkata Siva Vijayendra Bhamidipati wrote: > > --- > This is an automatically generated e-mail.

Re: [ACS41] Help needed clearing bugs!

2013-04-11 Thread Min Chen
Hi Ryan, Maybe I overlooked something, but I could not understand how your patch is fixing CLOUDSTACK-1987. It would be appreciated if you can address my comment in that patch. Thanks -min On 4/11/13 5:52 PM, "Prachi Damle" wrote: >Hi Ryan, > >Yes Sure, my bad, will lo

Re: Review Request: Fix for CLOUDSTACK-1987

2013-04-11 Thread Min Chen
lves a related problem. (Marcus, I thought this was Scott's issue) Right now, domain users cannot query service offerings by ID's. Should I file a different bug then? It's pretty simple to replicate. As a domain user, call listServiceOfferings, then make the same call with an ID o

Re: Review Request: Fix for CLOUDSTACK-1987

2013-04-11 Thread Min Chen
tps://reviews.apache.org/r/10426/#comment39602> For domain users, they should not be able to query system offerings. This fix didn't guard that case. - Min Chen On April 12, 2013, 4:57 a.m., Ryan Dietrich wrote: > > ---

Re: [ACS41] Help needed clearing bugs!

2013-04-11 Thread Min Chen
It seems to me that this patch will fix 1989, not 1987. Thanks -min On 4/11/13 6:24 PM, "Marcus Sorensen" wrote: >Does it fix 1989, or perhaps both? > >https://issues.apache.org/jira/browse/CLOUDSTACK-1989 >On Apr 11, 2013 7:08 PM, "Min Chen" wrote: >

Re: Review Request: Fix for CLOUDSTACK-1987

2013-04-11 Thread Min Chen
> On April 12, 2013, 5:28 a.m., Min Chen wrote: > > server/src/com/cloud/api/query/QueryManagerImpl.java, line 2111 > > <https://reviews.apache.org/r/10426/diff/1/?file=280571#file280571line2111> > > > > For domain users, they should not be able to quer

Re: Review Request: Fix for CLOUDSTACK-1987

2013-04-12 Thread Min Chen
I will provide a patch for 1987 today. Thanks -min On 4/12/13 6:06 AM, "Chip Childers" wrote: >On Fri, Apr 12, 2013 at 05:38:28AM +0000, Min Chen wrote: >> >> >> > On April 12, 2013, 5:28 a.m., Min Chen wrote: >> > > server/src/com/cl

Review Request: Fix CLOUDSTACK-1987: service offerings owned by a domain show up to domain users

2013-04-12 Thread Min Chen
. This addresses bug https://issues.apache.org/jira/browse/CLOUDSTACK-1987. Diffs - server/src/com/cloud/api/query/QueryManagerImpl.java b96691a Diff: https://reviews.apache.org/r/10432/diff/ Testing --- Validated the failed scenario in local setup. Thanks, Min Chen

Re: [ACS41] Help needed clearing bugs!

2013-04-12 Thread Min Chen
Chip, patch for CLOUDSTACK-1987 is submitted for review: https://reviews.apache.org/r/10432/. Thanks -min On 4/12/13 5:52 AM, "Chip Childers" wrote: >On Thu, Apr 11, 2013 at 10:36:59PM -0700, Min Chen wrote: >> It seems to me that this patch will fix 1989, not 1987. >&

Re: [ACS41] Help needed clearing bugs!

2013-04-12 Thread Min Chen
Thanks. Applied to master as well. -min On 4/12/13 11:56 AM, "Chip Childers" wrote: >On Fri, Apr 12, 2013 at 11:40:35AM -0700, Min Chen wrote: >> Chip, patch for CLOUDSTACK-1987 is submitted for review: >> https://reviews.apache.org/r/10432/. >> >> Thanks

Re: database creator

2013-04-17 Thread Min Chen
In my opinion, rolling upgrade may not be an accurate term here for this task. In my previous experience, rolling upgrade is defined as a upgrade process that allows users to keep their existing distributed system running while they upgrade each system gradually. I don't think that this is the case

Re: [ACS41] API Compatibility Bug?

2013-04-18 Thread Min Chen
Chip, I am looking into the bug to see if this is a real issue. Thanks -min On 4/18/13 9:00 AM, "Chip Childers" wrote: >Min / Rohit, > >Joe ran into some changes (documented in CLOUDSTACK-2091) when reviewing >the generated API documentation for 4.1 vs 4.0.x. Can either of you look >at the >bu

Re: [ACS41] API Compatibility Bug?

2013-04-18 Thread Min Chen
understand how we got those differences. Joe, can you clarify this a bit? Did you run those 3 apis and found out those differences? Thanks -min On 4/18/13 9:29 AM, "Min Chen" wrote: >Chip, I am looking into the bug to see if this is a real issue. > >Thanks >-min > &

Re: [ACS41] API Compatibility Bug?

2013-04-18 Thread Min Chen
I myself haven't used ApiXmlDocReader tool yet. Where are the old and new xml file in our repo? Thanks -min On 4/18/13 9:47 AM, "David Nalley" wrote: >On Thu, Apr 18, 2013 at 12:41 PM, Min Chen wrote: > >> I updated on the bug, I compared both 4.0 and 4.1 code, &

Re: [ACS41] API Compatibility Bug?

2013-04-18 Thread Min Chen
I am not quite familiar with this. Alena, are you able to chip in to take a look? As far as code and functionality is concerned, I don't see API compatibility issue here. Thanks -min On 4/18/13 9:56 AM, "David Nalley" wrote: >On Thu, Apr 18, 2013 at 12:52 PM, Min Chen wr

Re: [ACS41] API Compatibility Bug?

2013-04-18 Thread Min Chen
lass) > > >The same issue for DeleteUser. Basically there are no changes in the >commands, they were just documented wrong in 4.0. > > >-Alena. > > > > >On 4/18/13 10:03 AM, "David Nalley" wrote: > >>On Thu, Apr 18, 2013 at 1:01 PM, Min Chen wro

Re: [ACS41] API Compatibility Bug?

2013-04-18 Thread Min Chen
I can provide a quick fix on 4.0 branch. For 4.0 branch, what is the process to commit a fix? Submit a patch for Joe to commit, or I can directly commit? Thanks -min On 4/18/13 11:08 AM, "Chip Childers" wrote: >On Thu, Apr 18, 2013 at 11:07:18AM -0700, Alena Prokharchyk wrote: >> On 4/18/13 11:

Re: [ACS41] API Compatibility Bug?

2013-04-18 Thread Min Chen
Done. Commit number f797d450261d8a4d2734db2cfc7e40c67167cd14. -min On 4/18/13 11:20 AM, "Joe Brockmeier" wrote: >On Thu, Apr 18, 2013, at 01:15 PM, Min Chen wrote: >> I can provide a quick fix on 4.0 branch. For 4.0 branch, what is the >> process to commit a fix?

Re: [ASFCS42] Proposed schedule for our next release

2013-04-19 Thread Min Chen
I like this idea too, this will be a much better channel for community to understand features to be planned. +1 to this. -min On 4/19/13 7:02 AM, "Prasanna Santhanam" wrote: >On Fri, Apr 19, 2013 at 10:16:35AM +, Murali Reddy wrote: >> >> Pardon my ignorance of project management, but it a

Re: New PMC Member: Prasanna Santhanam

2013-04-22 Thread Min Chen
Congratulations, Prasanna. -min On 4/22/13 8:51 AM, "Chip Childers" wrote: >Hi folks > >The Project Management Committee (PMC) for Apache CloudStack >has asked Prasanna Santhanam to become a member of the PMC and >we are pleased to announce that he has accepted. > >Please join me in congratulat

Re: [DIscuss]Storage image store plugin framework refactoring

2013-04-22 Thread Min Chen
ril 10, 2013 12:42 PM >To: dev@cloudstack.apache.org >Subject: Re: [DIscuss]Storage image store plugin framework refactoring > >On Tue, Apr 09, 2013 at 09:19:11AM -0700, Min Chen wrote: >> Not yet at this moment. But we are planning to provide a sample plugin >> implementation f

Re: [DIscuss]Storage image store plugin framework refactoring

2013-04-22 Thread Min Chen
both NFS and S3 co-existing for this release. You raised a very good point, honestly I don't have an answer at this point. Edison, any thoughts on this? > >Thanks, >Sanjeev > >-Original Message- >From: Min Chen [mailto:min.c...@citrix.com] >Sent: Wednesday, April 10

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

2013-04-22 Thread Min Chen
Same feelings here. -min On 4/22/13 4:18 PM, "Frank Zhang" wrote: >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 >>

Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-24 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10294/#review19680 --- Ship it! Ship It! - Min Chen On April 23, 2013, 9:45 p.m

Re: Review Request: Remove 2k limitation for user data on a deployVMCmd issued as an HTTP POST request

2013-04-24 Thread Min Chen
> On April 24, 2013, 11:29 p.m., Min Chen wrote: > > Ship It! Prasanna has fixed the marvin test and pushed to master. These are the rebased commits on master: 4e9e7937ed5e56cbd7f678b2ed9863a56af16635 Fix Vijay's test to use the marvin integra

Re: [ACS41] Blocker and critical issues stopping me from spinning out a new RC for 4.1

2013-04-25 Thread Min Chen
I can take a look at CLOUDSTACK-2126. Thanks -min On 4/25/13 6:54 AM, "Chip Childers" wrote: >Hi all, > >The following bugs are currently blocking me from starting another 4.1.0 >voting round. Help please! > >CLOUDSTACK-2158 'userconcentratedpod_firstfit failed to find alternate >host to run V

Re: [ACS41] Blocker and critical issues stopping me from spinning out a new RC for 4.1

2013-04-25 Thread Min Chen
lized fields, I don't see why marvin has issues in deserialization in this case. Did I miss anything? Thanks -min On 4/25/13 9:18 AM, "Min Chen" wrote: >I can take a look at CLOUDSTACK-2126. > >Thanks >-min > >On 4/25/13 6:54 AM, "Chip Childers" wrote: &

Re: Exposing APIs that carry POST data

2013-04-25 Thread Min Chen
REST based service is postponed, no active progress from mysids either. Thanks -min On 4/25/13 1:10 AM, "Rohit Yadav" wrote: >On Thu, Apr 25, 2013 at 10:59 AM, Prasanna Santhanam >wrote: > >> On Wed, Apr 24, 2013 at 06:08:49AM -0400, Sebastien Goasguen wrote: >> > >> > On Apr 24, 2013, at 4:38

Re: [ACS41] Blocker and critical issues stopping me from spinning out a new RC for 4.1

2013-04-25 Thread Min Chen
. But I would argue that we should not return jobStatus in ListVmsCmd as well, this will also be a change from 4.0 release. Thanks -min On 4/25/13 9:48 AM, "Prasanna Santhanam" wrote: >On Thu, Apr 25, 2013 at 09:30:08AM -0700, Min Chen wrote: >> Prasanna, I updated CLOUDSTACK-2

Re: [ACS41] Blocker and critical issues stopping me from spinning out a new RC for 4.1

2013-04-25 Thread Min Chen
added in >(prior?) 4.0. I'd like to take that out too if there's no real reason >behind it. > >-- >Prasanna., > >On Thu, Apr 25, 2013 at 09:57:02AM -0700, Min Chen wrote: >> In 4.0, CS has special code to return job status for a VM returned from >&g

Re: [ACS41] Blocker and critical issues stopping me from spinning out a new RC for 4.1

2013-04-25 Thread Min Chen
Thanks Chip, will do that. -min On 4/25/13 10:37 AM, "Chip Childers" wrote: >On Thu, Apr 25, 2013 at 10:32:15AM -0700, Min Chen wrote: >> Totally agree. >> >> Chip and others, Prasanna and I exchanged ideas about CLOUDSTACK--2126, >> and we both agree th

Re: [ACS41] Blocker and critical issues stopping me from spinning out a new RC for 4.1

2013-04-25 Thread Min Chen
gt;> >we suggest that this can be fixed in 4.2 since current behavior is not >> >breaking functionalities, just return more information. Any thoughts? >> > >> >Thanks >> >-min >> > >> > >> > >> > >> > >>

Review Request: CLOUDSTACK-2196 and CLOUDSTACK-2126: fix missing async job status in listXXX api and incorrect async job status in other async api.

2013-04-25 Thread Min Chen
/UserVmJoinVO.java 33c49cd server/src/com/cloud/api/query/vo/VolumeJoinVO.java 4f6b35b Diff: https://reviews.apache.org/r/10789/diff/ Testing --- Tested locally using devcloud. Thanks, Min Chen

[ACS41]Patch Request

2013-04-25 Thread Min Chen
Hi Chip, https://reviews.apache.org/r/10789 is the patch for CLOUDSTACK-2196 and CLOUDSTACK-2126: fix missing async job status in listXXX api and incorrect async job status in other async api. Can you please commit it to 4.1? Thanks -min

Re: [ACS41]Patch Request

2013-04-25 Thread Min Chen
Thanks. Applied to master. -min On 4/25/13 4:59 PM, "Chip Childers" wrote: >On Thu, Apr 25, 2013 at 04:24:06PM -0700, Min Chen wrote: >> Hi Chip, >> >> https://reviews.apache.org/r/10789 is the patch for CLOUDSTACK-2196 >>and CLOUDSTACK-2126: fix missin

Re: [MERGE] CS-2163

2013-04-29 Thread Min Chen
Nitin, I have several questions after reading your FS: 1. The API to be modified should also include some list APIs to control which resources to be displayed to the end user based on your new flag, right? You didn't mention such APIs in your FS, so I am asking for clarification. 2. If ans

Re: [MERGE] CS-2163

2013-04-30 Thread Min Chen
+1 to Prasanna's argument. Tags provide more flexibility and extensibility than adding column to existing db schema. What happens if later on we want to add finer control to these resources not specific to end user? Adding another column? Also, I haven't seen your answer to my questions in this t

Re: New Committer: Clayton Weise

2013-05-01 Thread Min Chen
Congratulations, Clayton. On 4/30/13 9:53 AM, "Chip Childers" wrote: >The Project Management Committee (PMC) for Apache CloudStack >has asked Clayton Weise to become a committer and we are pleased to >announce that he has accepted. > >Clayton is exceptionally active on the #cloudstack IRC channe

Re: [Proposal]CloudStack IAM plugin feature (CLOUDSTACK-5920)

2014-01-22 Thread Min Chen
Hi Koushik, See my answers in line. Thanks. -min On 1/22/14 12:30 AM, "Koushik Das" wrote: >Some questions: > >- Is there a concept of generic permission (any action, any resource >etc.)? There shouldn't be a need to define hundreds of explicit >permissions for admin ac

Re: [Proposal]CloudStack IAM plugin feature (CLOUDSTACK-5920)

2014-01-22 Thread Min Chen
Hi Rajani, See my answers in line. Thanks On 1/22/14 6:29 AM, "Rajani Karuturi" wrote: >some questions I have: >1. Do we need groups and policies? Cant we derive group information from >policy applied? ie) any user can become domain admin if he is given the >right policies. [Min

Re: [ISSUES] Master VMware is broken

2014-01-22 Thread Min Chen
FYI, I checked in a fix 206c35c620a8e7a707f371e5a9e5dfd795912f5b for this to unblock my testing. Thanks -min On 1/22/14 5:43 PM, "Kelven Yang" wrote: >Master VMware was broken by this commit > >author Hugo >Trippaersh;h=b20add

Re: [ISSUES] Master VMware is broken

2014-01-23 Thread Min Chen
ry about that :-( Thanks for finding and fixing it. > >Where did 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 206c35c620a8e7a707f371e5a9e

4.3 cherry-pick request

2014-01-27 Thread Min Chen
Animesh, Can you please cherry-pick the following commit (https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=6d1475 182cd9ea17010a142043dd9fe9c923cf81) from 4.3-forward to 4.3 branch? Thanks. 6d1475182cd9ea17010a142043dd9fe9c923cf81 CLOUDSTACK-5960: Domain admin or user

cherry pick commit to 4.3

2014-02-06 Thread Min Chen
Hi Animesh, I fixed several S3 related bugs on 4.3-forward, can you please cherry-pick them to 4.3? Thanks. commit b11c13d4a120a6361231ee415082576dac6fe385 CLOUDSTACK-6019:CreateVolume from snapshot in NFS is throwing Warning "sync snapshot from cache to region wide store failed for imag

Re: cherry pick commit to 4.3

2014-02-06 Thread Min Chen
One more commit: commit e00241f41d6542cf1c5d93216fed55609cd7746a CLOUDSTACK-6049: CreateTemplate from snapshot is failing on a multi-zone setup after migration of NFS to S3. Thanks -min On 2/6/14 3:43 PM, "Min Chen" wrote: >Hi Animesh, > > I fixed several S3

Re: [PROPOSAL] List VM API enhancement

2014-02-07 Thread Min Chen
Hi Koushik, I agree with the idea of supporting multiple IDs. But I may not like the idea of introducing another different query parameter "ids" for this purpose. Why cannot we just change current "id" parameter to take a list of values? This way, user will not need to use two different pa

Re: [PROPOSAL] List VM API enhancement

2014-02-07 Thread Min Chen
ed for multiple IDs that breaks the >> parameter naming convention. >> >> I am all in favour of using the existing 'id' parameter if there is no >> issues with breaking the naming convention. >> >> >>> On 07-Feb-2014, at 11:25 PM, Min Chen wrote

Re: S3 based secondary Storage

2014-02-10 Thread Min Chen
Can you try with s3cmd? If still not working, please post your .s3cfg file. Thanks -min On 2/10/14 12:41 AM, "Paul Angus" wrote: >Yes, I'm using S3Browser, no problem uploading/downloading files > >Regards, > >Paul Angus >Cloud Architect >S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus

Re: Template for Java 7 in Master branch

2014-02-10 Thread Min Chen
Hugo, is it possible to also build a 32 bit systemvm template using Java 7? Some of us are using VM provisioned from Vmware ESX as our hypervisor host due to resource limitation, unfortunately it only supports 32-bit template not 64 bit. Thanks -min On 2/8/14 4:05 AM, "Hugo Trippaers" wrote: >S

Re: Template for Java 7 in Master branch

2014-02-10 Thread Min Chen
vm-master > > >Sent from my iPhone > >> On 10 feb. 2014, at 18:26, Min Chen wrote: >> >> Hugo, is it possible to also build a 32 bit systemvm template using Java >> 7? Some of us are using VM provisioned from Vmware ESX as our hypervisor >> host due to resou

Re: failing to build noredist on 4.3 RC4

2014-02-10 Thread Min Chen
I just tried to build noredist on 4.3 branch, it worked fine for me. Is this the first time you are building noredist on your machine? If so, you need to make sure that you have installed Vmware 5.1 SDK jar into your maven repo. Go to your .m2/repository, you should find this: localhost:reposit

Re: S3 based secondary Storage

2014-02-10 Thread Min Chen
_mode = normal use_https = False verbosity = WARNING Regards, Paul Angus Cloud Architect S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus paul.an...@shapeblue.com<mailto:paul.an...@shapeblue.com> -Original Message- From: Min Chen [mailto:min.c...@citrix.com] Sent:

Re: S3 based secondary Storage

2014-02-10 Thread Min Chen
d it from the paste for security - it was there:) > >Regards > >Paul Angus >Cloud Architect >S: +44 20 3603 0540 | M: +447711418784 | T: CloudyAngus >paul.an...@shapeblue.com > >-----Original Message- >From: Min Chen [mailto:min.c...@citrix.com] >Sent: 10 Febr

Re: failing to build noredist on 4.3 RC4 [solved]

2014-02-11 Thread Min Chen
l Angus >Cloud Architect >S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus >paul.an...@shapeblue.com > >-Original Message- >From: Min Chen [mailto:min.c...@citrix.com] >Sent: 10 February 2014 19:24 >To: dev@cloudstack.apache.org >Subject: Re: failing to build noredis

Re: API Calls with key value pairs (Details)

2014-02-12 Thread Min Chen
AddImageStore has no such issues of breaking existing scripts at all. There has no change on that API in 4.3. Thanks -min Sent from my iPhone > On Feb 12, 2014, at 9:36 AM, "Animesh Chaturvedi" > wrote: > > Can you call out which scripts in particular? Copying min for S3 comment. May > be i

Re: API Calls with key value pairs (Details)

2014-02-12 Thread Min Chen
Paul, AddImageStore is a new API added in 4.2, it has been designed like that to handle different meta information required by different image store provider. This API does not have any backwards compatibility issue. I was never aware of any community consensus to pass extra detail informat

Re: copying a template from random zone

2014-02-13 Thread Min Chen
Edison has already fixed this in ACS 4.3 and master, See commit https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=1b06451 5ac9c7f87e1780ab2210e60723f9d541c. Thanks -min On 2/12/14 8:16 AM, "Daan Hoogland" wrote: >LS, > >It seems that when instantiating a vm from a template clo

Re: copying a template from random zone

2014-02-13 Thread Min Chen
One correction, fix is not present in master, Edison will cherry-pick it there soon. Thanks -min On 2/13/14 10:47 AM, "Min Chen" wrote: >Edison has already fixed this in ACS 4.3 and master, See commit >https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=co

Re: copying a template from random zone

2014-02-13 Thread Min Chen
was wrong. I have not devised a way to verify yet. thanks for >the confirmation, Min > >On Thu, Feb 13, 2014 at 7:47 PM, Min Chen wrote: >> Edison has already fixed this in ACS 4.3 and master, See commit >> >>https://git-wip-us.apache.org/

Re: Anti-patterns

2014-02-25 Thread Min Chen
Thanks Hugo for the nice tutorial. In the last example, are you trying to saying this (luckily this time == works although not recommended) public class App { public static void main(String[] args) { Integer a = 1; Integer b = 1; if

Re: [4.3] [Cherry-pick] CLOUDSTACK-6174, CLOUDSTACK-6175

2014-02-26 Thread Min Chen
Animesh, Can you cherry pick the following commits from 4.3-forward to 4.3? 4c83f80c809674cb8ff9561d546a6ade25d983f8 CLOUDSTACK-6174: Provide separate option for Windows Server 2012 R2 as an OS type when registering template. 94257c0fc85c7178e6ca3146662de08feee2c290 CLOUDSTACK-6175:s3.

Re: com.vmware.vim25 Dependency Issue?

2014-03-06 Thread Min Chen
No. Those two jars are out-of-dated, we have upgraded to vmware SDK 5.1, those are for older version of SDK. Thanks -min On 3/6/14 4:44 AM, "Alex Hitchins" wrote: >I've just downloaded and run the install-non-oss.sh script however I'm >getting lots of errors like the below. > >java: cannot find

Re: com.vmware.vim25 Dependency Issue?

2014-03-06 Thread Min Chen
K. > >So when I download the SDK the jars are part of the install? > > >Regards > >Alex Hitchins > >D: +44 1892 523 587 | S: +44 2036 030 540 | M: +44 7788 423 969 > >alex.hitch...@shapeblue.com > >-Original Message- >From: Min Chen [mailto:m

[Merge] CloudStack IAM branch to master

2014-03-10 Thread Min Chen
Hi, Prachi and I would like to merge CloudStack Identity and Access Management(IAM) plugin services to the master branch. Development for this effort has been done by Prachi and me on ACS rbac branch (https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/rbac). Checkl

Re: [Merge] CloudStack IAM branch to master

2014-03-10 Thread Min Chen
No new jar dependencies. -min Sent from my iPhone > On Mar 10, 2014, at 7:22 PM, "Chiradeep Vittal" > wrote: > > Any new jar dependencies? > >> On 3/10/14, 11:34 AM, "Min Chen" wrote: >> >> Hi, >> >> Prachi and I wou

Re: [Merge] CloudStack IAM branch to master

2014-03-11 Thread Min Chen
Also, have already run FingBugs on our branch and addressed all new findings introduced by our branch. Thanks. -min On 3/10/14 7:33 PM, "Min Chen" wrote: >No new jar dependencies. > >-min > >Sent from my iPhone > >> On Mar 10, 2014, at 7:22 PM, "Chirade

Re: [Merge] CloudStack IAM branch to master

2014-03-13 Thread Min Chen
Since we haven't heard of any objections to this merge for 3 days, I am going to merge it to master today. Thanks -min On 3/11/14 12:23 PM, "Hugo Trippaers" wrote: > >On 11 mrt. 2014, at 19:52, Min Chen wrote: > >> Also, have already run FingBugs on ou

Re: Master is broken?

2014-03-13 Thread Min Chen
Yes, that is problem. Parameter name is changed to lowercase, which will not be able to retrieve parameters from API job. Thanks -min On 3/13/14 4:31 PM, "Daan Hoogland" wrote: >It's half passed midnight here so you won't get an answer soon from >this side of the ocean. > >The line >at com.clou

Re: Master is broken?

2014-03-13 Thread Min Chen
Fixed with commit d3fd66e9f555616530e483d1eacf2525556bf14e on master. Thanks -min On 3/13/14 4:34 PM, "Min Chen" wrote: >Yes, that is problem. Parameter name is changed to lowercase, which will >not be able to retrieve parameters from API job. > >Thanks >-min >

Re: [Merge] CloudStack IAM branch to master

2014-03-13 Thread Min Chen
IAM branch is now merged to master. Thanks -min On 3/13/14 10:13 AM, "Min Chen" wrote: >Since we haven't heard of any objections to this merge for 3 days, I am >going to merge it to master today. > >Thanks >-min > >On 3/11/14 12:23 PM, "Hugo Trippaers&

Anybody saw this error on master?

2014-03-13 Thread Min Chen
Hi there, Anybody has seen this error on master? It is not blocking, but still an eyesore to see it in the log. 2014-03-13 16:31:54,357 DEBUG [c.c.h.x.r.CitrixResourceBase] (DirectAgent-8:ctx-87d8612e) Vm cpu utilization 0.08 2014-03-13 16:31:54,357 DEBUG [c.c.h.x.r.CitrixResourceBase] (Direct

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Min Chen
reserve >history. It's nice to preserve history, but it's a lot easier to undo >a squashed merge. > >On Thu, Mar 13, 2014 at 5:56 PM, Min Chen wrote: >> IAM branch is now merged to master. >> >> Thanks >> -min >> >> On 3/13/14 10:13 AM,

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Min Chen
gt;:1145) >> at >>java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.jav >>a:615) >> at java.lang.Thread.run(Thread.java:744) >> >> On Thu, Mar 13, 2014 at 11:30 PM, Marcus wrote: >>> Min, in looking at this branch merge, I need to b

Re: [Merge] CloudStack IAM branch to master

2014-03-14 Thread Min Chen
Marcus, What is the latest commit you have picked up on your local setup from master? Our QA reports similar issues caused by recent VMSync bug fix, just want to make sure that your local code has that fix. Thanks -min On 3/14/14 9:37 AM, "Min Chen" wrote

Re: Anybody saw this error on master?

2014-03-14 Thread Min Chen
I am using XenServer 6.2 Thanks -min On 3/14/14 12:45 AM, "Sanjay Tripathi" wrote: >Hi Min, > >Which version of XS you are working on? > >--Sanjay > >-----Original Message- >From: Min Chen [mailto:min.c...@citrix.com] >Sent: Friday, March 14, 2014 6

Re: IAM guidelines for CS APIs

2014-03-17 Thread Min Chen
Hi David, Thanks a lot for your interest and willing to help on this feature. For the Phase I work laid out in https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+Identity+ and+Access+Management+%28IAM%29+Plugin, we have pretty much completed most of implementations and merge

Re: [MERGE] resize-root branch

2014-03-17 Thread Min Chen
Hi Marcus, I found an issue with this merge, my previous marvin test is broken in deployVirtualMachine if no rootdisksize is passed. All previous marvin tests are written without passing this parameter. Can you take a look at base.py to make sure that old marvin tests without passing those

Re: [MERGE] resize-root branch

2014-03-18 Thread Min Chen
tests... >> >> commit 3f7b628c707cd9c30542efd670416be5fbb473f0 >> Author: Gaurav Aradhye >> Date: Fri Feb 28 01:46:24 2014 -0500 >> >> CLOUDSTACK-6147: Adding first set of test caess for Dynamic >> Compute Offering feature >> >> Signed

Re: [MERGE] resize-root branch

2014-03-18 Thread Min Chen
Mar 17, 2014 at 11:01 PM, Marcus wrote: >> I'll take a look. I wasn't the one who added the details map to >> base.py, although I did add to it and can probably figure out what's >> wrong. The original commit didn't change any deployVirtualMachine >> cal

Re: IAM guidelines for CS APIs

2014-03-18 Thread Min Chen
Review board with patches that are linked to JIRA should be the right way to go. Do let us know what you are going to work on to avoid overlapping. Thanks -min From: David Grizzanti mailto:david.grizza...@sungard.com>> Date: Tuesday, March 18, 2014 7:46 AM To: Min Chen mailto

Re: [MERGE] resize-root branch

2014-03-18 Thread Min Chen
or Dynamic >> Compute Offering feature >> >> Signed-off-by: SrikanteswaraRao Talluri >> >> >> On Mon, Mar 17, 2014 at 11:01 PM, Marcus wrote: >>> I'll take a look. I wasn't the one who added the details map to >>> base.py, although I did ad

Re: random template from zones

2014-03-18 Thread Min Chen
Edison, did you cherry-pick https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=1b06451 5ac9c7f87e1780ab2210e60723f9d541c to 4.4 and master? It is in 4.3. Thanks -min On 3/18/14 3:45 AM, "Daan Hoogland" wrote: >H Min, Edison, > >As discussed in a prior mailthread [1] templates b

Re: [ACS4.4] Are tests still failing for people?

2014-03-20 Thread Min Chen
Mike, with your latest commit 7a7d9cff0d5bb40284506faa6e34138f97a12428 on 4.4, test failed in cloud-server: Tests in error: testRestoreVMF2(com.cloud.vm.UserVmManagerTest) testRestoreVMF3(com.cloud.vm.UserVmManagerTest) testRestoreVMF4(com.cloud.vm.UserVmManagerTest) testRestoreVMF5(com.c

Re: [ACS4.4] Are tests still failing for people?

2014-03-20 Thread Min Chen
014 at 5:19 PM, Mike Tutkowski < >> mike.tutkow...@solidfire.com> wrote: >> >>> Sure, I'll look into that. Thanks >>> >>> >>> On Thu, Mar 20, 2014 at 5:16 PM, Min Chen wrote: >>> >>>> Mike, with your latest commit >&

Re: [ACS4.4] Are tests still failing for people?

2014-03-20 Thread Min Chen
gt; On Thu, Mar 20, 2014 at 5:19 PM, Mike Tutkowski < >> mike.tutkow...@solidfire.com> wrote: >> >>> Sure, I'll look into that. Thanks >>> >>> >>> On Thu, Mar 20, 2014 at 5:16 PM, Min Chen wrote: >>> >>>> Mike, with

Re: [ACS4.4] Are tests still failing for people?

2014-03-20 Thread Min Chen
To revert a commit, you can just do: git revert git commit Thanks -min From: Mike Tutkowski mailto:mike.tutkow...@solidfire.com>> Date: Thursday, March 20, 2014 4:37 PM To: "dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org>" mailto:dev@cloudstack.apa

Re: Review Request 20210: List templates returns destroyed temples when copying the same template after deleting.

2014-04-10 Thread Min Chen
estroyed'. The much simpler fix to me is to remove entries from template_store_ref table at the end of deleting template flow when the deletion is successful (since they have already been deleted from secondary storage) instead of finding and then manually updating entry done in the patch.

Can we create IAM component in ACS JIRA?

2014-04-17 Thread Min Chen
Hi there, Can somebody with permission create a "IAM" component in ACS JIRA? This will help us to track issues filed specifically related to IAM feature. Thanks -min

Re: Can we create IAM component in ACS JIRA?

2014-04-17 Thread Min Chen
Thanks, David. -min On 4/17/14 10:12 AM, "David Nalley" wrote: >Done, enjoy! > >On Thu, Apr 17, 2014 at 1:05 PM, Min Chen wrote: >> Hi there, >> >> Can somebody with permission create a "IAM" component in ACS JIRA? This >>will help us to t

Anybody experienced test failure on master?

2014-04-17 Thread Min Chen
Hi there, Did anybody experience test failure on master branch? I got this: T E S T S --- Running com.cloud.utils.DbUtilTest log4j:WARN No appenders could be found for logger (com.cloud.utils.crypt.EncryptionSecretKeyChecker). log4j:WARN Plea

Re: Anybody experienced test failure on master?

2014-04-18 Thread Min Chen
this caused the problem though. Same structure in 4.4 works for me on the machine. Thanks -min Sent from my iPhone > On Apr 18, 2014, at 4:50 AM, "Koushik Das" wrote: > > 'mvm clean install' worked for me. Are you trying noredist? > > >> On 18-Apr-2014

Re: Anybody experienced test failure on master?

2014-04-18 Thread Min Chen
ava:80) at com.cloud.utils.db.TransactionLegacy.(TransactionLegacy.java:1024) On 4/18/14 2:18 AM, "Laszlo Hornyak" wrote: >Hi Min, > >Can you send details, stack traces? > >Thank you, >Laszlo > > >On Fri, Apr 18, 2014 at 2:33 AM, Min Chen wrote: > >> Hi there, >&g

Re: Build failed in Jenkins: build-master-slowbuild #630

2014-04-18 Thread Min Chen
Daan, I cannot tell from this email what is breaking the slow build? And which file? These commits didn't touch these files in cloud/bridge file at all. Can you tell us how to read slow build failures? Thanks -min On 4/18/14 2:57 PM, "Daan Hoogland" wrote: >Min, > >Can

Re: Build failed in Jenkins: build-master-slowbuild #630

2014-04-18 Thread Min Chen
loud.org/job/build-master-slowbuild/630/findbugsResu >lt/new/ > > >On Sat, Apr 19, 2014 at 12:58 AM, Min Chen wrote: >> Daan, >> >> I cannot tell from this email what is breaking the slow build? >>And which >> file? These commits didn't to

Re: Build failed in Jenkins: build-master-slowbuild #630

2014-04-18 Thread Min Chen
uite some >but I'd sugest running findbugs against those files and triage those. >Or revert the commits one by one to see which one contains the >culprit. That will take all weekend though. It is not called slowbuid >for nothing. > >On Sat, Apr 19, 2014 at 1:07 AM, Min Chen

<    1   2   3   4   5   >