Review Request 13424: CLOUDSTACK-4123: 3.0.6 to ASF 4.2 Upgrade: On Adding VMWare Cluster to the Setup, 3.0.6 System VM Template for VMWare is downloaded instead of 4.2

2013-08-08 Thread Harikrishna Patnala
. If new 4.2 system template is registered before upgrade then marking the old templates as removed during upgrade. Diffs - engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 0941408 Diff: https://reviews.apache.org/r/13424/diff/ Testing --- Thanks, Harikrishna Patnala

Re: [Discuss] Using XenServer 6.2's Clone-on-Boot feature on CloudStack

2013-08-09 Thread Harikrishna Patnala
If we can control fall back of IntelliCache reads and writes back to the storage then it won't hinder with host maintenance mode and Xenmotion, and I think it helps handling volatile VMs without recreating the root disks. Thanks, Harikrishna On 09-Aug-2013, at 11:39 AM, Prasanna Santhanam

Re: Review Request 13424: CLOUDSTACK-4123: 3.0.6 to ASF 4.2 Upgrade: On Adding VMWare Cluster to the Setup, 3.0.6 System VM Template for VMWare is downloaded instead of 4.2

2013-08-09 Thread Harikrishna Patnala
/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 17cbf5c Diff: https://reviews.apache.org/r/13424/diff/ Testing --- Thanks, Harikrishna Patnala

Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-09 Thread Harikrishna Patnala
/XenServer56FP1Resource.java d230be1 Diff: https://reviews.apache.org/r/13441/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 13424: CLOUDSTACK-4123: 3.0.6 to ASF 4.2 Upgrade: On Adding VMWare Cluster to the Setup, 3.0.6 System VM Template for VMWare is downloaded instead of 4.2

2013-08-10 Thread Harikrishna Patnala
/ResourceManagerImpl.java e5441af server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 17cbf5c Diff: https://reviews.apache.org/r/13424/diff/ Testing --- Thanks, Harikrishna Patnala

Review Request 13468: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm; Zone host is ready, but secondary storage vm template: 3 is not ready on secondary storage: 2

2013-08-10 Thread Harikrishna Patnala
/ConsoleProxyManagerImpl.java c5faccb server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java d1c148b Diff: https://reviews.apache.org/r/13468/diff/ Testing --- Tested the scenario mentioned in the bug ticket Thanks, Harikrishna Patnala

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-10 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/#review24968 --- On Aug. 9, 2013, 1 p.m., Harikrishna Patnala wrote

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-10 Thread Harikrishna Patnala
e100211 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java d230be1 Diff: https://reviews.apache.org/r/13441/diff/ Testing --- Thanks, Harikrishna Patnala

Re: db exception?

2013-08-10 Thread Harikrishna Patnala
Please check you cluster_details table for memory and cpu overcommit values. This statement expects to have these details in cluster_details table. -Harikrishna On 11-Aug-2013, at 4:45 AM, Pedro Roque Marques pedro.r.marq...@gmail.com wrote: I've bumped into exceptions with similar signature

Re: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-11 Thread Harikrishna Patnala
Congratulations -Harikrishna On 11-Aug-2013, at 12:55 PM, Abhinav Roy abhinav@citrix.com wrote: Congrats Vijay. Regards, Abhinav -Original Message- From: Sanjay Tripathi [mailto:sanjay.tripa...@citrix.com] Sent: Sunday, August 11, 2013 12:53 PM To:

Re: [ACS42] Listtemplates API respone without any Zone configuration

2013-08-12 Thread Harikrishna Patnala
We mark all the system and default templates as downloaded and during template sync we remove the template entries that are not there. I guess this we have introduced recently, this may cause listing all the templates. Thanks Harikrishna On 12-Aug-2013, at 1:46 PM, Sailaja Mada

Review Request 13525: CLOUDSTACK-4271: ASF 4.1 to 4.2 Upgrade: Missing Global Configuration parameter vmware.nested.virtualization on the Upgraded Setup.

