Review Request 14247: added support for ispersistent parameter in networkoffering create method in marvin

2013-09-20 Thread Anshul Gangwar
--- Thanks, Anshul Gangwar

Review Request 14467: fix for Invalid SMTP breaks HA

2013-10-03 Thread Anshul Gangwar
alert/AlertManagerImpl.java c437a8c Diff: https://reviews.apache.org/r/14467/diff/ Testing --- tested manually on my local setup Thanks, Anshul Gangwar

Re: Review Request 14467: fix for Invalid SMTP breaks HA

2013-10-03 Thread Anshul Gangwar
On Thursday 03 October 2013 08:44 PM, Chip Childers wrote: > On Thu, Oct 03, 2013 at 11:31:50AM +0000, Anshul Gangwar wrote: >> Description >> --- >> >> added the connection and socket timeout parameters for SMTP and sending >> message in new thread so that H

Review Request 14489: Wrong error notification is generated when Primary storage (Cluster wide) is added with wrong path

2013-10-04 Thread Anshul Gangwar
/CloudStackPrimaryDataStoreLifeCycleImpl.java b9b7424 Diff: https://reviews.apache.org/r/14489/diff/ Testing --- tested on my local setup Thanks, Anshul Gangwar

Re: Review Request 14489: Wrong error notification is generated when Primary storage (Cluster wide) is added with wrong path

2013-10-04 Thread Anshul Gangwar
/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java b9b7424 Diff: https://reviews.apache.org/r/14489/diff/ Testing --- tested on my local setup Thanks, Anshul Gangwar

Re: Wiki access

2013-10-04 Thread Anshul Gangwar
need edit permissions , username: anshulg Regards, Anshul On Friday 04 October 2013 02:47 PM, Saksham Srivastava wrote: > Please provide me edit permissions, username: saksham > > Regards, > Saksham > > -Original Message- > From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] > Sen

Re: Review Request 14489: Wrong error notification is generated when Primary storage (Cluster wide) is added with wrong path

2013-10-04 Thread Anshul Gangwar
/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java b9b7424 Diff: https://reviews.apache.org/r/14489/diff/ Testing --- tested on my local setup Thanks, Anshul Gangwar

RE: [ACS4.2.1] bug fixes and code freeze

2013-10-16 Thread Anshul Gangwar
https://issues.apache.org/jira/browse/CLOUDSTACK-4792 (Invalid SMTP breaks HA) is missing from this. I have already submitted patch on review board. Thanks, Anshul -Original Message- From: Abhinandan Prateek [mailto:abhinandan.prat...@citrix.com] Sent: Thursday, October 17, 2013 10:21

Re: Review Request 14147: marvin tests for Persistent networks without running a VM

2013-10-18 Thread Anshul Gangwar
--- marvin tests for Persistent networks without running a VM Diffs (updated) - test/integration/component/test_persistent_networks.py PRE-CREATION Diff: https://reviews.apache.org/r/14147/diff/ Testing --- ran successfully on my local advanced zone setup Thanks, Anshul

Re: Review Request 14467: fix for Invalid SMTP breaks HA

2013-10-21 Thread Anshul Gangwar
sting --- tested manually on my local setup Thanks, Anshul Gangwar

Re: Review Request 14467: fix for Invalid SMTP breaks HA

2013-10-21 Thread Anshul Gangwar
setup Thanks, Anshul Gangwar

Review Request 14827: hyperv unit tests

2013-10-22 Thread Anshul Gangwar
PRE-CREATION Diff: https://reviews.apache.org/r/14827/diff/ Testing --- Thanks, Anshul Gangwar

Re: Review Request 14827: hyperv unit tests

2013-10-23 Thread Anshul Gangwar
://reviews.apache.org/r/14827/diff/ Testing --- Thanks, Anshul Gangwar

Review Request 14905: Add global configuration parameters for SMTP timeout

2013-10-24 Thread Anshul Gangwar
3aa9695 server/src/com/cloud/configuration/Config.java cefcdd5 Diff: https://reviews.apache.org/r/14905/diff/ Testing --- tested on local setup Thanks, Anshul Gangwar

Re: Review Request 14827: hyperv unit tests

2013-10-28 Thread Anshul Gangwar
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14827/#review27604 --- On Oct. 23, 2013, 11:05 a.m., Anshul Gangwar wrote: > > -

