Re: Review Request 23410: Improvised CloudByte Storage Plugin for 4.5 and above. jira id : CLOUDSTACK-7098

2014-07-18 Thread punith s
> On July 18, 2014, 9:36 a.m., Mike Tutkowski wrote: > > Ship It! > > Mike Tutkowski wrote: > Changes were put into ACS repo with this commit: > > f3cbb699955243007ffb53b335d9508b2e422a06 sure thanks mike :) - punith --- T

Re: Review Request 23547: CLOUDSTACK-7113: Ability to reset password when Router VM is stopped

2014-07-18 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23547/#review48115 --- Commit 0069037573247e2cbe545e8d01d609040444e453 in cloudstack's bran

Re: Review Request 22927: Bugfix CLOUDSTACK-6261: remove the forceful timeout setting when login to NetScaler.

2014-07-18 Thread Rajesh Battala
> On July 9, 2014, 4:45 p.m., ASF Subversion and Git Services wrote: > > Commit e07d0ddabfedad137856c3c9ae7de3a96fe25feb in cloudstack's branch > > refs/heads/master from Sujaya Maiyya > > [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e07d0dd ] > > > > fixed CLOUDSTACK-6261: remo

Re: Review Request 22799: Golden (Base) Primary Storage feature

2014-07-18 Thread Hieu LE
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22799/ --- (Updated July 18, 2014, 10:26 a.m.) Review request for cloudstack, Mike Tutkows

[Issue]: Wrong status of System VM after destroy them

2014-07-18 Thread huangchunmei
Hi Alex, For cloudStack 4.3, I found the status of SSVM and CPVM are always in Destroyed status after destroyed them. This would lead to an issue that zone could be deleted if there are VMs with Destroyed status. Would you please help to check why we change the status to "Expunging", then upda

Re: Review Request 23617: Add Nic UUID to the context so that we can read the same in event bus after create a nic

2014-07-18 Thread Damodar Reddy Talakanti
> On July 17, 2014, 4:55 p.m., Nitin Mehta wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 960 > > > > > > Why is it made create=true when it is not a BaseAyncCreate cmd ? > > create=true should be add

Review Request 23677: CLOUDSTACK-7024: Resolved cleanup issue in test script

2014-07-18 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23677/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-7024 h

Re: What to modify to increase instance password length?

2014-07-18 Thread Harikrishna Patnala
Hi Ian, The commit was made only in 4.4 and not in master. Can you cherry pick the same to master. Thanks, Harikrishna On 26-Jun-2014, at 12:23 am, Ian Duffy wrote: > Just pushed a change for this to the 4.4-forward branch. > > Daan, will you review / cherrypick? 96412e3e58fd1ced9d269e4552aa

[JENKINS] devcloud-continuous-tests

2014-07-18 Thread Hugo Trippaers
Hey Talluri, Do you know what happend to this jenkins slave? The console log tells me that the system was halted? Cheers, Hugo

Re: Review Request 23282: CLOUDSTACK-6845 : First Code drop for NuageVsp Network plugin

2014-07-18 Thread Hugo Trippaers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23282/#review48121 --- * Unit tests complete successfully * Findbugs output reports no pro

Re: Review Request 23677: CLOUDSTACK-7024: Resolved cleanup issue in test script

2014-07-18 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23677/#review48122 --- Commit ec49669f1cf525b207889ae13a8a2b01a0271933 in cloudstack's bran

Re: Review Request 23677: CLOUDSTACK-7024: Resolved cleanup issue in test script

2014-07-18 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23677/#review48124 --- Commit 5f87056e950e0cae042d0c836f10a091800b8dbc in cloudstack's bran

RE: [PROPOSAL] Adding a plugin to check the password strength of all users

2014-07-18 Thread Demetrius Tsitrelis
Will the plugin merely show the strength of the password or will the plugin prevent the use of weak passwords? From: Damoder Reddy [damoder.re...@citrix.com] Sent: Thursday, July 17, 2014 11:02 PM To: dev@cloudstack.apache.org Subject: [PROPOSAL] Adding a

Review Request 23679: CLOUDSTACK-7130: Tagging failed test cases with product bug Id

2014-07-18 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23679/ --- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-7130 h