2013-08-13 Thread Harikrishna Patnala
--- Thanks, Harikrishna Patnala

Re: [ACS42]Regarding Scaleup ( CPU/Memory )

2013-08-14 Thread Harikrishna Patnala
Yes. we need to the improve the error message. Previously it was put to say any resource value should not less than the old value. On 14-Aug-2013, at 12:13 PM, Sailaja Mada sailaja.m...@citrix.com wrote: Hi, I am trying to scale up my VM memory from 512 MB to 1024 MB. It does not allow

Re: [ACS42]Regarding Scaleup ( CPU/Memory )

2013-08-14 Thread Harikrishna Patnala
Created a ticket for tracking https://issues.apache.org/jira/browse/CLOUDSTACK-4313 On 14-Aug-2013, at 12:37 PM, Harikrishna Patnala harikrishna.patn...@citrix.com wrote: Yes. we need to the improve the error message. Previously it was put to say any resource value should not less than

Re: [Doc] Please review: VM root disk reset on reboot doc

2013-08-15 Thread Harikrishna Patnala
Thank you Jessica, doc looks good for me. -Harikrishna On 15-Aug-2013, at 8:54 AM, Jessica Tomechak jessica.tomec...@gmail.commailto:jessica.tomec...@gmail.com wrote: Since no comments have been made against this doc, I am resolving the bug. Jessica T. On Thu, Aug 8, 2013 at 11:16 PM,

Re: [Doc] Developer Guide API Section for Review

2013-08-16 Thread Harikrishna Patnala
Hi Radhika, Thank you for the doc. Some changes are required, 1) updateCluster: The following request parameters are added: cpuovercommitratio, ramovercommitratio - These parameters are removed from the updateCluster API. Please remove from the doc as well. - Same doc entry exists in Page 24

Review Request 13682: CLOUDSTACK-2247: Automation - Reset a VM on Reboot

2013-08-20 Thread Harikrishna Patnala
/test_reset_vm_on_reboot.py PRE-CREATION Diff: https://reviews.apache.org/r/13682/diff/ Testing --- Thanks, Harikrishna Patnala

Re: registerSSHKeyPair

2013-08-22 Thread Harikrishna Patnala
Hi, RegisterSSHKey API is used when we have SSH key pair and if we want to register that key pair in Cloudstack. So when we provide public key it just calculates fingerprint and registers the SSH key pair. CS doesn't know about private key. It just returns null as private key. Our doc is

Re: System VMs not running

2013-08-22 Thread Harikrishna Patnala
Please follow the link to seed the system template http://cloudstack.apache.org/docs/en-US/Apache_CloudStack/4.1.1/html/Installation_Guide/management-server-install-flow.html#prepare-system-vm-template -Harikrishna On 22-Aug-2013, at 5:08 PM, Amit Das amit@cloudbyte.com wrote: Hi, I

Re: registerSSHKeyPair

2013-08-22 Thread Harikrishna Patnala
We generate docs based on the response object. Since we use the same response object for create/register ssh key pair APIs, doc shows same return values. We follow this for all API responses. Say list virtual machine API response does not include all the fields that are in api doc. -Harikrishna

Re: registerSSHKeyPair

2013-08-26 Thread Harikrishna Patnala
22, 2013, at 7:57 AM, Harikrishna Patnala harikrishna.patn...@citrix.com wrote: We generate docs based on the response object. Since we use the same response object for create/register ssh key pair APIs, doc shows same return values. We follow this for all API responses. Say list virtual

Review Request 13842: CLOUDSTACK-4482: getVMPassword() API call does not return password for Vms that are deployed with password enabled templates.

2013-08-27 Thread Harikrishna Patnala
for password enabled template Diffs - server/src/com/cloud/server/ManagementServerImpl.java d1e4e5f Diff: https://reviews.apache.org/r/13842/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 13842: CLOUDSTACK-4482: getVMPassword() API call does not return password for Vms that are deployed with password enabled templates.

