[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

2014-11-17 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/41#discussion_r20489860 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -725,6 +725,21 @@ private String validateConfigurationValue(String nam

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

2014-11-17 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/41#discussion_r20489652 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -725,6 +725,21 @@ private String validateConfigurationValue(String nam

[GitHub] cloudstack pull request: CLOUDSTACK-7932: Fixed wrong semantics fo...

2014-11-17 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/39#issuecomment-63432886 No. Because agent is reporting status of host. We cannot determine the status of VMs from that. --- If your project is set up for it, you can reply to this email a

Re: Multiple local storage pools?

2014-11-17 Thread Marcus
Not directly, but one way you could make this happen would be to create a 'sharedmountpoint' primary storage, and use tags to ensure that it doesn't try to use that storage on any host other than the desired one. Sharedmountpoint is basically telling CS that you have done the work of mounting a clu

[GitHub] cloudstack pull request: CLOUDSTACK-7932: Fixed wrong semantics fo...

2014-11-17 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/39#issuecomment-63430873 In the second return statement, should it return false since the agent is alive and returned some status? I didnt understand the argument for not adding unitt

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

2014-11-17 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/41#issuecomment-63430386 Can you add unittests for this? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does n

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

2014-11-17 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/41#discussion_r20488404 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -725,6 +725,21 @@ private String validateConfigurationValue(String name,

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

2014-11-17 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/41#discussion_r20488370 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -725,6 +725,21 @@ private String validateConfigurationValue(String name,

[GitHub] cloudstack pull request: CLOUDSTACK-7929: Unhandled exception when...

2014-11-17 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/40#discussion_r20488120 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -3723,6 +3723,10 @@ public NetworkOffering createNetworkOffering(Crea

[GitHub] cloudstack pull request: CLOUDSTACK-7932: Fixed wrong semantics fo...

2014-11-17 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/39#issuecomment-63429460 Actually null has different meaning than false. False means that vm is not alive while null means it cannot be determined whether it is alive or not. This

[GitHub] cloudstack pull request: CLOUDSTACK-7929: Unhandled exception when...

2014-11-17 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/40#discussion_r20487864 --- Diff: server/src/com/cloud/configuration/ConfigurationManagerImpl.java --- @@ -3723,6 +3723,10 @@ public NetworkOffering createNetworkOffering(Create

[GitHub] cloudstack pull request: CLOUDSTACK-7932: Fixed wrong semantics fo...

2014-11-17 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/39#issuecomment-63428603 From the method definition (isVmAlive), looks like what it is doing is right. ideally, we should change the return type from Boolean to boolean in the interface.

Review Request 28161: CLOUDSTACK-7933: test_escalations_instances.py - Fixed test_13_vm_nics for Vmware

2014-11-17 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28161/ --- Review request for cloudstack and SrikanteswaraRao Talluri. Bugs: CLOUDSTACK-79

Review Request 28160: CloudStack-6282: Added event test cases in Ipaddresses

2014-11-17 Thread Monis Majeed
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28160/ --- Review request for cloudstack and Santhosh Edukulla. Bugs: cloudstack-6282

[GitHub] cloudstack pull request: Merge 4.5 to master

2014-11-17 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/42 Merge 4.5 to master You can merge this pull request into a Git repository by running: $ git pull https://github.com/karuturi/cloudstack merge-45-to-master Alternatively you can review and

Re: Multiple local storage pools?

2014-11-17 Thread Andrija Panic
Ok, waiting for someone to answer this :) Sent from Google Nexus 4 On Nov 18, 2014 12:35 AM, "Nux!" wrote: > Andrija, but I want a more straight forward way to use the SSDs. ZFS I'm > sure it's awesome, but it would just complicate my setup. Thanks though. > > -- > Sent from the Delta quadrant u

[GitHub] cloudstack pull request: CLOUDSTACK-7930, CLOUDSTACK-7931: Do not ...

2014-11-17 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/41 CLOUDSTACK-7930, CLOUDSTACK-7931: Do not allow to set invalid values for global settings which are of type integer and float Do not allow to set invalid values for global settings which are o

[GitHub] cloudstack pull request: CLOUDSTACK-7929: Unhandled exception when...

