Re: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Rajani Karuturi
Interesting. Incase of KVM, its the reverse. The command which is supposed to goto SSVM ended up on KVM host. ~Rajani On 23-Oct-2015, at 10:49 am, Mike Tutkowski wrote: > Hi Rajani, > > I seem to get the same error. > > "Are you sure you got the right type of server?" is listed again in t

[GitHub] cloudstack pull request: CLOUDSTACK-8985: Deleted volume's removed...

2015-10-22 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/968#issuecomment-150490749 checked with the regular tests of the SBP integration test suits and there all pass (except for the vp offering things) @mike-tutkowski Do you have any recommen

Re: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Mike Tutkowski
So...I suspect my system template is out of date in some significant way. If I set a breakpoint in VMwareGuru and force the CopyCommand to go to the ESXi server resource that's running in my CS MS (instead of the one in the SSVM), then it works. On Thu, Oct 22, 2015 at 11:19 PM, Mike Tutkowski <

Re: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Mike Tutkowski
Hi Rajani, I seem to get the same error. "Are you sure you got the right type of server?" is listed again in the output. It seems VMwareGuru is invoked and it passes the CopyCommand on to the SSVM, which claims it doesn't support the command. Thanks, Mike On Thu, Oct 22, 2015 at 10:56 PM, Raja

RE: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Rajani Karuturi
Yes. This is the pr. We have some design issues to fix to make sure hypervisor gurus doesn't cross boundaries. But, until then, we have to fix the respective hv gurus. You can find more details on the bug which nux shared. You could also test with hypervisor gurus exclude global Param and exclu

Re: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Mike Tutkowski
Hi Rajani, Just checking to make sure this is the PR you intended. It looks related to OVM3 only. Thanks! Mike On Thu, Oct 22, 2015 at 9:39 PM, Rajani Karuturi wrote: > Mike, > Can you please test on PR https://github.com/apache/cloudstack/pull/962? > It fixes this issue. > > ~Rajani > > > > O

Build failed in Jenkins: build-master-slowbuild #2520

2015-10-22 Thread jenkins
See -- [...truncated 781 lines...] 2015-10-23 04:11:44,041 DEBUG [utils.script.Script] (main:) System resource: null 2015-10-23 04:11:44,041 DEBUG [utils.script.Script] (main:) Classpath reso

Re: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Rajani Karuturi
Mike, Can you please test on PR https://github.com/apache/cloudstack/pull/962? It fixes this issue. ~Rajani On 23-Oct-2015, at 7:29 am, Mike Tutkowski wrote: > I guess I should point out that my system VMs are all running on XenServer, > but I don't think that should be a problem. > > On Th

Re: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Mike Tutkowski
I seem to be having a similar problem with deploying a VM on VMware (on 4.6). I first tried with my (SolidFire) plug-in and then with just local storage. In both cases, I get an error saying "Are you sure you got the right type of server?" On Sun, Oct 18, 2015 at 2:29 AM, Nux! wrote: > All I get

Re: [4.6] Can't create template or volume from snapshot

2015-10-22 Thread Mike Tutkowski
I guess I should point out that my system VMs are all running on XenServer, but I don't think that should be a problem. On Thu, Oct 22, 2015 at 7:57 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > I seem to be having a similar problem with deploying a VM on VMware (on > 4.6). I first

[GitHub] cloudstack pull request: Fixed return type Void to void in DataMot...

2015-10-22 Thread rodrigo93
GitHub user rodrigo93 opened a pull request: https://github.com/apache/cloudstack/pull/969 Fixed return type Void to void in DataMotionStrategy. The main changes are: - Changing methods “Void” to “void”. - Removal of the method “Void copyAsync(DataObject srcDat

PR 968

2015-10-22 Thread Mike Tutkowski
Hi, A new integration test I wrote yesterday caught an issue that related to the "removed" column in the "volumes" table not being updated. I went ahead and corrected the issue (just a few lines in my plug-in): https://github.com/apache/cloudstack/pull/968 Can a couple people take a look? It sh

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Mike Tutkowski
Anyone want to take a look at this PR? https://github.com/apache/cloudstack/pull/968 It's quite short. I just needed to make sure that the volume that is to be deleted is not counted in the used-space calculation (this was throwing an exception because it wasn't supposed to be counting the volume

[GitHub] cloudstack pull request: CLOUDSTACK-8985: Deleted volume's removed...

2015-10-22 Thread mike-tutkowski
GitHub user mike-tutkowski opened a pull request: https://github.com/apache/cloudstack/pull/968 CLOUDSTACK-8985: Deleted volume's removed column not updated I found this issue when a SolidFire integration test yesterday wasn't able to delete primary storage because it claimed there