2013-08-27 Thread Harikrishna Patnala
to get the encrypted password and included a check for password enabled template Diffs - server/src/com/cloud/server/ManagementServerImpl.java d1e4e5f Diff: https://reviews.apache.org/r/13842/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 13842: CLOUDSTACK-4482: getVMPassword() API call does not return password for Vms that are deployed with password enabled templates.

2013-08-27 Thread Harikrishna Patnala
be assinged to VM to get the encrypted password. Diffs - server/src/com/cloud/server/ManagementServerImpl.java d1e4e5f Diff: https://reviews.apache.org/r/13842/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 13842: CLOUDSTACK-4482: getVMPassword() API call does not return password for Vms that are deployed with password enabled templates.

2013-08-27 Thread Harikrishna Patnala
to VM to get the encrypted password. Diffs (updated) - server/src/com/cloud/server/ManagementServerImpl.java d1e4e5f Diff: https://reviews.apache.org/r/13842/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12545: CLOUDSTACK-1800: Add automation tests for reset sshkey to access VM

2013-08-27 Thread Harikrishna Patnala
in test_vm_passwdenabled.py 2) We have 10 tests in this file and each test tries to create separate account, networks, router vms and guest vms. Could you please try clubbing the resources in the tests that use common resources. - Harikrishna Patnala On Aug. 13, 2013, 7:41 a.m., Girish Shilamkar wrote

Re: [Doc] Please review: Granular configuration parameters doc

2013-08-28 Thread Harikrishna Patnala
Thank you Likitha for the doc. Some text need to be added. 1) There is no need of restarting the management server after setting the local configuration parameters. 2) If we set value 'null' in the local configuration then CS considers the value from global configuration. Could you please make

Wiki on Adding config parameters at different scopes

2013-09-01 Thread Harikrishna Patnala
Hi all, As we know we have now the configuration parameters at different scopes (account, zone, cluster, or primary storage). https://issues.apache.org/jira/browse/CLOUDSTACK-741 I have prepared a wiki on how to use and add configuration parameters at different scopes. LINK:

Re: cpvm and ssvm but no host entries

2013-09-04 Thread Harikrishna Patnala
Seems like in both the cases agent is not up. please try troubleshooting SSVM https://cwiki.apache.org/CLOUDSTACK/ssvm-templates-secondary-storage-troubleshooting.html -Harikrishna On 05-Sep-2013, at 8:13 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: I've been trying to set up a CS

Re: Question about CentOS 5.5(64-bit) no GUI (KVM)

2013-09-10 Thread Harikrishna Patnala
Hi, Have you set the global parameter secstorage.allowed.internal.sites ? On 11-Sep-2013, at 10:34 AM, Mike Tutkowski mike.tutkow...@solidfire.com wrote: Hi, I'm new to using KVM with CS. I noticed under Templates there is CentOS 5.5(64-bit) no GUI (KVM). When I drill down to look at

Re: Creating an instance with ssh key pair

2013-09-13 Thread Harikrishna Patnala
SSH script pulls only the ssh public key that is associated with the vm instance. There is an API to reset the SSH key pair associated to the VM. This is will associate new ssh key pair to the vm and on reboot of vm this ssh script gets the new ssh public key into the VM. If you can modify the

Re: Review Request 12545: CLOUDSTACK-1800: Add automation tests for reset sshkey to access VM

2013-09-16 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12545/#review26131 --- Ship it! Ship It! - Harikrishna Patnala On Sept. 14, 2013, 7:46

Review Request 14227: CLOUDSTACK-4707: sourcetemplateid field is not getting set for derived templates

2013-09-19 Thread Harikrishna Patnala
sourcetemplateId is set in vm_template table Thanks, Harikrishna Patnala

