test_nicira_controller.py is failing on all Advzone - XS, KVM

2015-09-30 Thread Raja Pullela
Hi Miguel or someone's familiar with this test, Can you please provide documentation around how to get these tests running ? BTW - looks like this is a new test that was added on Aug 25th 2015. Thanks, Raja === TestName: None | Status : EXCEPTION === ERROR ==

Re: BVT report 9/29

2015-09-30 Thread Milamber
The Advanced Network with SG don't works with me (perhaps mistake in my setup). The error on wizard final step is: string '' has an unknown BroadcastDomainType Perhaps need a VLAN ID? (mandatory?) Extract of log: 2015-09-30 06:53:01,070 DEBUG [c.c.a.ApiServlet] (catalina-exec-5:ctx-85a55cd

RE: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Koushik Das
Raja, The earlier scale vm test couldn't run on simulator because of the validation steps (IIRC it tries to connect to the real VM created during test). Since it wasn't possible to run with simulator, the new test was added to run just on the simulator. Check the required_hardware tag. -Ori

Re: BVT report 9/29

2015-09-30 Thread Milamber
The advanced network installation without SG seems works, but the final screen of setup wizard stills blank until the activation zone pop-up. A lot of error in the browser console. I need make more test (clean my browser cache before setup) See screenshot: http://s27.postimg.org/qr7pijre

Re: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Sebastien Goasguen
> On Sep 30, 2015, at 9:10 AM, Koushik Das wrote: > > Raja, > The earlier scale vm test couldn't run on simulator because of the validation > steps (IIRC it tries to connect to the real VM created during test). Since it > wasn't possible to run with simulator, the new test was added to run jus

[GitHub] cloudstack pull request: CLOUDSTACK-89027 Restart Network fails in...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/898#issuecomment-144314489 In issues like this, we should track which automated test fails. If no test can check for that, then we need to write one and submit it in the PR. BTW: Th

[GitHub] cloudstack pull request: CLOUDSTACK-8879: Depend in rados-java 0.2...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/889#issuecomment-144315290 LGTM, small change. @wido what tests can be run to check for this. --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: jenkins disk full ?

2015-09-30 Thread David Nalley
The below is before I cleaned up a bit. We now have 3.2GB free. [root@domU-12-31-39-15-22-2D jobs]# du -h -d 1 1.3G ./package-rhel63-master 200M ./cloudstack-4.3-maven-build-noredist 638M ./package-rhel63-4.5 1.1G ./cloudstack-4.4-systemvm64 1.3M ./build-4.5 8.0K ./cloudstack-master-with-patch 186

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8880: calculate free m...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/847#issuecomment-144315690 Hi @kishankavala if you can answer @remibergsma and @borisroman questions, we can move forward with your PR. --- If your project is set up for it, you can reply to t

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8882: calculate networ...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/859#issuecomment-144316351 Hi @kishankavala can we run some simulator tests to check this ? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHu

[GitHub] cloudstack pull request: Cloudstack-8885 added blocked connection ...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/857#issuecomment-144317611 @remibergsma that one is tough to test as @karuturi mentions. so I am with @wido here +1 on code review. Small enough. We need to think further on how to handle co

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/861#issuecomment-144320963 @harikrishna-patnala and @koushik-das I agree with @remibergsma please advise on how you want to proceed, considering #883 reimplements the 602 resource. --- If you

Re: CloudStack Ceph S3 Secondary Storage on KVM

2015-09-30 Thread Wido den Hollander
On 09/30/2015 02:44 AM, ilya musayev wrote: > Hi Wido, > > Hope your day is going well. > > Would you know if CloudStack Ceph S3 Secondary Storage on KVM requires > NFS backing or can it be bypassed? > Yes, it still requires a staging NFS secondary store. Wido > Thanks > ilya >

Re: BVT report 9/29

2015-09-30 Thread Miguel Ferreira
Hi Sebastian, On 29 Sep 2015, at 20:42, Sebastien Goasguen mailto:run...@gmail.com>> wrote: @miguel, looks like smoke/test_scale_vm is reporting errors. It seems you were the last one to touch that test. Could you try to run the tests on some cloudstack setup you have and report back to use yo