RE: Review Request 14827: hyperv unit tests

2013-10-28 Thread Anshul Gangwar
: dev@cloudstack.apache.org; Prasanna Santhanam; Rajesh Battala; Devdeep Singh; Chiradeep Vittal Cc: Anshul Gangwar Subject: RE: Review Request 14827: hyperv unit tests The mvn build flags are: -P hyperv-plugin for the C# portion of plugin -DskipTests=false for unit tests (platform independent

Review Request 15448: add the timeout configuration for upgrade

2013-11-11 Thread Anshul Gangwar
/15448/diff/ Testing --- Thanks, Anshul Gangwar

Re: Review Request 15448: add the timeout configuration for upgrade

2013-11-11 Thread Anshul Gangwar
- setup/db/db/schema-420to421.sql 1d28485 Diff: https://reviews.apache.org/r/15448/diff/ Testing --- Thanks, Anshul Gangwar

Review Request 15512: patch to support install and uninstall Hyperv agent as windows service

2013-11-14 Thread Anshul Gangwar
/ServerResource/AgentShell/ProjectInstaller.cs PRE-CREATION Diff: https://reviews.apache.org/r/15512/diff/ Testing --- tested manually on my local setup Thanks, Anshul Gangwar

Review Request 15476: fix for Invalid SMTP breaks HA (master and 4.3)

2013-11-14 Thread Anshul Gangwar
r/src/com/cloud/alert/AlertManagerImpl.java ed0c9fd server/src/com/cloud/configuration/Config.java bc805b7 setup/db/db/schema-420to421.sql 1d28485 Diff: https://reviews.apache.org/r/15476/diff/ Testing --- tested manually on my local setup Thanks, Anshul Gangwar

Review Board giving 502 proxy error on creating review request with patch

2013-08-08 Thread Anshul Gangwar
Without patch I am able to create review request but then I am not able to upload patch. Thanks, Anshul

Review Request 13423: fix for CLOUDSTACK-2187: catalina log filled with ClassNotFoundException for snmp and syslog

2013-08-08 Thread Anshul Gangwar
awsapi webapp as those plugins are not added as dependency for awsapi. Diffs - awsapi/pom.xml 9463f24 Diff: https://reviews.apache.org/r/13423/diff/ Testing --- verfied by manually copying the plugin jars to awsapi webapp lib. Thanks, Anshul Gangwar

Re: [ANNOUNCE] New Committer: Vijay Bhamidipati

2013-08-11 Thread Anshul Gangwar
Congrats Vijay! On Saturday 10 August 2013 10:30 AM, Prasanna Santhanam wrote: > The Project Management Committee (PMC) for Apache CloudStack has asked Vijay > Bhamidipati to become a committer and we are pleased to announce that they > have > accepted. > > Being a committer allows many contribut

Re: [Doc] Please review: External SNMP and Syslog manager alerts

2013-08-21 Thread Anshul Gangwar
Doc looks good to me. Thanks, Anshul On Wednesday 21 August 2013 04:36 AM, Jessica Tomechak wrote: > Hi, > Documentation on sending administrator alerts to external SNMP or Syslog > managers is ready for review. You can view the updated text at the link > below. This is checked in to both master

Review Request 13730: CLOUDSTACK-1853 Alerts-SNMP:No alerts generated for router vm when VM stopped unexpectedly

2013-08-21 Thread Anshul Gangwar
/HighAvailabilityManagerImpl.java 6b0d3c4 Diff: https://reviews.apache.org/r/13730/diff/ Testing --- Tested manually on my setup Thanks, Anshul Gangwar

Review Request 13805: CLOUDSTACK-3806: OS Preference can not be set

2013-08-26 Thread Anshul Gangwar
, Anshul Gangwar

Review Request 14079: marvin tests for IP Address reservation within a network

2013-09-11 Thread Anshul Gangwar
Diff: https://reviews.apache.org/r/14079/diff/ Testing --- ran successfully on my local advanced zone setup Thanks, Anshul Gangwar

Re: Review Request 14079: marvin tests for IP Address reservation within a network

2013-09-12 Thread Anshul Gangwar
/integration/component/test_ip_reservation.py PRE-CREATION tools/marvin/marvin/integration/lib/base.py fa4cc82 Diff: https://reviews.apache.org/r/14079/diff/ Testing --- ran successfully on my local advanced zone setup Thanks, Anshul Gangwar

