RE: [ANNOUNCE] Rohit Yadav as new PMC member of CloudStack

2015-03-27 Thread Devdeep Singh
Congrats Rohit.

Regards,
Devdeep

 -Original Message-
 From: Sebastien Goasguen [mailto:run...@gmail.com]
 Sent: Friday, March 27, 2015 1:38 PM
 To: dev@cloudstack.apache.org
 Subject: [ANNOUNCE] Rohit Yadav as new PMC member of CloudStack
 
 The Project Management Committee (PMC) for Apache CloudStack are
 pleased to announce that Rohit Yadav has accepted our invitation to join the
 PMC.
 
 Please join me in congratulating him.
 
 On behalf of the Apache CloudStack PMC


[Discuss] Disabling storage pool for provisioning of volumes

2015-03-16 Thread Devdeep Singh
Hi,

I would like to propose a new feature Disabling storage pool for 
provisioning.  This allows an administrator to disable a storage pool from 
being used for provisioning of new volumes. Instances with volumes on a 
disabled pool will continue to run; however, no new volumes will get placed on 
the storage pool. This feature has been asked for in the community [1]. An 
initial draft of the FS can also be looked up here [2].

Kindly provide your feedback.

[1] 
http://markmail.org/message/5jouqvojuhgnlzxt?q=list:org%2Eapache%2Eincubator%2Ecloudstack-%2A+Disable+primary+storage
[2] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Disabling+Storage+Pool+for+Provisioning

Regards,
Devdeep


RE: [ANNOUNCE] New committer: Wilder Rodrigues

2015-03-01 Thread Devdeep Singh
Congrats Wilder.

Regards,
Devdeep

 -Original Message-
 From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
 Sent: Sunday, March 1, 2015 3:44 PM
 To: dev
 Subject: [ANNOUNCE] New committer: Wilder Rodrigues
 
 The Project Management Committee (PMC) for Apache CloudStack has asked
 Wilder Rodrigues to become a committer and we are pleased to announce
 that he has accepted.
 
 Keep up the good work and your contagious good spirit Wilder.
 
 Being a committer allows many contributors to contribute more
 autonomously. For developers, it makes it easier to submit changes and
 eliminates the need to have contributions reviewed via the patch submission
 process. Whether contributions are development-related or otherwise, it is a
 recognition of a contributor's participation in the project and commitment to
 the project and the Apache Way.
 
 Please join me in congratulating Wilder.
 
 --
 on behalf of the CloudStack PMC,
 Daan


RE: Disable primary storage, not delete...

2015-02-24 Thread Devdeep Singh
If a storage pool is put in disabled state then it will not be picked up as a 
possible destination when a volume is being migrated or when a virtual machine 
is being migrated along with its volumes.

Regards,
Devdeep

 -Original Message-
 From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
 Sent: Monday, February 23, 2015 7:44 PM
 To: dev@cloudstack.apache.org
 Cc: us...@cloudstack.apache.org
 Subject: Re: Disable primary storage, not delete...
 
 Yeah, probably option 2 is better.
 
 I would think you couldn't use this primary storage as the target of a
 migration or for HA.
 
 On Monday, February 23, 2015, Harikrishna Patnala 
 harikrishna.patn...@citrix.com wrote:
 
  Hi,
 
  I think it is better to go with option 2, as it makes sense to be part
  of one of the states.
 
  I believe that VMs using that storage continue to work, but what are
  the plans for the operations related to the storage on those VMs like
  migration/HA/... ?
 
  Thanks,
  Harikrishna
 
  On 23-Feb-2015, at 3:01 pm, Devdeep Singh devdeep.si...@citrix.com
  javascript:; wrote:
 
   Hi,
  
   I was looking into implementing the ability to disable a storage
   pool
  for provisioning. I could think of two ways to do it
  
   1. CloudStack admin could add a 'disabled' tag on the storage pool.
   The
  storage pool allocators would skip the pools for allocating volumes
  with the 'disabled' tag.
   2. Other option would be to add 'Disabled' as one of the states of
  storage pool. This could be a state in addition to Up, Maintenance etc.
  A storage pool in Disabled state will not be picked up for
  provisioning volumes. This would require new apis for disabling/enabling
 storage pool.
  
   Thoughts, comments?
  
   Regards,
   Devdeep
  
   -Original Message-
   From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com
  javascript:;]
   Sent: Monday, February 23, 2015 3:48 AM
   To: dev@cloudstack.apache.org javascript:;
   Cc: us...@cloudstack.apache.org javascript:;
   Subject: Re: Disable primary storage, not delete...
  
   For my suggestion to work, though, your compute and disk offerings
   have
  to
   be set up currently to make use of one or more storage tags.
  
   The idea is then that these offerings would require your primary
  storage to
   have a given tag or tags and it never will (effectively disabling
   that
  primary
   storage from being used with your offerings).
  
   On Sunday, February 22, 2015, Mike Tutkowski
   mike.tutkow...@solidfire.com javascript:;
   wrote:
  
   What about changing the storage tags field of your primary storage
   so it doesn't serve as a match for any compute or disk offering?
  
   On Sunday, February 22, 2015, Andrija Panic
   andrija.pa...@gmail.com
  javascript:;
   javascript:_e(%7B%7D,'cvml','andrija.pa...@gmail.com
   javascript:;');
  wrote:
  
   Hi folks,
  
   I was wondering is it safe to change the Cluster Wide primary
   storage to a Zone Wide primary storage (change the database,
   cloud.storage_pool). This is ACS 4.3.0
  
  
   Or actually better question - since I have some old NFS servers
   exist as Primary Storage - is there any way to exclude this NFS
   (Cluster
   Wide)  - or disable it, so the new Volume Uploads will not use
   that NFS storage...
  
   This is some trail from the history, that I can't really delete,
   but would like to completely disable further usage of this NFS 
   server...
   if possible at all...
  
   THanks,
  
  
   --
  
   Andrija Panić
  
  
  
   --
   *Mike Tutkowski*
   *Senior CloudStack Developer, SolidFire Inc.*
   e: mike.tutkow...@solidfire.com javascript:;
   javascript:_e(%7B%7D,'cvml','mike.tutkow...@solidfire.com
  javascript:;');
   o: 303.746.7302
   Advancing the way the world uses the cloud
   http://solidfire.com/solution/overview/?video=play*™*
  
  
  
   --
   *Mike Tutkowski*
   *Senior CloudStack Developer, SolidFire Inc.*
   e: mike.tutkow...@solidfire.com javascript:;
   o: 303.746.7302
   Advancing the way the world uses the cloud
   http://solidfire.com/solution/overview/?video=play*™*
 
 
 
 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the cloud
 http://solidfire.com/solution/overview/?video=play*™*


RE: Disable primary storage, not delete...

2015-02-23 Thread Devdeep Singh
Hi,

I was looking into implementing the ability to disable a storage pool for 
provisioning. I could think of two ways to do it

1. CloudStack admin could add a 'disabled' tag on the storage pool. The storage 
pool allocators would skip the pools for allocating volumes with the 'disabled' 
tag. 
2. Other option would be to add 'Disabled' as one of the states of storage 
pool. This could be a state in addition to Up, Maintenance etc. A storage 
pool in Disabled state will not be picked up for provisioning volumes. This 
would require new apis for disabling/enabling storage pool.

Thoughts, comments?

Regards,
Devdeep

 -Original Message-
 From: Mike Tutkowski [mailto:mike.tutkow...@solidfire.com]
 Sent: Monday, February 23, 2015 3:48 AM
 To: dev@cloudstack.apache.org
 Cc: us...@cloudstack.apache.org
 Subject: Re: Disable primary storage, not delete...
 
 For my suggestion to work, though, your compute and disk offerings have to
 be set up currently to make use of one or more storage tags.
 
 The idea is then that these offerings would require your primary storage to
 have a given tag or tags and it never will (effectively disabling that primary
 storage from being used with your offerings).
 
 On Sunday, February 22, 2015, Mike Tutkowski
 mike.tutkow...@solidfire.com
 wrote:
 
  What about changing the storage tags field of your primary storage so
  it doesn't serve as a match for any compute or disk offering?
 
  On Sunday, February 22, 2015, Andrija Panic andrija.pa...@gmail.com
  javascript:_e(%7B%7D,'cvml','andrija.pa...@gmail.com'); wrote:
 
  Hi folks,
 
  I was wondering is it safe to change the Cluster Wide primary storage
  to a Zone Wide primary storage (change the database,
  cloud.storage_pool). This is ACS 4.3.0
 
 
  Or actually better question - since I have some old NFS servers exist
  as Primary Storage - is there any way to exclude this NFS (Cluster
  Wide)  - or disable it, so the new Volume Uploads will not use that
  NFS storage...
 
  This is some trail from the history, that I can't really delete, but
  would like to completely disable further usage of this NFS server...
  if possible at all...
 
  THanks,
 
 
  --
 
  Andrija Panić
 
 
 
  --
  *Mike Tutkowski*
  *Senior CloudStack Developer, SolidFire Inc.*
  e: mike.tutkow...@solidfire.com
  javascript:_e(%7B%7D,'cvml','mike.tutkow...@solidfire.com');
  o: 303.746.7302
  Advancing the way the world uses the cloud
  http://solidfire.com/solution/overview/?video=play*™*
 
 
 
 --
 *Mike Tutkowski*
 *Senior CloudStack Developer, SolidFire Inc.*
 e: mike.tutkow...@solidfire.com
 o: 303.746.7302
 Advancing the way the world uses the cloud
 http://solidfire.com/solution/overview/?video=play*™*


RE: Error while deploying System VMs

2014-10-21 Thread Devdeep Singh
If you dashboard is showing 0/0 that means the secondary storage has not been 
successfully added. What error do you get while adding secondary storage? Is 
there anything else in the logs besides the stack trace you have shared?

Regards,
Devdeep

 -Original Message-
 From: Ashutosh Kelkar [mailto:ashut...@clogeny.com]
 Sent: Tuesday, October 21, 2014 11:51 AM
 To: cloudstack
 Subject: Error while deploying System VMs
 
 Hello all,
 
 I set up CS on latest master code, and I observe error while starting the
 system VMs. Also, the secondary storage is shown as 0/0 on Dashboard. I
 can see the error while mounting secondary storage, could not identify the
 cause.
 
 I have executed cloud-install-sys-tmplt command it successfully seeds the
 system templates.
 
 Please help me in identifying the exact issue, here are the management
 server logs.
 
 WARN  [c.c.u.d.Merovingian2] (Work-Job-Executor-19:ctx-0e65266f
 job-306/job-336 ctx-b37c0c6e) Was unable to find lock for the key
 template_spool_ref126 and thread id 4
 18267521
 com.cloud.utils.exception.CloudRuntimeException: Was unable to find lock
 for the key template_spool_ref126 and thread id 418267521
 at com.cloud.utils.db.Merovingian2.release(Merovingian2.java:274)
 at
 com.cloud.utils.db.TransactionLegacy.release(TransactionLegacy.java:397)
 at
 com.cloud.utils.db.GenericDaoBase.releaseFromLockTable(GenericDaoBase.
 java:1045)
 at sun.reflect.GeneratedMethodAccessor157.invoke(Unknown Source)
 at
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccess
 orImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:606)
 at
 org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(
 AopUtils.java:317)
 at
 org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoi
 npoint(ReflectiveMethodInvocation.java:183)
 at
 org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(
 ReflectiveMethodInvocation.java:150)
 at
 com.cloud.utils.db.TransactionContextInterceptor.invoke(TransactionContex
 tInterceptor.java:34)
 at
 org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(
 ReflectiveMethodInvocation.java:161)
 at
 org.springframework.aop.interceptor.ExposeInvocationInterceptor.invoke(E
 xposeInvocationInterceptor.java:91)
 at
 org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(
 ReflectiveMethodInvocation.java:172)
 at
 org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDyna
 micAopProxy.java:204)
 at com.sun.proxy.$Proxy73.releaseFromLockTable(Unknown Source)
 at
 org.apache.cloudstack.storage.volume.VolumeServiceImpl.createBaseImag
 eAsync(VolumeServiceImpl.java:514)
 at
 org.apache.cloudstack.storage.volume.VolumeServiceImpl.createVolumeFro
 mTemplateAsync(VolumeServiceImpl.java:748)
 at
 org.apache.cloudstack.engine.orchestration.VolumeOrchestrator.recreateV
 olume(VolumeOrchestrator.java:1227)
 at
 org.apache.cloudstack.engine.orchestration.VolumeOrchestrator.prepare(V
 olumeOrchestrator.java:1297)
 at
 com.cloud.vm.VirtualMachineManagerImpl.orchestrateStart(VirtualMachine
 ManagerImpl.java:974)
 at
 com.cloud.vm.VirtualMachineManagerImpl.orchestrateStart(VirtualMachine
 ManagerImpl.java:4600)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.j
 ava:57)
 at
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccess
 orImpl.java:43)
 at java.lang.reflect.Method.invoke(Method.java:606)
 at
 com.cloud.vm.VmWorkJobHandlerProxy.handleVmWorkJob(VmWorkJobHan
 dlerProxy.java:107)
 at
 com.cloud.vm.VirtualMachineManagerImpl.handleVmWorkJob(VirtualMachi
 neManagerImpl.java:4756)
 at
 com.cloud.vm.VmWorkJobDispatcher.runJob(VmWorkJobDispatcher.java:10
 2)
 at
 org.apache.cloudstack.framework.jobs.impl.AsyncJobManagerImpl$5.runIn
 Context(AsyncJobManagerImpl.java:513)
 at
 org.apache.cloudstack.managed.context.ManagedContextRunnable$1.run(
 ManagedContextRunnable.java:49)
 at
 org.apache.cloudstack.managed.context.impl.DefaultManagedContext$1.ca
 ll(DefaultManagedContext.java:56)
 at
 org.apache.cloudstack.managed.context.impl.DefaultManagedContext.call
 WithContext(DefaultManagedContext.java:103)
 at
 org.apache.cloudstack.managed.context.impl.DefaultManagedContext.run
 WithContext(DefaultManagedContext.java:53)
 at
 org.apache.cloudstack.managed.context.ManagedContextRunnable.run(Ma
 nagedContextRunnable.java:46)
 at
 org.apache.cloudstack.framework.jobs.impl.AsyncJobManagerImpl$5.run(As
 yncJobManagerImpl.java:470)
 at
 java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
 at java.util.concurrent.FutureTask.run(FutureTask.java:262)
 at
 

RE: [ANNOUNCE] New PMC Member: Rajani Karuturi

2014-10-17 Thread Devdeep Singh
Congrats Rajani.

Regards,
Devdeep

 -Original Message-
 From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
 Sent: Friday, October 17, 2014 3:52 PM
 To: dev
 Subject: [ANNOUNCE] New PMC Member: Rajani Karuturi
 
 The Project Management Committee (PMC) for Apache CloudStack are
 pleased to announce that Rajani Karuturi has accepted our invitation to join
 the PMC.
 
 Please join me in congratulating her.
 
 On behalf of the Apache CloudStack PMC
 --
 Daan


Re: Review Request 25989: CLOUDSTACK-7620: Added SNMP mib file for snp-alerts plugin

2014-09-26 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25989/#review54655
---


License header? It'll probably not pass the license checks. You'll either need 
to include the apache license or skip license check on it.

- Devdeep Singh


On Sept. 24, 2014, 7:29 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25989/
 ---
 
 (Updated Sept. 24, 2014, 7:29 a.m.)
 
 
 Review request for cloudstack and Devdeep Singh.
 
 
 Bugs: CLOUDSTACK-7620
 https://issues.apache.org/jira/browse/CLOUDSTACK-7620
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Added SNMP mib file for snmp-alerts plugin
 
 
 Diffs
 -
 
   plugins/alert-handlers/snmp-alerts/CS-ROOT-MIB.mib PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/25989/diff/
 
 
 Testing
 ---
 
 validates the MIB file using http://www.simpleweb.org/ietf/mibs/validate/
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 25934: Fixed various bugs in AlertsSyslogAppender

2014-09-26 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25934/#review54657
---

Ship it!


Committed in 4eeae5ad51ce8d35701509cde9b99cee4ecaccc7

- Devdeep Singh


