Re: NPE while adding a host

2014-12-04 Thread Min Chen
Yes. I checked in a fix yesterday with both code and schema update. If you updated code, you need to update schema as well. Thanks -min Sent from my iPhone > On Dec 4, 2014, at 5:20 AM, "Kuang-Ching Wang" > wrote: > > Ah - I know what happened - need to flush the DB. > > KC > >> On Dec 3,

Re: [QUESTION] @ReflectionUse

2014-11-27 Thread Min Chen
If I understand this clearly, this annotation was introduced by Kelven to prevent people from mistakenly removing those annotated methods if they find from IDE that those methods are not explicitly called anywhere. These methods are actually invoked through reflection. Thanks -min > On Nov 2

Re: [DISCUSS] Major business logic refactoring: Move from Account to UserAccount

2014-11-21 Thread Min Chen
If I understood correctly, (account, domainId) passed into deployVMCmd is used for impersonation-like behavior, that is, caller is deploying a VM on behalf of an account. Personally I don't like this kind of putting so many parameters in one API to perform several different functionalities, imperso

Re: No event publish can be wrapped within db transaction...why?

2014-11-18 Thread Min Chen
hat central system. > >I imagine if you have an external identity provider, you plug in a >different implementation of AccountManager (LDAPAccountManager? etc.), >and then a CreateAccountCmd would fail with an error saying the server is >configured to use <> so account creation

Re: No event publish can be wrapped within db transaction...why?

2014-11-18 Thread Min Chen
Hi Leo, "NO EVENT PUBLISH CAN BE WRAPPED WITHIN DB TRANSACTION!" is along the same line as "NO AGENT COMMAND CAN BE WRAPPED WITHIN DB TRANSACTION!". The rationale behind this is simple: event subscriber execution or agent command handling at resource layer may take too long, and we don't w

Re: [DISCUSS] Major business logic refactoring: Move from Account to UserAccount

2014-11-14 Thread Min Chen
>>>> Hi Min, >>>> >>>> Good to know. What do you propose we do moving forward. Do a >>>>refactoring >>>> run to fix it or leave it as it is and perhaps add user_id columns to >>>>few >>>> resources that are more usef

Re: [DISCUSS] Major business logic refactoring: Move from Account to UserAccount

2014-11-14 Thread Min Chen
orward. Do a refactoring >run to fix it or leave it as it is and perhaps add user_id columns to few >resources that are more useful for sysadmins such as vm_instance table. > >> On 14-Nov-2014, at 11:49 pm, Min Chen wrote: >> >> Rohit, >> >> I think that the h

Re: [DISCUSS] Major business logic refactoring: Move from Account to UserAccount

2014-11-14 Thread Min Chen
Rohit, I think that the historic reason for this is that CloudStack is only doing IAM access permission check on account level, user is only login authentication purpose. That is why we will see that all our CloudStack resource owner field is an account, since that is the only information

Re: Cloudstack 4.3.1 - NullPointerException when deleting VM

2014-11-06 Thread Min Chen
gt;Mike > >On Thu, Nov 6, 2014 at 11:08 AM, Min Chen wrote: > >> Yes, that is the workaround, just set pool_id to some dummy pool. >> >> >> On 11/6/14 10:01 AM, "Rafael Weingartner" >> wrote: >> >> >Thanks, After I looked at the

Re: Cloudstack 4.3.1 - NullPointerException when deleting VM

2014-11-06 Thread Min Chen
Yes, that is the workaround, just set pool_id to some dummy pool. On 11/6/14 10:01 AM, "Rafael Weingartner" wrote: >Thanks, After I looked at the CS source code. I worked around it by >setting >a pool_id in the database in the volume table. > >On Thu, Nov 6, 2014 at

Re: Cloudstack 4.3.1 - NullPointerException when deleting VM

2014-11-06 Thread Min Chen
This is a bug https://issues.apache.org/jira/browse/CLOUDSTACK-6934, which has been fixed in 4.5.0. The VM to be expunged did not allocate volume on primary storage. Thanks -min On 11/6/14 4:57 AM, "Rafael Weingartner" wrote: >HI, > >Have already someone seen a null pointer when deleting a VM?

Re: Heart Beat messages...

2014-10-15 Thread Min Chen
This is not error, that is why it is printed as DEBUG. There is job heartbeat thread that runs regularly trying to pick up jobs from each VM queue. When there is already a currently active job running for that queue, this message will be printed. Thanks -min On 10/15/14 12:58 PM, "Mo" wrote: >M

Re: [ANNOUNCE] New PMC Member: Pierre-Luc Dion