Re: Review Request 14079: marvin tests for IP Address reservation within a network

2013-09-12 Thread Anshul Gangwar
, Anshul Gangwar

Review Request 14147: marvin tests for Persistent networks without running a VM

2013-09-16 Thread Anshul Gangwar
/lib/base.py 9e46065 Diff: https://reviews.apache.org/r/14147/diff/ Testing --- ran successfully on my local advanced zone setup Thanks, Anshul Gangwar

Re: Review Request 14079: marvin tests for IP Address reservation within a network

2013-09-16 Thread Anshul Gangwar
) - test/integration/component/test_ip_reservation.py PRE-CREATION tools/marvin/marvin/integration/lib/base.py fa4cc82 Diff: https://reviews.apache.org/r/14079/diff/ Testing --- ran successfully on my local advanced zone setup Thanks, Anshul Gangwar

Re: Review Request 14147: marvin tests for Persistent networks without running a VM

2013-09-16 Thread Anshul Gangwar
Thanks, Anshul Gangwar

Re: Review Request: Feature Syslog Enhancements

2013-03-29 Thread Anshul Gangwar
://reviews.apache.org/r/9870/diff/ Testing --- Units tests for functionality are written and manual testing for syslog message generation for few alerts Thanks, Anshul Gangwar

Re: Review Request: Syslog messages not getting generated for the "alertType:: 0"

2013-04-10 Thread Anshul Gangwar
ers/syslog-alerts/src/org/apache/cloudstack/syslog/AlertsSyslogAppender.java d2f2565 Diff: https://reviews.apache.org/r/10392/diff/ Testing --- tested manually Thanks, Anshul Gangwar

Re: Review Request: Syslog messages not getting generated for the "alertType:: 0"

2013-04-10 Thread Anshul Gangwar
ira/browse/CLOUDSTACK-1993. Diffs - plugins/alert-handlers/syslog-alerts/src/org/apache/cloudstack/syslog/AlertsSyslogAppender.java d2f2565 Diff: https://reviews.apache.org/r/10392/diff/ Testing --- tested manually Thanks, Anshul Gangwar

RE: [DISCUSS] HyperV msi

2014-01-24 Thread Anshul Gangwar
Hi I started working on packaging (making msi) the HyperV agent code at Github - https://github.com/anshul1886/CloudStackAgentSetup. It does the following - packages all the needed files - Prompts for user\password the HyperV agent should run under - Option to create and install an SSL certifica

RE: [DISCUSS] HyperV msi

2014-01-27 Thread Anshul Gangwar
I can generate msi. Where can I share it? Regards, Anshul -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Monday, January 27, 2014 2:38 PM To: dev@cloudstack.apache.org Cc: Anshul Gangwar; Rajesh Battala; Paul Angus; Wido den Hollander Subject: Re: [DISCUSS] HyperV msi

RE: [DISCUSS] HyperV msi

2014-01-27 Thread Anshul Gangwar
@cloudstack.apache.org Cc: Donal Lafferty; Anshul Gangwar; Rajesh Battala; Wido den Hollander Subject: RE: [DISCUSS] HyperV msi The files in the directory are: AgentShell.exe Newtonsoft.Json.dll AgentShell.exe.config NSubstitute.dll AgentShell.exe.mdb System.Net.Http.Formatting.dll AWSSDK.dll

Review Request 17583: [Hyper-v] can't get console access after vm live migration for 3 minutes

2014-01-30 Thread Anshul Gangwar
.java d5a3fcd Diff: https://reviews.apache.org/r/17583/diff/ Testing --- verified by opening console and then doing migration and agin opening console. It is working fine now. Thanks, Anshul Gangwar

Review Request 17842: fixed vmsync for hyperv

2014-02-07 Thread Anshul Gangwar
verifying their state got updated correctly in cloudstack Thanks, Anshul Gangwar

Re: Review Request 17842: fixed vmsync for hyperv

2014-02-10 Thread Anshul Gangwar
Diff: https://reviews.apache.org/r/17842/diff/ Testing --- Performed basic vm operations externally and verifying their state got updated correctly in cloudstack Thanks, Anshul Gangwar

Review Request 19451: added missing HintPath for for System.Web.Http dependency in HypervResource and ServerResource.Tests project