Re: [ACS460] QA days (part one)

2015-10-22 Thread Pierre-Luc Dion
Hi, Just as a quick reminder, we will do the First QA days tomorrow. We should use the Slack channel: #qa_days and IRC channel #cloudstack-dev for live communications. The idea, tests new and current features of master branches, might worth to use jenkins builds and packages to speed up QA process.

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Mike Tutkowski
Anyone who uses managed storage, but this actually effectively means just SolidFire at this point (and this code has to be implemented in the plug-in). On Thu, Oct 22, 2015 at 1:07 PM, Daan Hoogland wrote: > Mik, isn't it more generic then the solidfire plugin? > > On Thu, Oct 22, 2015 at 8:20 P

Re: Getting Started With Documentation

2015-10-22 Thread Tim Mackey
On Tue, Oct 20, 2015 at 7:08 PM, Eric wrote: > Thanks, Erik! > Is there a discussion forum where contributors can discuss documentation? > I believe that, like all things in technology, there should be a set of > conventions that guide authors in the use of common terms. e.g., Xen? Or > Xen Proje

[GitHub] cloudstack pull request: CLOUDSTACK-8981 coded a more obscure host...

2015-10-22 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/965#issuecomment-150333242 usual ssvm and vpc tests were executed with the usual vpc offering failures only. --- If your project is set up for it, you can reply to this email and have yo

Re: Getting Started With Documentation

2015-10-22 Thread Pierre-Luc Dion
Hi Eric, To discuss CloudStack Documentation, best is dev@ mailing list. there is some Jira issues related to documentations too. related to guidelines, as Erik said it's all in github and if you want to enhance the look and feel, feel free to change it as the sphinx theme is now included in Git

[GitHub] cloudstack pull request: CLOUDSTACK-8964: Fix KVM snap to vol and ...

2015-10-22 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/962#issuecomment-150325540 @snuf The org.apache.cloudstack.utils.hypervisor.HypervisorUtilsTest.checkVolumeFileForActivityTest(HypervisorUtilsTest.java:70) test is IMHO a known issue. Are y

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Daan Hoogland
Mik, isn't it more generic then the solidfire plugin? On Thu, Oct 22, 2015 at 8:20 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > It looks like this is an issue with managed storage. > > I can actually put a fix in the SolidFire plug-in to correct what's going > on. > > I opened a JI

[GitHub] cloudstack pull request: CLOUDSTACK-8935 - Cannot remove VPC netwo...

2015-10-22 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/967#issuecomment-150324755 @wilderrodrigues i'm on it when it's ready! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] cloudstack pull request: CLOUDSTACK-8935 - Cannot remove VPC netwo...

2015-10-22 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/967#issuecomment-150323162 @wilderrodrigues ping me when you want me to run some tests! --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Mike Tutkowski
It looks like this is an issue with managed storage. I can actually put a fix in the SolidFire plug-in to correct what's going on. I opened a JIRA ticket and will submit a PR once I test my changes (there are only a few lines changed). On Thu, Oct 22, 2015 at 1:02 AM, Remi Bergsma wrote: > Hi

[GitHub] cloudstack pull request: CLOUDSTACK-8935 - Cannot remove VPC netwo...

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/967#issuecomment-150295398 Hi all, Please keep it on hold: I tested the network deletion manually before the component/test_vpc_offerings.py was completed. Deleting the network

[GitHub] cloudstack pull request: CLOUDSTACK-8935 - Cannot remove VPC netwo...

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/967#issuecomment-150294488 Ping @remibergsma @wido @borisroman @miguelaferreira @karuturi @bhaisaab @DaanHoogland Additional non-hardware required tests still running, but I w

[GitHub] cloudstack pull request: CLOUDSTACK-8935 - Cannot remove VPC netwo...

2015-10-22 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/967 CLOUDSTACK-8935 - Cannot remove VPC networks due to RTNETLINK error This PR fixes the "sequence item 0: expected string, NoneType found" error found in the CsDhcp.py file when attempting

Re: BVT report - 10/22

2015-10-22 Thread Wilder Rodrigues
Hi Raja, Test is done and we have failures: Test for port forwarding on source NAT ... === TestName: test_01_port_fwd_on_src_nat | Status : SUCCESS === ok Test for port forwarding on non source NAT ... === TestName: test_02_port_fwd_on_non_src_nat | Status : SUCCESS === ok Test for reboot route

Re: BVT report - 10/22

