[GitHub] cloudstack pull request: CLOUDSTACK-8940: Wrong value is inserted ...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/916#issuecomment-152449289 LGTM, based on a set of tests that I run on this branch (which I rebased myself first): ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -

[GitHub] cloudstack pull request: CLOUDSTACK-8793 Enable s2s VPN connection...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/879#issuecomment-152450103 @pdube I trust you, but the commit hash changed so I just run them again. LGTM, based on a set of tests that I run on this branch (which I rebased myself

Re: Build error

2015-10-30 Thread Daan Hoogland
@wido, is this one due to the temp dir path on windows thing as well? On Fri, Oct 30, 2015 at 7:55 AM, B Prakash wrote: > I am new to this and hence might be missing something, please could > someone help me with this build error? I am having the latest build and > getting the following error;

[GitHub] cloudstack pull request: CLOUDSTACK-8793 Enable s2s VPN connection...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/879 --- 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: Rename xapi plugins for s3 and swift to m...

2015-10-30 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/982#issuecomment-152451246 Just tested it! LGTM :+1: ``` [root@xen1 ~]# ls -la /etc/xapi.d/plugins/ | grep xenserver -rwxr-xr-x 1 root root 12714 Oct 30 07:52 s3xenser

[GitHub] cloudstack pull request: CLOUDSTACK-8866 : restart.retry.interval ...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/834#issuecomment-152452258 LGTM, based on a set of tests that I run on this branch (which I rebased myself first): ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -

Add Debian 8.0 in OS List for Template

2015-10-30 Thread Milamber
Hello, There are any special reason to not add Debian 8.0 32/64 bits on the list of OS in the template form ? like missing support from kvm or xenserver? Or we just need to add the INSERT into the sql schema like this commit [1] ? That would be great if the next release 4.6 could have the De

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

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/898#issuecomment-152452596 LGTM, based on a set of tests that I run on this branch (which I rebased myself first): ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -

[GitHub] cloudstack pull request: another typo that appears when monitoring...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/913#issuecomment-152452982 LGTM, based on a set of tests that I run on this branch (which I rebased myself first): ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -

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

2015-10-30 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/823#issuecomment-152453158 I tested this with this scenario. There are three service offerings. { "listserviceofferingsresponse": { "count": 3,

[GitHub] cloudstack pull request: CLOUDSTACK-8958: add dedicated ips to dom...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1007#issuecomment-152453373 LGTM, based on a set of tests that I run on this branch (which I rebased myself first): ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1005#issuecomment-152453969 @borisroman Fixed, that was indeed the problem --- 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

Re: Build error

2015-10-30 Thread Remi Bergsma
Please propose it as a new PR and we will include it soon. Easy enough to review, like last time. On 30/10/15 08:34, "Daan Hoogland" wrote: >@wido, is this one due to the temp dir path on windows thing as well? > >On Fri, Oct 30, 2015 at 7:55 AM, B Prakash wrote: > >> I am new to this and he

Re: Build fails on LocalTemplateDownloadTest

2015-10-30 Thread Wido den Hollander
On 29-10-15 15:36, Josh Harshman wrote: > Build environment isn't Windows, it's Ubuntu 14.04 that I am getting this > build failure on. > Is there something special about the /tmp on your system? Eg not enough space for example. Wido > > From: Wido de

[GitHub] cloudstack pull request: CLOUDSTACK-8928: While adding VMs to LB r...

2015-10-30 Thread nitin-maharana
Github user nitin-maharana commented on the pull request: https://github.com/apache/cloudstack/pull/903#issuecomment-152456961 Hi @runseb @remibergsma I don't understand how to write a test for this. If you have any idea of how to write one, please help me out. Thanks. --- If your pr

