Re: Fault percentage value of CPU usage in Cloud Platform

2016-11-21 Thread Bharat Kumar
Hi, There may be a difference in what you have allocated and what is being actually used. The dashboard shows what is allocated. Regards, Bharat. On 11/21/16, 9:44 PM, "williamstev...@gmail.com on behalf of Will Stevens" wrote: >You will have to contact Accelerite for support with ACP (pre

Re: [VOTE] Split Marvin to its own repository

2016-07-19 Thread Bharat Kumar
Hi Rohit, what are we trying to achieve by moving marvin into a separate repo.? --Bharat. From: Raja Pullela Sent: Tuesday, July 19, 2016 5:30:20 PM To: dev@cloudstack.apache.org Subject: Re: [VOTE] Split Marvin to its own repository Hi Rohit, same question

Incorrect system vm templates for master build.

2016-04-28 Thread Bharat Kumar
Hi All, looks like the systemvm templates are not getting built correctly in http://jenkins.buildacloud.org/job/build-systemvm64-master. The build version is in correctly set to 4.6, Im not sure but i think the build version in /etc/cloudstck-release is also getting wrongly set inside the temp

Re: [DISCUSS] PR testing process

2016-03-30 Thread Bharat Kumar
that. > > Most likely github already reports that PR as 'unstable' so you could also > check for that. > > Regards, Remi > > Sent from my iPhone > >> On 28 Mar 2016, at 09:28, Bharat Kumar wrote: >> >> Hi Sanjeev, >> >> Thanks for bri

Re: [DISCUSS] PR testing process

2016-03-28 Thread Bharat Kumar
Hi Sanjeev, Thanks for bringing this up. This is happening because the PR code and the master have deviated since the pr has been created(assuming that PR was rebased with master at the time of creation) and we start testing it. Ideally we should start testing as soon as the pr is created but

Re: introduction

2016-03-23 Thread Bharat Kumar
Welcome back Murali. —Bharat. > On 23-Mar-2016, at 9:50 PM, Rajesh Battala > wrote: > > Welcome Back Murali. Good to see you again on CloudStack :) > > -Original Message- > From: Murali Reddy [mailto:murali.re...@shapeblue.com] > Sent: Wednesday, March 23, 2016 5:21 PM > To: dev@clou

RE: [VOTE] Move 'apache/cloudstack' -> 'apache-cloudstack/cloudstack'

2016-03-20 Thread Bharat Kumar
+1 --Bharat On 21 Mar 2016 11:51 am, Kishan Kavala wrote: +1 for the move. Regards, Kishan Kavala Chief Product Engineer, CloudPlatform Development, Accelerite. www.accelerite.com -Original Message- From: chunfeng tian [mailto:tianyunqu...@gmail.com] Sent: 20

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-19 Thread Bharat Kumar
, Remi Bergsma mailto:rberg...@schubergphilis.com>> wrote: Hi Bharat I’d say remove the tests that fail, to get a report with only tests that pass. Regards, Remi From: Bharat Kumar mailto:bharat.ku...@accelerite.com>> Date: Tuesday 15 March 2016 at 10:40 To: Remi Bergsma

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-15 Thread Bharat Kumar
ht now, there are so many failures that soon people don’t even care to look at it. Once we have a stable set that works, we can start fixing tests and 1-by-1 add them to the mix. Just my $0.02 Regards, Remi On 15/03/16 07:06, "Bharat Kumar" mailto:bharat.ku...@accelerite.com>&g

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-14 Thread Bharat Kumar
Hi Paul, I totally agree with you. For now we should ignore the tests which fail all the time while we try to address them. Now that we have a CI in place we need go and fix issues with the tests. I have tried to identify a few of them and have mentioned them in the known issues part of the C

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-07 Thread Bharat Kumar
t, > > Good to see the list of failed and skipped ones in the report. Is it possible > to add some generic comment/reason to know why these test cases are > failed/skipped? > > Thanks, > Suresh > > > > > On 08/03/16 11:05 am, "Bharat Kumar" wrot

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-07 Thread Bharat Kumar
ber of hosts, etc) Report indicates few tests are skipped. is this due to setup limitations? can the skipped tests be also listed so that some of us can run those specific tests (say manually) and post the results. Thanks, Srinivas On Mon, Mar 7, 2016 at 11:06 PM, Bharat Kumar mailto:bha

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-07 Thread Bharat Kumar
Hi guys, I am also working on the similar reporting problem, here is what i did link to the report https://github.com/bvbharatk/cloud-stack/pull/1 I am thinking this is good enough for now, I want to start posting the results on each pr as shown in the above link. please give me your comments o

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-03 Thread Bharat Kumar
made things infinitely easier. > > Let me know if you have other questions. > > Cheers, > > Will > > > On Thu, Mar 3, 2016 at 11:29 PM, Bharat Kumar > wrote: > >> Hi will, >> >> we have a solution to post the results to the community by email. We