2014-10-01 Thread Min Chen
Congrats, Pierre-Luc! -min On 9/30/14 10:43 PM, "Prasanna Santhanam" wrote: >Congratulations Pierre-Luc! > >On Tue, Sep 30, 2014 at 7:20 PM, Chip Childers >wrote: >> The Project Management Committee (PMC) for Apache CloudStack has asked >> Pierre-Luc Dion to join the PMC and we are pleased to

Re: [ANNOUNCE] New PMC Member: Ian Duffy

2014-10-01 Thread Min Chen
Congratulations, Ian. -min On 9/30/14 10:42 PM, "Prasanna Santhanam" wrote: >Congratulations Ian! > >On Tue, Sep 30, 2014 at 7:20 PM, Chip Childers >wrote: >> The Project Management Committee (PMC) for Apache CloudStack has asked >> Ian Duffy to join the PMC and we are pleased to announce that

Re: Review Request 25722: CLOUDSTACK-7567 - Automate ACL test cases relating to depoying VM in shared network with different scopes - All/Domain/Domain with subdomain/Account for Admin, domain admin a

2014-09-16 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25722/#review53633 --- Ship it! Ship It! - Min Chen On Sept. 17, 2014, 12:42 a.m

Re: Review Request 22707: Test suite contains test cases relating to access checks for listSnapshot() with parameters - id, listall, isrecursive, account and domainid executed as ROOT admin, domain ad

2014-09-09 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22707/#review52788 --- Ship it! Ship It! - Min Chen On Sept. 9, 2014, 7:21 p.m

Re: Review Request 22712: This Test suite has test cases relating to acess checks for deleteNetwork() for Admin, domain admin and regular users

2014-09-09 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22712/#review52787 --- Ship it! Ship It! - Min Chen On Sept. 9, 2014, 8:39 p.m

Re: Review Request 22712: This Test suite has test cases relating to acess checks for deleteNetwork() for Admin, domain admin and regular users

2014-09-09 Thread Min Chen
of mixing them with single-line comments. - Min Chen On Sept. 8, 2014, 7:52 p.m., sangeetha hariharan wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 22707: Test suite contains test cases relating to access checks for listSnapshot() with parameters - id, listall, isrecursive, account and domainid executed as ROOT admin, domain ad

2014-09-09 Thread Min Chen
your method comments? - Min Chen On Sept. 9, 2014, 12:27 a.m., sangeetha hariharan wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 22705: Test cases relating to access checks for listVirtualMachine() with parameters - id, listall, isrecursive, account and domainid executed as ROOT admin, domain admin and regula

2014-09-05 Thread Min Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22705/#review52530 --- Ship it! Looks good to me. - Min Chen On Sept. 5, 2014, 7:02

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
Backported to 4.4 branch. -min From: Daan Hoogland mailto:daan.hoogl...@gmail.com>> Date: Wednesday, September 3, 2014 11:52 AM To: dev mailto:dev@cloudstack.apache.org>>, Min Chen mailto:min.c...@citrix.com>> Subject: Re: Commit cd8af6a guess you should go ahead and backpor

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
This is a bug that will impact reference counting only for old volume on NFS to S3 object store migration. Thanks -min On 9/3/14 11:25 AM, "Daan Hoogland" wrote: >H Min, > >what is this fix for? > > >On Wed, Sep 3, 2014 at 8:20 PM, Min Chen wrote: > >> Fix

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
Fix is checked into master branch. Can I back port it to 4.4? Thanks -min On 9/3/14 9:42 AM, "Min Chen" wrote: >Thanks Hugo for pointing that out. That is a bug caused by copy-and-paste, >the code should be in decrRefCnt routine like other SnapshotDataStoreVO >and TemplateD

Re: Commit cd8af6a

2014-09-03 Thread Min Chen
Thanks Hugo for pointing that out. That is a bug caused by copy-and-paste, the code should be in decrRefCnt routine like other SnapshotDataStoreVO and TemplateDataStoreVO. I will fix this in a minute. -min On 9/3/14 12:30 AM, "Hugo Trippaers" wrote: >Min, > >I¹m checking some findbugs reports

[BLOCKED] Unable to connect to management server on current master builds

2014-08-29 Thread Min Chen
CC Rohit here in case he didn't see this email. Rohit, can you fix this? Thanks -min On 8/29/14 9:21 AM, "Alex Brett" wrote: >Hello all, > >On current master builds (such as >http://jenkins.buildacloud.org/job/package-rhel63-master/3202/), I can't >connect to the management server, either via

Re: S3/Swift Problem around Virtual Size

2014-08-22 Thread Min Chen
No. For S3/Swift, register template will directly upload templates to S3 without going through staging NFS. It will only be copied to staging NFS when we first use that template to provision a VM. Thanks -min On 8/22/14 2:25 PM, "Francois Gaudreault" wrote: >Edison, > >Isnt the templates downlo