Re: Review Request 14227: CLOUDSTACK-4707: sourcetemplateid field is not getting set for derived templates

2013-09-19 Thread Harikrishna Patnala
In both the cases sourcetemplateId is set in vm_template table Thanks, Harikrishna Patnala

Re: Volunteers to Complete the 4.2 Release Notes

2013-09-24 Thread Harikrishna Patnala
, Abhinav -Original Message- From: Milamber [mailto:milam...@apache.org] Sent: Monday, September 23, 2013 10:53 PM To: us...@cloudstack.apache.org; dev@cloudstack.apache.org; Radhika Puthiyetath; Harikrishna Patnala Subject: Re: Volunteers to Complete the 4.2 Release Notes Le 23/09

Re: Database upgrade

2013-09-24 Thread Harikrishna Patnala
My comments inline -Harikrishna On 25-Sep-2013, at 10:18 AM, Indra Pramana in...@sg.or.id wrote: Dear all, During my (failed) upgrade attempt from CloudStack 4.1.1 to 4.2.0 early this week, I noted that CloudStack management server will automatically detect the database version and

Re: [ANNOUNCE] New committer: Rajesh Battala

2013-09-24 Thread Harikrishna Patnala
Congratulations Rajesh :) On 25-Sep-2013, at 10:26 AM, Venkata SwamyBabu Budumuru venkataswamybabu.budum...@citrix.com wrote: Congrats Rajesh! On 25/09/13 10:00 AM, Abhinav Roy abhinav@citrix.com wrote: Congrats Rajesh :) -Original Message- From: Sanjay Tripathi

RE: [4.2] [xenserver] [system vms] Xentools

2013-10-08 Thread Harikrishna Patnala
Hi, Earlier there was discussion on putting xen tools in systemvms. Please look into that. http://mail-archives.apache.org/mod_mbox/cloudstack-dev/201309.mbox/%3cce676527.4398b%25abhinandan.prat...@citrix.com%3E From: Paul Angus [mailto:paul.an...@shapeblue.com] Sent: Tuesday, October 08, 2013

RE: Dynamically scalable?

2013-10-09 Thread Harikrishna Patnala
Hi , While registering template we can specify whether xenserver/vmware tools installed in the template using this flag(dynamically scalable). This is required while deploying vm to set some parameters for dynamic scaling of cpu and ram feature to work. The same flag is reflected in the vm

Re: Review Request 14227: CLOUDSTACK-4707: sourcetemplateid field is not getting set for derived templates

2013-10-09 Thread Harikrishna Patnala
is set in vm_template table Thanks, Harikrishna Patnala

Review Request: Fix for CLOUDSTACK-2728: 41-42 DB upgrade: add step to upgrade system templates

2013-06-03 Thread Harikrishna Patnala
, Harikrishna Patnala

Review Request: Fix for CLOUDSTACK-1622, CLOUDSTACK-1704

2013-06-10 Thread Harikrishna Patnala
in the query. This addresses bugs CLOUDSTACK-1622 and CLOUDSTACK-1704. Diffs - engine/schema/src/com/cloud/capacity/dao/CapacityDaoImpl.java 0b9ff1a Diff: https://reviews.apache.org/r/11780/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request: fix for CLOUDSTACK-2347: Zone filter for listSnapshots API

2013-06-12 Thread Harikrishna Patnala
/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request: Fix for CLOUDSTACK-2502 and CLOUDSTACK-2939

2013-06-13 Thread Harikrishna Patnala
/HypervisorGuruBase.java d8945af server/src/com/cloud/vm/VirtualMachineManagerImpl.java 568fe55 server/test/com/cloud/vm/VirtualMachineManagerImplTest.java dd51e74 Diff: https://reviews.apache.org/r/11860/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
setup/db/db/schema-410to420.sql 272fc42 Diff: https://reviews.apache.org/r/11910/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
-mail. To reply, visit: https://reviews.apache.org/r/11910/#review21982 --- On June 17, 2013, 2:44 p.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-17 Thread Harikrishna Patnala
/cloud/vm/VirtualMachineManagerImpl.java f946cd1 server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 8715c9e setup/db/db/schema-410to420.sql 272fc42 Diff: https://reviews.apache.org/r/11910/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Re: [ANNOUNCE] New committer: Jayapal Reddy Uradi