On Sept. 23, 2014, 9:30 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25934/
 ---
 
 (Updated Sept. 23, 2014, 9:30 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Sateesh Chodapuneedi.
 
 
 Bugs: CLOUDSTACK-7610, CLOUDSTACK-7611 and CLOUDSTACK-7612
 https://issues.apache.org/jira/browse/CLOUDSTACK-7610
 https://issues.apache.org/jira/browse/CLOUDSTACK-7611
 https://issues.apache.org/jira/browse/CLOUDSTACK-7612
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Fixed following bugs in AlertsSyslogAppender
 1. Added sync alert.
 2. Changed unrecognised alerts are send as unknown instead of null.
 3. Added unit tests to cover some more scenarios.
 
 
 Diffs
 -
 
   
 plugins/alert-handlers/syslog-alerts/src/org/apache/cloudstack/syslog/AlertsSyslogAppender.java
  5f6e8ec 
   
 plugins/alert-handlers/syslog-alerts/test/org/apache/cloudstack/syslog/AlertsSyslogAppenderTest.java
  5799348 
 
 Diff: https://reviews.apache.org/r/25934/diff/
 
 
 Testing
 ---
 
 Added unit tests and all were passing fine
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 25820: CLOUDSTACK-7575: Improve performance of cleanString method in StringUtils

2014-09-26 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25820/#review54658
---

Ship it!


Committed changes in 7972f53c8d35b78bd27cbd2c8bb1beb763b69c00

- Devdeep Singh


On Sept. 19, 2014, 5:51 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25820/
 ---
 
 (Updated Sept. 19, 2014, 5:51 a.m.)
 
 
 Review request for cloudstack and Devdeep Singh.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 We are using two regular expressions to clean sensitive data from logged 
 string. One regular expression is cleaning from middle while other from end. 
 This patch combines those two regular expressions in one regular expression
 
 
 Diffs
 -
 
   utils/src/com/cloud/utils/StringUtils.java 7014e5c 
 
 Diff: https://reviews.apache.org/r/25820/diff/
 
 
 Testing
 ---
 
 tested the strings are cleaned up properly of passwords after the changes
 
 
 Thanks,
 
 Anshul Gangwar
 




RE: Regarding commit 7f440854f7bcd41a1bd6581c0239cde2e98261b7

2014-09-25 Thread Devdeep Singh
Yes, I have made a similar fix and committed the changes.

Regards,
Devdeep

 -Original Message-
 From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
 Sent: Thursday, September 25, 2014 1:05 AM
 To: Devdeep Singh
 Cc: dev@cloudstack.apache.org
 Subject: Re: Regarding commit
 7f440854f7bcd41a1bd6581c0239cde2e98261b7
 
 Devdeep,
 
 Guess I made a booboo. It should have read like this , agree?
 
 cmd = resource.getCurrentStatus(_id);
 while ( cmd == null  ++retried 
 _HostPingRetryCount.value())
 {
 cmd = resource.getCurrentStatus(_id);
 Thread.sleep(1000*_HostPingRetryTimer.value());
 }
 
 
 On Wed, Sep 24, 2014 at 4:22 PM, Devdeep Singh
 devdeep.si...@citrix.com
 wrote:
 
  Hi Daan,
 
  I am looking into bug [1] which reports that the vmsync functionality
  is broken on master. If a vm deployed by cloudstack is stopped
  directly on the hypervisor, its state is not updated in cloudstack. I
  see that in commit 7f440854f7bcd41a1bd6581c0239cde2e98261b7, a
 change
  was made to retry retrieving the resource status (PingCommand) to deal
 with network glitches.
  However, there is an issue with the commit and it has caused a
  regression with the vmsync functionality. Now, by default, the
  PingTask is not checking for the status of the resource. This breaks
  vmsync for all direct connected agents, which includes xenserver, hyper-v
 etc.
 
  I'll be submitting a fix for this issue. Do let me know if you have
  any concerns with it.
 
  [1] https://issues.apache.org/jira/browse/CLOUDSTACK-7598
 
  Regards,
  Devdeep
 
 
 
 
 --
 Daan


Regarding commit 7f440854f7bcd41a1bd6581c0239cde2e98261b7

2014-09-24 Thread Devdeep Singh
Hi Daan,

I am looking into bug [1] which reports that the vmsync functionality is broken 
on master. If a vm deployed by cloudstack is stopped directly on the 
hypervisor, its state is not updated in cloudstack. I see that in commit 
7f440854f7bcd41a1bd6581c0239cde2e98261b7, a change was made to retry retrieving 
the resource status (PingCommand) to deal with network glitches. However, there 
is an issue with the commit and it has caused a regression with the vmsync 
functionality. Now, by default, the PingTask is not checking for the status of 
the resource. This breaks vmsync for all direct connected agents, which 
includes xenserver, hyper-v etc.

I'll be submitting a fix for this issue. Do let me know if you have any 
concerns with it.

[1] https://issues.apache.org/jira/browse/CLOUDSTACK-7598

Regards,
Devdeep


Re: Review Request 25292: CLOUDSTACK-6898: [Hyper-V] fixed rdp console freezing during reboot.

2014-09-10 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25292/#review52824
---

Ship it!


Committed in 4c9507d21e82e9c55ac64e75165329565ec6a14f to master

- Devdeep Singh


On Sept. 3, 2014, 9:07 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25292/
 ---
 
 (Updated Sept. 3, 2014, 9:07 a.m.)
 
 
 Review request for cloudstack and Devdeep Singh.
 
 
 Bugs: CLOUDSTACK-6898
 https://issues.apache.org/jira/browse/CLOUDSTACK-6898
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 
 
 Console was freezing because we read data from socket in blocking mode.
 During reboot it was blocking infintely.
 To fix issue, now we are reading data in non-blocking mode.
 In non-blocking mode I set the timeout to 5 seconds.
 
 
 Diffs
 -
 
   services/console-proxy-rdp/rdpconsole/src/main/java/streamer/SyncLink.java 
 94281d2 
   
 services/console-proxy-rdp/rdpconsole/src/main/java/streamer/apr/AprSocketSource.java
  0298349 
 
 Diff: https://reviews.apache.org/r/25292/diff/
 
 
 Testing
 ---
 
 tested console is working fine during reboot. Now it freezes 5-6 seconds 
 before becoming responsive again.
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 25427: fixed few coverity issues

2014-09-10 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25427/#review52825
---

Ship it!


Committed in a400eecf6c868c05bacee21689238a1e0ce2 to master

- Devdeep Singh


On Sept. 8, 2014, 8:57 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25427/
 ---
 
 (Updated Sept. 8, 2014, 8:57 a.m.)
 
 
 Review request for cloudstack and Devdeep Singh.
 
 
 Bugs: CLOUDSTACK-7503
 https://issues.apache.org/jira/browse/CLOUDSTACK-7503
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 fixed few coverity issues. Not logging the proper string.
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
  76f8e75 
 
 Diff: https://reviews.apache.org/r/25427/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 25426: CLOUDSTACK-7370: Fixed password visible in plain text if password passed is in the end in url

2014-09-10 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25426/#review52827
---


Change looks ok but Utils unit test is failing. Kindly address the issue with 
failed unit tests too.

- Devdeep Singh


On Sept. 8, 2014, 8:22 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25426/
 ---
 
 (Updated Sept. 8, 2014, 8:22 a.m.)
 
 
 Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
 
 
 Bugs: CLOUDSTACK-7370
 https://issues.apache.org/jira/browse/CLOUDSTACK-7370
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 updated the regular expression which is used to clean password
 
 
 Diffs
 -
 
   utils/src/com/cloud/utils/StringUtils.java 17b8fc0 
 
 Diff: https://reviews.apache.org/r/25426/diff/
 
 
 Testing
 ---
 
 tested by passing affected strings through regular expressions.
 
 
 Thanks,
 
 Anshul Gangwar
 




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

2014-09-10 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25430/#review52852
---

Ship it!


Ship It!

- Devdeep Singh


On Sept. 8, 2014, 1:55 p.m., bharat kumar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25430/
 ---
 
 (Updated Sept. 8, 2014, 1:55 p.m.)
 
 
 Review request for cloudstack, Alena Prokharchyk, Devdeep Singh, edison su, 
 Koushik Das, Mike Tutkowski, and Nitin Mehta.
 
 
 Bugs: CLOUDSTACK-6099
 https://issues.apache.org/jira/browse/CLOUDSTACK-6099
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 live migration is failing for vm deployed using dynaic compute offerings with 
 NPE
 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-10 Thread Devdeep Singh


 On Sept. 10, 2014, 10:29 a.m., Rohit Yadav wrote:
  Just want to check with other reviewers, if this contribution is good to 
  merge on master? Bharat, if you may confirm that it is well tested on 
  Xen/KVM?

The fix looks correct and it can be merged. The issue was that with custom 
offerings the profile wasn't getting initialized correctly and it lacked the 
cpu and memory details. This would cause a null pointer exception in the 
allocators. I can apply the patch if no one else is working on it.


- Devdeep


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25430/#review52846
---


On Sept. 8, 2014, 1:55 p.m., bharat kumar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/25430/
 ---
 
 (Updated Sept. 8, 2014, 1:55 p.m.)
 
 
 Review request for cloudstack, Alena Prokharchyk, Devdeep Singh, edison su, 
 Koushik Das, Mike Tutkowski, and Nitin Mehta.
 
 
 Bugs: CLOUDSTACK-6099
 https://issues.apache.org/jira/browse/CLOUDSTACK-6099
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 live migration is failing for vm deployed using dynaic compute offerings with 
 NPE
 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 24444: CLOUDSTACK-7277: Removed the Hyper-V Agent service dependency on english locale to sync VMs properly

2014-08-18 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2/#review50859
---

Ship it!


Ship It!

- Devdeep Singh


On Aug. 7, 2014, 6:13 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/2/
 ---
 
 (Updated Aug. 7, 2014, 6:13 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-7277
 https://issues.apache.org/jira/browse/CLOUDSTACK-7277
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 For vmsync first VMs are listed and than their power state is determined.
 To exclude hosted system we filter the result on Caption='Virtual Machine' 
 but this string is locale dependent so it may not
 not work properly for locales other than english. To overcome this now we 
 started using ProcessId = 0 filter
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  195a964 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 b2a7b45 
 
 Diff: https://reviews.apache.org/r/2/diff/
 
 
 Testing
 ---
 
 tested on my local system with ProcessId = 0 filter VMs are syncing up 
 properly
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 24228: CLOUDSTACK-7235: Fixed ModifyStoragePoolCommand returns unsupported command answer if provided wrong path

2014-08-18 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24228/#review50861
---

Ship it!


Committed in d1796c5f585e79d9f833b414023352e5df84fb03 to asf master

- Devdeep Singh


On Aug. 4, 2014, 10:10 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/24228/
 ---
 
 (Updated Aug. 4, 2014, 10:10 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-7235
 https://issues.apache.org/jira/browse/CLOUDSTACK-7235
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
  Now ModifyStoragePoolCommand returns appropriate error if provided wrong path
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  195a964 
 
 Diff: https://reviews.apache.org/r/24228/diff/
 
 
 Testing
 ---
 
 verified by adding the primary storage with wrong path. Appropriate error is 
 returned from the agent
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 24444: CLOUDSTACK-7277: Removed the Hyper-V Agent service dependency on english locale to sync VMs properly

2014-08-18 Thread Devdeep Singh


 On Aug. 18, 2014, 6:29 a.m., Devdeep Singh wrote:
  Ship It!

Committed in master in commit 1c688f40e134e2f65fe11bb37946e550feb896ca


- Devdeep


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/2/#review50859
---


On Aug. 7, 2014, 6:13 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/2/
 ---
 
 (Updated Aug. 7, 2014, 6:13 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-7277
 https://issues.apache.org/jira/browse/CLOUDSTACK-7277
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 For vmsync first VMs are listed and than their power state is determined.
 To exclude hosted system we filter the result on Caption='Virtual Machine' 
 but this string is locale dependent so it may not
 not work properly for locales other than english. To overcome this now we 
 started using ProcessId = 0 filter
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  195a964 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 b2a7b45 
 
 Diff: https://reviews.apache.org/r/2/diff/
 
 
 Testing
 ---
 
 tested on my local system with ProcessId = 0 filter VMs are syncing up 
 properly
 
 
 Thanks,
 
 Anshul Gangwar
 




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

2014-08-18 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24226/#review50862
---

Ship it!


Pushed to master in commit 0ecc9f79af0f915ac347f856c3c2f8ce40f636af

- Devdeep Singh


On Aug. 4, 2014, 6:08 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/24226/
 ---
 
 (Updated Aug. 4, 2014, 6:08 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-7234
 https://issues.apache.org/jira/browse/CLOUDSTACK-7234
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 We no longer uses SMB credentials on Hyper-V Agent so we are stopping to send 
 SMB credentials to Hyper-V
 
 
 Diffs
 -
 
   
 engine/schema/src/org/apache/cloudstack/storage/datastore/db/StoragePoolVO.java
  1508ce0 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/CloudStackTypes.cs
  4516191 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 
 b40cb8f 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/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
 




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

2014-08-18 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/23877/#review50863
---

Ship it!


Pushed to master in commit 97f5281cf46e686d23df0fe0b1db0a6b6d591944

- Devdeep Singh


On July 24, 2014, 5:39 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/23877/
 ---
 
 (Updated July 24, 2014, 5:39 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh 
 Chodapuneedi.
 
 
 Bugs: CLOUDSTACK-7177
 https://issues.apache.org/jira/browse/CLOUDSTACK-7177
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 was filtering the ip address with ports as invalid address and hence the 
 alerts were not sent to non-default ports
 
 
 Diffs
 -
 
   
 plugins/alert-handlers/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: Review Request 24174: CLOUDSTACK-7220: fixed hyper-v agent broken in 4.4 branch

2014-08-01 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24174/#review49336
---

Ship it!


Ship It!

- Devdeep Singh


On Aug. 1, 2014, 5:21 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/24174/
 ---
 
 (Updated Aug. 1, 2014, 5:21 a.m.)
 
 
 Review request for cloudstack, daan Hoogland and Devdeep Singh.
 
 
 Bugs: CLOUDSTACK-7220
 https://issues.apache.org/jira/browse/CLOUDSTACK-7220
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 the commits were not applied in correct order while cherry picking from 
 4.4-forward to 4.4
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/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-08-01 Thread Devdeep Singh
Applied the patch to 4.4 branch and pushed. Thanks Anshul.

Regards,
Devdeep

 -Original Message-
 From: Daan Hoogland [mailto:daan.hoogl...@gmail.com]
 Sent: Friday, August 1, 2014 2:13 PM
 To: dev
 Subject: Re: Building of Hyper-V Agent seems broken in 4.4 branch
 
 Anshul, go ahead.
 
 On Fri, Aug 1, 2014 at 7:24 AM, Anshul Gangwar anshul.gang...@citrix.com
 wrote:
  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@cloudstack.apache.org
  Subject: RE: Building of Hyper-V Agent seems broken in 4.4 branch
 
  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,
  Anshul
 
  -Original Message-
  From: Devdeep Singh [mailto:devdeep.si...@citrix.com]
  Sent: Friday, August 01, 2014 10:01 AM
  To: dev@cloudstack.apache.org
  Subject: RE: Building of Hyper-V Agent seems broken in 4.4 branch
 
  Taking a look at it. Maybe a cherry-pick from 4.4-forward to 4.4 was missed.
 
  Regards,
  Devdeep
 
  -Original Message-
  From: Erik Weber [mailto:terbol...@gmail.com]
  Sent: Thursday, July 31, 2014 9:18 PM
  To: dev
  Subject: Building of Hyper-V Agent seems broken in 4.4 branch
 
  4.4.0 tag is working correctly, as is master branch.
 
  Any fix or advice appreciated.
 
 
  Erik
 
  -- compile output 
 
   Target CoreCompile:
  Tool
  C:\PROGRA~2\MONO-3~1.3\bin\mcs.bat
  execution started with arguments: /noconfig /debug:full /debug+
  /optimize- /out:obj\Debug\HypervResource.dll CloudStackTypes.cs
  IWmiCallsV2.cs WmiCallsV2.cs Properties\AssemblyInfo.cs
  HypervResourceController.cs Utils.cs /target:library
 /define:DEBUG;TRACE
  /reference:..\packages\AWSSDK.1.5.23.0\lib\AWSSDK.dll
  /reference:..\packages\DotNetZip.1.9.1.8\lib\net20\Ionic.Zip.dll
  /reference:..\packages\log4net.2.0.0\lib\net40-full\log4net.dll
 
 /reference:..\packages\Newtonsoft.Json.4.5.11\lib\net40\Newtonsoft.Json.
  dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Confi
  g
  ura
  tion.dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Manag
  e
  m
  ent.dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Net.Http.
  dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Web.d
  l
  l
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Web.H
  t
  tp
  .dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Xml.Linq.
  dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Data.
  D
  ata
  SetExtensions.dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\Microsoft.CSharp.
  dll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Data.
  d
  ll
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Xml.d
  l
  l
  /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
  frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Core.
  d
  ll
 
 /reference:C:\source\cloudstack\plugins\hypervisors\hyperv\DotNet\Ser
  v e rResource\WmiWrappers\bin\Debug\\WmiWrappers.dll
  /warn:4
  WmiCallsV2.cs(336,27): error CS0117: `HypervResource.Utils' does not
  contain a definition for `ConnectToRemote'
  Utils.cs(32,18): (Location of the
  symbol related to previous
  error)
  WmiCallsV2.cs(2448,69): error CS0103: The name `jobObj' does not
  exist in the current context
  HypervResourceController.cs(239,31): error CS0117:
 `HypervResource.Utils'
  does not contain a definition for `ConnectToRemote'
  Utils.cs(32,18): (Location of the
  symbol related to previous
  error)
  HypervResourceController.cs(250,35): error CS0117:
 `HypervResource.Utils'
  does not contain a definition for `ConnectToRemote

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

2014-07-31 Thread Devdeep Singh
Taking a look at it. Maybe a cherry-pick from 4.4-forward to 4.4 was missed.

Regards,
Devdeep

 -Original Message-
 From: Erik Weber [mailto:terbol...@gmail.com]
 Sent: Thursday, July 31, 2014 9:18 PM
 To: dev
 Subject: Building of Hyper-V Agent seems broken in 4.4 branch
 
 4.4.0 tag is working correctly, as is master branch.
 
 Any fix or advice appreciated.
 
 
 Erik
 
 -- compile output 
 
  Target CoreCompile:
 Tool C:\PROGRA~2\MONO-3~1.3\bin\mcs.bat
 execution started with arguments: /noconfig /debug:full /debug+ /optimize-
 /out:obj\Debug\HypervResource.dll CloudStackTypes.cs IWmiCallsV2.cs
 WmiCallsV2.cs Properties\AssemblyInfo.cs HypervResourceController.cs
 Utils.cs /target:library /define:DEBUG;TRACE
 /reference:..\packages\AWSSDK.1.5.23.0\lib\AWSSDK.dll
 /reference:..\packages\DotNetZip.1.9.1.8\lib\net20\Ionic.Zip.dll
 /reference:..\packages\log4net.2.0.0\lib\net40-full\log4net.dll
 /reference:..\packages\Newtonsoft.Json.4.5.11\lib\net40\Newtonsoft.Json.
 dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Configura
 tion.dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Managem
 ent.dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Net.Http.
 dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Web.dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Web.Http
 .dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Xml.Linq.
 dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Data.Data
 SetExtensions.dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\Microsoft.CSharp.
 dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Data.dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Xml.dll
 /reference:C:\PROGRA~2\MONO-3~1.3\lib\mono\4.5\..\xbuild-
 frameworks\.NETFramework\v4.5\RedistList\..\..\..\..\4.5\System.Core.dll
 /reference:C:\source\cloudstack\plugins\hypervisors\hyperv\DotNet\Serve
 rResource\WmiWrappers\bin\Debug\\WmiWrappers.dll
 /warn:4
 WmiCallsV2.cs(336,27): error CS0117: `HypervResource.Utils' does not
 contain a definition for `ConnectToRemote'
 Utils.cs(32,18): (Location of the symbol 
 related to previous
 error)
 WmiCallsV2.cs(2448,69): error CS0103: The name `jobObj' does not exist in
 the current context
 HypervResourceController.cs(239,31): error CS0117: `HypervResource.Utils'
 does not contain a definition for `ConnectToRemote'
 Utils.cs(32,18): (Location of the symbol 
 related to previous
 error)
 HypervResourceController.cs(250,35): error CS0117: `HypervResource.Utils'
 does not contain a definition for `ConnectToRemote'
 Utils.cs(32,18): (Location of the symbol 
 related to previous
 error)
 HypervResourceController.cs(992,27): error CS0117: `HypervResource.Utils'
 does not contain a definition for `ConnectToRemote'
 Utils.cs(32,18): (Location of the symbol 
 related to previous
 error)
 HypervResourceController.cs(1284,31): error CS0117: `HypervResource.Utils'
 does not contain a definition for `ConnectToRemote'
 Utils.cs(32,18): (Location of the symbol 
 related to previous
 error)
 HypervResourceController.cs(1560,39): error CS0117: `HypervResource.Utils'
 does not contain a definition for `ConnectToRemote'
 Utils.cs(32,18): (Location of the symbol 
 related to previous
 error)
 HypervResourceController.cs(1567,35): error CS0117: `HypervResource.Utils'
 does not contain a definition for `ConnectToRemote'
 Utils.cs(32,18): (Location of the symbol 
 related to previous
 error)
 Task Csc execution -- FAILED
 Done building target CoreCompile in project
 C:\source\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\
 HypervResource\HypervResource.csproj.--
 FAILED
 Done building project
 C:\source\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\
 HypervResource\HypervResource.csproj.--
 FAILED
 Task MSBuild execution 

RE: [ANNOUNCE] Santhosh Edukulla as a committer...

2014-06-30 Thread Devdeep Singh
Congrats Santhosh.

 -Original Message-
 From: Alex Huang [mailto:alex.hu...@citrix.com]
 Sent: Monday, June 30, 2014 9:55 PM
 To: dev@cloudstack.apache.org
 Subject: [ANNOUNCE] Santhosh Edukulla as a committer...
 
 Hi All,
 
 The Project Management Committee (PMC) for Apache CloudStack has
 asked Santhosh Edukulla to become a committer and we are pleased to
 announce that he has accepted.
 
 Being a committer allows many contributors to contribute more
 autonomously. For developers, it makes it easier to submit changes and
 eliminates the need to have contributions reviewed via the patch submission
 process. Whether contributions are development-related or otherwise, it is a
 recognition of a contributor's participation in the project and commitment to
 the project and the Apache Way.
 
 Please join me in congratulating Santhosh!
 
 --Alex, on behalf of the CloudStack PMC


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

2014-06-23 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22807/#review46389
---

Ship it!


Ship It!

- Devdeep Singh


On June 23, 2014, 5:30 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22807/
 ---
 
 (Updated June 23, 2014, 5:30 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh 
 Chodapuneedi.
 
 
 Bugs: CLOUDSTACK-6965
 https://issues.apache.org/jira/browse/CLOUDSTACK-6965
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Fix for CLOUDSTACK-6935 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 cluster is null for ZoneWideStoragePool and Hence the exception.
 
 
 Diffs
 -
 
   
 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-23 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22808/#review46390
---

Ship it!


Ship It!

- Devdeep Singh


On June 23, 2014, 6:01 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22808/
 ---
 
 (Updated June 23, 2014, 6:01 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh 
 Chodapuneedi.
 
 
 Bugs: CLOUDSTACK-6968
 https://issues.apache.org/jira/browse/CLOUDSTACK-6968
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
  allowing cluster scope volumes to attach to any VM. If migration is needed 
 then first they will be migrated to appropriate cluster before attaching
 
 
 Diffs
 -
 
   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
 




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 Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22256/#review46267
---

Ship it!


Ship It!

- Devdeep Singh


On June 20, 2014, 10:21 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22256/
 ---
 
 (Updated June 20, 2014, 10:21 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh 
 Chodapuneedi.
 
 
 Bugs: CLOUDSTACK-6830
 https://issues.apache.org/jira/browse/CLOUDSTACK-6830
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 fixed vm migration failing between different clusters if volumes are on zone 
 primary store. During VM migration, volumes on zone wide primary store 
 requiring storage migration resulting in failure of VM migration
 
 This also improves the hostsformigration api. Firstly we were ttrying to 
 list all hosts and then finding suitable storage pools for all volumes
 and then we were checking for whether vm migration requires storage 
 migration
 Now the process is reversed, We are checking for only those volumes which 
 are not in zone wide primary store and only verifying
 that volumes-poolid-clusterid is different from host clusterid and 
 usesLocal then only it requires storage migration
 
 
 Diffs
 -
 
   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
 4502365 
   server/src/com/cloud/server/ManagementServerImpl.java dd4ce0f 
 
 Diff: https://reviews.apache.org/r/22256/diff/
 
 
 Testing
 ---
 
 tested following scenarios on my local setup.
 1. VM migration for volumes 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: Review Request 22807: CLOUDSTACK-6965: Fix for CLOUDSTACK-6935 introduces the NullPointerException in ZoneWideStorageAllocator filter method

2014-06-20 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22807/#review46268
---



engine/storage/src/org/apache/cloudstack/storage/allocator/AbstractStoragePoolAllocator.java
https://reviews.apache.org/r/22807/#comment81579

For a zone wide storage pool, the clusterId would be null. In that case you 
should check with the storage pool hypervisor type. A zone wide storage pool 
has a hypervisor field associated with it.


- Devdeep Singh


On June 20, 2014, 10:21 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22807/
 ---
 
 (Updated June 20, 2014, 10:21 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh 
 Chodapuneedi.
 
 
 Bugs: CLOUDSTACK-6965
 https://issues.apache.org/jira/browse/CLOUDSTACK-6965
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Fix for CLOUDSTACK-6935 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 cluster is null for ZoneWideStoragePool and Hence the exception.
 
 
 Diffs
 -
 
   
 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 22415: CLOUDSTACK-6867: added the option for vhdx image format in upload volume

2014-06-13 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22415/#review45596
---

Ship it!


Ship It!

- Devdeep Singh


On June 11, 2014, 5:20 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22415/
 ---
 
 (Updated June 11, 2014, 5:20 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6867
 https://issues.apache.org/jira/browse/CLOUDSTACK-6867
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 added the option for vhdx image format in upload volume
 
 
 Diffs
 -
 
   ui/scripts/storage.js 604f09d 
 
 Diff: https://reviews.apache.org/r/22415/diff/
 
 
 Testing
 ---
 
 verified by opening the upload volume dialog in UI
 
 
 Thanks,
 
 Anshul Gangwar
 




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

2014-06-13 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22414/#review45595
---

Ship it!


Ship It!

- Devdeep Singh


On June 10, 2014, 11:23 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22414/
 ---
 
 (Updated June 10, 2014, 11:23 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6865 and CLOUDSTACK-6868
 https://issues.apache.org/jira/browse/CLOUDSTACK-6865
 https://issues.apache.org/jira/browse/CLOUDSTACK-6868
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 while attaching the volume we were changing the volume's Image format to 
 hypervisor's default Image format, hence it was failing to find the volume 
 with vhdx format
 Now changed the behavior to set Image Format for volume only when it is 
 not set
 
 
 Diffs
 -
 
   
 engine/orchestration/src/org/apache/cloudstack/engine/orchestration/VolumeOrchestrator.java
  064ffca 
 
 Diff: https://reviews.apache.org/r/22414/diff/
 
 
 Testing
 ---
 
 verified by uploading the vhd volume and attaching it to VM
 
 
 Thanks,
 
 Anshul Gangwar
 




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

2014-06-13 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22452/#review45597
---

Ship it!


Ship It!

- Devdeep Singh


On June 11, 2014, 7:31 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22452/
 ---
 
 (Updated June 11, 2014, 7:31 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6872
 https://issues.apache.org/jira/browse/CLOUDSTACK-6872
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Removed the redundant connectTORemote method in Hyper-v Agent. We dont't need 
 this method as share is domain joined so it has all the required permissions. 
 Removal of this method fixes this bug also.
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  0ad95b8 
   plugins/hypervisors/hyperv/DotNet/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
 




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 Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22500/#review45600
---

Ship it!


Ship It!

- Devdeep Singh


On June 13, 2014, 9:08 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/22500/
 ---
 
 (Updated June 13, 2014, 9:08 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6897
 https://issues.apache.org/jira/browse/CLOUDSTACK-6897
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 when we try to attach the uploaded/allocated volume to a VM on zwps,
  then we were passing the podId of VM instead of storage pool to storage 
 allocator.
  This resulting in use of Clusterscope storage allocator, allocating a 
 storage pool for VM on zwps beacuse of pod id not null.
   This was resulting in scope conflict later
 
 
 Diffs
 -
 
   
 engine/orchestration/src/org/apache/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
 




RE: [ANNOUNCE] Amogh Vasekar as committer

2014-06-02 Thread Devdeep Singh
Congrats Amogh.

 -Original Message-
 From: John Kinsella [mailto:j...@stratosec.co]
 Sent: Monday, June 2, 2014 11:44 PM
 To: dev@cloudstack.apache.org
 Subject: [ANNOUNCE] Amogh Vasekar as committer
 
 The Project Management Committee (PMC) for Apache CloudStack has
 asked Amogh Vasekar to become a committer and we are pleased to
 announce that he has accepted.
 
 Being a committer allows many contributors to contribute more
 autonomously. For developers, it makes it easier to submit changes and
 eliminates the need to have contributions reviewed via the patch submission
 process. Whether contributions are development-related or otherwise, it is a
 recognition of a contributor's participation in the project and commitment to
 the project and the Apache Way.
 
 Please join me in congratulating Amogh!
 
 -John, on behalf of the CloudStack PMC


[ACS 4.4] Cherry-pick for CLOUDSTACK-6802 and CLOUDSTACK-6810

2014-05-30 Thread Devdeep Singh
Hi Daan,

Could you please cherry-pick commit f5ce280da32c4e0837fa39651dc801ba3f906e41 
from 4.4-forward branch to 4.4 branch.

commit f5ce280da32c4e0837fa39651dc801ba3f906e41
Author: Devdeep Singh devd...@gmail.com
Date:   Tue May 20 11:40:51 2014 +0530

CLOUDSTACK-6810: Fix storage migration of a vm with volume on local was 
failing. When a plan
with hostid included was passed to the local storage pool allocator, it 
returned all the local
storage pools in the cluster, instead of just the local pool on the given 
host in the plan.
This was happening the search at a host level was happening only for data 
disk. Fixed this.
Additionally, the query to list the storage pools on a host was failing if 
the pool did have
tags. Fixed the query too.

CLOUDSTACK-6802: Fix for not being able to attach data disk on local. This 
issue gets fixed
with the above issue too. The query to list pools on a host was failing if 
there were no
tags on the storage pool.

Regards,
Devdeep


Regarding local storage pool allocator

2014-05-28 Thread Devdeep Singh
Hi,

I am looking into an issue where migration of a vm with its volume (storage 
motion) fails. The root volume of the vm is on local storage. The issue is that 
when LocalStoragePoolAllocator::select gets called and the hostid is included 
in the plan, the function ends up returning local storage pools from other 
hosts in the cluster too. This is happening because we list local storage pools 
on a host only if the volume type is 'Data'; otherwise we list all the storage 
pools in the cluster.

code snippet
// 4.4-forward branch, 
./engine/storage/src/org/apache/cloudstack/storage/allocator/LocalStoragePoolAllocator.java,
 Line 76
if (dskCh.getType() == Volume.Type.DATADISK  plan.getHostId() != null) {
/code snippet

Anyone aware why the check for data disk type was added in the first place? 
Will it break any scenario if I remove this check?

Regards,
Devdeep


Re: 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-27 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21429/#review43951
---

Ship it!


Ship It!

- Devdeep Singh


On May 14, 2014, 8:02 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/21429/
 ---
 
 (Updated May 14, 2014, 8:02 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6663
 https://issues.apache.org/jira/browse/CLOUDSTACK-6663
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Before starting Hyper-V agent we initialize HypervResourceController. We try 
 to initialize it with IP address 0.0.0.0. We try to get nic info of 0.0.0.0. 
 To get nic info we iterate through all nics and return the last NIC in the 
 list if it doesn't match with any IP address. So in case last NIC doesn't 
 have unicastAddress, Hyper-V agent will fail to start.
 
 We don't need IP address during initialization. It get initialized with 
 startupcommand later
 
 
 Diffs
 -
 
   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
 




Re: 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-16 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21260/#review42568
---

Ship it!


Ship It!

- Devdeep Singh


On May 9, 2014, 9:26 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/21260/
 ---
 
 (Updated May 9, 2014, 9:26 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6620
 https://issues.apache.org/jira/browse/CLOUDSTACK-6620
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 [Hyper-v] when all vms on host are deleted from outside, then there is null 
 pointer exception in hyperv agent in getting vmstats. As hyperv agent tries 
 to get the stats of non-existent vm
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 7dbb8c1 
 
 Diff: https://reviews.apache.org/r/21260/diff/
 
 
 Testing
 ---
 
 Created the needed environment. Verified that after application of patch 
 there was no nullpointerexception
 
 
 Thanks,
 
 Anshul Gangwar
 




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

2014-05-16 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21259/#review42567
---

Ship it!


Ship It!

- Devdeep Singh


On May 9, 2014, 9:07 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/21259/
 ---
 
 (Updated May 9, 2014, 9:07 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6399
 https://issues.apache.org/jira/browse/CLOUDSTACK-6399
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 when there are no vms running on hyperv host, then was returning null which 
 has different meaning. Now returning empty list of hostvmstatereport
 
 
 Diffs
 -
 
   
 plugins/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
 




[ACS44] Cherry pick for CLOUDSTACK-6510 and CLOUDSTACK-6622

2014-05-13 Thread Devdeep Singh
Hi Daan,

Please cherry pick the following commits to 4.4 branch.

commit 681e628543f40fcf1f3cb89592c2dc8e89b5eeb1
Author: Devdeep Singh devd...@gmail.com
Date:   Mon May 5 16:59:39 2014 +0530

CLOUDSTACK-6510: Fix gson serialization exception in storage migration. 
Gson couldn't serialize
a map with volume and storagepool objects for logging. Fixed by using 
volume and storage pool
ids instead of objects in the map.

commit 98df3e55b93df9f30b5767b20c0a4a6f1336406a
Author: Devdeep Singh devd...@gmail.com
Date:   Fri May 9 15:54:56 2014 +0530

CLOUDSTACK-6622: After a volume was live migrated, the destination smb 
storage path was added to
the folder column. For an smb share the smb credentials are in the query 
string of the path.
Before adding the path, smb shares query string should be cleaned up.

Regards,
Devdeep


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

2014-04-25 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20651/#review41455
---

Ship it!


Ship It!

- Devdeep Singh


On April 25, 2014, 8:28 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/20651/
 ---
 
 (Updated April 25, 2014, 8:28 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6470
 https://issues.apache.org/jira/browse/CLOUDSTACK-6470
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 When we stop VM in case of hyper-v, then it is always force shut downed i.e. 
 turn off. Even if the integration services are installed in hyper-v. Directly 
 turning of VM may result in corruption of disk. This patch fixes this issue. 
 I have used new management class ShutdownComponent to achieve this.
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 4795073 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/WmiWrappers/ROOT.virtualization.v2.Msvm_ShutdownComponent.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/WmiWrappers/WmiWrappers.csproj
  70ae8e8 
 
 Diff: https://reviews.apache.org/r/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
 




RE: Multiple hypervisor through single management server

2014-04-22 Thread Devdeep Singh
Hi Tejas,

For Hyper-V you need to create a zone in which Hyper-V is the only hypervisor 
type. Reason being, currently in cloudstack Hyper-V only works with SMB as a 
secondary storage. You can have a zone where a mixed set of other hypervisors 
are present, for example, xenserver, kvm etc, but for hyper-v you'll have to 
create a separate zone in which hyper-v is the only hypervisor. For a hyper-v 
zone; you'll have to register a secondary storage of type SMB.

Regards,
Devdeep

 -Original Message-
 From: Tejas Gadaria [mailto:refond.g...@gmail.com]
 Sent: Tuesday, April 22, 2014 2:22 PM
 To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
 Subject: Multiple hypervisor through single management server
 
 Hi
 
 I am trying manage multiple hypervisor through Management server.
 
 1 ) I have installed Xenserver 6.2 on CS 4.3 with nfs as primary  secondary
 storage, it was working fine.
 2) Created new cluster for hyperv and added hyperv host. Immediately
 hyperv Centos template download started.
 3) Added SMB/CIFS as primary storage local to hyperv cluster.
 4) while trying to create instance from ISO, log says ,
 
 2014-04-22 13:57:57,701 DEBUG
 [c.c.n.r.VirtualNetworkApplianceManagerImpl]
 (Job-Executor-4:ctx-d714df36 ctx-a482309d) Lock is released for network id
 206 as a part of router startup in
 Dest[Zone(Id)-Pod(Id)-Cluster(Id)-Host(Id)-Storage(Volume(Id|Type--
 Pool(Id))]
 : Dest[Zone(1)-Pod(1)-Cluster(2)-Host(4)-Storage(Volume(9|ROOT--
 Pool(2))]
 2014-04-22 13:57:57,701 ERROR [c.c.v.VirtualMachineManagerImpl]
 (Job-Executor-4:ctx-d714df36 ctx-a482309d) Failed to start instance
 VM[User|CentOS65-x64-Hyperv]
 com.cloud.utils.exception.CloudRuntimeException: Template 9 has not been
 completely downloaded to zone 1
 at
 com.cloud.template.TemplateManagerImpl.getTemplateSize(TemplateMan
 agerImpl.java:1745)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 
 
 Don't understand how to upload hyperv systemvm template to secondary
 storage where already one systemvm template for xen is available.
 
 Regards,
 Tejas


RE: Multiple hypervisor through single management server

2014-04-22 Thread Devdeep Singh
Hi Tejas,

It should work but I would recommend that you seed the vmware systemvm template 
too on the secondary storage.

Regards,
Devdeep

 -Original Message-
 From: Tejas Gadaria [mailto:refond.g...@gmail.com]
 Sent: Tuesday, April 22, 2014 3:05 PM
 To: us...@cloudstack.apache.org
 Cc: dev@cloudstack.apache.org
 Subject: Re: Multiple hypervisor through single management server
 
 Hi Devdeep,
 
 Thanks for clarification about hyperv,
 
 If I undo all the changes I made for hyperv setup and go ahead to deploy
 vmware on existing xen setup, Do I need to upload vmware systemvm or it
 won't be necessary.
 
 Regards,
 Tejas
 
 
 On Tue, Apr 22, 2014 at 2:40 PM, Devdeep Singh
 devdeep.si...@citrix.comwrote:
 
  Hi Tejas,
 
  For Hyper-V you need to create a zone in which Hyper-V is the only
  hypervisor type. Reason being, currently in cloudstack Hyper-V only
  works with SMB as a secondary storage. You can have a zone where a
  mixed set of other hypervisors are present, for example, xenserver,
  kvm etc, but for hyper-v you'll have to create a separate zone in
  which hyper-v is the only hypervisor. For a hyper-v zone; you'll have
  to register a secondary storage of type SMB.
 
  Regards,
  Devdeep
 
   -Original Message-
   From: Tejas Gadaria [mailto:refond.g...@gmail.com]
   Sent: Tuesday, April 22, 2014 2:22 PM
   To: dev@cloudstack.apache.org; us...@cloudstack.apache.org
   Subject: Multiple hypervisor through single management server
  
   Hi
  
   I am trying manage multiple hypervisor through Management server.
  
   1 ) I have installed Xenserver 6.2 on CS 4.3 with nfs as primary 
  secondary
   storage, it was working fine.
   2) Created new cluster for hyperv and added hyperv host. Immediately
   hyperv Centos template download started.
   3) Added SMB/CIFS as primary storage local to hyperv cluster.
   4) while trying to create instance from ISO, log says ,
  
   2014-04-22 13:57:57,701 DEBUG
   [c.c.n.r.VirtualNetworkApplianceManagerImpl]
   (Job-Executor-4:ctx-d714df36 ctx-a482309d) Lock is released for
   network
  id
   206 as a part of router startup in
   Dest[Zone(Id)-Pod(Id)-Cluster(Id)-Host(Id)-Storage(Volume(Id|Type--
   Pool(Id))]
   : Dest[Zone(1)-Pod(1)-Cluster(2)-Host(4)-Storage(Volume(9|ROOT--
   Pool(2))]
   2014-04-22 13:57:57,701 ERROR [c.c.v.VirtualMachineManagerImpl]
   (Job-Executor-4:ctx-d714df36 ctx-a482309d) Failed to start instance
   VM[User|CentOS65-x64-Hyperv]
   com.cloud.utils.exception.CloudRuntimeException: Template 9 has not
   been completely downloaded to zone 1
   at
  
 com.cloud.template.TemplateManagerImpl.getTemplateSize(TemplateMan
   agerImpl.java:1745)
   at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  
  
   Don't understand how to upload hyperv systemvm template to
 secondary
   storage where already one systemvm template for xen is available.
  
   Regards,
   Tejas
 


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

2014-04-21 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19868/#review40871
---

Ship it!


Ship It!

- Devdeep Singh


On April 15, 2014, 4:45 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/19868/
 ---
 
 (Updated April 15, 2014, 4:45 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6287 and CLOUDSTACK-6315
 https://issues.apache.org/jira/browse/CLOUDSTACK-6287
 https://issues.apache.org/jira/browse/CLOUDSTACK-6315
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 CLOUDSTACK-6287, CLOUDSTACK-6315 fixed password visible in plain text during 
 some operation in hyperv
 
 
 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
 




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

2014-04-21 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20365/#review40873
---

Ship it!


Ship It!

- Devdeep Singh


On April 15, 2014, 8:44 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/20365/
 ---
 
 (Updated April 15, 2014, 8:44 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6411
 https://issues.apache.org/jira/browse/CLOUDSTACK-6411
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
  some files are not loading in visual studio due to license header at top 
 making them unparsable. removed the license header and put them in exclude 
 list of rat plugin
 
 
 Diffs
 -
 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.Config 
 5181cb8 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.settings
  fb5b962 
   pom.xml 6837717 
 
 Diff: https://reviews.apache.org/r/20365/diff/
 
 
 Testing
 ---
 
 verified with rat plugin that the license check is passed successfully
 
 
 Thanks,
 
 Anshul Gangwar
 




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

2014-04-15 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20311/#review40370
---

Ship it!


Ship It!

- Devdeep Singh


On April 14, 2014, 8:28 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/20311/
 ---
 
 (Updated April 14, 2014, 8:28 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6398
 https://issues.apache.org/jira/browse/CLOUDSTACK-6398
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Change auto shutdown and startup actions to shutdown and nothing so that they 
 are consistent with Cloudstack auto shutdown and startup behavior
 
 
 Diffs
 -
 
   
 plugins/hypervisors/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
 




Re: 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-15 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20313/#review40371
---

Ship it!


Ship It!

- Devdeep Singh


On April 14, 2014, 8:41 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/20313/
 ---
 
 (Updated April 14, 2014, 8:41 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6400
 https://issues.apache.org/jira/browse/CLOUDSTACK-6400
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 CLOUDSTACK-6400: Occasionally VM is not deleted from back-end when it is 
 stopped from Hyper-V manager and then destroyed-expunged from CS. With this 
 patch hyper-v agent will delete the stopped VMs on hyper-v host so that 
 hyper-v host is in consistent state with cloudstack. This patch will also 
 make sure that VMs which are in process of creation they are returned as in 
 PowerOn state so that they are not accidentally deleted. Notes are added to 
 vm to identify whether it is created by cloudstack or not
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  4f4c14e 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
  8c682ad 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 79fd0bc 
 
 Diff: https://reviews.apache.org/r/20313/diff/
 
 
 Testing
 ---
 
 verified
 1. if vm is stopped from hyper-v manager then it should be automatically 
 deleted
 2. vms which is in process of creation is not getting accidentally deleted
 3. proper notes are added to vm created by cloudstack
 
 
 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-15 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20315/#review40372
---

Ship it!


Ship It!

- Devdeep Singh


On April 15, 2014, 5:12 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/20315/
 ---
 
 (Updated April 15, 2014, 5:12 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6401
 https://issues.apache.org/jira/browse/CLOUDSTACK-6401
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 fixed correct host state in cloudstack if the hyper-v agent is down. Added 
 heartbeat file to monitor whether host is actually down or not
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResource.csproj
  ec44568 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  4f4c14e 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 
 5ddcbef 
   
 plugins/hypervisors/hyperv/DotNet/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 19906: CLOUDSTACK-6289: fixed storage migration failing in case of hyper-v if there are multiple disks attached to VM

2014-04-08 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19906/#review39773
---

Ship it!


Ship It!

- Devdeep Singh


On April 2, 2014, 8:35 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/19906/
 ---
 
 (Updated April 2, 2014, 8:35 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6289
 https://issues.apache.org/jira/browse/CLOUDSTACK-6289
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 fixed storage migration failing in case of hyper-v if there are multiple 
 disks attached to VM. 
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/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
 




RE: Password visible in plan text CS 4.3

2014-04-01 Thread Devdeep Singh
Hi Tejas,

Can you file a bug for these? We clean up a string before logging it; but it 
looks like we missed it here.

Regards,
Devdeep

 -Original Message-
 From: Tejas Gadaria [mailto:refond.g...@gmail.com]
 Sent: Friday, March 28, 2014 6:01 PM
 To: dev@cloudstack.apache.org
 Subject: Password visible in plan text CS 4.3
 
 Hi,
 
 While doing volume migration from one storage to another, password was
 visible in plan text.
 environment CS 4.3,
 Management server: CentOS 6.3 x64
 Hypervisor: Hyperv
 
 2014-03-28 17:53:39,059 DEBUG [c.c.h.h.r.HypervDirectConnectResource]
 (DirectAgent-216:ctx-89b9eb84) POST response is
 [{org.apache.cloudstack.storage.command.CopyCmdAnswer:{result:true
 ,details:null,newData:{org.apache.cloudstack.storage.to.VolumeObject
 TO:{dataStore:{org.apache.cloudstack.storage.to.PrimaryDataStoreTO:{
 uuid:7890d244-e307-320e-ac42-
 f90e925b32b8,id:5,poolType:SMB,host:10.129.150.24,path:/vol
 _cifs?user=administratordomain=
 nw.com,port:445,url:SMB://
 10.129.150.24//vol_cifs?user=administratordomain=nw.com/?ROLE=Primar
 ySTOREUUID=7890d244-e307-320e-ac42-f90e925b32b8
 }},format:VHD,name:ROOT-
 7,path:\\10.129.150.24\vol_cifs\ROOT-7.vhd,uuid:ca572e34-ffa6-
 4cce-bb24-
 44c989b4156e,size:10737418240,primaryDataStore:{host:10.129.150.2
 4,uri:cifs://
 10.129.150.24/vol_cifs?user=administratordomain=nw.com
 ,_role:null,Path:\\10.129.150.24/vol_cifs,UncPath:\\
 10.129.150.24/vol_cifs,User:administrator,*Password:C1sco123*
 ,Domain:nw.com
 ,isLocal:false},nfsDataStore:null,FullFileName:\\10.129.150.24\vol_cif
 s\ROOT-7.vhd}},contextMap:{}}}]
 2014-03-28 17:53:39,060 DEBUG [c.c.h.h.r.HypervDirectConnectResource]
 (DirectAgent-216:ctx-89b9eb84) executeRequest received response
 [Lcom.cloud.agent.api.Answer;@31c91ca
 
 Regards,
 Tejas


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

2014-04-01 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19713/#review39139
---

Ship it!


Ship It!

- Devdeep Singh


On March 27, 2014, 6:13 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/19713/
 ---
 
 (Updated March 27, 2014, 6:13 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6288
 https://issues.apache.org/jira/browse/CLOUDSTACK-6288
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Currently Hyper-V supports vhd Image Format for templates and volumes. This 
 patch will change default ImageFormat to vhdx for hyper-v and allow 
 registration of vhdx format templates.
 
 this patch conatins two patches revision 1 contains management server side 
 code  and revision 2 conatins agent side code
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  40609e4 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 e51bd89 
 
 Diff: https://reviews.apache.org/r/19713/diff/
 
 
 Testing
 ---
 
 peform following tests
 
 1. registration of vhdx templates
 2. attach and detach disk
 3. volume migration
 4. deploy machine with registered template
 
 
 Thanks,
 
 Anshul Gangwar
 




RE: Password visible in plan text CS 4.3

2014-04-01 Thread Devdeep Singh
Hi Tejas,

Thanks. I'll into both the issues.

Regards,
Devdeep

 -Original Message-
 From: Tejas Gadaria [mailto:refond.g...@gmail.com]
 Sent: Tuesday, April 1, 2014 2:38 PM
 To: dev@cloudstack.apache.org
 Subject: Re: Password visible in plan text CS 4.3
 
 Hi Devdeep,
 
 Thanks for reply,
 
 I don't have privilege to assign the bugs, So I have just raised the issue.
 
 https://issues.apache.org/jira/browse/CLOUDSTACK-6315
 
 Also I have raised a bug on 27th March which is not assigned to anybody yet.
 if possible please take into the consideration.
 
 https://issues.apache.org/jira/browse/CLOUDSTACK-6287
 
 Regards,
 Tejas
 
 
 On Tue, Apr 1, 2014 at 12:03 PM, Devdeep Singh
 devdeep.si...@citrix.comwrote:
 
  Hi Tejas,
 
  Can you file a bug for these? We clean up a string before logging it;
  but it looks like we missed it here.
 
  Regards,
  Devdeep
 
   -Original Message-
   From: Tejas Gadaria [mailto:refond.g...@gmail.com]
   Sent: Friday, March 28, 2014 6:01 PM
   To: dev@cloudstack.apache.org
   Subject: Password visible in plan text CS 4.3
  
   Hi,
  
   While doing volume migration from one storage to another, password
   was visible in plan text.
   environment CS 4.3,
   Management server: CentOS 6.3 x64
   Hypervisor: Hyperv
  
   2014-03-28 17:53:39,059 DEBUG
   [c.c.h.h.r.HypervDirectConnectResource]
   (DirectAgent-216:ctx-89b9eb84) POST response is
  
 [{org.apache.cloudstack.storage.command.CopyCmdAnswer:{result:tr
   ue
   ,details:null,newData:{org.apache.cloudstack.storage.to.VolumeO
   bject
   TO:{dataStore:{org.apache.cloudstack.storage.to.PrimaryDataStore
   TO:{
   uuid:7890d244-e307-320e-ac42-
   f90e925b32b8,id:5,poolType:SMB,host:10.129.150.24,path:
   /vol
   _cifs?user=administratordomain=
   nw.com,port:445,url:SMB://
  
 10.129.150.24//vol_cifs?user=administratordomain=nw.com/?ROLE=Prima
   r
   ySTOREUUID=7890d244-e307-320e-ac42-f90e925b32b8
   }},format:VHD,name:ROOT-
   7,path:\\10.129.150.24\vol_cifs\ROOT-7.vhd,uuid:ca572e34-ffa
   6-
   4cce-bb24-
   44c989b4156e,size:10737418240,primaryDataStore:{host:10.129.
   150.2
   4,uri:cifs://
   10.129.150.24/vol_cifs?user=administratordomain=nw.com
   ,_role:null,Path:\\10.129.150.24/vol_cifs,UncPath:\\
   10.129.150.24/vol_cifs,User:administrator,*Password:C1sco123
   *
   ,Domain:nw.com
  
  ,isLocal:false},nfsDataStore:null,FullFileName:\\10.129.150.24
  \vol_cif
   s\ROOT-7.vhd}},contextMap:{}}}]
   2014-03-28 17:53:39,060 DEBUG
   [c.c.h.h.r.HypervDirectConnectResource]
   (DirectAgent-216:ctx-89b9eb84) executeRequest received response
   [Lcom.cloud.agent.api.Answer;@31c91ca
  
   Regards,
   Tejas
 


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

2014-03-27 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19521/#review38719
---

Ship it!


Ship It!

- Devdeep Singh


On March 21, 2014, 6:47 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/19521/
 ---
 
 (Updated March 21, 2014, 6:47 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Rajesh Battala, and Sateesh 
 Chodapuneedi.
 
 
 Bugs: CLOUDSTACK-6267
 https://issues.apache.org/jira/browse/CLOUDSTACK-6267
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Unblocking SMB as Zone wide primary storage for hyperv
 
 
 Diffs
 -
 
   server/src/com/cloud/storage/StorageManagerImpl.java 3b29b33 
   ui/scripts/system.js 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
 




Re: Review Request 17531: CLOUDSTACK-5440 - Added new guest OS types.

2014-03-24 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17531/#review38285
---

Ship it!


Pushed to 4.4 and master after minor updates to the patch.
1. Windows Server 2012 R2 wasn't needed as it was already added in 421to430 
script.
2. Split CentoOS 6.5 into 32 and 64 bit types.

- Devdeep Singh


On March 20, 2014, 11:50 a.m., Alex Hitchins wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17531/
 ---
 
 (Updated March 20, 2014, 11:50 a.m.)
 
 
 Review request for cloudstack and Devdeep Singh.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Added new guest OS types in upgrade script.
 
 
 Diffs
 -
 
   setup/db/db/schema-430to440.sql acc7e21 
 
 Diff: https://reviews.apache.org/r/17531/diff/
 
 
 Testing
 ---
 
 Ran new SQL, no errors, checked table, contains correct data.
 
 
 Thanks,
 
 Alex Hitchins
 




RE: [Merge] Storage motion for hyper-v

2014-03-14 Thread Devdeep Singh
I'll merge these changes later today.

Regards,
Devdeep

 -Original Message-
 From: Devdeep Singh [mailto:devdeep.si...@citrix.com]
 Sent: Wednesday, March 12, 2014 7:32 AM
 To: dev@cloudstack.apache.org
 Subject: [Merge] Storage motion for hyper-v
 
 Hi,
 
 I would like to merge storage motion for hyper-v targeted for 4.4 to master.
 Feature has been proposed on the dev list [1]. The support has been
 developed in branch [2]. Branch has been rebased to master.  Jira ticket for
 the bug is [3]. The changes are rat clean. I am running firebugs to make sure
 there are no failures because of these changes. Existing storage migration
 integration tests [4] can be run against these changes for functional testing.
 
 [1]
 http://markmail.org/message/xjdiylhe3a4fvflm?q=list:org%2Eapache%2Einc
 ubator%2Ecloudstack-%2A+HyperV+supporting+features+in+4%2E4
 [2] https://git-wip-
 us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=shortlog;h=ref
 s/heads/hyperv-storagemotion
 [3] https://issues.apache.org/jira/browse/CLOUDSTACK-6143
 [4] test/integration/component/test_storage_motion.py
 
 Regards,
 Devdeep


[Merge] Storage motion for hyper-v

2014-03-11 Thread Devdeep Singh
Hi,

I would like to merge storage motion for hyper-v targeted for 4.4 to master. 
Feature has been proposed on the dev list [1]. The support has been developed 
in branch [2]. Branch has been rebased to master.  Jira ticket for the bug is 
[3]. The changes are rat clean. I am running firebugs to make sure there are no 
failures because of these changes. Existing storage migration integration tests 
[4] can be run against these changes for functional testing.

[1] 
http://markmail.org/message/xjdiylhe3a4fvflm?q=list:org%2Eapache%2Eincubator%2Ecloudstack-%2A+HyperV+supporting+features+in+4%2E4
[2] 
https://git-wip-us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=shortlog;h=refs/heads/hyperv-storagemotion
[3] https://issues.apache.org/jira/browse/CLOUDSTACK-6143
[4] test/integration/component/test_storage_motion.py

Regards,
Devdeep


Re: Review Request 17889: CLOUDSTACK-4762: Enabling vGPU support for XenServer.

2014-03-10 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17889/#review36759
---

Ship it!


Ship It!

- Devdeep Singh


On Feb. 27, 2014, 12:35 p.m., Sanjay Tripathi wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17889/
 ---
 
 (Updated Feb. 27, 2014, 12:35 p.m.)
 
 
 Review request for cloudstack, Alex Huang, Devdeep Singh, and Koushik Das.
 
 
 Bugs: CLOUDSTACK-4760 and CLOUDSTACK-4762
 https://issues.apache.org/jira/browse/CLOUDSTACK-4760
 https://issues.apache.org/jira/browse/CLOUDSTACK-4762
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 CLOUDSTACK-4760 : Enabling GPU support for XenServer.
 CLOUDSTACK-4762 : Enabling VGPU support for XenServer.
 
 This feature is to enable the GPU-passthrough and vGPU functionality,
 with the help of this feature, admins/users will be able to leverage
 the GPU graphics unit power by deploying a virtul machine with GPU or
 vGPU support or by changing the service offering of an existing VM
 at any later point of time. There GPU/vGPU enabled VMs are able to run
 graphical applications.
 For now, this feature is only supported with XenServer hypervisor but
 can be extended to add the support of other hypervisors.
 
 
 Diffs
 -
 
   api/src/com/cloud/agent/api/to/GPUDeviceTO.java PRE-CREATION 
   api/src/com/cloud/agent/api/to/VirtualMachineTO.java bed3e1d 
   api/src/com/cloud/gpu/GPU.java PRE-CREATION 
   api/src/org/apache/cloudstack/api/ApiConstants.java 7b7f9ca 
   
 api/src/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java
  1d8cbff 
   api/src/org/apache/cloudstack/api/response/GpuResponse.java PRE-CREATION 
   api/src/org/apache/cloudstack/api/response/HostResponse.java e2d8eb5 
   api/src/org/apache/cloudstack/api/response/UserVmResponse.java 84d532b 
   api/src/org/apache/cloudstack/api/response/VgpuResponse.java PRE-CREATION 
   core/src/com/cloud/agent/api/GetGPUStatsAnswer.java PRE-CREATION 
   core/src/com/cloud/agent/api/GetGPUStatsCommand.java PRE-CREATION 
   core/src/com/cloud/agent/api/StartupRoutingCommand.java 626c87f 
   core/src/com/cloud/agent/api/StopCommand.java 6a29aa6 
   engine/components-api/src/com/cloud/resource/ResourceManager.java 95fb385 
   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
 d19fc38 
   
 engine/schema/resources/META-INF/cloudstack/core/spring-engine-schema-core-daos-context.xml
  08efb83 
   engine/schema/src/com/cloud/gpu/HostGpuGroupsVO.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/VGPUTypesVO.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/HostGpuGroupsDao.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/HostGpuGroupsDaoImpl.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/VGPUTypesDao.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/VGPUTypesDaoImpl.java PRE-CREATION 
   engine/schema/src/com/cloud/host/HostVO.java 56c066b 
   engine/schema/src/com/cloud/host/dao/HostDaoImpl.java 08a4366 
   engine/schema/src/com/cloud/service/ServiceOfferingDetailsVO.java e1a1e93 
   
 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
  48ae3ea 
   server/src/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java 
 b77f8ac 
   server/src/com/cloud/api/ApiDBUtils.java a23244b 
   server/src/com/cloud/api/query/dao/HostJoinDaoImpl.java 1b95d9b 
   server/src/com/cloud/api/query/dao/UserVmJoinDaoImpl.java 08478e2 
   server/src/com/cloud/configuration/ConfigurationManagerImpl.java 1c1da1f 
   server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java 3c87b24 
   server/src/com/cloud/hypervisor/HypervisorGuruBase.java 7fb79fa 
   server/src/com/cloud/network/NetworkUsageManagerImpl.java e9b0393 
   server/src/com/cloud/resource/ResourceManagerImpl.java adad85c 
   server/src/com/cloud/server/ManagementServerImpl.java 2a08ddc 
   server/src/com/cloud/server/StatsCollector.java 548587c 
   server/src/com/cloud/storage/VolumeApiServiceImpl.java c95d316 
   server/src/com/cloud/vm/UserVmManagerImpl.java 97e3ae7 
   server/test/com/cloud/resource/MockResourceManagerImpl.java 5599e8c 
   server/test/com/cloud/vm/DeploymentPlanningManagerImplTest.java 751a3bd 
   server/test/resources/createNetworkOffering.xml c6228da 
   setup/db/db/schema-430to440.sql 0ded7a9 
   test/integration/smoke/test_deploy_vgpu_enabled_vm.py PRE-CREATION 
   tools/marvin/marvin/integration/lib/base.py 0a7ad94 
   ui/scripts/configuration.js 869b876 
   ui/scripts/instances.js 53c9e98 
 
 Diff: https://reviews.apache.org/r/17889/diff/
 
 
 Testing
 ---
 
 Tests:
 
 Hosts:
 1) Add a XS hosts with GPU cards.
 2) Create a pool of XS hosts with GPU cards.
 3) Create a pool of XS hosts

RE: 4.4 Feature Freeze

2014-03-03 Thread Devdeep Singh
I was looking into adding support for iSCSI (CLOUDSTACK-6109) and HA of guest 
vms (CLOUDSTACK-6144) for hyper-v. I don't think I'll be able to finish it by 
14th.



Regards,

Devdeep


From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers
Sent: Friday, February 28, 2014 7:40 PM
To: dev@cloudstack.apache.org
Subject: Re: 4.4 Feature Freeze

i'm all for being flexible, but i find a lot of the arguments used here 
debatable.

It causes developers to rush their development to meet the deadline. This 
will happen anyway, every time we've extended the deadline we got new features 
coming in at the last minute. Actually i'm under the impression that when we 
move the deadline people will actually try to get more features in instead of 
working on stabilizing existing features.

We can't deliver features on the roadmap. There is validity to this point, 
but on the other hand we already know the entire release schedule way ahead, 
this feature freeze date should not come as a surprise. But as i mentioned in 
an earlier mail, lets have this discussion. Post which features might not make 
it into the release so we can have a discussion if we should slip the release 
date to get this feature in. I think we all now that there are commercial 
parties working with this software to build releases and have customers 
demanding features, but if we don't discuss that on list it's hard for us to 
take it into account.

Feature freeze wasn't called True, i wasn't even aware that this was a 
requirement. We should add this to the procedure here 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Releases so release 
managers know this is expected of them. It should not impact the dates as the 
dates are already fixed by the release schedule (every 4 months)


I'm still -1 on extending the feature freeze. I would rather extend the 
test/stability phase to we have some more time to fix issues before we get into 
the RC spinning.


This is the list of current features targeted for 4.4 according to our Jira. 
Which features would be impacted if we don't move the feature freeze?

ASF JIRA
Project: CloudStack
Type: New Feature
Fix Version: 4.4.0
Resolution: Unresolved
Sorted by: Updated descending
1-15 of 15 as at: 28/Feb/14 15:07
TKey Summary Assignee  Reporter  P  
Status  Resolution   Created  Updated  Due
[New Feature]  CLOUDSTACK-6181
Root resize
UnassignedNux [Major] [Open]  Open  Unresolved  
27/Feb/1427/Feb/14
[New Feature]  CLOUDSTACK-6161
distributed routing and network ACL with OVS plug-in
Murali ReddyMurali Reddy  [Major] [Open]  Open  
Unresolved  24/Feb/1424/Feb/14
[New Feature]  CLOUDSTACK-6092
Storage OverProvisioning as a Per Primary Basis
Saksham Srivastava   Saksham Srivastava[Major] [Open]  Open 
 Unresolved  13/Feb/1420/Feb/14
[New Feature]  CLOUDSTACK-6144
HA for guest VMs running Hyper-V
UnassignedRajesh Battala [Major] [Open]  Open  Unresolved   
   20/Feb/1420/Feb/14
[New Feature]  CLOUDSTACK-6143
Storage Live-Migration support for Hyper-V
UnassignedRajesh Battala [Major] [Open]  Open  Unresolved   
   20/Feb/1420/Feb/14
[New Feature]  CLOUDSTACK-6142
Zone Wide Primary Store in Hyper-V
UnassignedRajesh Battala [Major] [Open]  Open  Unresolved   
   20/Feb/1420/Feb/14
[New Feature]  CLOUDSTACK-6104
PVLAN support for CloudStack deployment over Nexus 1000v in VMware environment
Sateesh Chodapuneedi  Sateesh Chodapuneedi   [Major] 
[Open]  Open  Unresolved  14/Feb/14  15/Feb/14
[New Feature]  CLOUDSTACK-6109
Support of iSCSI as primary store in Hyper-V
Rajesh Battala   Rajesh Battala [Major] [Open]  Open  
Unresolved  14/Feb/1414/Feb/14
[New Feature]  CLOUDSTACK-6106
Support of VPC in HyperV
Rajesh Battala   Rajesh Battala [Major] [Open]  Open  
Unresolved  14/Feb/1414/Feb/14
[New Feature]  CLOUDSTACK-6090
Virtual Router Service Failure Alerting
Harikrishna Patnala   Harikrishna Patnala [Major] [Open]  Open  
Unresolved  13/Feb/1413/Feb/14
[New Feature]  CLOUDSTACK-6052
List VM enhancement to support querying with multiple VM IDs
Koushik Das  Koushik Das   [Major] [Open]  Open  Unresolved 
 07/Feb/1407/Feb/14
[New Feature]  CLOUDSTACK-5569
enhance OVS plug-in to support region level VPC and guest networks that span 
zones
Murali ReddyMurali Reddy  [Major] [Open]  Open  
Unresolved  19/Dec/1319/Dec/13
[New Feature]  CLOUDSTACK-5568
introduce notion of guest network that spans multiple zones
Murali ReddyMurali Reddy  [Major] [Open]  Open  
Unresolved   

Re: Review Request 17889: CLOUDSTACK-4762: Enabling vGPU support for XenServer.

2014-02-28 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17889/#review35769
---



api/src/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java
https://reviews.apache.org/r/17889/#comment66505

Why did you change the function? Was it that only the first key value was 
being passed?



core/src/com/cloud/agent/api/GetGPUStatsAnswer.java
https://reviews.apache.org/r/17889/#comment66506

The license header is different from other files; namely no space after the 
comment starts. Does it pass RAT?



ui/scripts/configuration.js
https://reviews.apache.org/r/17889/#comment66507

Shouldn't the details index begin with 0? That is, when dedication mode is 
not passed, then the details index should begin with 0.


- Devdeep Singh


On Feb. 27, 2014, 12:35 p.m., Sanjay Tripathi wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17889/
 ---
 
 (Updated Feb. 27, 2014, 12:35 p.m.)
 
 
 Review request for cloudstack, Alex Huang, Devdeep Singh, and Koushik Das.
 
 
 Bugs: CLOUDSTACK-4760 and CLOUDSTACK-4762
 https://issues.apache.org/jira/browse/CLOUDSTACK-4760
 https://issues.apache.org/jira/browse/CLOUDSTACK-4762
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 CLOUDSTACK-4760 : Enabling GPU support for XenServer.
 CLOUDSTACK-4762 : Enabling VGPU support for XenServer.
 
 This feature is to enable the GPU-passthrough and vGPU functionality,
 with the help of this feature, admins/users will be able to leverage
 the GPU graphics unit power by deploying a virtul machine with GPU or
 vGPU support or by changing the service offering of an existing VM
 at any later point of time. There GPU/vGPU enabled VMs are able to run
 graphical applications.
 For now, this feature is only supported with XenServer hypervisor but
 can be extended to add the support of other hypervisors.
 
 
 Diffs
 -
 
   api/src/com/cloud/agent/api/to/GPUDeviceTO.java PRE-CREATION 
   api/src/com/cloud/agent/api/to/VirtualMachineTO.java bed3e1d 
   api/src/com/cloud/gpu/GPU.java PRE-CREATION 
   api/src/org/apache/cloudstack/api/ApiConstants.java 7b7f9ca 
   
 api/src/org/apache/cloudstack/api/command/admin/offering/CreateServiceOfferingCmd.java
  1d8cbff 
   api/src/org/apache/cloudstack/api/response/GpuResponse.java PRE-CREATION 
   api/src/org/apache/cloudstack/api/response/HostResponse.java e2d8eb5 
   api/src/org/apache/cloudstack/api/response/UserVmResponse.java 84d532b 
   api/src/org/apache/cloudstack/api/response/VgpuResponse.java PRE-CREATION 
   core/src/com/cloud/agent/api/GetGPUStatsAnswer.java PRE-CREATION 
   core/src/com/cloud/agent/api/GetGPUStatsCommand.java PRE-CREATION 
   core/src/com/cloud/agent/api/StartupRoutingCommand.java 626c87f 
   core/src/com/cloud/agent/api/StopCommand.java 6a29aa6 
   engine/components-api/src/com/cloud/resource/ResourceManager.java 95fb385 
   engine/orchestration/src/com/cloud/vm/VirtualMachineManagerImpl.java 
 d19fc38 
   
 engine/schema/resources/META-INF/cloudstack/core/spring-engine-schema-core-daos-context.xml
  08efb83 
   engine/schema/src/com/cloud/gpu/HostGpuGroupsVO.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/VGPUTypesVO.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/HostGpuGroupsDao.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/HostGpuGroupsDaoImpl.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/VGPUTypesDao.java PRE-CREATION 
   engine/schema/src/com/cloud/gpu/dao/VGPUTypesDaoImpl.java PRE-CREATION 
   engine/schema/src/com/cloud/host/HostVO.java 56c066b 
   engine/schema/src/com/cloud/host/dao/HostDaoImpl.java 08a4366 
   engine/schema/src/com/cloud/service/ServiceOfferingDetailsVO.java e1a1e93 
   
 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
  48ae3ea 
   server/src/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java 
 b77f8ac 
   server/src/com/cloud/api/ApiDBUtils.java a23244b 
   server/src/com/cloud/api/query/dao/HostJoinDaoImpl.java 1b95d9b 
   server/src/com/cloud/api/query/dao/UserVmJoinDaoImpl.java 08478e2 
   server/src/com/cloud/configuration/ConfigurationManagerImpl.java 1c1da1f 
   server/src/com/cloud/deploy/DeploymentPlanningManagerImpl.java 3c87b24 
   server/src/com/cloud/hypervisor/HypervisorGuruBase.java 7fb79fa 
   server/src/com/cloud/network/NetworkUsageManagerImpl.java e9b0393 
   server/src/com/cloud/resource/ResourceManagerImpl.java adad85c 
   server/src/com/cloud/server/ManagementServerImpl.java 2a08ddc 
   server/src/com/cloud/server/StatsCollector.java 548587c 
   server/src/com/cloud/storage/VolumeApiServiceImpl.java c95d316 
   server/src/com/cloud/vm/UserVmManagerImpl.java 97e3ae7 
   server

RE: hyper-v broken again?

2014-02-27 Thread Devdeep Singh
Hi Paul,

Can you remove and add the image and primary store? It seems the credentials 
stored for the image store are not encrypted and the management server is 
failing to decrypt it. Bug cloudstack-6030 addressed the problem of credentials 
not encrypted. Maybe, your initial setup was done before this issue was fixed 
and that is why management server fails to decrypt it.

Regards,
Devdeep

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Thursday, February 27, 2014 5:49 PM
To: Rajesh Battala; dev@cloudstack.apache.org
Subject: RE: hyper-v broken again?

Mgmt. server code from rc6, agent built today.



Regards

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: CloudyAngus
paul.an...@shapeblue.commailto:paul.an...@shapeblue.com

From: Rajesh Battala [mailto:rajesh.batt...@citrix.com]
Sent: 27 February 2014 11:59
To: Paul Angus; dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org
Subject: RE: hyper-v broken again?

Hi Paul,
Is your mgmt server and hyper-v agent are latest build?
Smb credentials are encrypted and stored.


Thanks
Rajesh Battala

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Thursday, February 27, 2014 5:10 PM
To: dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org; Rajesh Battala
Subject: hyper-v broken again?

I'm using what was rc6

And getting the following error - (the host is in an alert state)

Error while decrypting: hyperv13!

Hyperv13! Is the password for hyperv service account.

The agent is from Jenkins today and I have refreshed the certificate on the 
host.

2014-02-27 11:33:46,472 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-2:ctx-9c59c959) POST request to 
https://10.0.0.24:8250/api/HypervResource/com.cloud.agent.api.CheckNetworkCommand
 with contents 
{networkInfoList:[{physicalNetworkId:200,privateNetworkName:Mgmt,guestNetworkName:GuestPub},{physicalNetworkId:201,publicNetworkName:GuestPub}],contextMap:{},wait:0}
2014-02-27 11:33:46,478 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-2:ctx-9c59c959) Sending cmd to 
https://10.0.0.24:8250/api/HypervResource/com.cloud.agent.api.CheckNetworkCommand
 cmd 
data:{networkInfoList:[{physicalNetworkId:200,privateNetworkName:Mgmt,guestNetworkName:GuestPub},{physicalNetworkId:201,publicNetworkName:GuestPub}],contextMap:{},wait:0}
2014-02-27 11:33:46,552 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-2:ctx-9c59c959) POST response is 
[{com.cloud.agent.api.CheckNetworkAnswer:{result:true,details:null,contextMap:{}}}]
2014-02-27 11:33:46,553 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-2:ctx-9c59c959) executeRequest received response 
[Lcom.cloud.agent.api.Answer;@3b020b13
2014-02-27 11:33:46,553 DEBUG [c.c.a.m.DirectAgentAttache] 
(DirectAgent-2:ctx-9c59c959) Seq 2-1800798211: Response Received:
2014-02-27 11:33:46,553 DEBUG [c.c.a.t.Request] (DirectAgent-2:ctx-9c59c959) 
Seq 2-1800798211: Processing:  { Ans: , MgmtId: 345049353548, via: 2, Ver: v1, 
Flags: 110, 
[{com.cloud.agent.api.CheckNetworkAnswer:{_reconnect:false,result:true,wait:0}}]
 }
2014-02-27 11:33:46,553 DEBUG [c.c.a.t.Request] (AgentTaskPool-8:ctx-c0cabb01) 
Seq 2-1800798211: Received:  { Ans: , MgmtId: 345049353548, via: 2, Ver: v1, 
Flags: 110, { CheckNetworkAnswer } }
2014-02-27 11:33:46,553 DEBUG [o.a.c.e.o.NetworkOrchestrator] 
(AgentTaskPool-8:ctx-c0cabb01) Network setup is correct on Agent
2014-02-27 11:33:46,553 DEBUG [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-8:ctx-c0cabb01) Sending Connect to listener: 
DeploymentPlanningManagerImpl
2014-02-27 11:33:46,555 DEBUG [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-8:ctx-c0cabb01) Sending Connect to listener: 
ClusteredVirtualMachineManagerImpl
2014-02-27 11:33:46,555 DEBUG [c.c.v.VirtualMachineManagerImpl] 
(AgentTaskPool-8:ctx-c0cabb01) Received startup command from hypervisor host. 
host id: 2
2014-02-27 11:33:46,556 DEBUG [c.c.v.VirtualMachineManagerImpl] 
(AgentTaskPool-8:ctx-c0cabb01) Found 0 VMs for host 2
2014-02-27 11:33:46,556 DEBUG [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-8:ctx-c0cabb01) Sending Connect to listener: DownloadListener
2014-02-27 11:33:46,557 DEBUG [c.c.a.m.AgentAttache] 
(DirectAgent-2:ctx-9c59c959) Seq 2-1800798211: No more commands found
2014-02-27 11:33:46,569 INFO  [o.a.c.s.i.TemplateServiceImpl] 
(AgentTaskPool-8:ctx-c0cabb01) Downloading builtin template centos64-x64 to 
data center: 1
2014-02-27 11:33:46,571 DEBUG [c.c.u.c.DBEncryptionUtil] 
(AgentTaskPool-8:ctx-c0cabb01) Error while decrypting: hyperv13!
2014-02-27 11:33:46,571 ERROR [c.c.a.m.AgentManagerImpl] 
(AgentTaskPool-8:ctx-c0cabb01) Monitor DownloadListener says there is an error 
in the connect process for 2 due to null
org.jasypt.exceptions.EncryptionOperationNotPossibleException
at 
org.jasypt.encryption.pbe.StandardPBEByteEncryptor.decrypt(StandardPBEByteEncryptor.java:918)
at 
org.jasypt.encryption.pbe.StandardPBEStringEncryptor.decrypt(StandardPBEStringEncryptor.java:725)
at 

RE: API Calls with key value pairs (Details)

2014-02-12 Thread Devdeep Singh
The user name and password were never there in the key value pair for creating 
storage pool. This was done for adding smb as a primary storage and was done in 
4.3 (adding smb as a pool type wasn't available in earlier release).

Initially user details were passed in the url query string (for smb). However, 
it would mean adding a password with special characters could fail if the 
password wasn't encoded and then the entire url encoded again. So I updated it 
to pass smb user information in details. This is required only if a storage 
pool being added is of type smb.

Regards,
Devdeep

-Original Message-
From: sebgoa [mailto:run...@gmail.com] 
Sent: Wednesday, February 12, 2014 5:56 PM
To: Paul Angus
Cc: dev@cloudstack.apache.org
Subject: Re: API Calls with key value pairs (Details)


On Feb 12, 2014, at 11:25 AM, Paul Angus paul.an...@shapeblue.com wrote:

 Hi Devs,
  
 I've noticed that a few newer API commands are requiring a slightly different 
 format, which is to require a details parameter.
  
 # cloudmonkey create storagepool scope=cluster clusterid=$CLUSTERID name=Pri1 
 podid=$POD1ID zoneid=$ZONEID provider=smb 
 url='cifs://10.0.1.27/hypervPri/stor/Primary?user=hypervpassword=hyperv13!domain=angusnet.local'
  
 : Failed to add data store: Missing cifs user and password details. Add them 
 as details parameter.
  

Can you set the user and password as regular key value pair, or does it need to 
be in a detail dict ? 
How did it work with 4.2.x ?
Can you identify a commit that changed the behavior ?

 And from my S3 testing yesterday:
  
 addImageStoreresponse=jsonsessionkey=sctphRL69EJ5RZdAWIrzfZH7jIo%3D
 name=S3Storageprovider=S3details%5B0%5D.key=accesskeydetails%5B0%5D
 .value=AKIAJBYAOXAE7JJYBKEQdetails%5B1%5D.key=secretkeydetails%5B1%5
 D.value=3F76mmMubsZohZAnPutLkxK9adA1XQNBQAMUdv%2FOdetails%5B2%5D.key=
 bucketdetails%5B2%5D.value=cloudstack-sec-stordetails%5B3%5D.key=use
 httpsdetails%5B3%5D.value=truedetails%5B4%5D.key=endpointdetails%5B
 4%5D.value=s3.amazonaws.com_=1392125369653
 
 API Docs:
 addImageStoreExample: 
 details[0].key=accesskeydetails[0].value=s389ddssaadetails[1].key=se
 cretkeydetails[1].value=8dshfsss
  
 The format seems very cumbersome compared to accesskey=value 
 secretkey=value - has this been a community decision to move in this 
 direction that I've missed?
  

First time that I see this, and yes it seems cumbersome and against prior 
design.

-sebastien


 Regards
  
 Paul Angus
 Senior Consultant / Cloud Architect
  
 image001.png
  
 S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
 paul.an...@shapeblue.com | www.shapeblue.com | Twitter:@shapeblue 
 ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS
  
 Need Enterprise Grade Support for Apache CloudStack?
 Our CloudStack Infrastructure Support offers the best 24/7 SLA for CloudStack 
 Environments.
 
 Apache CloudStack Bootcamp training courses
 
 **NEW!** CloudStack 4.2.1 training
 18th-19th February 2014, Brazil. Classroom 17th-23rd March 2014, 
 Region A. Instructor led, On-line 24th-28th March 2014, Region B. 
 Instructor led, On-line 16th-20th June 2014, Region A. Instructor led, 
 On-line 23rd-27th June 2014, Region B. Instructor led, On-line
 
 This email and any attachments to it may be confidential and are intended 
 solely for the use of the individual to whom it is addressed. Any views or 
 opinions expressed are solely those of the author and do not necessarily 
 represent those of Shape Blue Ltd or related companies. If you are not the 
 intended recipient of this email, you must neither take any action based upon 
 its contents, nor copy or show it to anyone. Please contact the sender if you 
 believe you have received this email in error. Shape Blue Ltd is a company 
 incorporated in England  Wales. ShapeBlue Services India LLP is a company 
 incorporated in India and is operated under license from Shape Blue Ltd. 
 Shape Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
 operated under license from Shape Blue Ltd. ShapeBlue is a registered 
 trademark.



RE: API Calls with key value pairs (Details)

2014-02-12 Thread Devdeep Singh

 -Original Message-
 From: Chip Childers [mailto:chipchild...@apache.org]
 Sent: Wednesday, February 12, 2014 7:43 PM
 To: dev@cloudstack.apache.org
 Subject: Re: API Calls with key value pairs (Details)
 
 On Wed, Feb 12, 2014 at 8:24 AM, Devdeep Singh
 devdeep.si...@citrix.com wrote:
  The user name and password were never there in the key value pair for
 creating storage pool. This was done for adding smb as a primary storage and
 was done in 4.3 (adding smb as a pool type wasn't available in earlier 
 release).
 
  Initially user details were passed in the url query string (for smb). 
  However,
 it would mean adding a password with special characters could fail if the
 password wasn't encoded and then the entire url encoded again. So I
 updated it to pass smb user information in details. This is required only if a
 storage pool being added is of type smb.
 
  Regards,
  Devdeep
 
 Except it breaks backward compat, right?

No, it will not break backward compat with 4.2.x (or earlier). When smb was 
added as pool type (in 4.3), the user information was initially in the query 
string of the url parameter. I made a change to pass it in details. I am not 
aware of any storage pool type which takes user information in the url 
parameter. If there is any, the change only affects smb pool type (which has 
been added in 4.3).

Regards,
Devdeep


Re: Hyper-v via the UI drops the password

2014-02-12 Thread Devdeep Singh
Hi Paul,

Can you check if the hyperv agent is latest? The agent seems to be out of sync 
on the host.

Regards,
Devdeep


RE: Cloudstack 4.3 with hyper-v

2014-02-11 Thread Devdeep Singh
If you are using a development environment you may see multiple mount points 
getting created. I'll look into addressing this. You shouldn't see this if you 
have installed cloudstack from rpm. The management server will clean up the 
mount points.

Regards,
Devdeep

-Original Message-
From: Tejas Gadaria [mailto:refond.g...@gmail.com] 
Sent: Tuesday, February 11, 2014 4:51 PM
To: dev@cloudstack.apache.org
Subject: Re: Cloudstack 4.3 with hyper-v

Hi Rajesh,

Thanks for replay,

but multiple mount point are necessary?
because all three mount point contains files and sub-directory with same name 
and size.

Regards,
Tejas


On Tue, Feb 11, 2014 at 4:00 PM, Rajesh Battala
rajesh.batt...@citrix.comwrote:

 Mount point you are seeing is used to copy the systemvm.iso to 
 secondary storage.

 -Original Message-
 From: Tejas Gadaria [mailto:refond.g...@gmail.com]
 Sent: Tuesday, February 11, 2014 3:11 PM
 To: dev@cloudstack.apache.org
 Subject: Cloudstack 4.3 with hyper-v

 Hi,

 Secondary storage mount point entry recorded multiple time on 
 management server which is I think not necessary.
 I have created one instance, It's working fine.


 [root@hyperv ~]# df -hT
 FilesystemTypeSize  Used Avail Use% Mounted on
 /dev/mapper/VolGroup-lv_root
   ext4 36G  4.1G   30G  13% /
 tmpfstmpfs1.9G 0  1.9G   0% /dev/shm
 /dev/sda1 ext4485M   37M  423M   8% /boot
 //10.129.151.55/Secondary
   cifs250G   24G  227G  10%
 /var/cloudstack/mnt/VM/345051498628.49893dd0
 //10.129.151.55/Secondary
   cifs250G   24G  227G  10%
 /var/cloudstack/mnt/VM/345051498628.4f2cf0e9
 //10.129.151.55/Secondary
   cifs250G   24G  227G  10%
 /var/cloudstack/mnt/VM/345051498628.7eb00501


 Regards,
 Tejas



Re: Review Request 17531: CLOUDSTACK-5440 - Added new guest OS types.

2014-02-10 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17531/#review34168
---


Can you make these changes in master branch on schema-430to440.sql file? Since 
a RC has already been cut for 4.3, we can put these changes in 4.4 release so 
that on upgrade from 4.3; these changes will come in.


setup/db/db/schema-421to430.sql
https://reviews.apache.org/r/17531/#comment64187

Can you fix the id values to 225, 226 and 227? The current values go till 
224. We can begin from there. Reason being the guest_os_hypervisor table refers 
the field on this table by 'id'. It would be good to have fixed values for the 
new os types.



setup/db/db/schema-421to430.sql
https://reviews.apache.org/r/17531/#comment64186

Could you make separate entries for Windows 8.1 (32 bit). and Windows 8.1 
(64 bit). Similarly for Windows Server 2012 R2. Similar pattern has been 
followed for all windows sku.


- Devdeep Singh


On Jan. 30, 2014, 11:18 a.m., Alexander Hitchins wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17531/
 ---
 
 (Updated Jan. 30, 2014, 11:18 a.m.)
 
 
 Review request for cloudstack.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Added new guest OS types in upgrade script.
 
 
 Diffs
 -
 
   setup/db/db/schema-421to430.sql afad06c 
 
 Diff: https://reviews.apache.org/r/17531/diff/
 
 
 Testing
 ---
 
 Ran new SQL, no errors, checked table, contains correct data.
 
 
 Thanks,
 
 Alexander Hitchins
 




Re: Review Request 17842: fixed vmsync for hyperv

2014-02-10 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17842/#review34173
---

Ship it!


Committed to 4.3-forward and master in commit ids 
eef65ac776de171d83f9f532ac2dda6902117178 and 
b86d45b0036482cfc9c9fc55f62954a1b9599188 respectively.

- Devdeep Singh


On Feb. 10, 2014, 11:07 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17842/
 ---
 
 (Updated Feb. 10, 2014, 11:07 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6054
 https://issues.apache.org/jira/browse/CLOUDSTACK-6054
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 fixed vmsync for hyperv. This fix adds an api call which return the 
 hostvmstatereport and made appropriate changes to 
 HypervDirectConnectResource.java 
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/CloudStackTypes.cs
  9637994 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  a132606 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
  5f814c5 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 f5a04dc 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
  a4dca6a 
 
 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
 




Re: Review Request 17842: fixed vmsync for hyperv

2014-02-09 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17842/#review34046
---



plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
https://reviews.apache.org/r/17842/#comment64013

There is already a WmiCallsV2::EnabledState class defined. Please use that.



plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
https://reviews.apache.org/r/17842/#comment64015

Hyperv agent should map the hyper-v state to cloudstack state and return an 
appropriate state in the answer. Lookat 
WmiCallsV2::EnabledState.ToCloudStackState


- Devdeep Singh


On Feb. 7, 2014, 11:22 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17842/
 ---
 
 (Updated Feb. 7, 2014, 11:22 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6054
 https://issues.apache.org/jira/browse/CLOUDSTACK-6054
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 fixed vmsync for hyperv. This fix adds an api call which return the 
 hostvmstatereport and made appropriate changes to 
 HypervDirectConnectResource.java 
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/CloudStackTypes.cs
  9637994 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  a132606 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
  5f814c5 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 f5a04dc 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
  a4dca6a 
 
 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
 




Re: Review Request 17842: fixed vmsync for hyperv

2014-02-09 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17842/#review34047
---



plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs
https://reviews.apache.org/r/17842/#comment64016

This will also return the root partition (computer system). Hyper-V agent 
should filter out the root partition entry and then return the updated vmsync 
state. 


- Devdeep Singh


On Feb. 7, 2014, 11:22 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17842/
 ---
 
 (Updated Feb. 7, 2014, 11:22 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6054
 https://issues.apache.org/jira/browse/CLOUDSTACK-6054
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 fixed vmsync for hyperv. This fix adds an api call which return the 
 hostvmstatereport and made appropriate changes to 
 HypervDirectConnectResource.java 
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/CloudStackTypes.cs
  9637994 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  a132606 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
  5f814c5 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 f5a04dc 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
  a4dca6a 
 
 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
 




RE: systemvm-4.3.0.iso

2014-02-07 Thread Devdeep Singh
Hi Paul,

For hyper-v, if older systemvm iso is present with the same name then it'll not 
get updated. You can remove the systemvm directory from secondary storage. Also 
on you hyper-v host, systemvm.iso gets put on 
RootDrive:\Users\Public\Documents\Hyper-V directory. You can delete it. When 
a system vm (ssvm or consolve proxy or router vm) is started, it'll get copied 
over.

Regards,
Devdeep

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Friday, February 7, 2014 4:35 PM
To: dev@cloudstack.apache.org
Subject: systemvm-4.3.0.iso

Have any changes been made to the deployment of the systemvm-4.3.0.iso between 
RC2 and RC3 ??

I can't get a system VMs to fully start in hyperv or xenserver - all issues 
seem to relate to the systemvm-4.3.0.iso - which doesn't seem to be making 
across to primary storage in hyper-v or properly configuring the secondary 
storage VM under xenserver...

Regards

Paul Angus
Senior Consultant / Cloud Architect

[cid:image002.png@01CE1071.C6CC9C10]

S: +44 20 3603 0540tel:+442036030540 | M: +4tel:+44796816158147711418784 | 
T: @CloudyAngus
paul.an...@shapeblue.commailto:paul.an...@shapeblue.com | 
www.shapeblue.comhttp://www.shapeblue.com/ | 
Twitter:@shapebluehttps://twitter.com/
ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS

Need Enterprise Grade Support for Apache CloudStack?
Our CloudStack Infrastructure 
Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers the 
best 24/7 SLA for CloudStack Environments.

Apache CloudStack Bootcamp training courses

**NEW!** CloudStack 4.2.1 traininghttp://shapeblue.com/cloudstack-training/
18th-19th February 2014, Brazil. 
Classroomhttp://shapeblue.com/cloudstack-training/
17th-23rd March 2014, Region A. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/
24th-28th March 2014, Region B. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/
16th-20th June 2014, Region A. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/
23rd-27th June 2014, Region B. Instructor led, 
On-linehttp://shapeblue.com/cloudstack-training/

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England  Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is operated under license from Shape Blue Ltd. Shape 
Blue Brasil Consultoria Ltda is a company incorporated in Brasil and is 
operated under license from Shape Blue Ltd. ShapeBlue is a registered trademark.


RE: systemvm-4.3.0.iso

2014-02-07 Thread Devdeep Singh
 
resource=org.apache.cloudstack.storage.resource.NfsSecondaryStorageResource 
instance=SecStorage sslcopy=true role=templateProcessor mtu=1500 
eth2ip=10.0.106.1 eth2mask=255.255.0.0 gateway=10.0.1.1 eth0ip=169.254.0.63 
eth0mask=255.255.0.0 eth1ip=10.0.106.248 eth1mask=255.255.0.0 
mgmtcidr=10.0.0.0/16 localgw=10.0.1.1 private.network.device=eth1 
eth3ip=10.0.106.249 eth3mask=255.255.0.0 storageip=10.0.106.249 
storagenetmask=255.255.0.0 storagegateway=10.0.1.1 internaldns1=10.0.1.2 
dns1=10.0.1.2,
  rebootOnCrash: false,
  enableHA: false,
  limitCpuUse: false,
  enableDynamicallyScaleVm: false,
  vncPassword: 38e8aeeea9cd60eb,
  params: {
memoryOvercommitRatio: 1.0,
cpuOvercommitRatio: 4.0
  },
  uuid: 6f2b4fce-ad17-46a5-a894-f7c92aa8122b,
  disks: [
{
  data: {
org.apache.cloudstack.storage.to.VolumeObjectTO: {
  uuid: 61ce74dc-6f44-4244-ad35-8c440e14cd0a,
  volumeType: ROOT,
  dataStore: {
org.apache.cloudstack.storage.to.PrimaryDataStoreTO: {
  uuid: 956abad9-77b4-33cc-9b6f-05a29b68bc3d,
  id: 3,
  poolType: NetworkFilesystem,
  host: 10.0.1.27,
  path: 
/hypervPri/stor/Primary?user=hypervdomain=angusnet.local,
  port: 445,
  url: 
NetworkFilesystem://10.0.1.27//hypervPri/stor/Primary?user=hypervdomain=angusnet.local/?ROLE=PrimarySTOREUUID=956abad9-77b4-33cc-9b6f-05a29b68bc3d
}
  },
  name: ROOT-27,
  size: 0,
  path: 10.0.1.27\\hypervPri\\stor\\Primary\\ROOT-27.vhd,
  volumeId: 27,
  vmName: s-27-VM,
  accountId: 1,
  format: VHD,
  id: 27,
  deviceId: 0,
  hypervisorType: Hyperv
}
  },
  diskSeq: 0,
  path: 10.0.1.27\\hypervPri\\stor\\Primary\\ROOT-27.vhd,
  type: ROOT,
  _details: {
managed: false,
storagePort: 445,
storageHost: 10.0.1.27,
volumeSize: 0
  }
}
  ],
  nics: [
{
  deviceId: 2,
  networkRateMbps: -1,
  defaultNic: true,
  uuid: 4d1e7e7d-4ea8-4caf-b50d-48388e157880,
  ip: 10.0.106.1,
  netmask: 255.255.0.0,
  gateway: 10.0.1.1,
  mac: 06:ad:e4:00:00:0f,
  dns1: 10.0.1.2,
  broadcastType: Native,
  type: Guest,
  broadcastUri: vlan://untagged,
  isSecurityGroupEnabled: false
},
{
  deviceId: 0,
  networkRateMbps: -1,
  defaultNic: false,
  uuid: 03cca24d-d6ff-4852-81a0-9004f30e5b76,
  ip: 169.254.0.63,
  netmask: 255.255.0.0,
  gateway: 169.254.0.1,
  mac: 0e:00:a9:fe:00:3f,
  broadcastType: LinkLocal,
  type: Control,
  isSecurityGroupEnabled: false
},
{
  deviceId: 1,
  networkRateMbps: -1,
  defaultNic: false,
  uuid: 6c5b1433-a390-4e0d-a4f6-37569463e9b4,
  ip: 10.0.106.248,
  netmask: 255.255.0.0,
  gateway: 10.0.1.1,
  mac: 06:08:d8:00:00:09,
  broadcastType: Native,
  type: Management,
  isSecurityGroupEnabled: false
},
{
  deviceId: 3,
  networkRateMbps: -1,
  defaultNic: false,
  uuid: c1bcc2d2-05f6-4d31-aeb9-31bb99f8d20a,
  ip: 10.0.106.249,
  netmask: 255.255.0.0,
  gateway: 10.0.1.1,
  mac: 06:29:7e:00:00:0a,
  broadcastType: Native,
  type: Storage,
  isSecurityGroupEnabled: false
}
  ]
},
contextMap: {}
  }
}

Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.com

-Original Message-
From: Devdeep Singh [mailto:devdeep.si...@citrix.com]
Sent: 07 February 2014 11:18
To: dev@cloudstack.apache.org
Subject: RE: systemvm-4.3.0.iso

Hi Paul,

For hyper-v, if older systemvm iso is present with the same name then it'll not 
get updated. You can remove the systemvm directory from secondary storage. Also 
on you hyper-v host, systemvm.iso gets put on 
RootDrive:\Users\Public\Documents\Hyper-V directory. You can delete it. When 
a system vm (ssvm or consolve proxy or router vm) is started, it'll get copied 
over.

Regards,
Devdeep

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Friday, February 7, 2014 4:35 PM
To: dev@cloudstack.apache.org
Subject: systemvm-4.3.0.iso

Have any changes been made to the deployment of the systemvm-4.3.0.iso between 
RC2 and RC3 ??

I can't get a system VMs to fully start in hyperv or xenserver - all issues 
seem to relate to the systemvm-4.3.0.iso - which doesn't seem to be making 
across to primary storage in hyper-v

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

2014-02-06 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17583/#review33773
---

Ship it!


Ship It!

- Devdeep Singh


On Jan. 31, 2014, 7:51 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17583/
 ---
 
 (Updated Jan. 31, 2014, 7:51 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6001
 https://issues.apache.org/jira/browse/CLOUDSTACK-6001
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 [Hyper-v] can't get console access after vm live migration for 3 minutes.
 
 
 Diffs
 -
 
   
 services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyRdpClient.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
 




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

2014-02-06 Thread Devdeep Singh


 On Feb. 6, 2014, 8:15 a.m., Devdeep Singh wrote:
  Ship It!

Committed to 4.3-forward branch in commit id 
fb87c85b2a313d75af7cd4b790118fea30a2dd1b


- Devdeep


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17583/#review33773
---


On Jan. 31, 2014, 7:51 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17583/
 ---
 
 (Updated Jan. 31, 2014, 7:51 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6001
 https://issues.apache.org/jira/browse/CLOUDSTACK-6001
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 [Hyper-v] can't get console access after vm live migration for 3 minutes.
 
 
 Diffs
 -
 
   
 services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyRdpClient.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
 




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

2014-02-06 Thread Devdeep Singh


 On Feb. 6, 2014, 8:15 a.m., Devdeep Singh wrote:
  Ship It!
 
 Devdeep Singh wrote:
 Committed to 4.3-forward branch in commit id 
 fb87c85b2a313d75af7cd4b790118fea30a2dd1b

Committed to master in commit id 13e0f293edc718a9ee03c56896b65240cdc86d34 


- Devdeep


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17583/#review33773
---


On Jan. 31, 2014, 7:51 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/17583/
 ---
 
 (Updated Jan. 31, 2014, 7:51 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and Rajesh Battala.
 
 
 Bugs: CLOUDSTACK-6001
 https://issues.apache.org/jira/browse/CLOUDSTACK-6001
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 [Hyper-v] can't get console access after vm live migration for 3 minutes.
 
 
 Diffs
 -
 
   
 services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyRdpClient.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
 




RE: Xen StorageMotion Fails

2014-01-30 Thread Devdeep Singh
Hi Francois,

This looks like an issue in cloudstack. Can you file a bug with the details?

Regards,
Devdeep

-Original Message-
From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com] 
Sent: Thursday, January 30, 2014 6:48 PM
To: Srikanteswararao Talluri
Cc: dev@cloudstack.apache.org
Subject: RE: Xen StorageMotion Fails

I used the UI for the vm migration. I can get you the exact api call but isnt 
the call in the pastebin output already..

Another thing, if there is a vm running on the same network on the destination 
host, I dont have the exception but the volume stay in migrate state forever.

Thanks

Francois
On Jan 30, 2014 2:19 AM, Srikanteswararao Talluri  
srikanteswararao.tall...@citrix.com wrote:

 Yes, I am able to migrate a VM on local storage(host) to another local 
 storage(host).

 Can you paste the API call you tried?

 Thanks,
 ~Talluri

 -Original Message-
 From: Francois Gaudreault [mailto:fgaudrea...@cloudops.com]
 Sent: Wednesday, 29 January 2014 6:31 PM
 To: dev@cloudstack.apache.org
 Subject: Xen StorageMotion Fails

 Hi,

 I posted this on the users@ mailing-list yesterday, but giving it more 
 thoughts I think this should have been posted here. We are running 
 4.2.1 with XenServer 6.2SP1. I tried to run a storage motion from a 
 local storage to another, and I am getting this error. Xen returns a 
 VM_REQUIRES_NETWORK when attempting the migration. This error, according to 
 the javadoc means:

 You attempted to run a VM on a host which doesn't have a PIF on a 
 Network needed by the VM. The VM has at least one VIF attached to the Network.

 http://pastebin.com/cHcAyPtP

 I believe this is supposed to be supported since CloudStack let me do 
 it from the UI. Right?

 Thanks!

 --
 Francois Gaudreault
 Architecte de Solution Cloud | Cloud Solutions Architect 
 fgaudrea...@cloudops.com
 514-629-6775
 - - -
 CloudOps
 420 rue Guy
 Montréal QC  H3J 1S6
 www.cloudops.com
 @CloudOps_




CLOUDSTACK-5942

2014-01-24 Thread Devdeep Singh
Hi Animesh,

Can I cherry-pick commit 6d997bd6893f0ab649899f88456d6e80b23bb873 from 
4.3-forward to 4.3 branch? This commit addresses issue CLOUDSTACK-5942 where 
the hyper-v agent was logging the password at few places.

Regards,
Devdeep


RE: Hyper-V Issues

2014-01-20 Thread Devdeep Singh
Can you check the systemvm.iso that is present. You may be having an old one 
because of the earlier failed attempts and since then you have rebuild 
management server. Check the following locations,
1. On the host at Root Drive:\Users\Public\Documents\Hyper-V\Virtual hard 
disks. Remove the iso file from this directory.
2. On secondary storage there will be a systemvm directory with the iso file. 
Delete the directory itself.

When cloudstack tries to create VR, it'll copy the iso file again.

Regards,
Devdeep

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com] 
Sent: Monday, January 20, 2014 4:36 PM
To: Devdeep Singh; dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar; Radhika Puthiyetath; 
Animesh Chaturvedi
Subject: RE: Hyper-V Issues

CPVM and SSVM are up, the SSVM downloaded the hyper-v demo template.

But I can't start an instance because of and ssh authentication error between 
the VR and CSMAN

The log suggests that the networking is ok as the ping succeeds (and ssvm 
downloaded the hyper-v demo template)

2014-01-20 10:47:09,694 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Ping command port, 10.0.107.17:3922
2014-01-20 10:47:09,694 INFO  [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Trying to connect to 10.0.107.17
2014-01-20 10:47:09,696 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Ping command port succeeded for vm r-38-VM
2014-01-20 10:47:09,696 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Execute network usage setup command on r-38-VM
2014-01-20 10:47:09,981 ERROR [c.c.u.s.SshHelper] (DirectAgent-30:ctx-f9cc6052) 
Failed to authentication SSH user root on host 10.0.107.17
2014-01-20 10:47:09,985 ERROR [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Unable to execute NetworkUsage command on DomR 
(10.0.107.17), domR may not be ready yet. failure due to java.lang.Exception: 
Failed to authentication SSH user root on host 10.0.107.17
2014-01-20 10:47:09,986 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Executing resource GetDomRVersionCmd: 
{accessDetails:{router.ip:10.0.107.17,router.name:r-38-VM},contextMap:{},wait:0}
2014-01-20 10:47:09,986 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Run command on domR 10.0.107.17, 
/opt/cloud/bin/get_template_version.sh
2014-01-20 10:47:09,986 DEBUG [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) Use router's private IP for SSH control. IP : 
10.0.107.17
2014-01-20 10:47:10,193 ERROR [c.c.u.s.SshHelper] (DirectAgent-30:ctx-f9cc6052) 
Failed to authentication SSH user root on host 10.0.107.17
2014-01-20 10:47:10,194 ERROR [c.c.h.h.r.HypervDirectConnectResource] 
(DirectAgent-30:ctx-f9cc6052) GetDomRVersionCmd failed due to 
java.lang.Exception: Failed to authentication SSH user root on host 10.0.107.17

I'll raise a ticket and post the ID

Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.com

-Original Message-
From: Devdeep Singh [mailto:devdeep.si...@citrix.com]
Sent: 20 January 2014 05:05
To: Paul Angus; dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar; Radhika Puthiyetath; 
Animesh Chaturvedi
Subject: RE: Hyper-V Issues

Glad to know it worked for you. Do let me know if you face any issue.

Regards,
Devdeep

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Monday, January 20, 2014 3:21 AM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Devdeep Singh; Rajesh Battala; Anshul Gangwar; Radhika 
Puthiyetath; Animesh Chaturvedi
Subject: RE: Hyper-V Issues

Forget that I built a new agent as well and I can connect again. I'll try an 
advanced network now.

Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.com

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: 19 January 2014 19:52
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Devdeep Singh; Rajesh Battala; Anshul Gangwar; Radhika 
Puthiyetath; Animesh Chaturvedi
Subject: RE: Hyper-V Issues

Thanks Sebastien.

@Hyperv-people

I ran the new commands in the wiki

PS C:\Users\Administrator New-SelfSignedCertificate -DnsName apachecloudstack 
-CertStoreLocation Cert:\LocalMachine\M
Directory: Microsoft.PowerShell.Security\Certificate::LocalMachine\My
ThumbprintSubject
-----
32BF5F505947B083E4CBDD9FDB2487CC8F133DF1  CN=apachecloudstack

PS C:\Users\Administrator netsh http add sslcert ipport=0.0.0.0:8250 
certhash=32BF5F505947B083E4CBDD9FDB2487CC8F133DF1 
appid={727beb1c-6e7c-49b2-8fbd-f03dbe481b08}
SSL Certificate successfully added

But I still get the same identical error, are there any other steps required? 
The windows

RE: Hyper-V Issues

2014-01-19 Thread Devdeep Singh
Glad to know it worked for you. Do let me know if you face any issue.

Regards,
Devdeep

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com] 
Sent: Monday, January 20, 2014 3:21 AM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Devdeep Singh; Rajesh Battala; Anshul Gangwar; Radhika 
Puthiyetath; Animesh Chaturvedi
Subject: RE: Hyper-V Issues

Forget that I built a new agent as well and I can connect again. I'll try an 
advanced network now.

Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.com

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: 19 January 2014 19:52
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Devdeep Singh; Rajesh Battala; Anshul Gangwar; Radhika 
Puthiyetath; Animesh Chaturvedi
Subject: RE: Hyper-V Issues

Thanks Sebastien.

@Hyperv-people

I ran the new commands in the wiki

PS C:\Users\Administrator New-SelfSignedCertificate -DnsName apachecloudstack 
-CertStoreLocation Cert:\LocalMachine\M
Directory: Microsoft.PowerShell.Security\Certificate::LocalMachine\My
ThumbprintSubject
-----
32BF5F505947B083E4CBDD9FDB2487CC8F133DF1  CN=apachecloudstack

PS C:\Users\Administrator netsh http add sslcert ipport=0.0.0.0:8250 
certhash=32BF5F505947B083E4CBDD9FDB2487CC8F133DF1 
appid={727beb1c-6e7c-49b2-8fbd-f03dbe481b08}
SSL Certificate successfully added

But I still get the same identical error, are there any other steps required? 
The windows firewall is shutdown.


Regards

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: CloudyAngus paul.an...@shapeblue.com

-Original Message-
From: Sebastien Goasguen [mailto:run...@gmail.com]
Sent: 19 January 2014 19:22
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Devdeep Singh; Rajesh Battala; Anshul Gangwar; Radhika 
Puthiyetath; Animesh Chaturvedi
Subject: Re: Hyper-V Issues


On Jan 19, 2014, at 2:14 PM, Paul Angus paul.an...@shapeblue.com wrote:

 Thanks Donal,

 I'll add some info in the wiki, although I'll check the validity of what I've 
 been doing with you guys before telling everyone else what they should be 
 doing.

 In the meantime - has something changed in the way CloudStack contacts 
 the agent in hyper-v in the last 3 days? I suddenly can't add a 
 hyper-v host to CloudStack. I get an error which suggests either an 
 SSL error or a host connectivity issue.  The only thing I changed was 
 using an updated build of the management server - the hyper-v host was 
 a rolled-back snapshot, so I know it is identical to the last time I 
 added it...

Paul, yes I saw this passing by:

https://issues.apache.org/jira/browse/CLOUDSTACK-5880 on jan 16


 http://pastebin.com/UJxTspcV

 Regards

 Paul Angus
 Cloud Architect
 S: +44 20 3603 0540 | M: +447711418784 | T: CloudyAngus 
 paul.an...@shapeblue.commailto:paul.an...@shapeblue.com

 From: Donal Lafferty [mailto:donal.laffe...@citrix.com]
 Sent: 17 January 2014 11:46
 To: dev@cloudstack.apache.org; Paul Angus
 Cc: Devdeep Singh; Rajesh Battala; Anshul Gangwar; Radhika 
 Puthiyetath; Animesh Chaturvedi
 Subject: Hyper-V Issues

 Hi Paul,

 Thanks for trying out the Hyper-V agent over the last couple of weeks!  I'm 
 not always able to help you with the problems, so I've compiled a list of 
 resources below to help with problems, collaboration and documentation.

 For problems you see, there is a team in Bangalore dedicated to polishing up 
 the Hyper-V agent.  They have been following up on the issues you've found.  
 However, I've included the email of team lead, Devdeep Singh, in the CC if 
 you want him to take a look at a specific JIRA issue you've created.

 For collaborating to provide user guidance, there is a wiki at 
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Hyper-V+Community+Tips+and+FAQ
It's a good place to accumulate gotchas that you, myself and other users 
 come across.  It's pretty empty at the moment, so make any additions or edits 
 you want.

 For documentation, there are Hyper-V sections in docs repo.  In 
 https://git-wip-us.apache.org/repos/asf?p=cloudstack-docs.git;a=tree;f=en-US;h=5fdde1c7134b675e4ede9f13b01e937406adc92e;hb=e8359806e3bbb49292deedb5bca11dc7ab0c9b7a
  files matching hyperv- tend to be Hyper-V related; however, I think a JIRA 
 issue with section name should point Radhika in the right direction for 
 making a fix.

 Look forward to seeing you at the CloudStack European User Group 
 Meeting in London next Thursday,


 DL
 Need Enterprise Grade Support for Apache CloudStack?
 Our CloudStack Infrastructure 
 Supporthttp://shapeblue.com/cloudstack-infrastructure-support/ offers the 
 best 24/7 SLA for CloudStack Environments.

 Apache CloudStack Bootcamp training courses

 **NEW!** CloudStack 4.2
 traininghttp://shapeblue.com/cloudstack-training/
 08/09 January 2014, Londonhttp://shapeblue.com/cloudstack-training

Re: Review Request 16540: CLOUDSTACK-5692: cleanup API response for primary/secondary storages

2014-01-17 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16540/#review32134
---

Ship it!


Ship It!

- Devdeep Singh


On Jan. 16, 2014, 1:52 p.m., Saksham Srivastava wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/16540/
 ---
 
 (Updated Jan. 16, 2014, 1:52 p.m.)
 
 
 Review request for cloudstack and Devdeep Singh.
 
 
 Bugs: CLOUDSTACK-5692
 https://issues.apache.org/jira/browse/CLOUDSTACK-5692
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Cleanup the API response while listing primary/secondary stores while using 
 cifs.
 Cleanup logs and remove passwords.
 
 
 Diffs
 -
 
   core/src/com/cloud/agent/transport/Request.java cbeb112 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java
  1edfea3 
   server/src/com/cloud/api/query/dao/ImageStoreJoinDaoImpl.java 8022871 
   server/src/com/cloud/api/query/dao/StoragePoolJoinDaoImpl.java 4d2aac2 
 
 Diff: https://reviews.apache.org/r/16540/diff/
 
 
 Testing
 ---
 
 Tested locally.
 
 The api response for list doesnot contain passwords:
 
 listimagestoresresponse : { count:1 ,imagestore : [  
 {id:182cfbfd-6343-4f35-804c-6b388fbf6a18,zoneid:1ae705a4-c9bc-4977-9260-ce128d7fd3d8,zonename:zone1,name:secondary1,url:cifs://10.102.192.151/SMB-Share/saksham/secondary?user=administratordomain=blr,protocol:cifs,providername:NFS,scope:ZONE,details:[]}
  ] } }
 
 The logs also do not contain passwords :
 
 2014-01-16 18:48:53,288 DEBUG [o.a.c.f.j.i.AsyncJobManagerImpl] 
 (Job-Executor-2:ctx-24ee5b9d ctx-b4e28b06) Complete async job-62, jobStatus: 
 SUCCEEDED, resultCode: 0, result: 
 org.apache.cloudstack.api.response.StoragePoolResponse/storagepool/{id:c59cc1c9-8d16-3090-95e7-d5c54839cf2c,zoneid:1ae705a4-c9bc-4977-9260-ce128d7fd3d8,zonename:zone1,podid:bd328cfc-692e-4c8c-8d32-e2a34abaaa37,podname:pod1,name:primary1,ipaddress:10.102.192.150,path:/SMB-Share/saksham/primary?user\u003dadministrator\u0026domain\u003dblr,created:2014-01-07T16:28:35+0530,type:NetworkFilesystem,clusterid:fc1df888-0e90-45c2-8555-5d4ed61c7bc3,clustername:cluster1,disksizetotal:500105736192,disksizeallocated:0,tags:sggss,state:Up,scope:CLUSTER,jobid:dfbd2072-48dc-457d-a417-312a74c517f9,jobstatus:0}
 
 
 Thanks,
 
 Saksham Srivastava
 




RE: Hyper-V

2014-01-17 Thread Devdeep Singh
Hi Paul,

Did you get a chance to try it in Advanced mode? Did it work for you? Anshul 
recently made a change to secure the communication between the management 
server and the agent. If you try with the latest changes you'll have to install 
a certificate so that the management server can talk to the agent. Following 
link [1] will tell you how to go about it.

[1] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Manually+Creating+and+installing+self+signed+certificate+for+CloudStack+Management+Server+communication+with+Hyper-V+agent

Regards,
Devdeep

-Original Message-
From: Devdeep Singh [mailto:devdeep.si...@citrix.com] 
Sent: Thursday, January 16, 2014 7:05 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: RE: Hyper-V

Hi Paul,

From the logs it looks like you have setup the zone in Basic mode. Basic mode 
is right now not supported for Hyper-V. Can you set it up in Advanced mode and 
see if it works?

Regards,
Devdeep

P.S. I just tried setting up the zone in Basic mode and confirmed I got the 
same exception.

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com] 
Sent: Thursday, January 16, 2014 5:25 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: RE: Hyper-V

I've got the System VMs creating, but not starting now...

Error seems to be:
POST response 
is[{com.cloud.agent.api.StartAnswer:{result:false,details:com.cloud.agent.api.StartCommand
 fail on exceptionObject reference not set to an instance of an object

Agent log:
http://pastebin.com/zhfXia9v

mgmt. log:
http://pastebin.com/ZUjBgqX8

Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.com

-Original Message-
From: Devdeep Singh [mailto:devdeep.si...@citrix.com]
Sent: 15 January 2014 02:57
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: RE: Hyper-V

Hi Paul,

I have put in a fix for the issue with local storage that you reported earlier 
[1]. Please pull the latest changes and try. It should work now. I'll look into 
the error you are facing with shared and let you know once it is fixed.

[1] https://issues.apache.org/jira/browse/CLOUDSTACK-5869

Regards,
Devdeep

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Tuesday, January 14, 2014 9:19 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: Hyper-V

Guys,

Primary storage local or shared is still giving me grief.

See the error below from the agent log when trying to create a system VM. the 
final error is 'org.apache.cloudstack.storage.command.CopyCommand failed on 
exception, net use of share 10.0.1.27/hypervPri/storfailed with Error: 
Unknown, 1219,'
The share is \\10.0.1.27\hypervPri\stor in UNC notation.


2014-01-14 15:40:12,566 [6] INFO  HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - 
org.apache.cloudstack.storage.command.CopyCommand{
  srcTO: {
org.apache.cloudstack.storage.to.TemplateObjectTO: {
  path: template/tmpl/1/9/,
  origUrl: 
http://download.cloud.com/templates/4.3/systemvm64template-2013-12-23-hyperv.vhd.bz2;,
  uuid: 18767718-7d1b-11e3-b742-0050568b4d4c,
  id: 9,
  format: VHD,
  accountId: 1,
  checksum: 5df45ee6ebe1b703a8805f4e1f4d0818,
  hvm: false,
  displayText: SystemVM Template (HyperV),
  imageDataStore: {
com.cloud.agent.api.to.NfsTO: {
 _url: 
cifs://10.0.100.5/stor_Cloud/Secondary/ACS43?user=hypervpassword=hypervdomain=angusnet,
  _role: Image
}
  },
  name: routing-9,
  hypervisorType: Hyperv
}
  },
  destTO: {
org.apache.cloudstack.storage.to.TemplateObjectTO: {
  origUrl: 
http://download.cloud.com/templates/4.3/systemvm64template-2013-12-23-hyperv.vhd.bz2;,
  uuid: 18767718-7d1b-11e3-b742-0050568b4d4c,
  id: 9,
  format: VHD,
  accountId: 1,
  checksum: 5df45ee6ebe1b703a8805f4e1f4d0818,
  hvm: false,
  displayText: SystemVM Template (HyperV),
  imageDataStore: {
org.apache.cloudstack.storage.to.PrimaryDataStoreTO: {
  uuid: 2bcff176-e5f3-350e-936a-7ee19a0e8658,
  id: 2,
  poolType: NetworkFilesystem,
  host: 10.0.1.27,
  path: 
/hypervPri/stor?user=hypervpassword=hyperv13!domain=angusnet.local,
  port: 445,
  url: 
NetworkFilesystem://10.0.1.27//hypervPri/stor?user=hypervpassword=hyperv13!domain=angusnet.local/?ROLE=PrimarySTOREUUID=2bcff176-e5f3-350e-936a-7ee19a0e8658
}
  },
  name: routing-9,
  hypervisorType: Hyperv
}
  },
  executeInSequence: false,
  options: {},
  contextMap: {},
  wait: 10800
}
2014-01-14 15:40:12,581 [6] ERROR HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - 
org.apache.cloudstack.storage.command.CopyCommand failed on exception, net use 
of share

RE: Hyper-V

2014-01-16 Thread Devdeep Singh
Hi Paul,

From the logs it looks like you have setup the zone in Basic mode. Basic mode 
is right now not supported for Hyper-V. Can you set it up in Advanced mode and 
see if it works?

Regards,
Devdeep

P.S. I just tried setting up the zone in Basic mode and confirmed I got the 
same exception.

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com] 
Sent: Thursday, January 16, 2014 5:25 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: RE: Hyper-V

I've got the System VMs creating, but not starting now...

Error seems to be:
POST response 
is[{com.cloud.agent.api.StartAnswer:{result:false,details:com.cloud.agent.api.StartCommand
 fail on exceptionObject reference not set to an instance of an object

Agent log:
http://pastebin.com/zhfXia9v

mgmt. log:
http://pastebin.com/ZUjBgqX8

Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.com

-Original Message-
From: Devdeep Singh [mailto:devdeep.si...@citrix.com]
Sent: 15 January 2014 02:57
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: RE: Hyper-V

Hi Paul,

I have put in a fix for the issue with local storage that you reported earlier 
[1]. Please pull the latest changes and try. It should work now. I'll look into 
the error you are facing with shared and let you know once it is fixed.

[1] https://issues.apache.org/jira/browse/CLOUDSTACK-5869

Regards,
Devdeep

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: Tuesday, January 14, 2014 9:19 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: Hyper-V

Guys,

Primary storage local or shared is still giving me grief.

See the error below from the agent log when trying to create a system VM. the 
final error is 'org.apache.cloudstack.storage.command.CopyCommand failed on 
exception, net use of share 10.0.1.27/hypervPri/storfailed with Error: 
Unknown, 1219,'
The share is \\10.0.1.27\hypervPri\stor in UNC notation.


2014-01-14 15:40:12,566 [6] INFO  HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - 
org.apache.cloudstack.storage.command.CopyCommand{
  srcTO: {
org.apache.cloudstack.storage.to.TemplateObjectTO: {
  path: template/tmpl/1/9/,
  origUrl: 
http://download.cloud.com/templates/4.3/systemvm64template-2013-12-23-hyperv.vhd.bz2;,
  uuid: 18767718-7d1b-11e3-b742-0050568b4d4c,
  id: 9,
  format: VHD,
  accountId: 1,
  checksum: 5df45ee6ebe1b703a8805f4e1f4d0818,
  hvm: false,
  displayText: SystemVM Template (HyperV),
  imageDataStore: {
com.cloud.agent.api.to.NfsTO: {
 _url: 
cifs://10.0.100.5/stor_Cloud/Secondary/ACS43?user=hypervpassword=hypervdomain=angusnet,
  _role: Image
}
  },
  name: routing-9,
  hypervisorType: Hyperv
}
  },
  destTO: {
org.apache.cloudstack.storage.to.TemplateObjectTO: {
  origUrl: 
http://download.cloud.com/templates/4.3/systemvm64template-2013-12-23-hyperv.vhd.bz2;,
  uuid: 18767718-7d1b-11e3-b742-0050568b4d4c,
  id: 9,
  format: VHD,
  accountId: 1,
  checksum: 5df45ee6ebe1b703a8805f4e1f4d0818,
  hvm: false,
  displayText: SystemVM Template (HyperV),
  imageDataStore: {
org.apache.cloudstack.storage.to.PrimaryDataStoreTO: {
  uuid: 2bcff176-e5f3-350e-936a-7ee19a0e8658,
  id: 2,
  poolType: NetworkFilesystem,
  host: 10.0.1.27,
  path: 
/hypervPri/stor?user=hypervpassword=hyperv13!domain=angusnet.local,
  port: 445,
  url: 
NetworkFilesystem://10.0.1.27//hypervPri/stor?user=hypervpassword=hyperv13!domain=angusnet.local/?ROLE=PrimarySTOREUUID=2bcff176-e5f3-350e-936a-7ee19a0e8658
}
  },
  name: routing-9,
  hypervisorType: Hyperv
}
  },
  executeInSequence: false,
  options: {},
  contextMap: {},
  wait: 10800
}
2014-01-14 15:40:12,581 [6] ERROR HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - 
org.apache.cloudstack.storage.command.CopyCommand failed on exception, net use 
of share \\10.0.1.27/hypervPri/storfailed with Error: Unknown, 1219
System.ArgumentException: net use of share \\10.0.1.27/hypervPri/storfailed 
with Error: Unknown, 1219
   at HypervResource.Utils.ConnectToRemote(String remoteUNC, String domain, 
String username, String password)
   at HypervResource.HypervResourceController.CopyCommand(Object cmd)
2014-01-14 15:40:12,581 [6] INFO  HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - {
  org.apache.cloudstack.storage.command.CopyCmdAnswer: {
result: false,
details: org.apache.cloudstack.storage.command.CopyCommand failed on 
exception, net use of share 10.0.1.27/hypervPri/storfailed with Error: 
Unknown, 1219,
newData: null,
contextMap: {}
  }
}

Regards

Paul Angus
Senior Consultant / Cloud Architect

RE: Hyper-V

2014-01-14 Thread Devdeep Singh
Hi Paul,

I have put in a fix for the issue with local storage that you reported earlier 
[1]. Please pull the latest changes and try. It should work now. I'll look into 
the error you are facing with shared and let you know once it is fixed.

[1] https://issues.apache.org/jira/browse/CLOUDSTACK-5869

Regards,
Devdeep

From: Paul Angus [mailto:paul.an...@shapeblue.com] 
Sent: Tuesday, January 14, 2014 9:19 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: Hyper-V

Guys,

Primary storage local or shared is still giving me grief.

See the error below from the agent log when trying to create a system VM. the 
final error is 
'org.apache.cloudstack.storage.command.CopyCommand failed on exception, net use 
of share 10.0.1.27/hypervPri/storfailed with Error: Unknown, 1219,'
The share is \\10.0.1.27\hypervPri\stor in UNC notation.


2014-01-14 15:40:12,566 [6] INFO  HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - 
org.apache.cloudstack.storage.command.CopyCommand{
  srcTO: {
    org.apache.cloudstack.storage.to.TemplateObjectTO: {
  path: template/tmpl/1/9/,
  origUrl: 
http://download.cloud.com/templates/4.3/systemvm64template-2013-12-23-hyperv.vhd.bz2;,
  uuid: 18767718-7d1b-11e3-b742-0050568b4d4c,
  id: 9,
  format: VHD,
  accountId: 1,
  checksum: 5df45ee6ebe1b703a8805f4e1f4d0818,
  hvm: false,
  displayText: SystemVM Template (HyperV),
  imageDataStore: {
    com.cloud.agent.api.to.NfsTO: {
 _url: 
cifs://10.0.100.5/stor_Cloud/Secondary/ACS43?user=hypervpassword=hypervdomain=angusnet,
  _role: Image
    }
  },
  name: routing-9,
  hypervisorType: Hyperv
    }
  },
  destTO: {
    org.apache.cloudstack.storage.to.TemplateObjectTO: {
  origUrl: 
http://download.cloud.com/templates/4.3/systemvm64template-2013-12-23-hyperv.vhd.bz2;,
  uuid: 18767718-7d1b-11e3-b742-0050568b4d4c,
  id: 9,
  format: VHD,
  accountId: 1,
  checksum: 5df45ee6ebe1b703a8805f4e1f4d0818,
  hvm: false,
  displayText: SystemVM Template (HyperV),
  imageDataStore: {
    org.apache.cloudstack.storage.to.PrimaryDataStoreTO: {
  uuid: 2bcff176-e5f3-350e-936a-7ee19a0e8658,
  id: 2,
  poolType: NetworkFilesystem,
  host: 10.0.1.27,
  path: 
/hypervPri/stor?user=hypervpassword=hyperv13!domain=angusnet.local,
  port: 445,
  url: 
NetworkFilesystem://10.0.1.27//hypervPri/stor?user=hypervpassword=hyperv13!domain=angusnet.local/?ROLE=PrimarySTOREUUID=2bcff176-e5f3-350e-936a-7ee19a0e8658
    }
  },
  name: routing-9,
  hypervisorType: Hyperv
    }
  },
  executeInSequence: false,
  options: {},
  contextMap: {},
  wait: 10800
}
2014-01-14 15:40:12,581 [6] ERROR HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - 
org.apache.cloudstack.storage.command.CopyCommand failed on exception, net use 
of share \\10.0.1.27/hypervPri/storfailed with Error: Unknown, 1219
System.ArgumentException: net use of share \\10.0.1.27/hypervPri/storfailed 
with Error: Unknown, 1219
   at HypervResource.Utils.ConnectToRemote(String remoteUNC, String domain, 
String username, String password)
   at HypervResource.HypervResourceController.CopyCommand(Object cmd)
2014-01-14 15:40:12,581 [6] INFO  HypervResource.HypervResourceController 
[cc482c9e-0d97-4be8-9dc8-2e546bdb00c1] - {
  org.apache.cloudstack.storage.command.CopyCmdAnswer: {
    result: false,
    details: org.apache.cloudstack.storage.command.CopyCommand failed on 
exception, net use of share 10.0.1.27/hypervPri/storfailed with Error: 
Unknown, 1219,
    newData: null,
    contextMap: {}
  }
}

Regards

Paul Angus
Senior Consultant / Cloud Architect



S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus
paul.an...@shapeblue.com | www.shapeblue.com | Twitter:@shapeblue
ShapeBlue Ltd, 53 Chandos Place, Covent Garden, London, WC2N 4HS

Need Enterprise Grade Support for Apache CloudStack?
Our CloudStack Infrastructure Support offers the best 24/7 SLA for CloudStack 
Environments.

Apache CloudStack Bootcamp training courses

**NEW!** CloudStack 4.2 training
08/09 January 2014, London
13-17 January 2014, GLOBAL. Instructor led, On-line
20-24 January 2014, GLOBAL. Instructor led, On-line

This email and any attachments to it may be confidential and are intended 
solely for the use of the individual to whom it is addressed. Any views or 
opinions expressed are solely those of the author and do not necessarily 
represent those of Shape Blue Ltd or related companies. If you are not the 
intended recipient of this email, you must neither take any action based upon 
its contents, nor copy or show it to anyone. Please contact the sender if you 
believe you have received this email in error. Shape Blue Ltd is a company 
incorporated in England  Wales. ShapeBlue Services India LLP is a company 
incorporated in India and is 

RE: Hyper-V agent

2014-01-06 Thread Devdeep Singh
Error seems to be in starting the service. Can you check under services 
(services.msc) if a service is present by the name CloudStack Hyper-V Agent? 
To debug the service start issue, can you open up the 8250 port (or try disable 
firewall) and check if the service starts up.

Regards,
Devdeep

-Original Message-
From: Paul Angus [mailto:paul.an...@shapeblue.com] 
Sent: Friday, January 3, 2014 11:19 PM
To: dev@cloudstack.apache.org; Donal Lafferty; Anshul Gangwar
Subject: RE: Hyper-V agent

So... updating .net 4.5.1 and a reboot and CloudAgent builds (with 19 warnings) 
http://pastebin.com/ahz5yJw2

I copy it to my hyper-v box and it bombs out immediately 
http://imgur.com/NMan0S2

Install log says:

Installing assembly 'B:\Microsoft\AgentShell\AgentShell.exe'.
Affected parameters are:
   assemblypath = B:\Microsoft\AgentShell\AgentShell.exe
   logfile = B:\Microsoft\AgentShell\AgentShell.InstallLog
Installing service CloudStack Hyper-V Agent...
Service CloudStack Hyper-V Agent has been successfully installed.
Creating EventLog source CloudStack Hyper-V Agent in log Application...
See the contents of the log file for the B:\Microsoft\AgentShell\AgentShell.exe 
assembly's progress.
The file is located at B:\Microsoft\AgentShell\AgentShell.InstallLog.
Committing assembly 'B:\Microsoft\AgentShell\AgentShell.exe'.
Affected parameters are:
   logtoconsole =
   assemblypath = B:\Microsoft\AgentShell\AgentShell.exe
   logfile = B:\Microsoft\AgentShell\AgentShell.InstallLog

agent log says:

2014-01-03 17:33:59,755 [1] DEBUG CloudStack.Plugin.AgentShell.Program [(null)] 
- CloudStack Hyper-V Agent arg is
2014-01-03 17:33:59,823 [1] INFO  CloudStack.Plugin.AgentShell.Program [(null)] 
- Installing and running CloudStack Hyper-V Agent
2014-01-03 17:34:02,185 [1] ERROR CloudStack.Plugin.AgentShell.Program [(null)] 
-  Error occured in starting service Cannot start service CloudStack Hyper-V 
Agent on computer '.'.


Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.com

From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: 03 January 2014 16:35
To: Donal Lafferty; dev@cloudstack.apache.org; Anshul Gangwar
Subject: RE: Hyper-V agent

When (trying) to build the hyper-v agent I get these errors:

Build FAILED.

Warnings:

C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\ServerResource.sln
 (default targets) - (Build target) - 
C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\WmiWrappers\WmiWrappers.csproj
 (default targets) - 
C:\PROGRA~2\MONO-3~1.3\lib\mono\4.0\Microsoft.Common.targets 
(ResolveAssemblyReferences target) -

C:\PROGRA~2\MONO-3~1.3\lib\mono\4.0\Microsoft.Common.targets:  warning 
: Reference 'AWSSDK' not resolved
C:\PROGRA~2\MONO-3~1.3\lib\mono\4.0\Microsoft.Common.targets:  warning 
: Reference 'Ionic.Zip' not resolved
C:\PROGRA~2\MONO-3~1.3\lib\mono\4.0\Microsoft.Common.targets:  warning 
: Reference 'log4net' not resolved
C:\PROGRA~2\MONO-3~1.3\lib\mono\4.0\Microsoft.Common.targets:  warning 
: Reference 'Newtonsoft.Json' not resolved
C:\PROGRA~2\MONO-3~1.3\lib\mono\4.0\Microsoft.Common.targets:  warning 
: Reference 'NSubstitute' not resolved
C:\PROGRA~2\MONO-3~1.3\lib\mono\4.0\Microsoft.Common.targets:  warning 
: Reference 'xunit' not resolved

Errors:

C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\ServerResource.sln
 (default targets) - (Build target) - 
C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\HypervResource\HypervResource.csproj
 (default targets) - 
C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\\.nuget\NuGet.targets
 (RestorePackages target) -


C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\\.nuget\NuGet.targets:
 error : Command 'mono --runtime=v4.0.30319 
C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\.nuget\NuGet.exe
 install 
C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\HypervResource\packages.config
 -source-RequireConsent -solutionDir 
C:\cygwin64\usr\local\cloudstack\plugins\hypervisors\hyperv\DotNet\ServerResource\\
 ' exited with code: 1.

 6 Warning(s)
 1 Error(s)

Time Elapsed 00:00:02.6896758



Any ideas?




Regards,

Paul Angus
Cloud Architect
S: +44 20 3603 0540 | M: +447711418784 | T: @CloudyAngus 
paul.an...@shapeblue.commailto:paul.an...@shapeblue.com

From: Donal Lafferty [mailto:donal.laffe...@citrix.com]
Sent: 02 January 2014 16:43
To: Paul Angus; dev@cloudstack.apache.orgmailto:dev@cloudstack.apache.org; 
Anshul Gangwar
Subject: RE: Hyper-V agent

I agree that we need a distro for the agent.

Based on what KVM does, what is the pattern for distributing non-Java agents?

DL


From: Paul Angus [mailto:paul.an...@shapeblue.com]
Sent: 02 

Re: Review Request 16605: fixed special characters not working in console view for hyperv

2014-01-06 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16605/#review31226
---

Ship it!


Committed in ef51def9fff3610b13f50be39c7610262d4a1c04. Kindly close the review 
request.

- Devdeep Singh


On Jan. 3, 2014, 11:42 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/16605/
 ---
 
 (Updated Jan. 3, 2014, 11:42 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh and rajeshbabu chintaguntla.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 this patch fixes the special characters  not working in cnsole view for 
 hyperv. Added the keymap for special keys and modified modifeir keys handling
 
 
 Diffs
 -
 
   
 services/console-proxy-rdp/rdpconsole/src/main/java/rdpclient/adapter/AwtRdpKeyboardAdapter.java
  36da0a3 
   
 services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyRdpClient.java
  6b317ff 
   
 services/console-proxy/server/src/com/cloud/consoleproxy/rdp/KeysymToKeycode.java
  10282ad 
 
 Diff: https://reviews.apache.org/r/16605/diff/
 
 
 Testing
 ---
 
 verified by typing special keys in console view of vm in hyperv
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 16551: put the hard coded rdp server port in host deatils

2014-01-01 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16551/#review31007
---



plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/discoverer/HypervServerDiscoverer.java
https://reviews.apache.org/r/16551/#comment59372

The rdp server port should be returned in details by the hyperv agent. You 
can update the agent to populate the port details in StartupCommand


- Devdeep Singh


On Jan. 1, 2014, 7:52 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/16551/
 ---
 
 (Updated Jan. 1, 2014, 7:52 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Donal Lafferty, and Rajesh 
 Battala.
 
 
 Bugs: CLOUDSTACK-5703
 https://issues.apache.org/jira/browse/CLOUDSTACK-5703
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 This patch removes the hard coded value of rdp server port and put it in host 
 details
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/discoverer/HypervServerDiscoverer.java
  2330da1 
   server/src/com/cloud/servlet/ConsoleProxyServlet.java e0aaa6d 
 
 Diff: https://reviews.apache.org/r/16551/diff/
 
 
 Testing
 ---
 
 tested on my local setup
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 16550: fixed rdp console mouse patch

2014-01-01 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16550/#review31008
---

Ship it!


Committed in a4d4d4b09aaff5b5606d1f9f58b2b80c92aee0a9

- Devdeep Singh


On Jan. 1, 2014, 7:45 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/16550/
 ---
 
 (Updated Jan. 1, 2014, 7:45 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Donal Lafferty, and Rajesh 
 Battala.
 
 
 Bugs: CLOUDSTACK-5702
 https://issues.apache.org/jira/browse/CLOUDSTACK-5702
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 this patch fixed the mouse events handling in rdp console
 
 
 Diffs
 -
 
   
 services/console-proxy/server/src/com/cloud/consoleproxy/ConsoleProxyRdpClient.java
  73c00be 
 
 Diff: https://reviews.apache.org/r/16550/diff/
 
 
 Testing
 ---
 
 tested on my local setup
 
 
 Thanks,
 
 Anshul Gangwar
 




Re: Review Request 16551: put the hard coded rdp server port in host deatils

2014-01-01 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16551/#review31009
---

Ship it!


Committed in e74889ac91c919ab26852da3d93133eaeae874c8

- Devdeep Singh


On Jan. 1, 2014, 7:52 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/16551/
 ---
 
 (Updated Jan. 1, 2014, 7:52 a.m.)
 
 
 Review request for cloudstack, Devdeep Singh, Donal Lafferty, and Rajesh 
 Battala.
 
 
 Bugs: CLOUDSTACK-5703
 https://issues.apache.org/jira/browse/CLOUDSTACK-5703
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 This patch removes the hard coded value of rdp server port and put it in host 
 details
 
 
 Diffs
 -
 
   
 plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/discoverer/HypervServerDiscoverer.java
  2330da1 
   server/src/com/cloud/servlet/ConsoleProxyServlet.java e0aaa6d 
 
 Diff: https://reviews.apache.org/r/16551/diff/
 
 
 Testing
 ---
 
 tested on my local setup
 
 
 Thanks,
 
 Anshul Gangwar
 




RE: Migrating a ROOT volume

2013-12-30 Thread Devdeep Singh
If the instance is stopped you can go to Instance - select the instance - and 
migrate the instance to another storage pool. This will move the root volume to 
the specified storage pool.

Regards,
Devdeep

-Original Message-
From: Girish Shilamkar [mailto:gir...@clogeny.com] 
Sent: Tuesday, December 31, 2013 11:40 AM
To: dev@cloudstack.apache.org
Subject: Migrating a ROOT volume

Hello,

As per CS admin guide ROOT volume can be migrated. But in UI I don't see an 
option to detach disk from a stopped disk.
And if I try to detach using API, it fails stating please specify a datadisk

Can someone please clarify.

Regards,
Girish



RE: [Jenkins] slave for building hyperv agent

2013-12-11 Thread Devdeep Singh
I am setting up an instance locally that can be used as a Jenkins slave for 
building the hyperv agent code. I'll reach out if I need assistance. Thanks.

Regards,
Devdeep
 
-Original Message-
From: Alex Hitchins [mailto:cloudst...@alexhitchins.com] 
Sent: Friday, December 6, 2013 6:20 PM
To: dev@cloudstack.apache.org
Subject: RE: [Jenkins] slave for building hyperv agent

C# should run under Mono without much fuss, unless there are core Windows 
library's used in the application.



Alexander Hitchins

Personal Email : a...@alexhitchins.com
Apache Email   : a...@alexhitchins.com
Website: http://alexhitchins.com
Mobile : 07788 423 969

-Original Message-
From: Hugo Trippaers [mailto:trip...@gmail.com]
Sent: 05 December 2013 20:09
To: dev@cloudstack.apache.org
Cc: dev@cloudstack.apache.org
Subject: Re: [Jenkins] slave for building hyperv agent

Hey devdeep,

I'll try. The trouble with windows boxes is that we need a license for it. I 
need to check if my company is ok with assigning one to this machine.

Do you need any other software besides just plain windows server standard 2012?

Cheers,

Hugo 

Sent from my iPhone

 On 5 dec. 2013, at 12:02, Devdeep Singh devdeep.si...@citrix.com wrote:
 
 Hi Hugo,
 
 The agent code is C# based. So I was looking for a windows based 
 server to
build it. Will it be possible to get one?
 
 Regards,
 Devdeep
 
 -Original Message-
 From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers
 Sent: Thursday, December 5, 2013 4:35 PM
 To: dev@cloudstack.apache.org
 Subject: Re: [Jenkins] slave for building hyperv agent
 
 Devdeep,
 
 what kind of instance would you need? I might be able to help you out 
 with
a Linux based server in our cloud?
 
 Or if you have a fixed set of instructions to prepare a slave i'm 
 happy to
add them to the template i use for the other builds from 
jenkins.buildacloud.org?
 
 Cheers,
 
 Hugo
 
 
 On 4 dec. 2013, at 12:24, Devdeep Singh devdeep.si...@citrix.com wrote:
 
 Hi,
 
 I am looking to setup a Jenkins slave to build the hyperv agent. The
hyperv agent code can be build using Mono and the instructions for building it 
are available at [1]. Can anyone help with providing an instance so that it can 
be configured as a jenkins slave for regular builds of the hyperv agent?
 
 [1]
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Creating+Hyperv+Agent
+Installer
 
 Regards,
 Devdeep
 



RE: [Jenkins] slave for building hyperv agent

2013-12-05 Thread Devdeep Singh
Hi Hugo,

The agent code is C# based. So I was looking for a windows based server to 
build it. Will it be possible to get one?

Regards,
Devdeep

-Original Message-
From: Trippie [mailto:trip...@gmail.com] On Behalf Of Hugo Trippaers
Sent: Thursday, December 5, 2013 4:35 PM
To: dev@cloudstack.apache.org
Subject: Re: [Jenkins] slave for building hyperv agent

Devdeep,

what kind of instance would you need? I might be able to help you out with a 
Linux based server in our cloud?

Or if you have a fixed set of instructions to prepare a slave i'm happy to add 
them to the template i use for the other builds from jenkins.buildacloud.org?

Cheers,

Hugo


On 4 dec. 2013, at 12:24, Devdeep Singh devdeep.si...@citrix.com wrote:

 Hi,
 
 I am looking to setup a Jenkins slave to build the hyperv agent. The hyperv 
 agent code can be build using Mono and the instructions for building it are 
 available at [1]. Can anyone help with providing an instance so that it can 
 be configured as a jenkins slave for regular builds of the hyperv agent?
 
 [1] 
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Creating+Hyperv+Agent+Installer
 
 Regards,
 Devdeep



[Jenkins] slave for building hyperv agent

2013-12-04 Thread Devdeep Singh
Hi,

I am looking to setup a Jenkins slave to build the hyperv agent. The hyperv 
agent code can be build using Mono and the instructions for building it are 
available at [1]. Can anyone help with providing an instance so that it can be 
configured as a jenkins slave for regular builds of the hyperv agent?

[1] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Creating+Hyperv+Agent+Installer

Regards,
Devdeep


RE: uploading to maven repository

2013-11-12 Thread Devdeep Singh
Hi Alex,

One of the reasons we have a copy of the java bindings is because cloudstack 
makes a customization to set the connection timeout. Will these customizations 
be kept when the stubs are uploaded to maven?

Regarding,
Devdeep

-Original Message-
From: Alex Huang [mailto:alex.hu...@citrix.com] 
Sent: Tuesday, November 12, 2013 9:35 PM
To: dev@cloudstack.apache.org
Subject: uploading to maven repository

Hi all,

I have a chance to convince xenserver to upload their java stubs to maven so 
that we don't have to keep a copy of it in ours source code.  They're stuck on 
the whole upload process as they've never dealt with maven before.  Is there 
anyone who can walk me through how to do it?

Reading the maven guide[1], it seems to easiest would be to just upload the 
artifact[2].  Is that the right approach?  

Thanks.

--Alex
[1] http://maven.apache.org/guides/mini/guide-central-repository-upload.html
[2] 
https://docs.sonatype.org/display/Repository/Uploading+3rd-party+Artifacts+to+The+Central+Repository


Re: Review Request 13922: Minimal Hyper-V Plugin

2013-11-08 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13922/#review28518
---

Ship it!


The feature was merged and changes committed to master in commit id 
f9f196a46ad820198894ef1ebba3d61413d5da59.

- Devdeep Singh


On Sept. 5, 2013, 8:29 a.m., Donal Lafferty wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/13922/
 ---
 
 (Updated Sept. 5, 2013, 8:29 a.m.)
 
 
 Review request for cloudstack, Alex Huang, Chiradeep Vittal, and Devdeep 
 Singh.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 A plugin for Hyper-V control is available for CloudStack.  The plugin 
 implements basic VM control;  however, its architecture allows additional 
 functionality to be easily added.  Incorporating the plugin in CloudStack 
 will allow the community to participate in improving the features available 
 with Hyper-V.
 
 The plugin uses a Director Connect Agent architecture described here: 
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Progress
 
 This links provides details of third party libraries used by the plugin 
 including their licenses.  No source for these libraries is used, and the 
 binaries are downloaded from their distributors at build time. 
 
 No proprietary tools are required for the build.  For instance, C# compiled 
 with Mono has been tested.  Therefore, the plugin has been added to the 
 default build and the default deployment.
 
 The plugin includes unit and functional tests that can be triggered at build 
 time.  See the link above for details.
 
 The Apache header is applied to source and where feasible to build config 
 files.
 
 The source is the https://github.com/lafferty/cloudstack/tree/hyperv_plugin
 
 
 Diffs
 -
 
   client/pom.xml 1afe5b898d328b06b5da7c132eaed14fefc78718 
   client/tomcatconf/applicationContext.xml.in 
 956c77c5fa6c3d909be1fbee8ba26e8d8ba1dce7 
   client/tomcatconf/componentContext.xml.in 
 f36d0ee02a2eb91733f6f2a76be60a1477529d4d 
   client/tomcatconf/simulatorComponentContext.xml.in 
 c46d1de086a5633b64c90884dbc02a03c3b2e8ad 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.gitignore PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.Config 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.targets 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentService.Designer.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentService.cs 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.Designer.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.settings
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShellException.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/App.config 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/Program.cs 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/Properties/AssemblyInfo.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/packages.config 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/App.config 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/CloudStackTypes.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResource.csproj
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Properties/AssemblyInfo.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCalls.cs 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/packages.config
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/App.config
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/HypervResourceControllerTest.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/Properties/AssemblyInfo.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/ServerResource.Tests.csproj
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/packages.config
  PRE-CREATION 
   plugins

Re: Review Request 13922: Minimal Hyper-V Plugin

2013-11-08 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13922/#review28519
---

Ship it!


The feature was merged and changes committed to master in commit id 
f9f196a46ad820198894ef1ebba3d61413d5da59.

- Devdeep Singh


On Sept. 5, 2013, 8:29 a.m., Donal Lafferty wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/13922/
 ---
 
 (Updated Sept. 5, 2013, 8:29 a.m.)
 
 
 Review request for cloudstack, Alex Huang, Chiradeep Vittal, and Devdeep 
 Singh.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 A plugin for Hyper-V control is available for CloudStack.  The plugin 
 implements basic VM control;  however, its architecture allows additional 
 functionality to be easily added.  Incorporating the plugin in CloudStack 
 will allow the community to participate in improving the features available 
 with Hyper-V.
 
 The plugin uses a Director Connect Agent architecture described here: 
 https://cwiki.apache.org/confluence/display/CLOUDSTACK/Progress
 
 This links provides details of third party libraries used by the plugin 
 including their licenses.  No source for these libraries is used, and the 
 binaries are downloaded from their distributors at build time. 
 
 No proprietary tools are required for the build.  For instance, C# compiled 
 with Mono has been tested.  Therefore, the plugin has been added to the 
 default build and the default deployment.
 
 The plugin includes unit and functional tests that can be triggered at build 
 time.  See the link above for details.
 
 The Apache header is applied to source and where feasible to build config 
 files.
 
 The source is the https://github.com/lafferty/cloudstack/tree/hyperv_plugin
 
 
 Diffs
 -
 
   client/pom.xml 1afe5b898d328b06b5da7c132eaed14fefc78718 
   client/tomcatconf/applicationContext.xml.in 
 956c77c5fa6c3d909be1fbee8ba26e8d8ba1dce7 
   client/tomcatconf/componentContext.xml.in 
 f36d0ee02a2eb91733f6f2a76be60a1477529d4d 
   client/tomcatconf/simulatorComponentContext.xml.in 
 c46d1de086a5633b64c90884dbc02a03c3b2e8ad 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.gitignore PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.Config 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.targets 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentService.Designer.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentService.cs 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.Designer.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.settings
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShellException.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/App.config 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/Program.cs 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/Properties/AssemblyInfo.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/packages.config 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/App.config 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/CloudStackTypes.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResource.csproj
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Properties/AssemblyInfo.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/Utils.cs 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCalls.cs 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/packages.config
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/App.config
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/HypervResourceControllerTest.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/Properties/AssemblyInfo.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/ServerResource.Tests.csproj
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/packages.config
  PRE-CREATION 
   plugins

RE: [Merge] hyperv branch to master

2013-11-04 Thread Devdeep Singh
The changes have been merged to the master branch.

Regards,
Devdeep

-Original Message-
From: Devdeep Singh [mailto:devdeep.si...@citrix.com] 
Sent: Thursday, October 31, 2013 3:38 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: RE: [Merge] hyperv branch to master

Thanks Darren. I'll start working on getting it merged into the master. I hope 
to finish it by Friday.

Regards,
Devdeep

-Original Message-
From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com]
Sent: Tuesday, October 29, 2013 10:45 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: Re: [Merge] hyperv branch to master

+1

When you merge you will get a conflict in the spring configuration.
componentContext.xml and others are gone now in master.  You will need to 
update to the new style.  The easiest will be to refer to XenServer spring 
config for an example of how to do it.  You can largely copy and paste the 
contents of plugins/hypervisors/xen/resources/META-INF/cloudstack and replace 
xenserver with hyperv.  You can also refer to 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Plug-ins%2C+Modules%2C+and+Extensions
for more in depth explaination of the changes.

Darren

On Tue, Oct 29, 2013 at 9:35 AM, David Nalley da...@gnsa.us wrote:
 On Mon, Oct 28, 2013 at 7:41 AM, Devdeep Singh devdeep.si...@citrix.com 
 wrote:
 Hi,

 I would like to merge the support for Hyperv to the master branch. 
 Development for this has been done by Donal, Rajesh, Anshul and I on branch 
 [1]. The feature was proposed for merge earlier [3] but unit tests for 
 hyperv agent code were requested [4].


 I know this has been discussed several times, but what was the 
 decision about making the agent code live in its own repo?
 It's C#, might need to be signed to actually run easily on Windows, etc.

 --David


RE: [Merge] hyperv branch to master

2013-10-31 Thread Devdeep Singh
Thanks Darren. I'll start working on getting it merged into the master. I hope 
to finish it by Friday.

Regards,
Devdeep

-Original Message-
From: Darren Shepherd [mailto:darren.s.sheph...@gmail.com] 
Sent: Tuesday, October 29, 2013 10:45 PM
To: dev@cloudstack.apache.org
Cc: Donal Lafferty; Rajesh Battala; Anshul Gangwar
Subject: Re: [Merge] hyperv branch to master

+1

When you merge you will get a conflict in the spring configuration.
componentContext.xml and others are gone now in master.  You will need to 
update to the new style.  The easiest will be to refer to XenServer spring 
config for an example of how to do it.  You can largely copy and paste the 
contents of plugins/hypervisors/xen/resources/META-INF/cloudstack and replace 
xenserver with hyperv.  You can also refer to 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Plug-ins%2C+Modules%2C+and+Extensions
for more in depth explaination of the changes.

Darren

On Tue, Oct 29, 2013 at 9:35 AM, David Nalley da...@gnsa.us wrote:
 On Mon, Oct 28, 2013 at 7:41 AM, Devdeep Singh devdeep.si...@citrix.com 
 wrote:
 Hi,

 I would like to merge the support for Hyperv to the master branch. 
 Development for this has been done by Donal, Rajesh, Anshul and I on branch 
 [1]. The feature was proposed for merge earlier [3] but unit tests for 
 hyperv agent code were requested [4].


 I know this has been discussed several times, but what was the 
 decision about making the agent code live in its own repo?
 It's C#, might need to be signed to actually run easily on Windows, etc.

 --David


Re: Review Request 13735: CLOUDSTACK-4450: Possibility of /tmp/xapilog filling up the Root disk on Xenserver.

2013-10-28 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13735/#review27596
---

Ship it!


Ship It!

- Devdeep Singh


On Aug. 22, 2013, 3:45 p.m., Sanjay Tripathi wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/13735/
 ---
 
 (Updated Aug. 22, 2013, 3:45 p.m.)
 
 
 Review request for cloudstack, Abhinandan Prateek and Devdeep Singh.
 
 
 Bugs: CLOUDSTACK-4450
 https://issues.apache.org/jira/browse/CLOUDSTACK-4450
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 CLOUDSTACK-4450: Possibility of /tmp/xapilog filling up the Root disk on 
 Xenserver.
 
 https://issues.apache.org/jira/browse/CLOUDSTACK-4450
 
 
 Diffs
 -
 
   scripts/vm/hypervisor/xenserver/hostvmstats.py 38609b1 
 
 Diff: https://reviews.apache.org/r/13735/diff/
 
 
 Testing
 ---
 
 Verified the fix locally on cloudstack setup with 4.2-forward branch code and 
 Xenserver 6.2.
 
 
 Thanks,
 
 Sanjay Tripathi
 




[Merge] hyperv branch to master

2013-10-28 Thread Devdeep Singh
Hi,

I would like to merge the support for Hyperv to the master branch. Development 
for this has been done by Donal, Rajesh, Anshul and I on branch [1]. The 
feature was proposed for merge earlier [3] but unit tests for hyperv agent code 
were requested [4].

Checklist:
The development for the feature was earlier done in branch [2] but later moved 
to [1].
Jira ticket for the feature is here [5].
The FS can be found at [6] and [7].
Unit tests for the feature are available at [8]

This link [7] also provides details of third party libraries used by the plugin 
including their licenses. No source for these libraries is used, and the 
binaries are downloaded from their distributors at build time. No proprietary 
tools are required for the build.  For instance, C# compiled with Mono has been 
tested. Mono setup details can be found here [9].

[1] 
https://git-wip-us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=shortlog;h=refs/heads/hyperv
[2] https://github.com/lafferty/cloudstack/tree/hyperv_plugin
[3] 
http://markmail.org/message/cm7kdwh6l2ea772e?q=list:org%2Eapache%2Eincubator%2Ecloudstack-%2A+Minimal+Hyperv+plugin
[4] 
http://markmail.org/message/vxknappvyovhmeyp?q=list:org%2Eapache%2Eincubator%2Ecloudstack-%2A+Minimal+Hyperv+pluginpage=3
[5] https://issues.apache.org/jira/browse/CLOUDSTACK-999
[6] 
https://cwiki.apache.org/confluence/display/CLOUDSTACK/Hyper-V+2012+%283.0%29+Support
[7] https://cwiki.apache.org/confluence/display/CLOUDSTACK/Progress
[8] 
https://git-wip-us.apache.org/repos/asf/cloudstack/repo?p=cloudstack.git;a=blob;f=plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/HypervResourceController1Test.cs;h=ec91e9ff21742bde78873ed7cee49f2423e425b9;hb=refs/heads/hyperv
[9] 
http://dlafferty.blogspot.co.uk/2013/08/building-your-microsoft-solution-with.html

Regards,
Devdeep




Re: Review Request 14827: hyperv unit tests

2013-10-27 Thread Devdeep Singh

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14827/#review27588
---

Ship it!


Ship It!

- Devdeep Singh


On Oct. 23, 2013, 11:05 a.m., Anshul Gangwar wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/14827/
 ---
 
 (Updated Oct. 23, 2013, 11:05 a.m.)
 
 
 Review request for cloudstack, Chiradeep Vittal, Devdeep Singh, Donal 
 Lafferty, and Rajesh Battala.
 
 
 Repository: cloudstack-git
 
 
 Description
 ---
 
 Hyperv Unit tests written using NSubstitute and XUnit. Description for this 
 can be found at 
 
 http://nsubstitute.github.io/help/getting-started/
 http://xunit.codeplex.com/wikipage?title=HowToUse
 http://xunit.codeplex.com/wikipage?title=Comparisons
 
 Currently packages have to be manually copied for Linux. buildagent script is 
 not downloading them.
 
 I have also changed the existing tests to xunit 
 
 
 Diffs
 -
 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.gitignore cf9cb85 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.Config 
 PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/.nuget/NuGet.targets 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.Designer.cs
  a73e6bb 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentSettings.settings
  435b8e0 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/AgentShell.csproj 
 fe055d0 
   plugins/hypervisors/hyperv/DotNet/ServerResource/AgentShell/packages.config 
 f5f47e6 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResource.csproj
  dbd7b15 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/HypervResourceController.cs
  7a0c2db 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCalls.cs 
 PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/IWmiCallsV2.cs
  PRE-CREATION 
   plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCalls.cs 
 1b9e073 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/WmiCallsV2.cs 
 7557320 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/HypervResource/packages.config
  b0f2ace 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/App.config
  1bf17d4 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/HypervResourceController1Test.cs
  PRE-CREATION 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/HypervResourceControllerTest.cs
  8a86727 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/ServerResource.Tests.csproj
  381245e 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/ServerResource.Tests/packages.config
  08ef691 
   
 plugins/hypervisors/hyperv/DotNet/ServerResource/WmiWrappers/WmiWrappers.csproj
  d3baab4 
   plugins/hypervisors/hyperv/buildagent.sh f2a4921 
   
 plugins/hypervisors/hyperv/var/test/storagepool/TestCopiedLocalTemplate.vhdx 
 PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/14827/diff/
 
 
 Testing
 ---
 
 
 Thanks,
 
 Anshul Gangwar
 




RE: [ANNOUNCE] New PMC member: Animesh Chaturvedi

2013-10-21 Thread Devdeep Singh
Congrats Animesh.

Regards,
Devdeep

-Original Message-
From: chip.child...@sungard.com [mailto:chip.child...@sungard.com] On Behalf Of 
Chip Childers
Sent: Tuesday, October 22, 2013 12:31 AM
To: dev@cloudstack.apache.org
Subject: [ANNOUNCE] New PMC member: Animesh Chaturvedi

The Project Management Committee (PMC) for Apache CloudStack has asked Animesh 
Chaturvedi to join the PMC and we are pleased to announce that they have 
accepted.

Join me in congratulating Animesh!

-The CloudStack PMC


  1   2   3   >