[GitHub] cloudstack pull request: CLOUDSTACK-8879: Depend in rados-java 0.2...

2015-09-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/889#issuecomment-144322421 @runseb No, not really. All my Unit tests are succeeding on rados-java itself. It should work as expected. But I have no in CloudStack test --- If your project is set

[GitHub] cloudstack pull request: CLOUDSTACK-8893: Fixing script as per the...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/871#issuecomment-144322464 @pavanb018 it always helps other folks reviewing to know what you did. For example. Did you just check the Travis green light ? Did you just check the code throu

RE: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Raja Pullela
Sebastien, any BVT tests failing is a blocker and needs to be fixed - which is the reason for Blocker. We already have a fix. Koushik, modified the test "test_01_scale_vm" with required_hardware=false on simulator setup and it works. So, I think we can let the second method go. I will al

[GitHub] cloudstack pull request: CLOUDSTACK-8894: Restrict vGPU enabled VM...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/868#issuecomment-144322859 @anshul1886 Can you add a bit of a description here. Is that fixing a bug, is that a new feature ? Can you add Travis tests to check this ? many thanks --- I

RE: BVT report 9/29

2015-09-30 Thread Raja Pullela
Miguel, please see the defect https://issues.apache.org/jira/browse/CLOUDSTACK-8924 Let me know if you have any questions, Raja -Original Message- From: Miguel Ferreira [mailto:mferre...@schubergphilis.com] Sent: Wednesday, September 30, 2015 1:43 PM To: Sebastien Goasguen Cc: dev ; Bor

Re: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Sebastien Goasguen
> On Sep 30, 2015, at 10:15 AM, Raja Pullela wrote: > > Sebastien, any BVT tests failing is a blocker and needs to be fixed - which > is the reason for Blocker. We already have a fix. > Right, but that’s a fix in the test itself, no an issue with the cloudstack “core”. In any case, grea

Build failed in Jenkins: build-master-jdk18 #358