2013-06-17 Thread Harikrishna Patnala
Congratulations Jayapal. On 17-Jun-2013, at 10:29 PM, Abhinandan Prateek cloudst...@aprateek.com wrote: Congrats Jayapal ! Well deserved. On 17/06/13 9:00 PM, Chip Childers chip.child...@sungard.com wrote: The Project Management Committee (PMC) for Apache CloudStack has asked Jayapal

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-18 Thread Harikrishna Patnala
, Harikrishna Patnala

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-18 Thread Harikrishna Patnala
. To reply, visit: https://reviews.apache.org/r/11910/#review21982 --- On June 18, 2013, 7:05 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply

Review Request: Fix for CLOUDSTACK-2181: Scale down is allowed, which is not expected

2013-06-18 Thread Harikrishna Patnala
: https://reviews.apache.org/r/11936/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-19 Thread Harikrishna Patnala
/VirtualMachineManagerImpl.java f946cd1 server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 8715c9e setup/db/db/schema-410to420.sql 272fc42 Diff: https://reviews.apache.org/r/11910/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Re: Review Request: Fix for CLOUDSTACK-2987 Ensure XStools to be there in template inorder to enable dynamic scaling of vm

2013-06-19 Thread Harikrishna Patnala
/VirtualMachineManagerImpl.java 5814075 server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 8715c9e setup/db/db/schema-410to420.sql c782234 Diff: https://reviews.apache.org/r/11910/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request: Fix for CLOUDSTACK-2916, CLOUDSTACK-2457: validation for weight based configuration parameters

2013-06-20 Thread Harikrishna Patnala
://reviews.apache.org/r/11989/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request: Fix for CLOUDSTACK-2835: Registering router template and usage of router.template.* configuration parameter

2013-06-24 Thread Harikrishna Patnala
the configuration parameter routing.template.xenserver to the new ROUTING template. Tested the default behavior of router deployment from SYSTEM template. Tested upgrade case from 4.1 to 4.2 Thanks, Harikrishna Patnala

Review Request 12078: Fix for CLOUDSTACK-3187: cluster.(cpu/memory).allocated.capacity.disablethreshold in not considering overcommit factor.

2013-06-25 Thread Harikrishna Patnala
Thanks, Harikrishna Patnala

Review Request 12079: Fix for CLOUDSTACK-2794: Global parameter router.template.idshould be removed.

2013-06-25 Thread Harikrishna Patnala
/TemplateManagerImpl.java 273614c setup/db/db/schema-410to420.sql d2add32 Diff: https://reviews.apache.org/r/12079/diff/ Testing --- Tested with latest master, also upgrade case. Thanks, Harikrishna Patnala

Review Request 12102: Fix for CLOUDSTACK-3208: showing xenhost username and password details in plain text in logs when we pefrom removehost from CS

2013-06-26 Thread Harikrishna Patnala
: https://reviews.apache.org/r/12102/diff/ Testing --- Thanks, Harikrishna Patnala

Review Request 12136: Fix for CLOUDSTACK-3245: non admin user not able to register template

2013-06-27 Thread Harikrishna Patnala
. Diffs - api/src/org/apache/cloudstack/api/command/user/template/RegisterTemplateCmd.java 13bd5da server/src/com/cloud/template/TemplateAdapterBase.java 7835680 Diff: https://reviews.apache.org/r/12136/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Unable to register templates and ISOs