Re: [PROPOSAL] Minimum Viable CI Integration

2016-03-03 Thread Bharat Kumar
Hi will, we have a solution to post the results to the community by email. We also have a github integration to fetch the Prs , run tests against them and post the consolidated results by email and share the logs using dropbox. We are facing some setup delays to get this up and running. I am sur

Re: PR validation using proposed CI.

2016-03-02 Thread Bharat Kumar
; *Will STEVENS* > Lead Developer > > *CloudOps* *| *Cloud Solutions Experts > 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 > w cloudops.com *|* tw @CloudOps_ > > On Fri, Feb 26, 2016 at 2:27 PM, Sebastien Goasguen > wrote: > >> >>> On Feb 26, 2016, at 7:

Re: PR validation using proposed CI.

2016-02-26 Thread Bharat Kumar
rhaps you can explain the workflow and extensiveness of your tests. > > Thanks > ilya > > > > On 2/24/16 9:23 PM, Bharat Kumar wrote: >> Hi, >> >> As all of you know from earlier discussions in the community, we have been >> working on

Re: PR validation using proposed CI.

2016-02-25 Thread Bharat Kumar
u, Feb 25, 2016 at 6:23 AM, Bharat Kumar > wrote: > >> Hi, >> >> As all of you know from earlier discussions in the community, we have been >> working on implementing a CI to test github PRs and publish results. We >> have implemented this >> internally a

Re: ACS CI BVT RUN: xenserver: Pr_number: 1417