[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152457228 @davidamorimfaria due to a known bug in jenkins/maven integration the build in jenkins failed. Could you repush with git push --force to your branch? I'm revie

[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152457639 @davidamorimfaria I saw you didn't go for the tomcat instance creation, right? changes look good and as we discussed off-line. Can you add a jira ticket

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1005#discussion_r43480510 --- Diff: plugins/hypervisors/kvm/test/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtUtilitiesHelperTest.java --- @@ -0,0 +1,52 @@ +// Licen

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1005#issuecomment-152459236 very onky indeed. code looks good, though I don't like the hard coded paths --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/985#issuecomment-152460936 @ustcweizhou Which version of libvirt are you using? If you use libvirt 1.0.6 or newer, you can omit the path='...' attribute of the element, and libvirt will

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/985#issuecomment-152461778 @wido ja, you got it. The issue happened on a host running with Ubuntu 12.04 (QEMU 1.2.1 and libvirt 0.9.13) There is no issue on Ubuntu 14.04 (QEMU 2.0.0 and

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/985#issuecomment-152462216 @wido by the way, I just remember I have implemented some codes for qemu-guest-agent support , based on cloudstack 4.2.0 maybe. It is not fully tested. I will

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1005#issuecomment-152462532 @DaanHoogland I don't like them either, but these paths are currently hardcoded in LibvirtComputingResource and I want to verify that they don't change and break some

[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152471048 Thanks for the fix and the PR, @davidamorimfaria I will try to test/review this one today. Cheers, Wilder --- If your project is set

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

2015-10-30 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-152474694 In tomcat I see the following errors: ``` Oct 30, 2015 9:39:13 AM org.apache.catalina.core.ApplicationDispatcher invoke SEVERE: Servlet.service()

[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-9000: logrotate c...

2015-10-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/992#issuecomment-152476140 @remibergsma should we merge this on master now? (jenkins failed due to some jvm issue) --- If your project is set up for it, you can reply to this email and have

Re: Git with Eclipse

2015-10-30 Thread Sebastien Goasguen
Ron, I don’t know how to use Github with eclipse, I use the command line for everything. But the documentation should be good: https://github.com/apache/cloudstack-docs-admin#contributing-to-the-documentation Basically you need an account on GitHub. You fork https://github.com/apache/cloudstack

[GitHub] cloudstack pull request: adjust packaging for centos7

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152479205 Seems to start fine now, thanks @davidamorimfaria Haven't setup a cloud with this or tested it further. Just built RPM packages and started management

[GitHub] cloudstack pull request: CLOUDSTACK-8993: DHCP fails with "no addr...

2015-10-30 Thread serbaut
Github user serbaut commented on the pull request: https://github.com/apache/cloudstack/pull/981#issuecomment-152481954 Afaict, the DHCP code is rewritten in 4.6 so this exact issue shouldn't affect 4.5. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-9000: logrotate c...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/992#issuecomment-152486552 Double checked a compile: ``` [INFO] [INFO] BUILD SUCCESS [INFO] --

[GitHub] cloudstack pull request: [master/4.6] CLOUDSTACK-9000: logrotate c...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/992 --- 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: [4.5] CLOUDSTACK-9000: logrotate cloudsta...

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

applying fixes, after 4.6 is out

2015-10-30 Thread Daan Hoogland
H, Being kind of an optimist sometimes I want to start talking on how to handle PRs from 4.6.1 onward. If you don't understand what I'm talking about, please feel free to bother me with questions. I had a discussion with Remi last night about refusing PRs or not based on the premiss they are fixi

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

2015-10-30 Thread miguelaferreira
Github user miguelaferreira commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-152490396 After digging in a bit more, I've debugged the browser session when hitting the home page with `session="false"` what I see is that the page is not even comp

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

2015-10-30 Thread terbolous
Github user terbolous commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-152491165 just proves that we really need a new web ui :-) --- 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: Fixed user_vm_view to only display keypai...

2015-10-30 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/1006#issuecomment-152493863 Manual Testing: Before: ssh_keypairs ``` mysql> select * from ssh_keypairs; +++---+--+--

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add VirtIO channel to al...

2015-10-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/985#issuecomment-152494985 @ustcweizhou @remibergsma I just pushed a new version of the commit. On Ubuntu AppArmor needs to be disabled since the default profile for libvirt doesn't allow

[GitHub] cloudstack pull request: Fixed: Network Update from RVR offering t...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/818#issuecomment-152496955 LGTM I ran the compile on this one and did a code review. @remi, no changes since your last check, you can merge at your will --- If your project is set up

[GitHub] cloudstack pull request: Rename xapi plugins for s3 and swift to m...

2015-10-30 Thread karuturi
Github user karuturi commented on the pull request: https://github.com/apache/cloudstack/pull/982#issuecomment-152497680 manually tested by upgrading an existing xenserver setup and force reconnecting the host Before ``` [root@rjn-xen65-host plugins]# ls -la /etc/xapi.

Re: [PROPOSAL] remove travis pull request checks from github

2015-10-30 Thread Daan Hoogland
On Fri, Oct 30, 2015 at 7:05 AM, Rajani Karuturi wrote: ​+1'ed and further > On Fri, Oct 30, 2015 at 4:55 AM, David Nalley wrote: > > > 2 hours before this message was sent, there was a message about > > jenkins being unreliable. > ​both are not perfect, travis is costing RMs and their minions,

Re: Automatically trigger a new Jenkins build

2015-10-30 Thread Daan Hoogland
So Rajani, you suggest to remove the automatic trigger and leave that to reviewer. Sound fine to me. On Fri, Oct 30, 2015 at 7:35 AM, Rajani Karuturi wrote: > Thats a good idea. I have seen other open source projects do it this way. > (example: netty project) > On demand build is a better way th

Re: Add Debian 8.0 in OS List for Template

2015-10-30 Thread Daan Hoogland
Milamber, there are api's for this: http://cloudstack.apache.org/api/apidocs-4.5/root_admin/addGuestOs.html and http://cloudstack.apache.org/api/apidocs-4.5/root_admin/addGuestOsMapping.html Or am I missing your point? On Fri, Oct 30, 2015 at 8:48 AM, Milamber wrote: > Hello, > > There are any

[GitHub] cloudstack pull request: another typo that appears when monitoring...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/913#issuecomment-152499596 valid English replacement of strings I see @bhaisaab 's point in his comment but had eclipse check java references to the call and is never made. It must be imp

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1005#issuecomment-152499836 @wido good answer based on code review LGTM so far --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we

[GitHub] cloudstack pull request: CLOUDSTACK-8793 Enable s2s VPN connection...

2015-10-30 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/879#issuecomment-152500179 Thanks @remibergsma I haven't got time to retest that branch again, now that it's in master will retry... --- If your project is set up for it, you can repl

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1005#issuecomment-152501126 @DaanHoogland I've just pushed a version with a comment in there to explain what I'm doing Code-wise nothing changed --- If your project is set up for it, yo

Re: Git with Eclipse

2015-10-30 Thread Daan Hoogland
Ron, in eclipse i avoid doing any work involving multiple remotes. I do those things from the commandline. Other things work fine and the part with remotes might work well as well these days. If you use only your fork from eclipse and fetch and rebase from apache or github from out side eclipse yo

Re: applying fixes, after 4.6 is out

2015-10-30 Thread Daan Hoogland
valid question but is this more work in comparison to having the contribution be refused on the grounds of not being on the right version? We encounter a conflict and then bring it back to the contributor... further flow is untouched. agree? On Fri, Oct 30, 2015 at 11:55 AM, Boris Schrijver wrote

[GitHub] cloudstack pull request: Fixed user_vm_view to only display keypai...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1006 --- 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: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1005#issuecomment-152503018 @remibergsma can you spare a bubble for this one? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] cloudstack pull request: CLOUDSTACK-8677: use wrapper classes for ...

2015-10-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/987#issuecomment-152505938 @borisroman Can we run the tests on this one? --- 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 pr

[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2015-10-30 Thread shwetaag
GitHub user shwetaag opened a pull request: https://github.com/apache/cloudstack/pull/1011 CLOUDSTACK-9012 :automation of cores feature test path https://issues.apache.org/jira/browse/CLOUDSTACK-9012 Automated a full scenario of coreos guest OS support: it includes registerin

[GitHub] cloudstack pull request: CLOUDSTACK-9012 :automation of cores feat...

2015-10-30 Thread shwetaag
Github user shwetaag commented on the pull request: https://github.com/apache/cloudstack/pull/1011#issuecomment-152510674 pasting result.txt test1_coreos_VM_creation (integration.component.test_coreos.TestDeployVmWithCoreosTemplate) ... === TestName: test1_coreos_VM_creation

[GitHub] cloudstack pull request: CLOUDSTACK-9003 Make VirtualMachineName i...

2015-10-30 Thread ProjectMoon
Github user ProjectMoon commented on the pull request: https://github.com/apache/cloudstack/pull/988#issuecomment-152513166 While I haven't pushed new work to this PR yet, I do have a prototype of what I intend to do, and am looking for thoughts on it here. My plan: * Move

[GitHub] cloudstack pull request: CLOUDSTACK-9003 Make VirtualMachineName i...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/988#issuecomment-152516868 @ProjectMoon : sounds good let me know of any more PoC code or further design. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/1012 Use java.io.tmpdir instead of hardcoded /tmp This was submitted earlier in PR #884 but that did not merge properly. This is a new PR with the same change. You can merge this pull requ

Re: Building SNAPSHOT debian packages

2015-10-30 Thread Wido den Hollander
On 29-10-15 14:20, Jeff Hair wrote: > Hi, > > We are building Debian packages on unstable branches (4.5 branch to be > specific). The Debian source format 30. rejects the version 4.5.3-SNAPSHOT > with an error "Can't build with source format 3.0 native: native package > version may not have a re

New PMC member: Erik Weber

2015-10-30 Thread Giles Sirett
All The PMC is delighted to announce that we have asked Erik weber to join the PMC and he has accepted Please join me in congratulating Erik Kind Regards Giles D: +44 20 3603 0541 | M: +44 796 111 2055 giles.sir...@shapeblue.com Find out more about ShapeBlue

[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1012#issuecomment-152536606 this lgtm but I am not running on any windows machines. @wido do you remember who the people that were bothered were and can you ping thm to verify? basical

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152536728 I haven't installed a cloud with the packages made with this change, but so far the management servers start and talk to each other: Oct 30 13:54:0

Re: New PMC member: Erik Weber

2015-10-30 Thread Mike Tutkowski
Congratulations, Erik! On Friday, October 30, 2015, Giles Sirett wrote: > All > The PMC is delighted to announce that we have asked Erik weber to join the > PMC and he has accepted > > Please join me in congratulating Erik > > Kind Regards > Giles > > D: +44 20 3603 0541 | M: +44 796 111 > 2055

Re: New PMC member: Erik Weber

2015-10-30 Thread Daan Hoogland
enloy the ride, Erik. Welcome. On Fri, Oct 30, 2015 at 3:17 PM, Mike Tutkowski < mike.tutkow...@solidfire.com> wrote: > Congratulations, Erik! > > On Friday, October 30, 2015, Giles Sirett > wrote: > > > All > > The PMC is delighted to announce that we have asked Erik weber to join > the > > PMC

[GitHub] cloudstack pull request: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1012#issuecomment-152541675 @DaanHoogland I think it was @SudharmaJain As long as the build still works on Linux/Mac we didn't break anything. I'm positive it's now also fixed on Windows.

Re: New PMC member: Erik Weber

2015-10-30 Thread Wido den Hollander
Congrats! Welcome :) On 30-10-15 15:01, Giles Sirett wrote: > All > The PMC is delighted to announce that we have asked Erik weber to join the > PMC and he has accepted > > Please join me in congratulating Erik > > Kind Regards > Giles > > D: +44 20 3603 0541 | M: +44 796 111 > 2055 > giles.s

Re: New PMC member: Erik Weber

2015-10-30 Thread Patrick Dube
Congratulations Erik! On Fri, Oct 30, 2015 at 10:33 AM Wido den Hollander wrote: > Congrats! Welcome :) > > On 30-10-15 15:01, Giles Sirett wrote: > > All > > The PMC is delighted to announce that we have asked Erik weber to join > the PMC and he has accepted > > > > Please join me in congratula

[GitHub] cloudstack pull request: CLOUDSTACK-9014 Rename xapi plugins for s...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/982#issuecomment-152553309 @karuturi Added Jira issue CLOUDSTACK-9014 --- 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-9014 Rename xapi plugins for s...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/982 --- 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: Use java.io.tmpdir instead of hardcoded /...

2015-10-30 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1012#issuecomment-152555222 Tested and that is not working on Windows. I remember having fixed that into a PR that @DaanHoogland submitted @wido, to fix the problem for windows

[GitHub] cloudstack pull request: CLOUDSTACK-8677: use wrapper classes for ...

2015-10-30 Thread borisroman
Github user borisroman commented on the pull request: https://github.com/apache/cloudstack/pull/987#issuecomment-152559399 @widi already building --- 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 ha

Re: [4.6] VM Snapshots no longer work with managed storage

2015-10-30 Thread Remi Bergsma
Hi Mike, Thanks for reporting this issue. Can you provide some more details please? The Jira issue has none. When you put a blocker, we need to have some information so we can understand what is going on and help with debugging / fixing. Even better would be to discuss this. Let’s focus on res

Re: New PMC member: Erik Weber

2015-10-30 Thread Remi Bergsma
Congrats Erik! On 30/10/15 15:01, "Giles Sirett" wrote: >All >The PMC is delighted to announce that we have asked Erik weber to join the PMC >and he has accepted > >Please join me in congratulating Erik > >Kind Regards >Giles > >D: +44 20 3603 0541 | M: +44 796 111 >2055 >giles.sir...@shape

[GitHub] cloudstack pull request: kvm: Add UnitTests for LibvirtUtilitiesHe...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1005#issuecomment-152564991 @DaanHoogland Tests are running. --- 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

[GitHub] cloudstack pull request: another typo that appears when monitoring...

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

CloudStack-8933 changes regressed HyperV and VMware BVTs - 8 tests in test_ssvm.py

2015-10-30 Thread Raja Pullela
Hi Wilder The changes from this commit are failing the HyperV and VMware BVTs. Can you please let me know if these tests were passing in your environment- VMware, Hyperv? Raja Error - "the cached link local should be same current local ip, but they are different!"

[GitHub] cloudstack pull request: Fixed: Network Update from RVR offering t...

2015-10-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/818 --- 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-8964: Ovm3HypervisorGuru wrong...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/975#issuecomment-152566972 @dahn are you OK to merge this as-is? --- 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 pro

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152568600 based on the review of the changes and looking at the deployment of the management servers i think this LGTM --- If your project is set up for it, you can rep

[GitHub] cloudstack pull request: CLOUDSTACK-8746: VM Snapshotting implemen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/977#issuecomment-152569385 FYI: Results of tests that I run on this branch: ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s -a tags=advanced,required_hardware=true

[GitHub] cloudstack pull request: CLOUDSTACK-9006 - ListTemplates API retur...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1009#issuecomment-152570782 LGTM, based on a set of tests that I run on this branch (which I rebased myself first): ``` nosetests --with-marvin --marvin-config=${marvinCfg} -s

Re: extending macchinina image

2015-10-30 Thread Nux!
Hello, Macchinina is based on buildroot.org, the configs I used can be found here: https://github.com/NuxRo/macchinina/tree/master/configs You can essentially build your own, the problem with this is that once you start adding new stuff it grows and grows. At some point it's not worth the troub

Re: New PMC member: Erik Weber

2015-10-30 Thread Nux!
It's a boy, congrats! :) -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Remi Bergsma" > To: dev@cloudstack.apache.org > Cc: "Erik Weber" > Sent: Friday, 30 October, 2015 15:48:40 > Subject: Re: New PMC member: Erik Weber > Congrats

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152573997 This PR fixes CentOS 7 packaging and obsoletes #888 and CLOUDSTACK-8812. I will merge this and close the other. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-152578560 @davidamorimfaria Final request, please squash your commits. We need them to be atomic. Please ping me when done, thanks! --- If your project is set up for it,

[GitHub] cloudstack pull request: CLOUDSTACk-9002: VM deployment is success...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/995#issuecomment-152581144 @wilderrodrigues did you run more tests? --- 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-9006 - ListTemplates API retur...

2015-10-30 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1009#discussion_r43523817 --- Diff: framework/db/src/com/cloud/utils/db/Filter.java --- @@ -89,7 +89,7 @@ public void addOrderBy(Class clazz, String field, boolean ascending) {

[GitHub] cloudstack pull request: CLOUDSTACK-8940: Wrong value is inserted ...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/916#issuecomment-152581514 @wilderrodrigues any update on your review? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

[GitHub] cloudstack pull request: CLOUDSTACK-9006 - ListTemplates API retur...

2015-10-30 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1009#discussion_r43523867 --- Diff: framework/db/src/com/cloud/utils/db/Filter.java --- @@ -89,7 +89,7 @@ public void addOrderBy(Class clazz, String field, boolean ascending) {

test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-10-30 Thread Raja Pullela
Hi Michael Can you please add a wait for the template to download before the VM creation? Raja Begin forwarded message: From: Raja Pullela mailto:raja.pull...@citrix.com>> Date: October 29, 2015 at 11:49:59 AM GMT+5:30 To: "CloudStack Dev (dev@cloudstack.apache.org

Re: 4.6 status update

2015-10-30 Thread Remi Bergsma
Hi all, The blocker mentioned below has been fixed. Its PR (1008) is almost ready to be merged. In the mean while, Mike reported an issue, that is now our single blocker. Waiting for more info on it. Lots of PRs were merged, thanks to all that tested, reviewed or otherwise helped out. Could y

Re: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-10-30 Thread Remi Bergsma
Hi Raja, Talked to Michael and he will improve the test soon. Regards, Remi On 30/10/15 17:44, "Raja Pullela" wrote: >Hi Michael > >Can you please add a wait for the template to download before the VM creation? > >Raja >Begin forwarded message: >From: Raja Pullela mailto:raja.pull...@citr

Re: test_vpc_vpn.py failing 2 BVTs on all Advnetworking config

2015-10-30 Thread Raja Pullela
Super, thanks Remi! > On Oct 30, 2015, at 10:35 PM, Remi Bergsma > wrote: > > Hi Raja, > > Talked to Michael and he will improve the test soon. > > Regards, > Remi > > > > > >> On 30/10/15 17:44, "Raja Pullela" wrote: >> >> Hi Michael >> >> Can you please add a wait for the template

RE: Build fails on LocalTemplateDownloadTest

2015-10-30 Thread Josh Harshman
Oh wow, I feel dumb for not noticing that... Builds fine now. Thanks :) Will submit a PR shortly. From: Wido den Hollander Sent: Friday, October 30, 2015 1:01 AM To: dev@cloudstack.apache.org Subject: Re: Build fails on LocalTemplateDownloadTest Josh Ha

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

2015-10-30 Thread K0zka
Github user K0zka commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-152597661 I do not know if a new web UI is needed, but tomcat 6 is a walking dead. @miguelaferreira yes correct, the session can not be created once the headers are out

Re: [4.6] VM Snapshots no longer work with managed storage

2015-10-30 Thread Mike Tutkowski
Hi Remi, Yeah, sorry...I should have provided more details in the ticket. It was late when I encountered the issue and just wanted to record that we have a situation where data is being destroyed on accident (which seemed like a blocker to me, but I'm OK if you'd like to change that status). I wen

[GitHub] cloudstack pull request: CLOUDSTACK-9004: Add features to HyperVEn...

2015-10-30 Thread jharshman
GitHub user jharshman opened a pull request: https://github.com/apache/cloudstack/pull/1013 CLOUDSTACK-9004: Add features to HyperVEnlightenmentFeatureDef Add function to set vapic, spinlock and retries Add function to get retry value Modify toString to output appropriate XML

[GitHub] cloudstack pull request: CLOUDSTACK-9013: Virtual router failed to...

2015-10-30 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1014 CLOUDSTACK-9013: Virtual router failed to start on KVM This fix a typo of commit 4a177031b055f3649e3b4a00c80eddb5cafa1dd7 You can merge this pull request into a Git repository by running:

[GitHub] cloudstack pull request: CLOUDSTACK-9013: Virtual router failed to...

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1014#issuecomment-152624797 Ping @wilderrodrigues to have a look --- 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 pro

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

2015-10-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/961#issuecomment-152631348 @K0zka FYI: it also didn't work in tomcat 7 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If yo

Re: [ANNOUNCE] CloudStack Certification !!!!

2015-10-30 Thread Erik Weber
My local test center can't find the exam, has anyone else had any issues? They can see LPI examns, but not LPI-japan ones.. -- Erik On Tue, Oct 27, 2015 at 6:16 PM, Giles Sirett wrote: > All > > [cross posting- sorry] > > *GET CERTIFICED ON APACHE CLOUDSTACK * > > As some of you may of heard,

Re: New PMC member: Erik Weber

2015-10-30 Thread Erik Weber
Thanks everyone! :-) -- Erik On Fri, Oct 30, 2015 at 3:32 PM, Wido den Hollander wrote: > Congrats! Welcome :) > > On 30-10-15 15:01, Giles Sirett wrote: > > All > > The PMC is delighted to announce that we have asked Erik weber to join > the PMC and he has accepted > > > > Please join me in

  1   2   >