2014-11-17 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/40 CLOUDSTACK-7929: Unhandled exception when setting negative value for throttling rate while creating network offering To fix issue while creating network offering if one specifies negative val

[GitHub] cloudstack pull request: CLOUDSTACK-7932: Fixed wrong semantics fo...

2014-11-17 Thread anshul1886
GitHub user anshul1886 opened a pull request: https://github.com/apache/cloudstack/pull/39 CLOUDSTACK-7932: Fixed wrong semantics for isVmAlive() method in HypervInvestigator Fixed wrong semantics for isVmAlive() method in HypervInvestigator Findbugs will report error on th

Jenkins build is still unstable: simulator-singlerun #647

2014-11-17 Thread jenkins
See

Build failed in Jenkins: build-master-simulator #1150

2014-11-17 Thread jenkins
See -- [...truncated 1135 lines...] [INFO] No sources to compile [INFO] [INFO] --- maven-surefire-plugin:2.12:test (default-test) @ cloud-framework-events --- [INFO] Surefire report director

Jenkins build is still unstable: simulator-singlerun #646

2014-11-17 Thread jenkins
See

Re: Multiple local storage pools?

2014-11-17 Thread Nux!
Andrija, but I want a more straight forward way to use the SSDs. ZFS I'm sure it's awesome, but it would just complicate my setup. Thanks though. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Andrija Panic" > To: dev@cloudstack.apa

Re: Review Request 27305: CLOUDSTACK-7810 : Fixed the script "test_vpc_vm_life_cycle.py"

2014-11-17 Thread Chandan Purushothama
> On Nov. 17, 2014, 10:45 p.m., sangeetha hariharan wrote: > > Ship It! Commit e712ac78d0814b973886ccdbd2c1edd88a0bfbb9 in cloudstack's branch refs/heads/master from Chandan Purushothama [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=e712ac7 ] CLOUDSTACK-7810 : Fixed the script '

Re: Review Request 27305: CLOUDSTACK-7810 : Fixed the script "test_vpc_vm_life_cycle.py"

2014-11-17 Thread sangeetha hariharan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27305/#review61829 --- Ship it! Ship It! - sangeetha hariharan On Nov. 15, 2014, 11:04

Jenkins build is still unstable: simulator-singlerun #645

2014-11-17 Thread jenkins
See

Re: Review Request 28043: CLOUDSTACK 7915: Remove hard-coded values for Load Balancer algorithms in UI

2014-11-17 Thread Brian Federle
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28043/#review61817 --- I am getting conflicts in the properties file applying to the latest

Re: Why a particular commit is found in 4.4 branch but not master and 4.5

2014-11-17 Thread Kuang-Ching Wang
I figured this out - the CLOUDSTACK-5505 header is dropped when the fixes are merged into master and 4.5. Then I don’t see why I am hitting the issue - created a VPC, when applying static NAT it failed to apply, and it is found the VpcVR does not have eth1. The log complains about “ethnull”, w

Build failed in Jenkins: build-4.5 #140

2014-11-17 Thread jenkins
See Changes: [jessicawang] CLOUDSTACK-7927: UI > Infrastructure > Primary Storage > detailView > add "View Volumes" link that will list all volumes under this primary storage when being clicked.

Why a particular commit is found in 4.4 branch but not master and 4.5

2014-11-17 Thread Kuang-Ching Wang
Hi, I am looking at an issue fixed a while back - https://issues.apache.org/jira/browse/CLOUDSTACK-5505 I am hitting the same issue with the 4.5 branch code, and after searching around found that the fix was only found in branch 4.4, not master, not 4.5 What does this mean? The commit is only

Re: Wasting Time !

2014-11-17 Thread Andrija Panic
Gents - my point of view on the cloudstack and the work arround it is the folowing: - Pretty lacking documentation - Working on new fancy features - instead of making current features work at all... - Few examples: -- new vxlan isolation method was introduced some release ago - default MTU of 14

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Andrija Panic
Upgrade fine for me - no issues at all... - from the RPMs snapshots On 17 November 2014 14:36, Daan Hoogland wrote: > On Mon, Nov 17, 2014 at 2:12 PM, Pierre-Luc Dion > wrote: > > Trying devcloud4 with 4.4 branch and I'm having this issue: > > > I don't see the schema-441to442.sql files in the