2016-02-25 Thread Bharat Kumar
enhance this to post directly on the PR. > > > >> On Feb 25, 2016, at 6:27 AM, Bharat Kumar wrote: >> >> >> On 24-Feb-2016, at 10:24 PM, >> jenkins-...@citrix.com<mailto:jenkins-...@citrix.com> wrote: >> >> [http://jenkins-ccp.citrix.com/sta

Re: ACS CI BVT RUN: xenserver: Pr_number: 1417

2016-02-24 Thread Bharat Kumar
On 24-Feb-2016, at 10:24 PM, jenkins-...@citrix.com wrote: [http://jenkins-ccp.citrix.com/static/e59dfe28/images/32x32/red.gif]BUILD FAILURE Build Start Date: Wed Feb 24 06:29:34 2016 Git Repo Url: https://github.com/apache/cloudstack.git Git Commit Id

PR validation using proposed CI.

2016-02-24 Thread Bharat Kumar
Hi, As all of you know from earlier discussions in the community, we have been working on implementing a CI to test github PRs and publish results. We have implemented this internally and will start testing PRs and publish the results by mail to dev list. At this point if a test fails, we do

Re: Important Pending Items

2016-02-11 Thread Bharat Kumar
ender if you believe you have received this email in error. -Original Message- From: Bharat Kumar [mailto:bharat.ku...@citrix.com] Sent: Thursday, February 11, 2016 9:57 AM To: dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org> Subject: Re: Important Pending Items Hi Se

Re: Important Pending Items

2016-02-11 Thread Bharat Kumar
Hi Sebastien, As Raja said, we are actively working on it and we will share the code on github soon. Thanks, Bharat. > On 11-Feb-2016, at 2:41 PM, Raja Pullela wrote: > > Hi Sebastien, > > On item (3) - the BVTs have been running consistently (with same passrates) > and haven't had the tim

Re: [PROPOSAL] Freeze everything until we get CI

2016-02-01 Thread Bharat Kumar
o hardware, and could use >> it to help test releases/PR >> >> >> Erik >> >> Den fredag 29. januar 2016 skrev Bharat Kumar >> følgende: >> >>> yes, we would be sharing it with the community and get this running in the >>> ACS infra. >

Re: [PROPOSAL] Freeze everything until we get CI

2016-01-28 Thread Bharat Kumar
keep this going. Thanks, Bharat. On 28-Jan-2016, at 7:37 PM, Erik Weber mailto:terbol...@gmail.com>> wrote: Why not share it as is, then the community could help improving this, rather than this being a single company effort? -- Erik On Thu, Jan 28, 2016 at 1:49 PM, Bharat

Re: [PROPOSAL] Freeze everything until we get CI

2016-01-28 Thread Bharat Kumar
Hi All, I agree that we need to have a CI to deal with the large volume of PRs. The current travis CI is not good enough as it runs only simulator tests. We identified this issue and came up with a effective CI for automating test runs for a each PR. This is already functional, with few github i

Re: [PROPOSAL] Freeze everything until we get CI

2016-01-27 Thread Bharat Kumar
Hi, I agree with Remi on the hurdles he mentioned. It is difficult to integrate 3rd party CI, If someone wants to implement their own CI, the link below gives one way to do it. https://cwiki.apache.org/confluence/display/CLOUDSTACK/Test+bed+orchestrator+and+test+runner+to+enable+continuos+integr

Re: User vm password gets update only the master VR.

2015-11-05 Thread Bharat Kumar
IMHO. Talked to Rajani and she agrees. > > Regards, > Remi > > > > On 05/11/15 10:29, "Bharat Kumar" wrote: > >> Hi Wilder, >> >> If we think that it is ok to have the work around for now fix this later, we >> can bring the priori

Re: User vm password gets update only the master VR.

2015-11-05 Thread Bharat Kumar
the issue > is not a blocker. > > Cheers, > Wilder > > >> On 05 Nov 2015, at 10:09, Bharat Kumar wrote: >> >> I have’t checked, but form the code it looks like it will work if we reissue >> the reset command after the slave becomes master. >> &

Re: User vm password gets update only the master VR.

2015-11-05 Thread Bharat Kumar
I have’t checked, but form the code it looks like it will work if we reissue the reset command after the slave becomes master. —Bharat. On 05-Nov-2015, at 12:59 pm, Erik Weber mailto:terbol...@gmail.com>> wrote: On Thu, Nov 5, 2015 at 7:15 AM, Bharat Kumar mailto:bharat.ku...@citr

User vm password gets update only the master VR.

2015-11-04 Thread Bharat Kumar
Hi, In my local setup i found this issue, The user VM password is not getting saved in the backup router. we send the save password command to both the VRs in a rvr enabled network, But the password gets saved only in the master VR. This happens because the password server is not running in th

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
rmed decision on whether to > use a particular release or not. I am not saying that postponing is the only > option. > > > > > -Koushik > > > On 28-Sep-2015, at 6:20 PM, Wilder Rodrigues > mailto:wrodrig...@schubergphilis.com>> > wrote: > > I a

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
ping the router(s), but not any further than that. > SNAT works fine in a normal/non-vpc network. > I'll try to look more into it later today. > > Have a nice day :) > > Lucian > > -- > Sent from the Delta quadrant using Borg technology! > > Nux! &

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
all those emails about > documentation/bugs when we all already know about it and are working hard to > get it in a better shape. > > Cheers, > Wilder > >> On 28 Sep 2015, at 13:54, Bharat Kumar wrote: >> >> Hi Remi, >> >> Whatever ever we think w

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
: > Dude, this is the final friendly email about his. All points have been made > in previous mails. This has nothing to do with ‘blameless’ and ‘learning’ > anymore. > > Read Seb’s mail. We will move on now. > > > Regards, Remi > > > > On 28/09/15 13

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
-Sep-2015, at 5:24 pm, Bharat Kumar mailto:bharat.ku...@citrix.com>> wrote: Hi Remi, Whatever ever we think we have discovered are all well known best practices while developing code in community. I agree that tests need to be run on a new PR, but i wonder why was this ignored when mergi

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
but we are not going to > revert it, this would not go cleanly as David mentioned. > > So let’s get back to blockers for 4.6, are there still VR related issues with > master ? > > > > >> Regards, >> Remi >> >> >> >> >> >

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
pass. Even better is we write (missing) tests before changing stuff so you > know they pass before and after the fact. > > When doing reviews, feel free to ask for design documentation if you feel it > is needed. > > Regards, Remi > > > > On 28/09/15 11:02, "B

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
en add a test :-) > > I do agree on the design documentation requirement for any (major?) change. I > found some design documentations on the subject you mention, but it should > have been more detailed. > > Regards, > Remi > > > > > > > On 28/09/15

Re: Blameless post mortem

2015-09-28 Thread Bharat Kumar
Hi Remi, Thank you for the Blame less postmortem. I think there is a bigger problem here than just the review process and running tests. Even if we run the tests we cannot be sure that every thing will work as intended. The tests will only give some level of confidence. The tests may not cove

Re: Proposal to get to 100% passrate on BVTs

2015-09-22 Thread Bharat Kumar
is any specific ask ? Thanks, Raja -Original Message- From: Bharat Kumar [mailto:bharat.ku...@citrix.com] Sent: Tuesday, September 22, 2015 10:03 AM To: dev@cloudstack.apache.org<mailto:dev@cloudstack.apache.org> Subject: Re: Proposal to get to 100% passrate on BVTs Hi, I am alr

Re: Proposal to get to 100% passrate on BVTs

2015-09-21 Thread Bharat Kumar
Hi, I am already working towards this. We can already replicate the test infra using puppet. But i need some help in generating reports, sharing the test results and archiving them. particularly archiving them since we need some space which is accessible by everyone. I have also made a propos

Re: [Blocker/Critical] VR related Issues

2015-09-03 Thread Bharat Kumar
K-8799<https://issues.apache.org/jira/browse/CLOUDSTACK-8799> Thanks, Bharat. On 12-Aug-2015, at 10:52 am, Bharat Kumar mailto:bharat.ku...@citrix.com>> wrote: Hi, looks like there is one more issue. Conntrackd fails to start in case of rvr enabled isolated networks. created a

Re: [Blocker/Critical] VR related Issues

2015-08-11 Thread Bharat Kumar
Hi, looks like there is one more issue. Conntrackd fails to start in case of rvr enabled isolated networks. created a bug to track this. https://issues.apache.org/jira/browse/CLOUDSTACK-8725 Thanks, Bharat. On 11-Aug-2015, at 3:03 pm, Kishan Kavala mailto:kishan.kav...@citrix.com>> wrote:

Re: [Proposal] Minimise VR downtime during network update.

2015-08-11 Thread Bharat Kumar
. Thanks, Bharat. On 11-Aug-2015, at 9:16 pm, Daan Hoogland wrote: > Can you make a ticket on apache instead of referring an internal ticket? > > Can you describe how this work relates to the rvpc refactor work done for > 4.6? > > > > On Tue, Aug 11, 2015 at 5:41 P

[Proposal] Minimise VR downtime during network update.

2015-08-11 Thread Bharat Kumar
Hi, Currently we use redundant virtual router enabled networks to prevent network outage even if one of the virtual router fails. This is achieved by using redundant virtual routers along with VRRP protocol. This has increased the network robustness. However updating a network with new network

Re: [Proposal] Continuos Integration Testing on ACS master.

2015-07-23 Thread Bharat Kumar
> > Good work Bharat, Keep in mind that many people work on this in many ways and > many attempts have failed due to nothing but lack of consensus. I have only > glanced but will take my time with it soon. > > On Thu, Jul 23, 2015 at 10:50 AM, Bharat Kumar > wrote: >&g

[Proposal] Continuos Integration Testing on ACS master.

2015-07-23 Thread Bharat Kumar
Hi All, Currently we do sanity checks on master using Travis, But travis runs only simulator tests (AFAIK). I think it will be better if we can run both BVTs and regression tests on real hardware. I am woking on implementing this. Functional spec and design details. https://cwiki.apache.org/con

Re: file /var/log/cloudstack/agent from install of cloudstack-agent-4.5.1-1.el7.centos.x86_64 conflicts with file from package cloudstack-management-4.5.1-1.el7.centos.x86_64

2015-07-15 Thread Bharat Kumar
Hi, I totally agree with Wido, This should be fixed in the ACS repo for every one to use, Instead of sharing this using non ACS private builds. Thanks, Bharat. On 15-Jul-2015, at 2:41 pm, Rohit Yadav mailto:rohit.ya...@shapeblue.com>> wrote: On 14-Jul-2015, at 9:23 pm, Wido den Hollander ma

Re: 4.5 clustered mgmt server

2015-04-15 Thread Bharat Kumar
Hi, I guss you are correct. we need to add the newly created channel ch1 to the map after the handshake is complete. _peers.put(peerName, ch) should be changed to _peers.put(peerName, ch1); and also return ch1 instead of ch. Thanks, Bharat. On 15-Apr-2015, at 12:21 pm, Marcus wrote: > Is

Regarding Cloudstack shutdown operations.

2015-04-13 Thread Bharat Kumar
Hi, Recently i noticed that when we stop cloudstack we do not mark the state of the management server as down in the ms_host table. On further investigation I came to know that the stop methods of the beans are not getting executed during shutdown. I could not see any logs related to cloudsta

Re: Review Request 29364: [Dashboard] Primary Storage Used(type tag with value 2) related tag is not showing in listCapacity api response.

2014-12-23 Thread bharat kumar
/29364/diff/ Testing --- tested on 4.5 Thanks, bharat kumar

Re: Review Request 29365: [Overcommit]Static Min/Max memory is not getting configured according to overcommit factor for vms on xen 6.2

2014-12-23 Thread bharat kumar
a090b1106ac4fb1acf3680991e990931d951fb65 Diff: https://reviews.apache.org/r/29365/diff/ Testing --- Thanks, bharat kumar

Review Request 29365: [Overcommit]Static Min/Max memory is not getting configured according to overcommit factor for vms on xen 6.2

2014-12-23 Thread bharat kumar
: https://reviews.apache.org/r/29365/diff/ Testing --- Thanks, bharat kumar

Review Request 29364: [Dashboard] Primary Storage Used(type tag with value 2) related tag is not showing in listCapacity api response.

2014-12-23 Thread bharat kumar
/diff/ Testing --- tested on 4.5 Thanks, bharat kumar

Review Request 28217: when a template is deleted and then copied over again , it is still marked as Removed in template_zone_ref table

2014-11-18 Thread bharat kumar
/VMTemplateZoneDaoImpl.java 8cadf61 Diff: https://reviews.apache.org/r/28217/diff/ Testing --- Thanks, bharat kumar

Review Request 28085: Fixed some coverity issues.

2014-11-14 Thread bharat kumar
://reviews.apache.org/r/28085/diff/ Testing --- Thanks, bharat kumar

Re: Review Request 27868: InvalidParameter Exception with stacktrace in MS log wile executing scale vm.

2014-11-11 Thread bharat kumar
) - server/src/com/cloud/vm/UserVmManagerImpl.java 77ace7a5f6c7f6e423a7e97bbf4c4bb5fdcb730a Diff: https://reviews.apache.org/r/27868/diff/ Testing --- Thanks, bharat kumar