Hotfix for CLOUDSTACK-7260 is back ported to 4.3

2014-08-21 Thread Min Chen
Hi Ilya, Per your request and instruction, I have back ported hot fix for CLOUDSTACK-7260 to ACS 4.3 branch. Thanks -min

Re: [VOTE] Adapting git workflow for release branches

2014-08-19 Thread Min Chen
in On 8/19/14 10:55 AM, "Rohit Yadav" wrote: >tl;dr? hope you read it; > >On 19-Aug-2014, at 7:10 pm, Min Chen wrote: > >> I will hesitate on this "No enforcement" approach in the flow. I bet >>that >> without some kind of enforcement, based on

Re: 4.5 RM

2014-08-19 Thread Min Chen
+1 on what David said. Shared the same feelings as him, we really need to focus on the high priority issue facing CloudStack. Gerrit/Github pull request will definitely help in this regards. Thanks -min On 8/19/14 10:15 AM, "David Nalley" wrote: >> >> IMHO we should not even release 4.5 until

Re: [VOTE] Adapting git workflow for release branches

2014-08-19 Thread Min Chen
uot;Rohit Yadav" wrote: >Hi, > >On 19-Aug-2014, at 6:48 pm, Min Chen wrote: > >> In that case, we should call out this procedure >> >> If (you¹re a committer) { >> Go create a hotfix branch and ask RM to pick it up >> } else { >> Go

Re: [VOTE] Adapting git workflow for release branches

2014-08-19 Thread Min Chen
a reason as to why this proposal is flexible, and it does not >introduce any policing but gives a guideline for people to follow. > >Lastly, checking out branches and working on them using git is not >expensive at all, just few keyboard strokes maybe so just don¹t be afraid. > >A

Re: [VOTE] Adapting git workflow for release branches

2014-08-19 Thread Min Chen
I would rather CI be considered together with this thread, since this thread needs to decide at what condition RM can merge a hotfix/bugfic branch to release branch. Thanks -min Sent from my iPhone > On Aug 19, 2014, at 8:09 AM, "Pierre-Luc Dion" wrote: > > +1 , CI shouldn't be another topic

Re: [VOTE] Adapting git workflow for release branches

2014-08-18 Thread Min Chen
See my comments inline. Thanks -min On 8/18/14 3:22 PM, "Rohit Yadav" wrote: >Hey, > >in-line; > >On 18-Aug-2014, at 11:09 pm, Min Chen wrote: >> Rohit, >> I read your proposal, but maybe I mistook your idea: >> >> "- After

Re: [VOTE] Adapting git workflow for release branches

2014-08-18 Thread Min Chen
instead of currently checking into some forward branch and RM then cherry-pick it up to release branch to control quality. What is your new proposed approach then? Please clarify. Thanks -min On 8/18/14 12:06 PM, "Rohit Yadav" wrote: >Min, > >On 18-Aug-2014

Re: [VOTE] Adapting git workflow for release branches

2014-08-18 Thread Min Chen
Rohit, I think that Edison and I have clearly indicated our objection reason in our previous email. Based on current cloudstack quality, RM needs to have control over what commits to be in release branch to get a release at least having some quality. With this proposed model, how can you g

Re: [VOTE] Adapting git workflow for release branches

2014-08-16 Thread Min Chen
I am -1 on this as well. Any git process change needs to be considered together with the quality problem we are facing now, which i believe that it is the root cause for RM manual cherry pick issue. We cannot just adopt a process by ignoring its root quality problem. Sorry that your answer here

Re: Cancel VM migration when cluster is facing issues

2014-08-15 Thread Min Chen
There should be a global configuration for this: enable.ha.storage.migration. You can turn that off. Thanks -min From: Amin Samir mailto:a...@opencloud.net.au>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Thursday, August 14,

Re: [SHOW] Authentication refactoring

2014-08-14 Thread Min Chen
Sorry, I might just get part of your commit yesterday when I looked. Now I saw them. Thanks for your clarification. -min On 8/14/14 10:01 AM, "Rohit Yadav" wrote: > >On 14-Aug-2014, at 6:54 pm, Min Chen wrote: > >> Hi Rohit, >> >> Any reason why you didn

Re: [SHOW] Authentication refactoring

2014-08-14 Thread Min Chen
Hi Rohit, Any reason why you didn't implement response class for login and logout like any other API cmd? I think that will be useful as mentioned in your FS. Thanks -min On 8/12/14 2:10 AM, "Rohit Yadav" wrote: >This was done: >https://cwiki.apache.org/confluence/displ

Re: [DISCUSS] Removing template URL format checking logic