2014-03-19 Thread Anshul Gangwar
/HypervResource/HypervResource.csproj a2dc323 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/ServerResource.Tests.csproj eec595e Diff: https://reviews.apache.org/r/19451/diff/ Testing --- verified by building hyper-v agent code on visual studio Thanks, Anshul

Review Request 19457: CLOUDSTACK-6262: Fixed network throttling for hyper-v

2014-03-20 Thread Anshul Gangwar
/WmiWrappers.csproj b4bf04b Diff: https://reviews.apache.org/r/19457/diff/ Testing --- verified by seeing the maximum bandwidth value of nic of vm in hyper-v manager Thanks, Anshul Gangwar

Review Request 19521: CLOUDSTACK-6267: Unblock SMB as Zone wide primary storage for hyperv

2014-03-20 Thread Anshul Gangwar
cdd02b2 ui/scripts/zoneWizard.js 988e0f9 Diff: https://reviews.apache.org/r/19521/diff/ Testing --- verified by creating zone wide primary storage and perfroming other basic operations Thanks, Anshul Gangwar

Review Request 19713: CLOUDSTACK-6288: Change default ImageFormat to vhdx for hyper-v and allow registration of vhdx format templates

2014-03-26 Thread Anshul Gangwar
tests 1. registration of vhdx templates 2. attach and detach disk 3. volume migration 4. deploy machine with registered template Thanks, Anshul Gangwar

Re: Review Request 19713: CLOUDSTACK-6288: Change default ImageFormat to vhdx for hyper-v and allow registration of vhdx format templates

2014-03-26 Thread Anshul Gangwar
Thanks, Anshul Gangwar

RE: Alert Email must be SSL?

2014-03-26 Thread Anshul Gangwar
Have you configured your smtp host and port correctly? If there is authentication enabled then you have set the appropriate global setting alert.smtp.useAuth etc. Regards, Anshul -Original Message- From: zcsu [mailto:szc...@hotmail.com] Sent: Thursday, March 27, 2014 11:20 AM To: dev@c

RE: Alert Email must be SSL?

2014-03-27 Thread Anshul Gangwar
PM To: dev@cloudstack.apache.org; Anshul Gangwar Subject: RE: Alert Email must be SSL? Thanks anshul The others global setting is the same if alert.smtp.useAuth is default value, Error Log is: 2014-03-27 14:48:29,819 ERROR [cloud.alert.AlertManagerImpl] (Email-Alerts-Sender-1:null) Failed to

Review Request 19868: CLOUDSTACK-6287, CLOUDSTACK-6315 fixed password visible in plain text during some operation in hyperv

2014-04-01 Thread Anshul Gangwar
Diffs - utils/src/com/cloud/utils/StringUtils.java 1600488 Diff: https://reviews.apache.org/r/19868/diff/ Testing --- verified by looking into logs whether password is logged for the commands mentioned in bugs Thanks, Anshul Gangwar

Review Request 19906: CLOUDSTACK-6289: fixed storage migration failing in case of hyper-v if there are multiple disks attached to VM

2014-04-02 Thread Anshul Gangwar
/WmiCallsV2.cs ff8ab90 Diff: https://reviews.apache.org/r/19906/diff/ Testing --- tested by doing migration of volume attached to VM which has multiple disks. This test was successfull Thanks, Anshul Gangwar

Review Request 19907: CLOUDSTACK-6325 : fixed cleaning of bin and obj directories when building hyper-v agent with mono

2014-04-02 Thread Anshul Gangwar
. It is building fine with visual studio also. Thanks, Anshul Gangwar

Review Request 19913: CLOUDSTACK-6326 : Fixed password visible in plain text in some of commands in Hyper-v Agent logs

2014-04-03 Thread Anshul Gangwar
/HypervResourceController.cs 66b9828 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 6ebc5bf Diff: https://reviews.apache.org/r/19913/diff/ Testing --- verified by seeing the logs for some commands Thanks, Anshul Gangwar

Review Request 20311: CLOUDSTACK-6398: [hyper-v] Change auto shutdown and startup actions of VMs to shutdown and nothing

2014-04-14 Thread Anshul Gangwar
/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 79fd0bc Diff: https://reviews.apache.org/r/20311/diff/ Testing --- tested by verifying the state of VMs after hyper-v host restart Thanks, Anshul Gangwar