2013-06-29 Thread Harikrishna Patnala
Can you try troubleshooting the SSVM by following this link https://cwiki.apache.org/CLOUDSTACK/ssvm-templates-secondary-storage-troubleshooting.html Probably agent is not running in the SSVM due to which no entry is created in the host table. please try building systemvm and recreate ssvm

Review Request 12232: CLOUDSTACK-3324: Fix/Improve upgrade system vm templates 41 to 42

2013-07-02 Thread Harikrishna Patnala
hypervisors Diffs - engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java da68348 Diff: https://reviews.apache.org/r/12232/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request 12233: CLOUDSTACK-3324: Fix/Improve upgrade system vm templates 41 to 42 (Stable master Patch)

2013-07-02 Thread Harikrishna Patnala
hypervisors Diffs - engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 90a311b Diff: https://reviews.apache.org/r/12233/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request 12242: CLOUDSTACK-3302: [Multiple_IP_Ranges][VmWare] VM deployement failed due to dnsmasq configuration issue in case of multiple subnets in a vlan

2013-07-03 Thread Harikrishna Patnala
46e5cf9 Diff: https://reviews.apache.org/r/12242/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Review Request 12263: CLOUDSTACK-3354: [automation] scale up VM is failing

2013-07-04 Thread Harikrishna Patnala
--- Thanks, Harikrishna Patnala

Review Request 12264: CLOUDSTACK-3351: Fix add cluster API to set right hypervisor type

2013-07-04 Thread Harikrishna Patnala
with hypervisor type with other tables in DB leading to failure of various DB queries. Diffs - server/src/com/cloud/resource/ResourceManagerImpl.java c6e8d7d Diff: https://reviews.apache.org/r/12264/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12264: CLOUDSTACK-3351: Fix add cluster API to set right hypervisor type

2013-07-04 Thread Harikrishna Patnala
/#review22759 --- On July 4, 2013, 8:40 a.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12264

Re: Review Request 12263: CLOUDSTACK-3354: [automation] scale up VM is failing

2013-07-08 Thread Harikrishna Patnala
) - setup/db/db/schema-410to420.sql 8130999 Diff: https://reviews.apache.org/r/12263/diff/ Testing --- Thanks, Harikrishna Patnala

Review Request 12324: CLOUDSTACK-3358: create template from a volume/snapshot should also take input from User/Admin whether xstools / vmware tools installed in the VM

2013-07-08 Thread Harikrishna Patnala
/CreateTemplateCmd.java 07aeb0a server/src/com/cloud/hypervisor/HypervisorGuruBase.java 50bfb30 server/src/com/cloud/template/TemplateManagerImpl.java 5a54656 Diff: https://reviews.apache.org/r/12324/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12324: CLOUDSTACK-3358: create template from a volume/snapshot should also take input from User/Admin whether xstools / vmware tools installed in the VM

2013-07-08 Thread Harikrishna Patnala
/cloudstack/api/command/user/template/CreateTemplateCmd.java 07aeb0a server/src/com/cloud/hypervisor/HypervisorGuruBase.java 50bfb30 server/src/com/cloud/template/TemplateManagerImpl.java 5a54656 Diff: https://reviews.apache.org/r/12324/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12358: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm

2013-07-09 Thread Harikrishna Patnala
--- tested locally Thanks, Harikrishna Patnala

Re: Review Request 12358: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm

2013-07-09 Thread Harikrishna Patnala
/MockResourceManagerImpl.java 18cff80 Diff: https://reviews.apache.org/r/12358/diff/ Testing --- tested locally Thanks, Harikrishna Patnala

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-10 Thread Harikrishna Patnala
with actual usage and over committed values for memory. Thanks, Harikrishna Patnala

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-10 Thread Harikrishna Patnala
server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 44e22e2 Diff: https://reviews.apache.org/r/12427/diff/ Testing --- Tested locally both with actual usage and over committed values for memory. Thanks, Harikrishna Patnala

Re: Review Request 12358: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm

2013-07-10 Thread Harikrishna Patnala
/secondary/SecondaryStorageManagerImpl.java 05256a8 server/test/com/cloud/resource/MockResourceManagerImpl.java 18cff80 Diff: https://reviews.apache.org/r/12358/diff/ Testing --- tested locally Thanks, Harikrishna Patnala

Review Request 12498: CLOUDSTACK-2987: Fixing dynamicallyscalable flag in register and update template API

2013-07-12 Thread Harikrishna Patnala
/src/com/cloud/storage/VMTemplateVO.java 3e71191 server/src/com/cloud/template/TemplateAdapterBase.java fce2506 server/src/com/cloud/template/TemplateManagerImpl.java 5a54656 Diff: https://reviews.apache.org/r/12498/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12498: CLOUDSTACK-2987: Fixing dynamicallyscalable flag in register and update template API

2013-07-12 Thread Harikrishna Patnala
--- Thanks, Harikrishna Patnala

Re: Review Request 12324: CLOUDSTACK-3358: create template from a volume/snapshot should also take input from User/Admin whether xstools / vmware tools installed in the VM

2013-07-15 Thread Harikrishna Patnala
--- Thanks, Harikrishna Patnala

Re: Review board is not accessible.

2013-07-16 Thread Harikrishna Patnala
Me too not able to access Review board. Fix for a blocker bug is in pending so attaching the patch to the bug ticket itself. https://issues.apache.org/jira/browse/CLOUDSTACK-3551 Can someone review my patch and commit. Thanks, Harikrishna On 16-Jul-2013, at 3:46 PM, Bharat Kumar

Re: Review Request 12324: CLOUDSTACK-3358: create template from a volume/snapshot should also take input from User/Admin whether xstools / vmware tools installed in the VM

2013-07-16 Thread Harikrishna Patnala
missing anything. Thankyou - Harikrishna Patnala On July 15, 2013, 12:10 p.m., Harikrishna Patnala wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12324

Review Request 12632: CLOUDSTACK-3551: Fix value over flow due to type conversion

2013-07-17 Thread Harikrishna Patnala
/CapacityManagerImpl.java d54624d server/src/com/cloud/vm/VirtualMachineManagerImpl.java 827e233 Diff: https://reviews.apache.org/r/12632/diff/ Testing --- Tested locally Thanks, Harikrishna Patnala

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-17 Thread Harikrishna Patnala
, Harikrishna Patnala

Re: Review Request 12427: Fix for CLOUDSTACK-3365 and CLOUDSTACK-2536

2013-07-17 Thread Harikrishna Patnala
generated e-mail. To reply, visit: https://reviews.apache.org/r/12427/#review23239 --- On July 10, 2013, 10:56 a.m., Harikrishna Patnala wrote: --- This is an automatically generated

Re: Review Request 12358: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm

2013-07-18 Thread Harikrishna Patnala
/SecondaryStorageManagerImpl.java 6859b0b Diff: https://reviews.apache.org/r/12358/diff/ Testing --- tested locally Thanks, Harikrishna Patnala

Review Request 12719: CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-18 Thread Harikrishna Patnala
developer upgrade does not fail in deploying router. Diffs - engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java e37ccd5 setup/db/db/schema-410to420.sql 40e3d2b Diff: https://reviews.apache.org/r/12719/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12719: CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-19 Thread Harikrishna Patnala
--- Thanks, Harikrishna Patnala

Review Request 12757: CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-19 Thread Harikrishna Patnala
developer upgrade does not fail in deploying router. Diffs - engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java e54a699 setup/db/db/schema-410to420.sql 921d5fa Diff: https://reviews.apache.org/r/12757/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12757: [4.2]CLOUDSTACK-3634: Adding router.template.xen/kvm/hyperv/kvm/lxc in upgrade setup