2014-08-12 Thread Min Chen
> >On 12-Aug-2014, at 8:53 pm, Min Chen wrote: > >> A better fix to address this issue has been committed to master: >> >>https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=e3564 >>65 >> 8befaa72cbe5fd510bea2a25b40f108f5. This should address

Re: [SHOW] Authentication refactoring

2014-08-12 Thread Min Chen
Hi Rohit, My understanding is that you will do this on your feature branch "auth-refactor", then merge them after passing at least some CI automation tests. Today, I saw all these commits already in master: 10 hours agoRohit Yadav DefaultLoginAPIAuthenticatorCmd: return userId as U

Re: [DISCUSS] Removing template URL format checking logic

2014-08-12 Thread Min Chen
ter iso/template/volume. It should be one util method. >But would you know all the valid formats a URL can be - or will you just >check for say 'vhd' in the URL ? > >Thanks, >-Nitin > >On 11/08/14 5:26 PM, "Min Chen" wrote: > >>Yes. But on secon

Re: [DISCUSS] Removing template URL format checking logic

2014-08-11 Thread Min Chen
Nitin > >On 11/08/14 3:01 PM, "Min Chen" wrote: > >>Just checked in a fix to remove similar format check for ISO for >>https://issues.apache.org/jira/browse/CLOUDSTACK-7312. >> >>Thanks >>-min >> >>On 8/11/14 9:59 AM, "Rohit Yadav&quo

Re: [DISCUSS] Removing template URL format checking logic

2014-08-11 Thread Min Chen
>discussing it with Marcus on the issue. > >Cheers. > >On 11-Aug-2014, at 6:54 pm, Min Chen wrote: > >> Yes, I have filed and fixed that bug a month ago. >> >> Thanks >> -min >> >> On 8/10/14 10:39 PM, "Harikrishna Patnala"

Re: [DISCUSS] Removing template URL format checking logic

2014-08-11 Thread Min Chen
Yes, I have filed and fixed that bug a month ago. Thanks -min On 8/10/14 10:39 PM, "Harikrishna Patnala" wrote: >Hi Rohit, > >This was already fixed for templates >https://issues.apache.org/jira/browse/CLOUDSTACK-6940 > >We have to do the same for registering ISOs as well. > > >Thanks, >Harikri

Re: [PROPOSAL] Solving the cherry-picking problem

2014-08-07 Thread Min Chen
Hi Rohit, IMHO, the root cause for RM cherry-pick problem is code quality. Without solving that first based on some kind of enforcement, this will not help much. The reason to use forward branch and RM cherry-picking is to control what can go to release branch. Your proposal removed that p

Re: [VOTE] git workflow

2014-08-05 Thread Min Chen
>> I don't see any gains from this, either. >> I vote "-1". >> > >It is much more than shifting pains, the wiki page is not discussion the >workflow quite extensively, with several pointers that we should all take >the time to view/read: > >

Re: [VOTE] git workflow

2014-08-05 Thread Min Chen
Agree with Animesh. Didn't see any gains from this, we just shift pain from one branch to another, so vote -1. -min On 8/2/14 9:50 PM, "Animesh Chaturvedi" wrote: >+0 > >While this protects master with only commits which are merges from >release branch and keeps it clean but the issues that we

Re: 4.4-forward picked empty

2014-07-30 Thread Min Chen
Daan, That commit is already there in 4.4 branch. See https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=6ba541a fb73475758a62a17dae1ec1be088da810. Thanks -min On 7/30/14 5:33 AM, "Daan Hoogland" wrote: >thanks Rajani > >On Wed, Jul 30, 2014 at 2:20

Re: download template -> delete vhd

2014-07-23 Thread Min Chen
In current ACS master, Template is not deleted from secondary storage when extractTemplate is called, just its symlink is deleted. Thanks -min On 7/23/14 4:15 AM, "Tomasz Zięba" wrote: >Hello, > >Could someone confirm that download template deletes the vhd file from >secondary storage. > >We ar

Re: Replace primary & secondary storage

2014-07-22 Thread Min Chen
rage id >information. it was quite helpful. > >I found this blog >http://stankirdey.com/content/cloudstack-merging-secondary-storages > >but he is using `template_host_ref`for ACS 4.2. > >Is there any documented procedure to replace secondary storage for ACS 4.3 >? > >Re

Re: [DISCUSS] [PROPOSAL] SAML2 plugin for SSO/SLO in CloudStack

2014-07-21 Thread Min Chen
+1. Very well-written FS and email, Rohit. Those open questions are very valid, I added a little comment in your FS regarding the flow. Thanks -min On 7/20/14 8:35 AM, "Rohit Yadav" wrote: >Hi, > >I'm assuming no one objects the proposal and the spec, I'll move forward >with the first implement

Re: Replace primary & secondary storage