Review Request 20313: CLOUDSTACK-6400:[hyper-v] [vmsync] Occasionally VM is not deleted from back-end when it is stopped from Hyper-V manager and then destroyed-expunged from CS

2014-04-14 Thread Anshul Gangwar
created by cloudstack Thanks, Anshul Gangwar

Review Request 20315: CLOUDSTACK-6401: [hyper-v] fixed correct host state in cloudstack if the hyper-v agent is down

2014-04-14 Thread Anshul Gangwar
/ServerResource/HypervResource/heartbeat.bat PRE-CREATION Diff: https://reviews.apache.org/r/20315/diff/ Testing --- verified various scenarios 1. agent is down but host is up 2. both are down 3. both are up Thanks, Anshul Gangwar

Re: Review Request 19868: CLOUDSTACK-6287, CLOUDSTACK-6315 fixed password visible in plain text during some operation in hyperv

2014-04-14 Thread Anshul Gangwar
whether password is logged for the commands mentioned in bugs Thanks, Anshul Gangwar

Re: Review Request 20315: CLOUDSTACK-6401: [hyper-v] fixed correct host state in cloudstack if the hyper-v agent is down

2014-04-14 Thread Anshul Gangwar
. both are down 3. both are up Thanks, Anshul Gangwar

Review Request 20365: CLOUDSTACK-6411: some files are not loading in visual studio due to license header at top making them unparsable

2014-04-15 Thread Anshul Gangwar
license check is passed successfully Thanks, Anshul Gangwar

Review Request 20651: CLOUDSTACK-6470: fixed while stopping vm hyper-v agent, vm is now gracefully shutting down

2014-04-23 Thread Anshul Gangwar
it shows that the shutdown process is initiated. Thanks, Anshul Gangwar

Re: Review Request 20651: CLOUDSTACK-6470: fixed while stopping vm hyper-v agent, vm is now gracefully shutting down

2014-04-24 Thread Anshul Gangwar
/20651/diff/ Testing --- verified on my local setup by looking into the console of VM. When stopping the VM it shows that the shutdown process is initiated. Thanks, Anshul Gangwar

Review Request 20701: CLOUDSTACK-6504: removed the warnings coming in building hyper-v agent code

2014-04-24 Thread Anshul Gangwar
/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 4795073 Diff: https://reviews.apache.org/r/20701/diff/ Testing --- tested by creating the fresh setup after the removal of warnings. Thanks, Anshul Gangwar

Re: Review Request 20651: CLOUDSTACK-6470: fixed while stopping vm hyper-v agent, vm is now gracefully shutting down

2014-04-25 Thread Anshul Gangwar
/diff/ Testing --- verified on my local setup by looking into the console of VM. When stopping the VM it shows that the shutdown process is initiated. Thanks, Anshul Gangwar

[ACS44] Cherry pick for CLOUDSTACK-6399

2014-05-11 Thread Anshul Gangwar
Hi Daan/Hugo, Can you please cherry-pick following from 4.4-forward to 4.4: commit 704f6d8279c314a165199804ee43eaac4b0d482a Author: Anshul Gangwar Date: Fri May 9 14:32:36 2014 +0530 CLOUDSTACK-6399: When there are no vms running on hyperv host, then agent was returning null. In

Review Request 21429: CLOUDSTACK-6663: Agent fails to start on some setups if last nic on host in nic list doesn't contain unicast address

2014-05-14 Thread Anshul Gangwar
plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentService.cs 9d66a5c Diff: https://reviews.apache.org/r/21429/diff/ Testing --- Hyper-V Agent starts up without any problem. Created the basic Hyper-V setup and it works fine Thanks, Anshul Gangwar

[PROPOSAL] Storage Load Balancing

2017-03-27 Thread Anshul Gangwar
Hi All, I would like to propose the storage load balancing feature. Please find the FS for the same at https://cwiki.apache.org/confluence/display/CLOUDSTACK/Storage+Load+balancing. Regards, Anshul DISCLAIMER == This e-mail may contain privileged and confidential information which is

Re: Strange size of template from snapshot on XenServer

2017-11-01 Thread Anshul Gangwar
Hi Lotic, Can you try with any release between 4.6 to 4.8 and see if this bug is there? Basically, any release which contains commit 2c4ea503f92bcf9c611f409d5cdecb42b0115b69 and missing commit 8a3fd10. On 02/11/17, 1:14 AM, "Lotic Lists" wrote: Rafael, I know and appreciate your help :)