Re: Multiple local storage pools?

2014-11-17 Thread Andrija Panic
Lucian, don't relly have the answer - but did you maybe consider ZFS on Linux, mounted on /var/lib/libvirt/images/ - there you can add SSD for writings and perhaps read cache (SLOG for writes and L2ARC for readings) ? On 17 November 2014 19:37, Nux! wrote: > Hi guys, > > Do you know how it would

Multiple local storage pools?

2014-11-17 Thread Nux!
Hi guys, Do you know how it would be possible to add multiple local primary storage pools? I know that if local storage is enabled /var/lib/libvirt/images (in KVM's case) is added automatically. But assuming I want to add an SSD or SATA array to this host and want to differentiate it within clo

Re: Review Request 28043: CLOUDSTACK 7915: Remove hard-coded values for Load Balancer algorithms in UI

2014-11-17 Thread Daniel Vega Simoes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28043/ --- (Updated Nov. 17, 2014, 6:26 p.m.) Review request for cloudstack. Changes ---

Jenkins build became unstable: simulator-singlerun #644

2014-11-17 Thread jenkins
See

Jenkins build is back to stable : simulator-singlerun #643

2014-11-17 Thread jenkins
See

Jenkins build is still unstable: simulator-singlerun #642

2014-11-17 Thread jenkins
See

Re: [TEST] DevCloud4 - rework / Cleanup

2014-11-17 Thread Ian Duffy
Hi Pierre, No that is correct. We cannot control the .1 interface which is on the host machine. We can however control .5 and get it to do forwarding On 17 Nov 2014 12:28, "Pierre-Luc Dion" wrote: > Hi Ian, > > I'm trying to run the binary-advanced, in the Public IPs of the zone the > gateway is

New Defects reported by Coverity Scan for cloudstack

2014-11-17 Thread scan-admin
Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. 11 new defect(s) introduced to cloudstack found with Coverity Scan. 18 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defec

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Daan Hoogland
On Mon, Nov 17, 2014 at 2:12 PM, Pierre-Luc Dion wrote: > Trying devcloud4 with 4.4 branch and I'm having this issue: > I don't see the schema-441to442.sql files in the db folder too. > > does 442 is on a different branch? no, I added the file locallly and forgot to commit. it is pushed now, so

Jenkins build became unstable: simulator-singlerun #641

2014-11-17 Thread jenkins
See

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Pierre-Luc Dion
Trying devcloud4 with 4.4 branch and I'm having this issue: [WARNING] java.lang.reflect.InvocationTargetException at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57) at sun.reflect.DelegatingMethodAccessor

Re: [TEST] DevCloud4 - rework / Cleanup

2014-11-17 Thread Pierre-Luc Dion
Hi Ian, I'm trying to run the binary-advanced, in the Public IPs of the zone the gateway is 192.168.23.5, should my box have the IP 192.168.23.5 instead of the current 192.168.23.1 ? thanks, On Sun, Nov 16, 2014 at 2:57 PM, Ian Duffy wrote: > Hi Pierre, > > So advanced and basic are running

Re: Show volumes - showing another VM's volume

2014-11-17 Thread Andrija Panic
Created improvement ticket - to consider implementing Volume purging, just like with VMs...so they are not deleted immediatey - so there is time to recover the volume... https://issues.apache.org/jira/browse/CLOUDSTACK-7926 On 12 November 2014 14:48, Logan Barfield wrote: > I've noticed this

Jenkins build is back to stable : simulator-singlerun #640

2014-11-17 Thread jenkins
See

[GitHub] cloudstack pull request: CLOUDSTACK-7541: Added restriction to not...

2014-11-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/28 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] cloudstack pull request: CLOUDSTACK-7758: Fixed although api calls...

2014-11-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/29 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[GitHub] cloudstack pull request: CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed d...

2014-11-17 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/30#issuecomment-63287919 closing this PR as patch got committed --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your proje

[GitHub] cloudstack pull request: CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed d...

2014-11-17 Thread anshul1886
Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/30 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] cloudstack pull request: CLOUDSTACK-7620: Added SNMP MIB file for ...

2014-11-17 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/31#issuecomment-63287776 closing pull request as this got merged to master --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: CLOUDSTACK-7620: Added SNMP MIB file for ...