2014-07-21 Thread Min Chen
That article only applied to ACS 4.1 and older versions. In ACS 4.2 storage refactor, db tables are changed. template_host_ref has been deprecated and replaced with template_store_ref. Thanks -min On 7/21/14 3:47 AM, "Tejas Gadaria" wrote: >Hi, > >I found this article http://support.citrix.com/

Re: Documentation on cloud-engine and cloudstack-framework

2014-07-14 Thread Min Chen
Hi Rohit, Regarding IAM, here is the FS for the work Prachi and I have done: https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+Identity+ and+Access+Management+%28IAM%29+Plugin, but we disable it on ACS 4.4 due to API and service layer gap mentioned in https://cwiki.apache.o

Re: Why to have API Commands for Admin actions?

2014-06-25 Thread Min Chen
hat used to check if the user is an Admin/Domain >Admin/regular user against the Db. > >With IAM, we cannot have such if-else conditions around hardcoded roles. >The design should work with custom roles and custom 'response views' >allocated to the user. >This change is a f

Re: Why to have API Commands for Admin actions?

2014-06-24 Thread Min Chen
Hi Wilder, This is a recent change introduced by IAM feature, see FS here https://cwiki.apache.org/confluence/display/CLOUDSTACK/CloudStack+Identity+ and+Access+Management+%28IAM%29+Plugin, particularly see details in Response View section. The intention of this is to eventually provide cu

[ACS4.4] Cherry pick CLOUDSTACK-6971

2014-06-23 Thread Min Chen
Hi Daan, Would you please cherry pick the following commit from 4.4-forward to 4.4 branch? Commit: 31e250a9d2adbf9ee59da66073497e38c02ded86 CLOUDSTACK-6971: createAutoScaleVmProfile failed with NPE due to lack of bean Injection. Thanks -min

Re: S3 use with simulator

2014-06-20 Thread Min Chen
ndary storage that is s3 based, and that the only way >was to use the updatecloud Api > >-Sebastien > >> On 20 Jun 2014, at 18:49, Min Chen wrote: >> >> Hi Sebastien, >> >>I am curious why you are using updateCloudToUseObjectStore api to >>

Re: ID instead of name in deployVirtualMachine

2014-06-20 Thread Min Chen
That seems a convention in all of our APIs. For APIs with support sort of on-behalf-of semantics, currently we are always using (account, domainId) parameter pair, where account requires accountName. You can see similar patterns in list APIs. Of course, we can change this, but may need to be in nex

Re: S3 use with simulator

2014-06-20 Thread Min Chen
Hi Sebastien, I am curious why you are using updateCloudToUseObjectStore api to add S3 as a secondary store in your data center? To set up a data center using S3, you need to use addImageStore api. Thanks -min On 6/20/14 7:47 AM, "Sebastien Goasguen" wrote: >I switch to

Re: Can we delete a template after a VM is instantiated from it?

2014-06-18 Thread Min Chen
This seems a bug to me, start vm should not rely on the template not deleted. Please file a bug for that. Thanks -min On 6/16/14 10:33 AM, "Rafael Weingartner" wrote: >This way, it seems that you have a bug in CS 4.3.0 when starting a machine >that was created from a template that has been dele

Can you please check resizeVolume bug CLOUSTACK-6797?

2014-06-12 Thread Min Chen
Hi Marcus, Would you mind checking this resizeVolume bug CLOUSTACK-6797? It is about resizeVolume not checking storage pool capacity. Thanks -min

Re: [DISCUSS] Release 4.4

2014-06-11 Thread Min Chen
Just looked at automation setup with Rayees, CLOUDSTACK-6825 is not really an issue of createTemplateFromSnapshot, the real issue is that the snapshot passed to createTemplate command has its owner removed in DB, thus causing checkAccess failure. As for why the snapshot is still not removed when th

Re: [DISCUSS] Release 4.4

2014-06-11 Thread Min Chen
I have assigned CLOUDSTACK-6825 to me. But from the stack trace, it seems failing in doing check access on the snapshot owner that is not active anymore, feel like a racing condition happening on automation setup. Rayees, several points to clarify: 1. Is this still happening on recent automation ru

[ACS4.4]Cherry-pick 09a357fb90b48ed6e2725ea60e632a2ad5529f79

2014-06-10 Thread Min Chen
Hi Daan, Would you please cherry-pick the following commit from 4.4-forward to 4.4? Commit: 09a357fb90b48ed6e2725ea60e632a2ad5529f79 CLOUDSTACK-6890:createVPC invoked by admin does not observe start flag. Thanks -min

Re: [DISCUSS] Introducing Gerrit for quality? was: [PROPOSAL] Using continuous integration to maintain our code quality...