Re: Strange size of template from snapshot on XenServer

2017-11-08 Thread Anshul Gangwar
3420873216 hidden=0 parent=none > Was it not working way before “8a3fd10”?! I think we will need help from more folks here to see if this is a problem that only affects XenServer deployments. I will also test on my 4.9.2.0 environment again to see if I miss anyt

Re: Strange size of template from snapshot on XenServer

2017-11-10 Thread Anshul Gangwar
he primary storage where all of the snapshots parent VHDs are stored is used to create the template. Also, I opened the PR only against master. If someone needs the fix in any other version we can cherry pick to 4.9.2.0. On Wed, Nov 8, 2017 at 9:41 AM, Anshul Gangwar <

Re: Strange size of template from snapshot on XenServer

2017-11-12 Thread Anshul Gangwar
nly point I changed is that, instead of using “selectOneHypervisorHostByZone”, I create a specific method for that, which selects a host according to the volume of the snapshot. On Fri, Nov 10, 2017 at 8:21 AM, Anshul Gangwar < anshul.gang...@accelerite.com> wrote:

Review Request 21259: CLOUDSTACK-6399: VMSync thread does not run if all the VMs on a host are stopped from outside CS

2014-05-15 Thread Anshul Gangwar
/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java 7888e43 Diff: https://reviews.apache.org/r/21259/diff/ Testing --- verified when there are no vms on host, then vmsync thread is running fine Thanks, Anshul Gangwar

Review Request 21260: CLOUDSTACK-6620: [Hyper-v] when all vms on host are deleted from outside, then there is null pointer exception in hyperv agent in getting vmstats

2014-05-15 Thread Anshul Gangwar
, Anshul Gangwar

[ACS44] Cherry pick for CLOUDSTACK-6663

2014-05-26 Thread Anshul Gangwar
Hi Daan/Hugo, Can you please cherry-pick following from 4.4-forward to 4.4: commit b85f687cd7f12cd14a1feeb80593ba55e05b2fa8 Author: Anshul Gangwar Date: Wed May 14 13:18:41 2014 +0530 CLOUDSTACK-6663: Fixed Hyper-V agent fails to start on some setups. During startup agent tries to

RE: [ANNOUNCE] Saksham Srivastava as committer

2014-05-29 Thread Anshul Gangwar
Congrats Saksham -Original Message- From: sebgoa [mailto:run...@gmail.com] Sent: Thursday, May 29, 2014 12:18 PM To: dev@cloudstack.apache.org Subject: [ANNOUNCE] Saksham Srivastava as committer The Project Management Committee (PMC) for Apache CloudStack has asked Saksham Srivastava to

Review Request 22256: CLOUDSTACK-6830 :[hyper-v] fixed vm migration failing between different clusters if volumes are on zone wide primary store

2014-06-05 Thread Anshul Gangwar
g fine Thanks, Anshul Gangwar

Re: Review Request 22256: CLOUDSTACK-6830 :[hyper-v] fixed vm migration failing between different clusters if volumes are on zone wide primary store

2014-06-05 Thread Anshul Gangwar
olumes on zone wide primary store is working fine 2. VM migration for volumes on cluster wide primary store is working fine 3. VM migration for volumes on local storage is working fine Thanks, Anshul Gangwar

RE: Access console VM via custom portal

2014-06-05 Thread Anshul Gangwar
Have you verified the sessionkey which you are passing is correct? You can sso (single sign on) to automatically login into cloudstack and hence can access VM console. You can also use methods as described at http://www.cloudops.com/2013/01/working-with-the-cloudstack-api/. Regards, Anshul

Review Request 22414: CLOUDSTACK-6865, CLOUDSTACK-6868: [hyper-v] attach volume of vhd format volume is failing

2014-06-10 Thread Anshul Gangwar
iff: https://reviews.apache.org/r/22414/diff/ Testing --- verified by uploading the vhd volume and attaching it to VM Thanks, Anshul Gangwar

Review Request 22415: CLOUDSTACK-6867: added the option for vhdx image format in upload volume

2014-06-10 Thread Anshul Gangwar
by opening the upload volume dialog in UI Thanks, Anshul Gangwar

Review Request 22452: CLOUDSTACK-6872 : removed the redundant connectToRemote method