Re: Review Request 27868: InvalidParameter Exception with stacktrace in MS log wile executing scale vm.

2014-11-11 Thread bharat kumar
-- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27868/#review60943 --- On Nov. 11, 2014, 2:19 p.m., bharat kumar wrote: > > --

Re: Review Request 27868: InvalidParameter Exception with stacktrace in MS log wile executing scale vm.

2014-11-11 Thread bharat kumar
- server/src/com/cloud/vm/UserVmManagerImpl.java 77ace7a5f6c7f6e423a7e97bbf4c4bb5fdcb730a Diff: https://reviews.apache.org/r/27868/diff/ Testing --- Thanks, bharat kumar

Review Request 27868: InvalidParameter Exception with stacktrace in MS log wile executing scale vm.

2014-11-11 Thread bharat kumar
--- Thanks, bharat kumar

Review Request 27021: Reservations for VMware VMs remain after dynamic scaling

2014-10-21 Thread bharat kumar
/VmwareResource.java 085b6bbed4918be3155fd44c8bed785983526e11 Diff: https://reviews.apache.org/r/27021/diff/ Testing --- Thanks, bharat kumar

Review Request 26969: During VM deployment and while choosing CUSTOM disk offering for data disk, the size of the data disk is not considered for checking resource limit of primary storage.