2014-06-09 Thread Min Chen
I like github pull request as well from past usage, it is very convenient for developers and reviewers to perform their tasks, compared to our current RB. Also agree with David, the pre-requisite for this enforcement is that we should have CI in place to make this happen. Thanks -min On 6/9/14 8:

Re: [ANNOUNCE] Demetrius Tsitrelis as committer

2014-06-09 Thread Min Chen
Congrats, Demetrius. -min On 6/6/14 4:17 PM, "John Kinsella" wrote: >Folks - this one¹s a little belated - we went through the invite process >around the >time of the mail issues, and somehow we didn¹t send the announcement to >dev@. >I noticed while doing some housekeeping this week, and wante

[ACS4.4]Cherry-pick CLOUDSTACK-6859

2014-06-07 Thread Min Chen
Hi Daan, Would you please cherry-pick the following commit from 4.4-forward to 4.4? Commit: 961cd7657e7dbb69d497123071db3156bd15d8e9 CLOUDSTACK-6859:Management Server PermGen run out of memory after some time due to class leak. Thanks -min

Re: [ACS4.4] [Issue] Unable to create a resource tag on ISO and Template resource

2014-06-04 Thread Min Chen
Santhosh, The API command to create a resource tag is createTagsCmd. From just ResourceObjectType enum listed in ResourceTag interface, Template/ISO is supported. Thanks -min On 6/4/14 7:57 AM, "Santhosh Edukulla" wrote: >Namitha, > >Actually, my point is mainly to say

Re: [ACS5.0] IAM feature postponed from 4.4 to 5.0?

2014-06-03 Thread Min Chen
1 >but keep them for account 2 is it possible? > >Thanks >Meghna. > > >On Thu, May 22, 2014 at 10:22 PM, Min Chen wrote: > >> Added API issues we found through IAM feature in the wiki page created >>by >> Demetrius: >> https://cwiki.apache.org/confluenc

Re: [ANNOUNCE] Amogh Vasekar as committer

2014-06-02 Thread Min Chen
Congratulations, Amogh. -min On 6/2/14 11:53 AM, "Ahmad Emneina" wrote: >oh dang! good stuff Amogh! > > >On Mon, Jun 2, 2014 at 11:29 AM, Rayees Namathponnan < >rayees.namathpon...@citrix.com> wrote: > >> Congrats Amogh. >> >> Regards, >> Rayees >> >> -Original Message- >> From: Nitin M

Re: [OFFLINE] Going away for some time

2014-06-02 Thread Min Chen
Welcome back, Rohit. -min On 6/2/14 9:56 AM, "Rohit Yadav" wrote: >Hi, > >I've found a way to sponsor myself to work full-time on the project again >in upcoming months, and will slowly and gradually start contributing to >the >project. Please do help me in catching up about the changes and >dev

Re: Erro: No remote endpoint to send command, check if host or ssvm is down?

2014-05-28 Thread Min Chen
ust need to stop the management servers, run that in every cluster >that I have and everything should be fine? > > >On Wed, May 28, 2014 at 5:42 PM, Min Chen wrote: > >> You should clear old systemvm.iso from xen server so that new >>systemvm.iso >> built can b

Re: [ACS 4.4] Blocker and Critical Defects 5/27

2014-05-28 Thread Min Chen
Done. Thanks -min On 5/28/14 2:01 PM, "Sudha Ponnaganti" wrote: >Can you resolve it so QA can pick it up. It is still in open state > >-Original Message----- >From: Min Chen [mailto:min.c...@citrix.com] >Sent: Tuesday, May 27, 2014 4:15 PM >To: dev@cloudstack.a

Re: Erro: No remote endpoint to send command, check if host or ssvm is down?

2014-05-28 Thread Min Chen
ce.NfsSecondaryStorageResource >>> Unable to start agent: Resource class not found: >>> org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource due >>>to: >>> java.lang.ClassNotFoundException: >>> org.apache.cloudstack.storage.resource.NfsSec

Re: Erro: No remote endpoint to send command, check if host or ssvm is down?

2014-05-28 Thread Min Chen
oked into the CS UI, infrastructure>System VMs and the column that has >some values about the agent-state has a -- in it. Is it normal? > >At the end, should the SSVM have an entry into the host table? > > >On Wed, May 28, 2014 at 2:05 PM, Min Chen wrote: > >> Check if ag

Re: Erro: No remote endpoint to send command, check if host or ssvm is down?

2014-05-28 Thread Min Chen
Check if agent is running your ssvm. Thanks -min On 5/28/14 7:26 AM, "Rafael Weingartner" wrote: >Hi all, >sorry, to bother you again with this thread. >Could at least someone that has the CS 4.3.0 deployed try to download a >volume and/or a template? > >Could someone also check the database, t