2015-10-22 Thread Wilder Rodrigues
Okay. We haven’t got any failure, but I’m curious about what you have to share. I started the test_network here and it’s going fine. However, we need to improve that one. It has a test that delete the PF rules and tries to SSH into the VM - failure is expected. But unfortunately it has a retrie

Re: BVT report - 10/22

2015-10-22 Thread Raja Pullela
Hi wilder, Internal Lb tests, I need to investigate the failures, some race condition when I ran it. Best Raja > On Oct 22, 2015, at 9:40 PM, Wilder Rodrigues > wrote: > > Hi Raja, > > Internal LB has been working since wilder PR 940 was merged and the test > itself has also been improve

Re: BVT report - 10/22

2015-10-22 Thread Wilder Rodrigues
Hi Raja, Internal LB has been working since PR 940 was merged and the test itself has also been improved to do proper testing. We have tested that for KVM. What do you mean by "// Issue to be investigated”? Concerning the SSVM/CPVM, yep… working fine again: I created the PR 959 (with extra tes

Re: BVT report - 10/22

2015-10-22 Thread Raja Pullela
I didn't run them this time. > On Oct 22, 2015, at 7:45 PM, sebgoa wrote: > > I presume simulator runs are still 100% ? > >> On Oct 22, 2015, at 4:01 PM, Raja Pullela wrote: >> >> Hi, >> >> Following is the latest BVT report - 10/22 >> Xen Basic : 94.0% >> KVM Basic: 97.1% - glad to see th

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-10-22 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-150268787 I will review the changes late today or early tomorrow. I have a few items I need to attend to first. > On Oct 22, 2015, at 11:16 AM, Kris Sterck

[GitHub] cloudstack pull request: Added support for KVM teamd devices to Li...

2015-10-22 Thread dmabry
GitHub user dmabry opened a pull request: https://github.com/apache/cloudstack/pull/966 Added support for KVM teamd devices to LibvirtComputingResource.java This will allow KVM users to utilize teamd nic teaming devices named team*. You can merge this pull request into a Git reposit

[GitHub] cloudstack pull request: CLOUDSTACK-8832 : Update Nuage VSP plugin...

2015-10-22 Thread KrisSterckx
Github user KrisSterckx commented on the pull request: https://github.com/apache/cloudstack/pull/801#issuecomment-150258402 @remibergsma how can we continue pls ? Could you run Travis again ? Can we do anything ? Thanks --- If your project is set up for it, you can reply to this ema

[GitHub] cloudstack pull request: CLOUDSTACK-8964: Fix KVM snap to vol and ...

2015-10-22 Thread snuf
Github user snuf commented on the pull request: https://github.com/apache/cloudstack/pull/962#issuecomment-150257035 @NuxRo interesting... I'm checking out my commit again and rebuilding to test again, I'm running both at the same time though... so that might explain something perhaps

Re: BVT report - 10/22

2015-10-22 Thread sebgoa
I presume simulator runs are still 100% ? On Oct 22, 2015, at 4:01 PM, Raja Pullela wrote: > Hi, > > Following is the latest BVT report - 10/22 > Xen Basic : 94.0% > KVM Basic: 97.1% - glad to see the reboot ssvm/cpvm are fixed > Xen Adv: 91.9% - test_internal_lb and test_network needs code f

BVT report - 10/22

2015-10-22 Thread Raja Pullela
Hi, Following is the latest BVT report - 10/22 Xen Basic : 94.0% KVM Basic: 97.1% - glad to see the reboot ssvm/cpvm are fixed Xen Adv: 91.9% - test_internal_lb and test_network needs code fixes, possibly! KVM Adv: 93.6% - glad to see the reboot ssvm/cpvm are fixed - test_internal_lb and test_

[GitHub] cloudstack pull request: CLOUDSTACK-8981 coded a more obscure host...

2015-10-22 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/965 CLOUDSTACK-8981 coded a more obscure host and clear failure message test fails when port is reachable so prevent it as much as possible making sure that all kinds of weird developers c

[GitHub] cloudstack pull request: CLOUDSTACK-8964: Fix KVM snap to vol and ...

2015-10-22 Thread NuxRo
Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/962#issuecomment-150225444 I've built EL6 RPMs from https://github.com/snuf/cloudstack/archive/6efe627e8e11c18acf62ff02b85007de122a917c and "yum reinstall" management and agent. (am I testi

Re: master build broken?