2014-10-21 Thread bharat kumar
limit of primary storage. Diffs - server/src/com/cloud/vm/UserVmManagerImpl.java 2636096d03d7cfa097605ac066b8bd66b24972b7 Diff: https://reviews.apache.org/r/26969/diff/ Testing --- tested on 4.5 Thanks, bharat kumar

Review Request 25991: user vm can get a gateway ip when gateway ip is a part of the guest ip range.

2014-09-24 Thread bharat kumar
/ConfigurationManagerImpl.java df6ce45 Diff: https://reviews.apache.org/r/25991/diff/ Testing --- Thanks, bharat kumar

Review Request 25768: changing value of cpu/mem.overprovisioning.factor for xen cluster is not affecting total memory at zone level

2014-09-17 Thread bharat kumar
/ Testing --- Tested on master. Thanks, bharat kumar

Regarding gateway ip getting allocated to user VMs in shared network.

2014-09-11 Thread Bharat Kumar
Hi All, I have found that the gateway ip can get allocated to the uservms in case of shared networks. This is a corner case and might happen accidentally. This happens because we allow addition guest ip ranges containing the gateway ip, while creating a shared network and internally there is n

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

2014-09-08 Thread bharat kumar
https://issues.apache.org/jira/browse/CLOUDSTACK-6099 Diffs - engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 8edef77 Diff: https://reviews.apache.org/r/25430/diff/ Testing --- Tested live migration on master using xenserver. Thanks, bharat kumar

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