2013-07-19 Thread Harikrishna Patnala
setup/db/db/schema-410to420.sql 921d5fa Diff: https://reviews.apache.org/r/12757/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12427: Fix for CLOUDSTACK-3365: cluster level parameters cluster.(cpu/memory).allocated.capacity.notificationthreshold is not considering overcommit value

2013-07-19 Thread Harikrishna Patnala
://reviews.apache.org/r/12427/diff/ Testing --- Tested locally both with actual usage and over committed values for memory. Thanks, Harikrishna Patnala

Re: 41-42 db upgrade

2013-07-19 Thread Harikrishna Patnala
: Checked in my part into 4.2 and master branch. Thanks -min From: Alena Prokharchyk alena.prokharc...@citrix.commailto:alena.prokharc...@citrix.com Date: Wednesday, July 17, 2013 1:26 PM To: Harikrishna Patnala harikrishna.patn...@citrix.commailto:harikrishna.patn...@citrix.com, Min Chen min.c

Review Request 12759: CLOUDSTACK-3669: Bringing down the dynamic scalable check (for VM whether it has xstools or vmware tools in it) to the management layer.

2013-07-19 Thread Harikrishna Patnala
://reviews.apache.org/r/12759/diff/ Testing --- Thanks, Harikrishna Patnala

Review Request 12808: Fix for CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and (cpu/memory).overprosioning.factor are redundant(cluster level)

2013-07-22 Thread Harikrishna Patnala
/ConfigurationManagerImpl.java 92178c9 server/src/com/cloud/resource/ResourceManagerImpl.java d7652f4 server/test/com/cloud/resource/MockResourceManagerImpl.java 2c66134 Diff: https://reviews.apache.org/r/12808/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

2013-07-22 Thread Harikrishna Patnala
adcf475 server/test/com/cloud/vm/UserVmManagerTest.java 0eb9a08 Diff: https://reviews.apache.org/r/12809/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12808: Fix for CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and (cpu/memory).overprosioning.factor are redundant(cluster level)

2013-07-23 Thread Harikrishna Patnala
Diff: https://reviews.apache.org/r/12808/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12809: Fix for CLOUDSTACK-3703: change service offering of stopped vm on kvm is failing

2013-07-24 Thread Harikrishna Patnala
0eb9a08 Diff: https://reviews.apache.org/r/12809/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12759: CLOUDSTACK-3669: Bringing down the dynamic scalable check (for VM whether it has xstools or vmware tools in it) to the management layer.

2013-07-25 Thread Harikrishna Patnala
server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 44a703d Diff: https://reviews.apache.org/r/12759/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12808: Fix for CLOUDSTACK-2536: updating parameters (cpu/memory)overcommit ratio and (cpu/memory).overprosioning.factor are redundant(cluster level)

2013-07-25 Thread Harikrishna Patnala
/ResourceManagerImpl.java 342b9fd server/test/com/cloud/resource/MockResourceManagerImpl.java 2c66134 Diff: https://reviews.apache.org/r/12808/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12937: CLOUDSTACK-3779: Support System vm scaling on VMWare

2013-07-25 Thread Harikrishna Patnala
/VirtualMachineManagerImplTest.java 44a703d setup/db/db/schema-410to420.sql 82ca403 Diff: https://reviews.apache.org/r/12937/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12937: CLOUDSTACK-3779: Support System vm scaling on VMWare

2013-07-25 Thread Harikrishna Patnala
://reviews.apache.org/r/12937/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12937: CLOUDSTACK-3779: Support System vm scaling on VMWare

2013-07-26 Thread Harikrishna Patnala
/schema-410to420.sql 82ca403 Diff: https://reviews.apache.org/r/12937/diff/ Testing --- Thanks, Harikrishna Patnala

Re: Review Request 12937: CLOUDSTACK-3779: Support System vm scaling on VMWare

2013-07-26 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12937/#review23914 --- On July 26, 2013, 8:41 a.m., Harikrishna Patnala wrote

  1   2   3   4   5   >