2015-10-22 Thread Daan Hoogland
Thanks Wei, As discussed on IM localhost works for me but only when not open ;) I changed it to 127.0.0.128 and it works. I do not get the idea behind the test and I don't think it is a unit test if a external host (or it's absense) is required. I will fail after the point the exception is expecte

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42740897 --- Diff: utils/src/main/java/com/cloud/utils/nicira/nvp/plugin/NiciraNvpApiVersion.java --- @@ -0,0 +1,60 @@ +// +// Licensed to the Apa

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42740931 --- Diff: utils/src/main/java/com/cloud/utils/nicira/nvp/plugin/NiciraNvpApiVersion.java --- @@ -0,0 +1,60 @@ +// +// Licensed to the Apa

[GitHub] cloudstack pull request: CLOUDSTACK-8964: Fix KVM snap to vol and ...

2015-10-22 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/962#issuecomment-150203517 Thanks for taking this up @snuf. I am out of office and will be able to test it only on Monday. @NuxRo Can you try this please? --- If your project is set up

Re: master build broken?

2015-10-22 Thread Wei ZHOU
Author: Wei Zhou AuthorDate: Thu Aug 27 10:32:57 2015 +0200 Commit: Wei Zhou CommitDate: Thu Aug 27 10:32:57 2015 +0200 use fakelocalhost in BasicRestClientTest --- .../com/cloud/utils/rest/BasicRestClientTest.java |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff -

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/935#discussion_r42740183 --- Diff: vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareHelper.java --- @@ -79,6 +79,52 @@ public static boolean isReservedScsiDeviceNumb

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150185163 The directory in which a test lives is irrelevant to me, so I would also +1 a PR that moves this to another directory. --- If your project is set up for it,

Re: BVT Report 10/14

2015-10-22 Thread Wilder Rodrigues
I will run test_network on latest master with the following environment now Advance Zone 2 KVM hosts Basic Zone 1 KVM host Cheers, Wilder > On 22 Oct 2015, at 09:01, Remi Bergsma wrote: > > Thanks! > > FYI: 1) is fixed by PR 959 that was merged last night. The tests should now > pass, as sh

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150178317 I think you guys agree on the problem but disagree on the solution. To me 100% success on BVT, if it includes ~20% (or whatever percentage) of skipped tests is not

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/952 --- 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

Re: master build broken?

2015-10-22 Thread Daan Hoogland
any clues out there? I backtracked master to 4 weeks ago and keep getting this failure!?! On Thu, Oct 22, 2015 at 12:11 PM, Daan Hoogland wrote: > git reset --hard > mvn -P developer,systemvm clean install > > leads to build failure in > BasicRestClientTest.testExecuteRequestWhenClientThrowsIOEx

[GitHub] cloudstack pull request: CLOUDSTACK-8970 Centos 6.{1,2,3,4,5} gues...

2015-10-22 Thread snuf
Github user snuf commented on the pull request: https://github.com/apache/cloudstack/pull/956#issuecomment-150176025 Values are inline with the guest_os table and inserts the inserts all work so LGTM! --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] cloudstack pull request: CLOUDSTACK-8979: add virtio support for C...