2014-06-10 Thread Anshul Gangwar
ource/WmiCallsV2.cs 372f848 Diff: https://reviews.apache.org/r/22452/diff/ Testing --- tested that volume operations are working fine after removal of this method Thanks, Anshul Gangwar

Re: Review Request 22452: CLOUDSTACK-6872 : removed the redundant connectToRemote method

2014-06-11 Thread Anshul Gangwar
otNet/ServerResource/HypervResource/Utils.cs c8e951e plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 372f848 Diff: https://reviews.apache.org/r/22452/diff/ Testing --- tested that volume operations are working fine after removal of this method Thanks, Anshul Gangwar

Review Request 22500: CLOUDSTACK-6897: In a Multi cluster setup attach of a uploaded volume to a VM on zwps is failing with scope conflict

2014-06-12 Thread Anshul Gangwar
/r/22500/diff/ Testing --- tested by uploading a volume and attaching it to vm on zwps in multi cluster setup Thanks, Anshul Gangwar

Re: Review Request 22500: CLOUDSTACK-6897: In a Multi cluster setup attach of a uploaded volume to a VM on zwps is failing with scope conflict

2014-06-13 Thread Anshul Gangwar
/cloudstack/engine/orchestration/VolumeOrchestrator.java 3cf8c6c Diff: https://reviews.apache.org/r/22500/diff/ Testing --- tested by uploading a volume and attaching it to vm on zwps in multi cluster setup Thanks, Anshul Gangwar

[ACS44] Cherry pick for CLOUDSTACK-6897, CLOUDSTACK-6868, CLOUDSTACK-6872, CLOUDSTACK-6867, CLOUDSTACK-6865

2014-06-13 Thread Anshul Gangwar
Hi Daan/Hugo, Can you please cherry-pick following commits from 4.4-forward to 4.4: commit 2dc9e2c530b36efc4c4684e13ae0412103925d05 Author: Anshul Gangwar Date: Thu Jun 12 13:54:56 2014 +0530 CLOUDSTACK-6897: when we try to attach the uploaded/allocated volume to a VM on zwps, then we

Re: Review Request 22256: CLOUDSTACK-6830 :[hyper-v] fixed vm migration failing between different clusters if volumes are on zone wide primary store

2014-06-16 Thread Anshul Gangwar
orking fine Thanks, Anshul Gangwar

Expected behavior for volume migration done after the download of volume

2014-06-19 Thread Anshul Gangwar
Hi, I am looking into fixing bug https://issues.apache.org/jira/browse/CLOUDSTACK-6900. When we download volume then we create entry for that volume in volume_store_ref table. We mark the volume in ready state. When we migrate that volume, then again one more entry is created with same volume

RE: ZoneWideStoragePoolAllocator#filter seems weird for me

2014-06-20 Thread Anshul Gangwar
This patch has introduced the NullPointerException in code. Since ZoneWideStoragePoolAllocator's filter method has been removed it is using AbstractStoragePoolAllocator's filter method. In the filter method there is check for cluster's hypervisor type and and disk profile hypervisor type. But

Re: Review Request 22256: CLOUDSTACK-6830 :[hyper-v] fixed vm migration failing between different clusters if volumes are on zone wide primary store

2014-06-20 Thread Anshul Gangwar
g fine Thanks, Anshul Gangwar

Review Request 22807: CLOUDSTACK-6965: Fix for CLOUDSTACK-6935 introduces the NullPointerException in ZoneWideStorageAllocator filter method

2014-06-20 Thread Anshul Gangwar
orage/allocator/AbstractStoragePoolAllocator.java 2d958a6 Diff: https://reviews.apache.org/r/22807/diff/ Testing --- Thanks, Anshul Gangwar

RE: ZoneWideStoragePoolAllocator#filter seems weird for me

2014-06-20 Thread Anshul Gangwar
I have submitted the patch for this https://reviews.apache.org/r/22807/. Can somebody have a look into the patch? -Original Message- From: Anshul Gangwar [mailto:anshul.gang...@citrix.com] Sent: Friday, June 20, 2014 3:32 PM To: dev@cloudstack.apache.org Subject: RE

Review Request 22808: CLOUDSTACK-6968: Allow Cluster scope volumes to attach to any VM