Re: [ACS 4.4] Blocker and Critical Defects 5/27

2014-05-27 Thread Min Chen
OVS] Can't create more than 7 GRE tunnel >networks in xen cluster Murali Reddy >CLOUDSTACK-6710 [Automation] VM snapshot failing with NPE in >vmware Likitha Shetty >CLOUDSTACK-6644 Unable to attach Volume to a VM as a System User >Min Chen >CLOUD

[ACS4.4] Cherry pick d5fbcafc2ff04e665c2ec8079c759fe4cabf9d1a

2014-05-23 Thread Min Chen
Hi Daan, Can you please cherry-pick commit d5fbcafc2ff04e665c2ec8079c759fe4cabf9d1a from 4.4-forward to 4.4 branch? This commit removes windows line ending from IAMService.java. Thanks -min

[ACS4.4] Cherry-pick da5ad74d5f4388c4aa1df2f2e5f9053bfb70d83d

2014-05-22 Thread Min Chen
Hi Daan, Would you please cherry-pick the following commit from 4.4-forward to 4.4? Commit: da5ad74d5f4388c4aa1df2f2e5f9053bfb70d83d CLOUDSTACK-6752: IAM command class separation caused ApiDoc warning of duplicated cmd class for the same api name. Thanks

[ACS4.4] Cherry-pick 0d243ec7f2e1a0e3508d385f7b793b103c56ca07

2014-05-22 Thread Min Chen
Hi Daan, Would you please cherry pick the following commit from 4.4-forward branch to 4.4 branch? Commit: 0d243ec7f2e1a0e3508d385f7b793b103c56ca07 CLOUDSTACK-6745:DomainAdmin is not able to deploy Vm for users in his domain/subdomain. Thanks -min

Re: [ACS5.0] IAM feature postponed from 4.4 to 5.0?

2014-05-22 Thread Min Chen
Added API issues we found through IAM feature in the wiki page created by Demetrius: https://cwiki.apache.org/confluence/display/CLOUDSTACK/API+changes Thanks -min On 5/14/14 9:34 AM, "Min Chen" wrote: >Thanks Daan. Yes, I saw that there is another thread about putting an API >

[ACS4.4] Cherry-pick 356f6121a78d147d72136044c90472234f667730

2014-05-21 Thread Min Chen
Hi Daan, Can you please cherry-pick this commit from 4.4-forward to 4.4? Commit: 356f6121a78d147d72136044c90472234f667730 CLOUDSTACK-6742: listVolumes - As regularuser , able to list Vms and volumes of other users. Thanks -min

Re: [ACS4.4] Cherry-pick f748a552e9546e91e18c574b375f3ea6c8d7e043

2014-05-20 Thread Min Chen
uot;git rm test/integration/smoke/test_vm_iam.py". I can do that if you allow me to check into 4.4. Thanks. -min On 5/20/14 2:12 PM, "Min Chen" wrote: >Ok. Let me try. Can I cherry pick this into 4.4 after resolving the >conflict? > >Thanks >-min

Re: [ACS4.4] Cherry-pick f748a552e9546e91e18c574b375f3ea6c8d7e043

2014-05-20 Thread Min Chen
Ok. Let me try. Can I cherry pick this into 4.4 after resolving the conflict? Thanks -min On 5/20/14 1:44 PM, "Daan Hoogland" wrote: >Sorry Min, > >I didn't check the conflict, just aborted. > >On Tue, May 20, 2014 at 6:20 PM, Min Chen wrote: >> Hi Daan, &g

Re: Patch missing in 4.4-forward

2014-05-20 Thread Min Chen
I ran into same issues in analyzing automation test. The related integration tests are merged into 4.4-forward along with marvin test framework merge, but this patch does not exist in 4.4-forward, so we should either disable these two integration tests, or pull that patch code into 4.4-forward. Th

Re: [ACS4.4] Cherry-pick f748a552e9546e91e18c574b375f3ea6c8d7e043

2014-05-20 Thread Min Chen
Hi Daan, What is the conflict? Do you need me to resolve the conflict? Sorry, just realized that the URL is internal to citrix, not visible to outside. Thanks -min On 5/20/14 1:16 AM, "Daan Hoogland" wrote: >Min, > >On Tue, May 20, 2014 at 1:30

[ACS4.4] Cherry-pick f748a552e9546e91e18c574b375f3ea6c8d7e043

2014-05-19 Thread Min Chen
Hi Daan, As we proposed earlier in community, since IAM feature is not ready for 4.4 release due to backwards compatibility issue, we need to disable IAM on 4.4. I have checked in commit f748a552e9546e91e18c574b375f3ea6c8d7e043 on 4.4-forward to disable IAM, would you please cherry-pick this c