2015-10-22 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/963#issuecomment-150173240 (1) make the change on mysql (2) create a template (CentOS 7) if you do not have one. (3) create an instance with the CentOS 7 template on KVM. (4) che

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread michaelandersen
Github user michaelandersen commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150173017 The exceptions: error: [Errno 113] No route to host error: [Errno 111] Connection refused until ===SSH to Host 192.168.23.12 port : 22 SUCCESS

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread pvr9711
Github user pvr9711 commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150171997 @miguelaferreira 100% passrate on the BVTs - if there are skipped cases you should/want to find out why they are getting skipped and fix them. If you are skipping t

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150171835 New test run, same result: ``` [mct-run-marvin-tests@3] Running shell script + nosetests --with-xunit --xunit-file=integration-test-results/smoke/

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150169879 Does 100% pass rate when some of the tests have been skipped mean more than 100% pass rate when the same tests have been excluded? I only hear about

master build broken?

2015-10-22 Thread Daan Hoogland
git reset --hard mvn -P developer,systemvm clean install leads to build failure in BasicRestClientTest.testExecuteRequestWhenClientThrowsIOException Expected exception: com.cloud.utils.rest.CloudstackRESTException ​anybody else seeing this?​ -- Daan

[GitHub] cloudstack pull request: FIX: Ovm3 physical network traffic labels...

2015-10-22 Thread snuf
GitHub user snuf opened a pull request: https://github.com/apache/cloudstack/pull/964 FIX: Ovm3 physical network traffic labels to work. The labeling was broken. Only labels assigned at zone creation were used, changing labels was not working. Tested with changing a label and

[GitHub] cloudstack pull request: CLOUDSTACK-8979: add virtio support for C...

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/963#issuecomment-150168660 Hi @ustcweizhou How did you test it? If you give us the steps, we can help testing and also see if we can automate it. Cheers, Wilder

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread pvr9711
Github user pvr9711 commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150168452 @miguelaferreira - we have discussed this on an earlier email. as Prashant said, In this case a test case is skipped because the setup does not have the configurati

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

2015-10-22 Thread jenkins
See Changes: [miguelferreira] cloud-apidoc module depends on cloud-client-ui -- [...truncated 398 lines...] Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.033 sec - in

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-22 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-150164133 Hi Daan, I have verified the result with curl -v, therefore manual verification. It would be nice to have automatic test to verify that unauthenticated users c

[GitHub] cloudstack pull request: cloud-apidoc module depends on cloud-clie...

2015-10-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/958 --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150162817 @nitt10prashant I rather not skip at all. There are just too many examples of tests being skipped and people not reviewing the output. And that's not even th

[GitHub] cloudstack pull request: CLOUDSTACK-8979: add virtio support for C...

2015-10-22 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/963 CLOUDSTACK-8979: add virtio support for CentOS 7 on KVM @borisroman @NuxRo could you test it? You can merge this pull request into a Git repository by running: $ git pull https://github

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150162235 @wilderrodrigues I think it was, in the end I ran only the smoke test suite and even that got errors so please ignore for this PR --- If your project is set up

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150161922 @miguelaferreira i was going though mail thread and found this "I think the test should fail if the config is not right, or otherwise be excluded if the

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150161422 That's what I thought. The JUnit report plugin in Jenkins sais the test passed. (See last line of table in picture) ![screen shot 2015-10-22 at 11 13

[GitHub] cloudstack pull request: Improve marvin test site2site VPN

2015-10-22 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/952#issuecomment-150160922 Hi @miguelaferreira The system-err is because the Marvin code prints all the ssh commands to the terminal. It doesn't mean it failed. All the SSH re

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread pvr9711
Github user pvr9711 commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150153497 LGTM - skipping testing is designed with a purpose and this is exactly we should be using it for. As a as person running/maintaining the tests, someone would have t

Re: CentOS 7 templates lack virtio

2015-10-22 Thread Nux!
@dev btw we should fix this in 4.6 as well https://issues.apache.org/jira/browse/CLOUDSTACK-8979 -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Nux!" > To: us...@cloudstack.apache.org > Cc: "dev" > Sent: Thursday, 22 October, 2015 0

Re: CentOS 7 templates lack virtio

2015-10-22 Thread Nux!
Ozhan, I think it's in the agent code somewhere. CC-ing dev@ - guys can you suggest a quick workaround for this? The CentOS7 KVM profile does not come with VirtIO, instead it exposes IDE disks and such. Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Origin

[GitHub] cloudstack pull request: CLOUDSTACK-8956: NSX/Nicira Plugin does n...

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/935#issuecomment-150145949 @nvazquez PR looks awesome now. Great work! I will pull in your PR and run the marvin test we already have against it to see if that is still good. I

[GitHub] cloudstack pull request: CLOUDSTACK-8964: Fix KVM snap to vol and ...

2015-10-22 Thread snuf
Github user snuf commented on the pull request: https://github.com/apache/cloudstack/pull/962#issuecomment-150142734 Eh that's odd, it's supposed to wait for 2 seconds, waited for 1.002 and then fails because it waited too short ? what am I missing here ? :) --- If your project is se

[GitHub] cloudstack pull request: CLOUDSTACK-8975:skip test case if niciraN...

2015-10-22 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/960#issuecomment-150141365 We have had a lengthy discussion about this on the mailing list. I didn't' agree that skipping tests is better than excluding them from a test run. I

[GitHub] cloudstack pull request: CLOUDSTACK-8977: remove session creation ...

2015-10-22 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-150131358 makes sense. lgtm . testing in batch with other ui fixes? It is rather small but not trivial and should be verified. --- If your project is set up for it, you

Re: [4.6] Issue with removed column in volumes table not being updated

2015-10-22 Thread Remi Bergsma
Hi Mike, Thanks for the heads up. Please file a jira issue with details and keep us posted. Thanks, Remi Sent from my iPhone > On 22 Oct 2015, at 08:27, Mike Tutkowski wrote: > > Hi, > > Just an FYI that I noticed tonight that the removed column in the volumes > table is not updated when

Re: BVT Report 10/14

2015-10-22 Thread Remi Bergsma
Thanks! FYI: 1) is fixed by PR 959 that was merged last night. The tests should now pass, as shown in the PR. The others we'll have to look into once we have the details. Will also see if I can run those on my dev environment. Regards, Remi Sent from my iPhone > On 22 Oct 2015, at 08:28,