2014-06-20 Thread Anshul Gangwar
- server/src/com/cloud/storage/VolumeApiServiceImpl.java 819cf89 Diff: https://reviews.apache.org/r/22808/diff/ Testing --- tested on my local setup Thanks, Anshul Gangwar

Re: Review Request 22807: CLOUDSTACK-6965: Fix for CLOUDSTACK-6935 introduces the NullPointerException in ZoneWideStorageAllocator filter method

2014-06-22 Thread Anshul Gangwar
eWideStoragePool and Hence the exception. Diffs (updated) - engine/storage/src/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java 2d958a6 Diff: https://reviews.apache.org/r/22807/diff/ Testing --- Thanks, Anshul Gangwar

Re: Review Request 22808: CLOUDSTACK-6968: Allow Cluster scope volumes to attach to any VM

2014-06-22 Thread Anshul Gangwar
needed then first they will be migrated to appropriate cluster before attaching Diffs (updated) - server/src/com/cloud/storage/VolumeApiServiceImpl.java 8a7b220 Diff: https://reviews.apache.org/r/22808/diff/ Testing --- tested on my local setup Thanks, Anshul Gangwar

[ACS44] Cherry pick Request

2014-06-23 Thread Anshul Gangwar
Hi Daan/Hugo, Can you please cherry-pick following commits from 4.4-forward to 4.4: commit ef45f06f88c955dcf47f1fb69a3a0e87eb817e1d Author: Anshul Gangwar Date: Mon Jun 23 10:58:44 2014 +0530 CLOUDSTACK-6965: fixed the NullPointerException introduced by fix for cloudstack 6935 in

RE: [ACS44] Cherry pick Request

2014-06-23 Thread Anshul Gangwar
Looks good -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Monday, June 23, 2014 1:27 PM To: Anshul Gangwar Cc: dev@cloudstack.apache.org Subject: Re: [ACS44] Cherry pick Request On Mon, Jun 23, 2014 at 9:52 AM, Daan Hoogland wrote: >>

Review Request 23877: CLOUDSTACK-7177: AlertSyslogAppender does not honor a non-default port specified in syslog host parameter

2014-07-23 Thread Anshul Gangwar
/syslog-alerts/src/org/apache/cloudstack/syslog/AlertsSyslogAppender.java 50ccbc1 Diff: https://reviews.apache.org/r/23877/diff/ Testing --- tested by starting the rsyslog on non-default port and sending the alerts to non-default port Thanks, Anshul Gangwar

RE: Building of Hyper-V Agent seems broken in 4.4 branch

2014-07-31 Thread Anshul Gangwar
The commits are not applied in correct order. This commit 8fb89cdc8e2dff60d49fecd3e51a9bf997061035 is applied after ef0cec938165fdf3531f92dc8f4c2930ff95fa5e. The later commit has resolve conflicts in cherry-picking. It was not resolved correctly which introduces this build failure. Regards, An

Review Request 24174: CLOUDSTACK-7220: fixed hyper-v agent broken in 4.4 branch

2014-07-31 Thread Anshul Gangwar
/HypervResourceController.cs 0ad95b8 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 2f404ff Diff: https://reviews.apache.org/r/24174/diff/ Testing --- verified building on my local setup Thanks, Anshul Gangwar

RE: Building of Hyper-V Agent seems broken in 4.4 branch

2014-07-31 Thread Anshul Gangwar
I have created the patch https://reviews.apache.org/r/24174/ to fix the issue. Is it ok to go ahead and commit it into 4.4 branch? Regards, Anshul -Original Message- From: Anshul Gangwar [mailto:anshul.gang...@citrix.com] Sent: Friday, August 01, 2014 10:22 AM To: dev

RE: [ACS441]please test

2014-08-01 Thread Anshul Gangwar
The order of cherry-picked commits is not maintained. i.e. the order in which commits are made in 4.4-forwrad is not same in 4.4. This will create more regressions instead of fixing. One fix(Hyper-V agent build is broken) which I made is occurred due to this. To verify fixes, which one has made

Review Request 24226: CLOUDSTACK-7234: stop sending smb credentials to Hyper-V Agent

2014-08-03 Thread Anshul Gangwar
/HypervDirectConnectResource.java 2671b20 Diff: https://reviews.apache.org/r/24226/diff/ Testing --- verified on Hyper-V agent that we are not receiving SMB credentials. Thanks, Anshul Gangwar

  1   2   >