Re: [PROPOSAL] Add getPathSeparator() to DataStoreTO.java

2014-05-19 Thread Min Chen
The proposal looks ok to me. But it may involve several changes in our code base, some places in our code we even used hard-coded separator instead of using constants. With this change, some global cleanup needs to be done to make sure that it will not bring regressions. Thanks -min On 5/13/14 10

Re: seeing "Unknown parameters : ctxdetails" for addResourceDetail/removeResourceDetail

2014-05-16 Thread Min Chen
Ctxdetails complained in your warning log is one of internal parameters added by ApiDispatcher, and is not publicly presented in the API Cmd class. For those parameters, they are not errors in the request and nothing to be fixed. Thanks -min On 5/14/14 12:46 AM, "Antonio Fornié Casarrubios" wrot

Re: [ACS4.4] Cherry-pick CLOUDSTACK-6600

2014-05-16 Thread Min Chen
Daan, we found the file with windows line endings. It will be fixed in our disabling IAM commit to be checked in next week. Thanks -min On 5/16/14 3:26 PM, "Min Chen" wrote: >Daan, this commit is already pulled into 4.4 by you. I didn't see trailing >white space on my ecl

Re: [ACS4.4] Cherry-pick CLOUDSTACK-6600

2014-05-16 Thread Min Chen
Daan, this commit is already pulled into 4.4 by you. I didn't see trailing white space on my eclipse? Which file are you talking about? Thanks -min On 5/16/14 12:54 AM, "Daan Hoogland" wrote: >On Thu, May 8, 2014 at 2:19 AM, Min Chen wrote: >> 500c99eef7f7a04d95ed95697

[ACS4.4] Cherry-pick 3 bugs

2014-05-16 Thread Min Chen
Hi Daan, Would you please cherry-pick the following 3 bug fixes from 4.4-forward to 4.4 branch? Commit: 500c99eef7f7a04d95ed95697a2678fef283e61d CLOUDSTACK-6600:IAM Security checker needs to have cache to improve checkAccess performance. Commit: b9bdaf9d31803cd080475e27da56695c3293c4ca CLOUDSTA

Re: [ACS5.0] IAM feature postponed from 4.4 to 5.0?

2014-05-14 Thread Min Chen
add to the discussion what it is >that is keeping you from implementing. And what requirements you have >for the 5.0 api so we can start devising the architectural guidelines >for the new api. more and more calls for a 5.0 are coming up lately so >let's move forward. (changing title) &

Re: Problem with mailing list....

2014-05-13 Thread Min Chen
I had the same issue, my email sent a while back is still not sent to community yet. -min On 5/13/14 4:57 PM, "Alex Huang" wrote: >I've find some of my emails did not make it to the mailing list. Does >anyone else have this problem? > >--Alex

[ACS 4.4] Proposal to Disable IAM feature in 4.4

2014-05-13 Thread Min Chen
Hi All, In the past several weeks, QA has done some testing on IAM feature and found several backward-compatibility issues. Even though Prachi and I have tried our best to fix bugs to maintain backward compatibility, we realized that in order to support true IAM model documented in our FS http

[ACS 4.4] Proposal to Disable IAM feature in 4.4

2014-05-13 Thread Min Chen
Hi All, In the past several weeks, QA has done some testing on IAM feature and found several backward-compatibility issues. Even though Prachi and I have tried our best to fix bugs to maintain backward compatibility, we realized that in order to support true IAM model documented in our FS http

Re: [QUERY] [IAM TEST CASE] Vritual Machine IAM test scenarios

2014-05-12 Thread Min Chen
it and also the teardown >once its done so that it leaves the system clean in the same state it >started with. >This also helps in easy understanding of what is being tested. >I think we should file a bug for this and fix it. > >~Rajani > > > >On 12-May-2014, at

[ACS44] Cherry pick requests

2014-05-12 Thread Min Chen
Hi Daan, Can you please cherry pick the following commits from 4.4-forward to 4.4 branch? Some of them are resending previous requests. Commit: 500c99eef7f7a04d95ed95697a2678fef283e61d CLOUDSTACK-6600:IAM Security checker needs to have cache to improve checkAccess performance. Commit:b

Re: [QUERY] [IAM TEST CASE] Vritual Machine IAM test scenarios

2014-05-12 Thread Min Chen
st_03, account_1B should have two grants effective, and thus he can see 3 VMs. Thanks -min From: Sonal Ojha mailto:sonal.o...@sungardas.com>> Date: Monday, May 12, 2014 5:04 AM To: Prachi Damle mailto:prachi.da...@citrix.com>>, Min Chen mailto:min.c...@citrix.com>> Cc: &qu

  1   2   3   4   5   >