rfc: newsystemvm

2014-07-18 Thread Leo Simons
Hey folks, https://github.com/schubergphilis/cloudstack/commit/f125f1564e8921def00dc0235ecca51470a2a22e https://github.com/schubergphilis/cloudstack/tree/f125f1564e8921def00dc0235ecca51470a2a22e/tools/appliance This started out as wanting the systemvm build to take systemvm/patches/debian/{debia

Re: Review Request 23679: CLOUDSTACK-7130: Tagging failed test cases with product bug Id

2014-07-18 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23679/#review48125 --- Commit 24da72f37395a6bb612ea1d073db0155289cf000 in cloudstack's bran

Review Request #22863 , #23314 : Brocade Plugin review request...

2014-07-18 Thread Ritu Sabharwal
Hi Hugo, I have fixed the 3 items as proposed by you: 1. Added the unit tests 2. Added integration test 3. Removed the configuration from properties file to networks service provider configuration using APIs or GUI. Can you please review the plugin code so that we can make i

Re: vms stopped while restarted by user

2014-07-18 Thread Tomasz Zięba
Hello, After analyzing the code, I am convinced that 10 minutes is indeed associated with the parameters that gave Daan, and specifically in: class: HighAvailabilityManagerImpl.java function: protected Long stopVM(final HaWorkVO work) throws ConcurrentOperationException .. work.setTimesTried

Re: [PROPOSAL] Adding a plugin to check the password strength of all users

2014-07-18 Thread Damoder Reddy
Will show the strength of the password as well. On 18-Jul-2014, at 6:53 pm, Demetrius Tsitrelis wrote: > Will the plugin merely show the strength of the password or will the plugin > prevent the use of weak passwords? > > > From: Damoder Reddy [damode

RE: [PROPOSAL] Adding a plugin to check the password strength of all users

2014-07-18 Thread Demetrius Tsitrelis
So the plugin will show the strength AND it will enforce the strength when a user is created or updates his password. Will it be possible for an administrator to disable either of these? For both of those capabilities is the plugin's behavior configurable for different authentication encoders?

Re: rfc: newsystemvm

2014-07-18 Thread Chiradeep Vittal
Thanks Leo. Can you summarize the changes (it is a lot of changes) From: Leo Simons mailto:lsim...@schubergphilis.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Friday, July 18, 2014 at 7:42 AM To: int-toolkit mailto:int-too

Re: Infiniband support

2014-07-18 Thread Chiradeep Vittal
Is this for storage (presumably)? You can use a “pre-setup” storage From: Dan Crowe mailto:dan.cr...@shapeblue.com>> Reply-To: "dev@cloudstack.apache.org" mailto:dev@cloudstack.apache.org>> Date: Wednesday, July 16, 2014 at 3:49 AM To: "dev@cloudstack.apache.org<

Re: Infiniband support

2014-07-18 Thread ilya musayev
Hi Dan In what sense do you see support for infiniband and cloudstack? CloudStack manages pre-set infrastructure. Infiniband is usually something that you setup initially, and then hopefully you never have to touch it again - until new addition comes thru. Regards ilya On 7/16/14, 3:49 AM, D

Re: Review Request 23617: Add Nic UUID to the context so that we can read the same in event bus after create a nic

2014-07-18 Thread Nitin Mehta
> On July 17, 2014, 4:55 p.m., Nitin Mehta wrote: > > server/src/com/cloud/vm/UserVmManagerImpl.java, line 960 > > > > > > Why is it made create=true when it is not a BaseAyncCreate cmd ? > > create=true should be add

Daily CI test summary 07/18/14

2014-07-18 Thread Rayees Namathponnan
Hi All, We enabled KVM in our CI environment, you can see the current status at https://cwiki.apache.org/confluence/x/RAGTAg Simulator 100% pass KVM 9 failures Xen 2 failures There is no new issue reported with lastets build, below falures are known issues and details are below S

Re: Review Request 23282: CLOUDSTACK-6845 : First Code drop for NuageVsp Network plugin

2014-07-18 Thread Suresh Ramamurthy
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23282/ --- (Updated July 19, 2014, 12:49 a.m.) Review request for cloudstack, Alena Prokha