2014-09-08 Thread bharat kumar
- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25430/#review52575 ------- On Sept. 8, 2014, 9:16 a.m., bharat kumar wrote: > >

Re: Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

2014-09-08 Thread bharat kumar
) - engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 8edef77 Diff: https://reviews.apache.org/r/25430/diff/ Testing --- Tested live migration on master using xenserver. Thanks, bharat kumar

Review Request 25430: live migration is failing for vm deployed using dynaic compute offerings with NPE

2014-09-08 Thread bharat kumar
--- Tested live migration on master using xenserver. Thanks, bharat kumar

Re: Review Request 24892: passwd_server attempts to start but terminates with the exit code 137

2014-08-20 Thread bharat kumar
/debian/config/opt/cloud/bin/passwd_server 0f4a772 Diff: https://reviews.apache.org/r/24892/diff/ Testing (updated) --- I was able to reproduce this in the older version but not on master. but iam putting this patch anyway as this will not effect the regular runs. Thanks, bharat kumar

Review Request 24892: passwd_server attempts to start but terminates with the exit code 137

2014-08-20 Thread bharat kumar
0f4a772 Diff: https://reviews.apache.org/r/24892/diff/ Testing --- Thanks, bharat kumar

Re: Review Request 23806: Re-copying templates to other zones doesn't work

2014-08-20 Thread bharat kumar
hanks, bharat kumar

Re: Review Request 23806: Re-copying templates to other zones doesn't work

2014-08-19 Thread bharat kumar
e/src/org/apache/cloudstack/storage/image/db/TemplateDataStoreDaoImpl.java 50334f4 server/src/com/cloud/template/HypervisorTemplateAdapter.java 72aaff5 server/src/com/cloud/template/TemplateManagerImpl.java f5ad97f Diff: https://reviews.apache.org/r/23806/diff/ Testing --- Thanks, b