2014-11-17 Thread anshul1886
Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/31 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Daan Hoogland
I think, till we o longer find bugs that bug us. Not sure, but as far as I know there is no bug in it now. I just don't want to go into a 72 hour vote period now untill I get some support for the release. On Mon, Nov 17, 2014 at 10:01 AM, Nux! wrote: > Cheers, will do. Till when do we have to do

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Daan Hoogland
So we will have to wait for you with starting;) On Mon, Nov 17, 2014 at 10:38 AM, Erik Weber wrote: > On Mon, Nov 17, 2014 at 9:56 AM, Daan Hoogland > wrote: > >> People, >> >> I want to make a new 4.4 release after not delivering the quality I >> would like for 4.4.0 and 4.4.1. I want to call o

Jenkins build is back to normal : build-4.5 #139

2014-11-17 Thread jenkins
See

[GitHub] cloudstack pull request: CLOUDSTACK-7703, CLOUDSTACK-7752: Fixed d...

2014-11-17 Thread rajesh-battala
Github user rajesh-battala commented on the pull request: https://github.com/apache/cloudstack/pull/30#issuecomment-63281107 merging this patch to Master as it had passed CI as well --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

Re: Review Request 28039: CLOUDSTACK-7912: Remove hardcoded netscaler info and read it from config file

2014-11-17 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28039/#review61707 --- Ship it! 5f99917991a59f8ecd6d8b0e17b497fe210e636e 4.5 and master -

[GitHub] cloudstack pull request: CLOUDSTACK-7620: Added SNMP MIB file for ...

2014-11-17 Thread rajesh-battala
Github user rajesh-battala commented on the pull request: https://github.com/apache/cloudstack/pull/31#issuecomment-63280334 Merging the commit to master as it had passed CI build --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

Re: Review Request 28032: CLOUDSTACK-7913 - Added reconnect functionality to Host class in base.py

2014-11-17 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28032/#review61706 --- Ship it! 19781e094b987cf65d05d890cd3cd86fc22cb873 4.5 and master -

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Erik Weber
On Mon, Nov 17, 2014 at 9:56 AM, Daan Hoogland wrote: > People, > > I want to make a new 4.4 release after not delivering the quality I > would like for 4.4.0 and 4.4.1. I want to call on all to test the head > of the 44 branch before I do. Both two prior releases have known > issues within one w

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Andrija Panic
Hi Dan, would you mind provide 4.4.2 RPMs somewhere so we can test the upgrade from 4.4.1 ? best On 17 November 2014 09:56, Daan Hoogland wrote: > People, > > I want to make a new 4.4 release after not delivering the quality I > would like for 4.4.0 and 4.4.1. I want to call on all to test the

Re: [ACS44] release 4.4.2 imminent

2014-11-17 Thread Nux!
Cheers, will do. Till when do we have to do this? Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Daan Hoogland" > To: "dev" , us...@cloudstack.apache.org > Sent: Monday, 17 November, 2014 08:56:22 > Subject: [ACS44] release 4

Re: did anyone try 4.4.2?

2014-11-17 Thread Daan Hoogland
That one is fixed (i hope) On Mon, Nov 17, 2014 at 3:52 AM, Pierre-Luc Dion wrote: > I'm trying to build 4.4.2 from 4.4 branch. I'm having the db deployment > error that schema-441to442.sql does not exist. Could it be the cause of the > problem you are seeing too ? > > On Fri, Nov 14, 2014 at 8:5

[ACS44] release 4.4.2 imminent

2014-11-17 Thread Daan Hoogland
People, I want to make a new 4.4 release after not delivering the quality I would like for 4.4.0 and 4.4.1. I want to call on all to test the head of the 44 branch before I do. Both two prior releases have known issues within one week after release. In my opinion it would be great if we can preven

Re: [QUESTION] How come we don't include Users@ in vote threads?

2014-11-17 Thread Chip Childers
On Sun, Nov 16, 2014 at 11:44 PM, Pierre-Luc Dion wrote: > I agree that more tests are welcome, we have to try then :) > >> Afaik non-pmc's aren't binding and any votes would merely be an > indication, >> or do i misunderstand? > > All votes are important [1] and count as indicator, who ever vote