2015-09-30 Thread jenkins
See Changes: [pavanbandarupally] Automation for multiple disk attachments to instance [pavanbandarupally] Imparting changes mentioned by nitt10prashant [pavanbandarupally] changed the testcase skip code into setup method [jaya

Re: jenkins disk full ?

2015-09-30 Thread Rajani Karuturi
Thanks david. Do you know if anyone else has access to this? if not, can you give access to PL please? ~Rajani On 30-Sep-2015, at 1:13 pm, David Nalley wrote: > The below is before I cleaned up a bit. We now have 3.2GB free. > > [root@domU-12-31-39-15-22-2D jobs]# du -h -d 1 > 1.3G ./packag

RE: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Raja Pullela
Yup... btw, I am on top of this and goal is to get to 100% passrate on BVTs! -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Wednesday, September 30, 2015 1:48 PM To: dev@cloudstack.apache.org Subject: Re: [Blocker] test duplicated in test_scale_vm.py > On

RE: jenkins disk full ?

2015-09-30 Thread Raja Pullela
Yes, please give access to PL or couple of other folks - Rajani or other PMC members? -Original Message- From: Rajani Karuturi [mailto:rajani.karut...@citrix.com] Sent: Wednesday, September 30, 2015 1:51 PM To: dev@cloudstack.apache.org Cc: Pierre-Luc Dion ; tall...@apache.org Subject: R

Build failed in Jenkins: build-4.5 #531

2015-09-30 Thread jenkins
See Changes: [Pierre-Luc Dion] Fix dockerbuild because hardcoded filename for marvin -- [...truncated 3296 lines...] [INFO] Compiling 44 source files to

Re: jenkins disk full ?

2015-09-30 Thread David Nalley
Right now these folks have access: ewanm:x:1002:1002::/home/ewanm:/bin/bash prasanna:x:1003:1003::/home/prasanna:/bin/bash salvatore:x:1004:1004::/home/salvatore:/bin/bash edison:x:1005:1005::/home/edison:/bin/bash pradeep:x:1006:1006::/home/pradeep:/bin/bash rayeesn:x:1007:1007::/home/rayeesn:/bi

Re: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Sebastien Goasguen
> On Sep 30, 2015, at 10:28 AM, Raja Pullela wrote: > > Yup... btw, I am on top of this and goal is to get to 100% passrate on BVTs! > Yes I got it, and we all have the same goal. It does appear that not all smoke tests are run by Travis, otherwise we would catch any errors on simulator fo

[GitHub] cloudstack pull request: CLOUDSTACK-8894: Restrict vGPU enabled VM...

2015-09-30 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/868#issuecomment-144328947 @runseb Updated the bug description. I am looking into travis tests. This test will require vGPU enabled hosts with different type of GPU cards. --- If your proj

RE: jenkins disk full ?

2015-09-30 Thread Raja Pullela
Btw, edison, pradeep, rayeesn - are not active anymore - talluri may be? I think. -Original Message- From: David Nalley [mailto:da...@gnsa.us] Sent: Wednesday, September 30, 2015 2:01 PM To: dev@cloudstack.apache.org Cc: Pierre-Luc Dion ; tall...@apache.org Subject: Re: jenkins disk f

Re: OVS Plugin documentation issues

2015-09-30 Thread Sebastien Goasguen
> On Sep 29, 2015, at 2:40 PM, Ron Wheeler > wrote: > > Don't mind making the changes. > I have a lot of SVN experience and have the git repo for Apache OFBiz > configured but am still trying to figure out git. > What do I have to setup in Eclipse to access the "right" bits on git. > > Seems

[GitHub] cloudstack pull request: CLOUDSTACK-8895: Verify if storage on sto...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/869#issuecomment-144331549 @pritisarap12 can this be run via simulator ? --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8896: allocated percentage of ...

2015-09-30 Thread runseb
Github user runseb commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/873#discussion_r40771703 --- Diff: server/src/com/cloud/storage/StorageManagerImpl.java --- @@ -1736,7 +1737,10 @@ public boolean storagePoolHasEnoughSpace(List volumes, StoragePoo

[GitHub] cloudstack pull request: CLOUDSTACK-8897: baremetal:addHost:make h...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/874#issuecomment-144332355 We won't be able to test this on simulator. @harikrishna-patnala can you answer @borisroman and then we can merge. LGTM +1 on code review alone --- If your pr

[GitHub] cloudstack pull request: Changes made to DeployDataCenter to suppo...

2015-09-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/809#issuecomment-144332699 @sanju1010 makes sense and then of course a test report and -description is needed. --- If your project is set up for it, you can reply to this email and have

RE: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Raja Pullela
The problem needs to be fixed - "have tested a solution that was discussed with koushik" - we will have to create PR with that change. Once the PR is pushed - we will have one less blocker/test failure. I agree this is not a cloudstack core code problem - but we should NOT make that a criter

[GitHub] cloudstack pull request: CLOUDSTACK-8847: ListServiceOfferings is ...

2015-09-30 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/823#issuecomment-144333001 Hi @remibergsma, I rebased my commit against the current master. I added unit test for the change. Thanks, Nitin --- If your project is

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-30 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/885#discussion_r40772428 --- Diff: engine/schema/src/com/cloud/vm/dao/VMInstanceDaoImpl.java --- @@ -805,6 +805,12 @@ public Boolean doInTransaction(TransactionStatus status)

[GitHub] cloudstack pull request: CLOUDSTACK-8913: Search box in Templates ...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/891#issuecomment-144333775 @nitin-maharana LGTM +1 based on code review. Maybe adding two snapshots to compare would help review (just a thought) --- If your project is set up for it, you c

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-144334476 I think this will work but it is a fix on a broken state machine. The statemachine expects a power report every so and so time-interval and when it doesn't come

Re: BVT report 9/29

2015-09-30 Thread Miguel Ferreira
All, Following the instructions I had posted in the original PR (https://github.com/apache/cloudstack/pull/741#issuecomment-135415998) I can reproduce the same result. That is, the test passes against the simulator with an Advanced zone: [root@cs1 cloudstack]# nosetests --with-marvin --marvin-

Re: BVT report 9/29

2015-09-30 Thread Sebastien Goasguen
Thanks a lot Miguel, > On Sep 30, 2015, at 11:15 AM, Miguel Ferreira > wrote: > > All, > > Following the instructions I had posted in the original PR > (https://github.com/apache/cloudstack/pull/741#issuecomment-135415998) I can > reproduce the same result. That is, the test passes against

Re: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Rajani Karuturi
I agree with Raja. BVT test case failures should be treated as blockers. Once the test issue is fixed, they might uncover new issues/regressions. ~Rajani On 30-Sep-2015, at 2:32 pm, Raja Pullela wrote: > The problem needs to be fixed - "have tested a solution that was discussed > with koush

Re: test_nicira_controller.py is failing on all Advzone - XS, KVM

2015-09-30 Thread Miguel Ferreira
Hi Raja, That test needs a NSX cluster. With that in place, the configuration used to run the test must define a section called NiciraNvp, and that section should have an array os hosts with that name (see line 46 of the test). In a Marvin config that materializes in something like this: "ni

Re: jenkins disk full ?

2015-09-30 Thread Daan Hoogland
H David, I just tried and it works indeed. Forgot about that. Does it make sense to have a script containing du mail to dev@ if something like /var/lib/jenkins/workspace or / exceeds a threshold? (yes me volunteering) On Wed, Sep 30, 2015 at 10:48 AM, Raja Pullela wrote: > Btw, edison, pradeep,

Re: jenkins disk full ?

2015-09-30 Thread David Nalley
SGTM, Feel free to. On Wed, Sep 30, 2015 at 5:34 AM, Daan Hoogland wrote: > H David, I just tried and it works indeed. Forgot about that. Does it make > sense to have a script containing du mail to dev@ if something like > /var/lib/jenkins/workspace or / exceeds a threshold? (yes me volunteerin

RE: BVT report 9/29

2015-09-30 Thread Raja Pullela
thanks Miguel for confirming! I will get the PR to close this out. -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: Wednesday, September 30, 2015 2:57 PM To: Miguel Ferreira Cc: dev@cloudstack.apache.org Subject: Re: BVT report 9/29 Thanks a lot Miguel, >

Re: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Daan Hoogland
I disagree, and let me carefully try to formulate why: BVT is useful and 'should' be run before any thing is declared good. The problem is the ability for the community at large to run it and to fix it for any kind of set-up. It should not be automatically treated as blockers but only if a full ana

[GitHub] cloudstack pull request: CLOUDSTACK-8894: Restrict vGPU enabled VM...

2015-09-30 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/868#issuecomment-144341943 @runseb It seems like simulator does't have support for vGPU. If I find some time then I will try to add support for it. --- If your project is set up for it, yo

[GitHub] cloudstack pull request: CLOUDSTACK-8911: VM start job got stuck i...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/895#issuecomment-144343011 LGTM. Have verified the scenario for max. guest limit. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/900 CLOUDSTACK-8924: Removed duplicate test from test_scale_vm.py Please go through CS-8924 for more details. You can merge this pull request into a Git repository by running: $ git pull http

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144344567 root@localhost:~/cloudstack# nosetests --with-marvin --marvin-config=/root/cloudstack/setup/dev/local1.cfg --zone=Sandbox-simulator --hypervisor=simulator -a tags=

[GitHub] cloudstack-docs pull request: The help topic created for the DHCP/...

2015-09-30 Thread rajsekharkpally
GitHub user rajsekharkpally opened a pull request: https://github.com/apache/cloudstack-docs/pull/11 The help topic created for the DHCP/DNS offload and Config Drive support feature The help topic that describes the DHCP/DNS offload and Config drive support feature has been added (

[GitHub] cloudstack pull request: Cloudstack-8885 added blocked connection ...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/857#issuecomment-144345855 @runseb In a prefect world, one would still run the tests against it to verify they still all pass. Maybe we can execute the basic VM life cycle test (smoke/test

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread pvr9711
Github user pvr9711 commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144346102 LGTM! Thanks Sanjeev! --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8879: Depend in rados-java 0.2...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/889#issuecomment-144346492 @wido maybe @borisroman can execute smoke/test_vm_life_cycle.py to do a quick check? --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: CLOUDSTACK-8879: Depend in rados-java 0.2...

2015-09-30 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/889#issuecomment-144346818 Will do. Best regards, Boris Schrijver TEL: +31633784542 MAIL: bo...@pcextreme.nl > On September 30, 2015 at 12:11

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-30 Thread resmo
Github user resmo commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-144346880 well I need it in 4.5.3. I would suggest we take this fix for now for 4.6 as well and make a proper refactor for 4.7/5.0. --- If your project is set up for it, you ca

[GitHub] cloudstack pull request: CLOUDSTACK-8879: Depend in rados-java 0.2...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/889#issuecomment-144346920 @borisroman thank you! --- 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 h

[GitHub] cloudstack pull request: CLOUDSTACK-8906: /var/log/cloud/ doesn't ...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/883#issuecomment-144347154 @SudharmaJain Why can't the XenServer600Resource be used to handle XS 6.0.2 hosts? The only change I see is the patch script (scripts/vm/hypervisor/xenserver/xen

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/861#issuecomment-144347634 I have updated #883 with my comments, looking at the changes it shouldn't be required. Lets wait for @SudharmaJain response --- If your project is set up for it

[GitHub] cloudstack pull request: CLOUDSTACK-8808: Successfully registered ...

2015-09-30 Thread karuturi
GitHub user karuturi opened a pull request: https://github.com/apache/cloudstack/pull/901 CLOUDSTACK-8808: Successfully registered VHD template is downloaded again due to missing virtualsize property in template.properties We have multiple file processors to process different types

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Removed duplicate test f...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/900#issuecomment-144348817 LGTM. Since now test_01_scale_vm can be run on simulator, no need for the other one. --- If your project is set up for it, you can reply to this email and have

Re: CloudStack networking documentation

2015-09-30 Thread Sebastien Goasguen
Pinging Paul Angus, who can surely answer these questions. We can then edit the docs. > On Sep 29, 2015, at 7:03 AM, Ron Wheeler > wrote: > > http://docs.cloudstack.apache.org/en/master/concepts.html#what-is-apache-cloudstack > > In the opening paragraph of About Physical Networks" it says > "

Re: OVS documentation

2015-09-30 Thread Sebastien Goasguen
> On Sep 29, 2015, at 2:06 PM, Ron Wheeler > wrote: > > Can someone fix this? > Honestly Ron, the issue is that very few people are actually using the GRE isolation. One would need to test it with the latest cloudstack 4.5.2 and latest OVS to see if it works with newer versions…. -sebasti

Re: jenkins disk full ?

2015-09-30 Thread Daan Hoogland
So I did, expressing my worst operator practices I created a small script: ! /bin/bash THRESHOLD=95 USAGE=`/bin/df -P /var/lib/jenkins/workspace | /bin/grep -v Capacity | /bin/cut -c 47- | /bin/cut -c 6-7 ` function maildev() { /bin/mail -s 'disk usage at jenkins.bac.o' dev@cloudstack.apach

[GitHub] cloudstack pull request: CLOUDSTACK-8913: Search box in Templates ...

2015-09-30 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/891#issuecomment-144361240 Hi @runseb, Added two snapshots of before change and after change. Before Change: ![pr_891_before_change](https://cloud.githubusercontent.com/asse

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-144361517 ok, no further comment (please see the possible null pointer one) --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack pull request: CLOUDSTACK-8913: Search box in Templates ...

2015-09-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/891#issuecomment-144361701 @nitin-maharana thanks for this. LGTM +1 based on code review and look at snapshot. Somehow might want to compile and test fresh. --- If your project is set

[GitHub] cloudstack pull request: CLOUDSTACK-8656: tests ignoring exception...

2015-09-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/850#issuecomment-144363576 @borisroman the expeted attribute is not there and as I understand with reason. The exceptions are part of the contract and need to be handled by the client. Th

[GitHub] cloudstack pull request: CLOUDSTACK-8808: Successfully registered ...

2015-09-30 Thread borisroman
Github user borisroman commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/901#discussion_r40784661 --- Diff: core/src/com/cloud/storage/template/QCOW2Processor.java --- @@ -75,6 +76,16 @@ public FormatInfo process(String templatePath, ImageFormat for

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-144377723 @resmo Just a heads-up that I am testing this as we speak. Running the BVT tests against this branch to verify it all works. Once everything is done I'll post re

RE: test_nicira_controller.py is failing on all Advzone - XS, KVM

2015-09-30 Thread Raja Pullela
thanks Miguel ! From: Miguel Ferreira [mailto:mferre...@schubergphilis.com] Sent: Wednesday, September 30, 2015 3:01 PM To: Raja Pullela Cc: dev Subject: Re: test_nicira_controller.py is failing on all Advzone - XS, KVM Hi Raja, That test needs a NSX cluster. With that in place, the configurat

[GitHub] cloudstack pull request: CLOUDSTACK-8656: tests ignoring exception...

2015-09-30 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/850#issuecomment-144384074 @DaanHoogland @borisroman The test needs to invoke a protected method from a class and so is done using reflection. The test case already asserts for the method

[GitHub] cloudstack pull request: CLOUDSTACK-8924: Enable dynamic scaling t...

2015-09-30 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/902 CLOUDSTACK-8924: Enable dynamic scaling to run test_scale_vm.py test on simulator Simulator setup uses the config file from following location: tools/marvin/marvin/config/setup.cfg Add

Re: OVS Plugin documentation issues

2015-09-30 Thread Ron Wheeler
That was exactly what I needed. I have cloned the master, tried to make a "dev". I will make the suggested changes to the "dev" branch. I am using Eclipse and seem to be having some trouble mapping the README to the corresponding Eclipse operations but that is a general "git/person in the chai

Re: OVS documentation

2015-09-30 Thread Ron Wheeler
On 30/09/2015 6:33 AM, Sebastien Goasguen wrote: On Sep 29, 2015, at 2:06 PM, Ron Wheeler wrote: Can someone fix this? Honestly Ron, the issue is that very few people are actually using the GRE isolation. One would need to test it with the latest cloudstack 4.5.2 and latest OVS to see if i

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-144430354 @resmo I tested a series of BVT tests (not all of them) and the result is fine. First test run: ``` marvinCfg=/data/shared/marvin/mct-zone1-kvm1.

[GitHub] cloudstack pull request: CLOUDSTACK-8808: Successfully registered ...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/901#issuecomment-144439566 Thanks @karuturi for picking this up. Will have a look soon! --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

RE: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Somesh Naidu
> that said we should still work towards a test set that can function as the So essentially, we are saying that the existing BVT isn't really a BVT and that we have to come up with it. From a user/customer standpoint this doesn't inspire confidence. Anyway, is anybody willing to do this, as in,

[GitHub] cloudstack pull request: CLOUDSTACK-8848: ensure power state is up...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/885#issuecomment-16693 The functionality of this PR also seems to work: ``` WARN [c.c.v.VirtualMachinePowerStateSyncImpl] (AgentManager-Handler-5:null) Detected missing VM

RE: VR refactoring, concerns and a way out ?

2015-09-30 Thread Somesh Naidu
> To sum it up: it’s easy to sit-down and wait for other people to work on > features/test/fixes. Wilder - essentially are you saying these failures are due to a. Ramnath and folks didn't contribute at the time and b. you did but weren't able to completely nail it down? Raja/Ramnath - would it

Re: OVS documentation

2015-09-30 Thread Sebastien Goasguen
> On Sep 30, 2015, at 4:20 PM, Ron Wheeler > wrote: > > On 30/09/2015 6:33 AM, Sebastien Goasguen wrote: >>> On Sep 29, 2015, at 2:06 PM, Ron Wheeler >>> wrote: >>> >>> Can someone fix this? >>> >> Honestly Ron, the issue is that very few people are actually using the GRE >> isolation. >>

Re: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Sebastien Goasguen
@somesh, again your message is not helping all. > On Sep 30, 2015, at 5:09 PM, Somesh Naidu wrote: > >> that said we should still work towards a test set that can function as the > > So essentially, we are saying that the existing BVT isn't really a BVT and > that we have to come up with it. F

Re: VR refactoring, concerns and a way out ?

2015-09-30 Thread Sebastien Goasguen
@somesh you message is not helping at all. The problems is being worked on by a lot of people. Folks know the current blockers and I am expecting to see PR come in to fix them in the coming days. We will then see what the results of BVT are. Please read the full thread before replying. > On S

RE: CloudStack networking documentation

2015-09-30 Thread Giles Sirett
Paul is travelling this week - I'll ping somebody else to answer Rons questions Regards Giles -Original Message- From: Sebastien Goasguen [mailto:run...@gmail.com] Sent: 30 September 2015 11:34 To: dev@cloudstack.apache.org; Paul Angus Subject: Re: CloudStack networking documentation P

Re: CloudStack networking documentation

2015-09-30 Thread Dag Sonstebo
Hi Ron, In the opening paragraph of About Physical Networks" it says "The network corresponds to a NIC on the hypervisor host." >> Yes - a physical network can be associated with a NIC or more typically >>in production environments - a bonded pair of NICs for resilience. In Basic Zone Network Tr

RE: [Blocker] test duplicated in test_scale_vm.py

2015-09-30 Thread Somesh Naidu
Ok, I will stop. I was only trying to help. I was just trying to say that we would greatly benefit from a CI practice that runs against an automated BVT and any failure should result in reverting the check-in (essentially a blocker). If the BVT is doing more than it should then tune it according

RE: test_nicira_controller.py is failing on all Advzone - XS, KVM

2015-09-30 Thread Raja Pullela
Hi Miguel, Can you please add some checking in the setup method for the configuration parameters and if it is not available, can you skip this test? Raja -Original Message- From: Raja Pullela [mailto:raja.pull...@citrix.com] Sent: Wednesday, September 30, 2015 5:52 PM To: Miguel Ferrei

[GitHub] cloudstack pull request: Strongswan vpn feature

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/872#issuecomment-144499793 @jayapalu Before you run the tests, please rebase as you'll otherwise find many false-positives. --- If your project is set up for it, you can reply to this ema

[GitHub] cloudstack pull request: CLOUDSTACK-8888: Xenserver 6.0.2 host stu...

2015-09-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/861#issuecomment-144500362 @koushik-das Thanks! Another benefit: this is an easier change that doesn't require much testing (all it does it execute a SQL query at upgrade time).

Re: VR refactoring, concerns and a way out ?

2015-09-30 Thread Abhinandan Prateek
I can understand Citrix concern of some critical scenarios being missed. Citrix supports several customers with sometimes weird setups requiring that extra step in fixing and making the code that works in all those scenarios. The current code has been contributed by community, by people who jus

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2015-09-30 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-144619954 So, I've been paging through the diff files to see what might be a good candidate for unit testing. Unfortunately, I don't really see much. W

[GitHub] cloudstack pull request: CLOUDSTACK-8808: Successfully registered ...

2015-09-30 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/901#discussion_r40881668 --- Diff: core/src/com/cloud/storage/template/QCOW2Processor.java --- @@ -75,6 +76,16 @@ public FormatInfo process(String templatePath, ImageFormat forma

[GitHub] cloudstack pull request: CLOUDSTACK-8896: allocated percentage of ...

2015-09-30 Thread karuturi
Github user karuturi commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/873#discussion_r40881830 --- Diff: server/src/com/cloud/storage/StorageManagerImpl.java --- @@ -1736,7 +1737,10 @@ public boolean storagePoolHasEnoughSpace(List volumes, StorageP