Re: Review Request 23806: Re-copying templates to other zones doesn't work

2014-08-19 Thread bharat kumar
hanks, bharat kumar

Review Request 24645: Disabling failed test cases.

2014-08-13 Thread bharat kumar
/integration/smoke/test_snapshots.py 5db3e40 test/integration/smoke/test_templates.py a757c2a Diff: https://reviews.apache.org/r/24645/diff/ Testing --- Thanks, bharat kumar

Re: Review Request 23808: listCapacity API missing types for certain zones

2014-07-22 Thread bharat kumar
/capacity/dao/CapacityDaoImpl.java b7784f2 server/src/com/cloud/server/ManagementServerImpl.java 99b12732 Diff: https://reviews.apache.org/r/23808/diff/ Testing --- Thanks, bharat kumar

Re: Review Request 23806: Re-copying templates to other zones doesn't work

2014-07-22 Thread bharat kumar
e/src/org/apache/cloudstack/storage/image/db/TemplateDataStoreDaoImpl.java 50334f4 server/src/com/cloud/template/HypervisorTemplateAdapter.java 20309d6 server/src/com/cloud/template/TemplateManagerImpl.java f5ad97f Diff: https://reviews.apache.org/r/23806/diff/ Testing --- Thanks, b

Review Request 23808: listCapacity API missing types for certain zones

2014-07-22 Thread bharat kumar
server/src/com/cloud/server/ManagementServerImpl.java 99b12732 Diff: https://reviews.apache.org/r/23808/diff/ Testing --- Thanks, bharat kumar

Review Request 23807: List templates API returns destroyed templates when recopying the template.

2014-07-22 Thread bharat kumar
/TemplateJoinDao.java 298be4d server/src/com/cloud/api/query/dao/TemplateJoinDaoImpl.java 4c8ada1 Diff: https://reviews.apache.org/r/23807/diff/ Testing --- Thanks, bharat kumar

Review Request 23806: Re-copying templates to other zones doesn't work

2014-07-22 Thread bharat kumar
sting --- Thanks, bharat kumar

Review Request 23193: mark the test_01_primary_storage_iscsi as test that requires hardware

2014-06-30 Thread bharat kumar
- test/integration/smoke/test_primary_storage.py 0312ad6 Diff: https://reviews.apache.org/r/23193/diff/ Testing --- Thanks, bharat kumar

Re: [ACS4.5] Can't access volumes on storage tab in GUI

2014-06-30 Thread Bharat Kumar
Hi Mike, I think the volume_view did not get created properly for some reason. I think recreating the volume_view will fix the problem. below commas should fix the issue. DROP VIEW IF EXISTS `cloud`.`volume_view`; CREATE VIEW `cloud`.`volume_view` AS select volumes.id, volum

Review Request 22375: tagging the test case test_deploy_vgpu_enabled_vm with the bugId.

2014-06-09 Thread bharat kumar
/integration/smoke/test_deploy_vgpu_enabled_vm.py 13fad61 Diff: https://reviews.apache.org/r/22375/diff/ Testing --- Thanks, bharat kumar

Review Request 22379: tagging the test case test_vpc_site2site_vpn with the bugId.

2014-06-09 Thread bharat kumar
/integration/smoke/test_vpc_vpn.py 03826e9 Diff: https://reviews.apache.org/r/22379/diff/ Testing --- Thanks, bharat kumar

Review Request 22378: tagging the test case test_01_snapshot_root_disk with the bugId.

2014-06-09 Thread bharat kumar
/integration/smoke/test_snapshots.py a960e70 Diff: https://reviews.apache.org/r/22378/diff/ Testing --- Thanks, bharat kumar

Review Request 22377: tagging the test case test_01_sys_vm_start with the bugId.

2014-06-09 Thread bharat kumar
/integration/smoke/test_secondary_storage.py 481d907 Diff: https://reviews.apache.org/r/22377/diff/ Testing --- Thanks, bharat kumar

Review Request 22376: tagging the test case test_09_delete_detached_volume with the bugId.

2014-06-09 Thread bharat kumar
- test/integration/smoke/test_volumes.py 73c2722 Diff: https://reviews.apache.org/r/22376/diff/ Testing --- Thanks, bharat kumar

Review Request 22229: Test case failure in test_iso.py

2014-06-04 Thread bharat kumar
/integration/smoke/test_iso.py a3e42f1 Diff: https://reviews.apache.org/r/9/diff/ Testing --- Thanks, bharat kumar

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-5077

2014-05-05 Thread Bharat Kumar
Hi Daan, updating the link to diff. https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=99b4cf788eded0f6f155ff8cda51aeb2e505f764 Thanks Bharat. On 05-May-2014, at 3:36 pm, Bharat Kumar mailto:bharat.ku...@citrix.com>> wrote: Hi Daan, please cherry pick the following comm

[ACS 4.4] cherry-pick request for CLOUDSTACK-5077

2014-05-05 Thread Bharat Kumar
Hi Daan, please cherry pick the following commit to 4.4 branch. commit 489bb0c7edf6ab011895d5f082328eb5fe48aac1 in cloudstack 4.4-forward branch. link to diff Cloudstack-5077: reserve cpu and memory only when vmware.reserve.cpu/mem are …

Re: Review Request 15349: Overprivisoning changes needed

2014-04-16 Thread bharat kumar
/com/cloud/hypervisor/vmware/resource/VmwareResource.java e89777d setup/db/db/schema-420to421.sql 42e9c1e Diff: https://reviews.apache.org/r/15349/diff/ Testing --- Thanks, bharat kumar

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

2014-04-16 Thread bharat kumar
pache/cloudstack/storage/image/db/TemplateDataStoreDaoImpl.java f5140e0 server/src/com/cloud/template/HypervisorTemplateAdapter.java 963aec9 server/src/com/cloud/template/TemplateManagerImpl.java 57f7ff6 Diff: https://reviews.apache.org/r/20210/diff/ Testing --- Tested on 4.2 Thanks, b

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

2014-04-10 Thread bharat kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20210/#review40039 ------- On April 10, 2014, 3:04 p.m., bharat kumar wrote: > > ---

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

2014-04-10 Thread bharat kumar
destroyed state. I think one should be enough. - bharat --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20210/#review40034 --- On Apr

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

2014-04-10 Thread bharat kumar
/ObjectInDataStoreManagerImpl.java d9a5164 engine/storage/src/org/apache/cloudstack/storage/image/db/TemplateDataStoreDaoImpl.java f5140e0 server/src/com/cloud/template/HypervisorTemplateAdapter.java 963aec9 Diff: https://reviews.apache.org/r/20210/diff/ Testing (updated) --- Tested on 4.2 Thanks, bharat kumar

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

2014-04-10 Thread bharat kumar
pache/cloudstack/storage/image/db/TemplateDataStoreDaoImpl.java f5140e0 server/src/com/cloud/template/HypervisorTemplateAdapter.java 963aec9 Diff: https://reviews.apache.org/r/20210/diff/ Testing --- Tested on 4.2 Thanks, bharat kumar

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

2014-03-10 Thread Bharat Kumar
if we're > using 'details' or rootdisksize as an api param. That's why I'm > asking. > > On Mon, Mar 10, 2014 at 1:43 AM, Bharat Kumar wrote: >> Hi All, >> the roodiskresize is no longer valid. as there is no code which is using >> rootdiskresiz

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

2014-03-10 Thread Bharat Kumar
Can you comment on why rootdisksize was made from >> a parameter into a part of the details map? >> >> On Tue, Mar 4, 2014 at 3:12 AM, Bharat Kumar wrote: >>> Hi ALL, >>> There are many other APIs that use Map like createNetworkOffering, >>> updateZo

Re: [PROPOSAL][QUESTION] Map parameters in API Commands

2014-03-04 Thread Bharat Kumar
Hi ALL, There are many other APIs that use Map like createNetworkOffering, updateZone, createTemplate, in most of the cases we do not say how to use maps, one way would be to write this in the description or to use the same way to access maps of all APIs. BTW the way to use details in deploy vm

  1   2   3   4   >