[GitHub] cloudstack pull request: CLOUDSTACK-8476-Disable enable pod:

2015-05-26 Thread pritisarap12
Github user pritisarap12 closed the pull request at: https://github.com/apache/cloudstack/pull/261 --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-8476-Disable enable host:

2015-05-26 Thread pritisarap12
Github user pritisarap12 closed the pull request at: https://github.com/apache/cloudstack/pull/278 --- 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 featu

[GitHub] cloudstack pull request: Fixed VM Deployment Issue in(integration....

2015-05-26 Thread pritisarap12
Github user pritisarap12 closed the pull request at: https://github.com/apache/cloudstack/pull/312 --- 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 featu

[GitHub] cloudstack pull request: Fixed VM Deployment Issue in(integration....

2015-05-26 Thread gauravaradhye
Github user gauravaradhye commented on the pull request: https://github.com/apache/cloudstack/pull/312#issuecomment-105776689 Looks good, and as discusses with you, this test case has been run and it is passing now successfully. Merging this review request. --- If your project is set

[GitHub] cloudstack pull request: Skip vm migration code from the test sinc...

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

[GitHub] cloudstack pull request: CLOUDSTACK-8516: Fixed syntax errors and ...

2015-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/313 --- 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-8516: Fixed syntax errors and ...

2015-05-26 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/313 CLOUDSTACK-8516: Fixed syntax errors and pep8 issues in testpath_storage_migration.py Checked with pep8 and pylint. No issues now. You can merge this pull request into a Git repository by

[GitHub] cloudstack pull request: UDSTACK-8513-Fixed-VM-Deployment-Issue:

2015-05-26 Thread pritisarap12
GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/312 UDSTACK-8513-Fixed-VM-Deployment-Issue: --Fixed the vm deployment issue You can merge this pull request into a Git repository by running: $ git pull https://github.com/pritisarap12

Re: [GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread Erik Weber
Thanks for filling in the blanks Rajani! -- Erik On Wed, May 27, 2015 at 7:32 AM, Rajani Karuturi wrote: > On Wed, May 27, 2015 at 3:23 AM, Erik Weber wrote: > > > This is a perfect example of why we should use Jira for (almost all) > > commits. > > > > We know what this commit does, but we h

[GitHub] cloudstack pull request: CLOUDSTACK-8515: Skip snapshot test cases...

2015-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/311 --- 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-8515: Skip snapshot test cases...

2015-05-26 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/311 CLOUDSTACK-8515: Skip snapshot test cases for HyperV and LXC Skip because unsupported feature. You can merge this pull request into a Git repository by running: $ git pull https://git

[GitHub] cloudstack pull request: Skip vm migration code from the test sinc...

2015-05-26 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/310 Skip vm migration code from the test since it is not supported on LXC You can merge this pull request into a Git repository by running: $ git pull https://github.com/sanju1010/cloudstack

Re: [GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread Rajani Karuturi
On Wed, May 27, 2015 at 3:23 AM, Erik Weber wrote: > This is a perfect example of why we should use Jira for (almost all) > commits. > > We know what this commit does, but we have to guess why it was done this > way. > And we don't know how to reproduce the original issue. > > It references what

[GitHub] cloudstack pull request: CLOUDSTACK-8514: test_haproxy.py - Verify...

2015-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/309 --- 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-8514: test_haproxy.py - Verify...

2015-05-26 Thread gauravaradhye
GitHub user gauravaradhye opened a pull request: https://github.com/apache/cloudstack/pull/309 CLOUDSTACK-8514: test_haproxy.py - Verifying router state before validating network rules The test case test_08_create_policy_router_stopped fails in automation runs because the router st

Re: SystemVM (kvm) : 3 different versions for the same CloudStack version (4.5.1)

2015-05-26 Thread Rajani Karuturi
On Wed, May 27, 2015 at 12:54 AM, Rohit Yadav wrote: > > > On 26-May-2015, at 6:27 am, Rajani Karuturi wrote: > > > > I dont think we should mention shapeblue.com/cloud.com templates in the > > docs. They may not really be from the point where RC is voted. I think it > > should be the apt-get.eu

Re: SystemVM (kvm) : 3 different versions for the same CloudStack version (4.5.1)

2015-05-26 Thread Rajani Karuturi
I said that because, I dont know from which commit the template is built from and am not sure if thats the one that is voted. Neither do I have access to check the same or correct the template incase of an error. So, I am not comfortable with putting a reference to that in the official documentatio

Re: [GitHub] cloudstack pull request: CLOUDSTACK-8231: fixed merge conflicts wi...

2015-05-26 Thread Rajani Karuturi
Yesterday I noticed some sync issues between github and git-wip-us.apache. github didnt have all the commits from apache. May be that is the reason why you are seeing additional commits in your pull request. ~Rajani On Tue, May 26, 2015 at 2:45 PM, Vadim Kimlaychuk wrote: > Rohit, > >

Re: refresh browser - logged out from ACS ?

2015-05-26 Thread ilya
But it was not refresh - to best of my recollection.. On 5/26/15 8:27 PM, ilya wrote: I vaguely recall Rohit mentioned it was some sort of security fix that was causing this side effect due to the way sessionids were handled.. On 5/26/15 8:15 AM, Andrija Panic wrote: Thx Rafael, as usuall :)

Re: refresh browser - logged out from ACS ?

2015-05-26 Thread ilya
I vaguely recall Rohit mentioned it was some sort of security fix that was causing this side effect due to the way sessionids were handled.. On 5/26/15 8:15 AM, Andrija Panic wrote: Thx Rafael, as usuall :) I remember there was some thread on this topic, but cant really find it... On 26 May 2

Re: CLOUDSTACK-8407: Presharedkey is not accessible

2015-05-26 Thread Andrija Panic
I can confirm it's there on 4.5.1 - so all fine for this one :) On 27 May 2015 at 00:22, Will Stevens wrote: > OK. I will test. Thanks. > On May 26, 2015 6:17 PM, "Rohit Yadav" wrote: > > > > > > On 27-May-2015, at 12:09 am, Will Stevens > wrote: > > > > > > What do you mean by "I had already

Re: CLOUDSTACK-8407: Presharedkey is not accessible

2015-05-26 Thread Will Stevens
OK. I will test. Thanks. On May 26, 2015 6:17 PM, "Rohit Yadav" wrote: > > > On 27-May-2015, at 12:09 am, Will Stevens wrote: > > > > What do you mean by "I had already fixed this on 4.5/master before 4.5.1 > > was released, therefore 4.5.1 users should not experience any issue.”? > > The regres

Re: CLOUDSTACK-8407: Presharedkey is not accessible

2015-05-26 Thread Rohit Yadav
> On 27-May-2015, at 12:09 am, Will Stevens wrote: > > What do you mean by "I had already fixed this on 4.5/master before 4.5.1 > was released, therefore 4.5.1 users should not experience any issue.”? The regression exists for 4.4 but just fixed. It’s not an issue for 4.5.1 release and latest 4.

Re: CLOUDSTACK-8407: Presharedkey is not accessible

2015-05-26 Thread Will Stevens
What do you mean by "I had already fixed this on 4.5/master before 4.5.1 was released, therefore 4.5.1 users should not experience any issue."? So is the preshared key available in the API now? Is there a way to get the preshared key if you don't have access to the DB? Will *Will STEVENS* Lead

[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/308#discussion_r31085151 --- Diff: ui/scripts/cloudStack.js --- @@ -117,22 +117,15 @@ } return cookieValue;

IPv6 support in CloudStack for VPC(Advanced Networking)

2015-05-26 Thread Suresh Ramamurthy
Hello Everyone, We at Schuberg Philis, had couple of discussions regarding the support for IPv6 in CloudStack for VPC and we have come up with first version of its design document. Please find the design document at https://cwiki.apache.org/confluence/display/CLOUDSTACK/IPv6+in+VPC+Router Design

[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/308#issuecomment-105682213 I searched around for that reference and found nothing useful, perhaps @bhaisaab can shed some light, since it was signed off by him :) If I understood correct

Re: CLOUDSTACK-8407: Presharedkey is not accessible

2015-05-26 Thread Rohit Yadav
Hi Will, I had already fixed this on 4.5/master before 4.5.1 was released, therefore 4.5.1 users should not experience any issue. I’ve just checked and cherry-picked the fix on 4.4 branch. > On 26-May-2015, at 10:57 pm, Will Stevens wrote: > > @Rohit: Do you have any suggestions for how to acc

[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/308#discussion_r31084802 --- Diff: ui/scripts/cloudStack.js --- @@ -329,7 +325,6 @@ g_regionsecondaryenabled = null; g_

Re: [GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread Erik Weber
This is a perfect example of why we should use Jira for (almost all) commits. We know what this commit does, but we have to guess why it was done this way. And we don't know how to reproduce the original issue. It references what I believe to be an internal Citrix Jira/tracking#, so if anyone tha

[VOTE] Apache Cloudstack 4.4.4

2015-05-26 Thread Daan Hoogland
Hi All, For those still interested in the 4.4 tree I've created a 4.4.4 release, with the following artifacts up for a vote: Git Branch and Commit SH:https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=shortlog;h=refs/heads/4.4-RC20150526T2323 Commit: c7600582a9c6a7441db3e26a5641d4d0ad7130

[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread terbolous
Github user terbolous commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/308#discussion_r31083230 --- Diff: ui/scripts/cloudStack.js --- @@ -329,7 +325,6 @@ g_regionsecondaryenabled = null; g_l

[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/pull/308#issuecomment-105672476 @bhaisaab please take 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 project do

[GitHub] cloudstack pull request: Reinstate working sessions in browser

2015-05-26 Thread rsafonseca
GitHub user rsafonseca opened a pull request: https://github.com/apache/cloudstack/pull/308 Reinstate working sessions in browser I've seen that session persistence stopped working due to https://github.com/apache/cloudstack/commit/19e3c0168e744a76b5e1dc24a5eafa776d342404 From

[GitHub] cloudstack pull request: CLOUDSTACK-6181: Allow RBD volumes to be ...

2015-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/281 --- 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-8407: Presharedkey is not accessible

2015-05-26 Thread Will Stevens
@Rohit: Do you have any suggestions for how to access the preshared key now? I see you removed the ability for it to accessed through the API. https://issues.apache.org/jira/browse/CLOUDSTACK-8407 Cheers, *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Mont

Re: SystemVM (kvm) : 3 different versions for the same CloudStack version (4.5.1)

2015-05-26 Thread Rohit Yadav
> On 26-May-2015, at 6:27 am, Rajani Karuturi wrote: > > I dont think we should mention shapeblue.com/cloud.com templates in the > docs. They may not really be from the point where RC is voted. I think it > should be the apt-get.eu / jenkins. I’m not sure why this is relevant to the specific iss

[GitHub] cloudstack pull request: CLOUDSTACK-7888: unable to create remote ...

2015-05-26 Thread primechuck
GitHub user primechuck opened a pull request: https://github.com/apache/cloudstack/pull/307 CLOUDSTACK-7888: unable to create remote vpn because of special character in password Always enter chap-secrets as a quoted field. In the event of special characters it creates and deletes t

RE: SystemVM (kvm) : 3 different versions for the same CloudStack version (4.5.1)

2015-05-26 Thread Paul Angus
Hi Ranji, I don't follow your logic - British Telecom have donated the infra specifically to host community's convenience binaries much as Wido does on apt-get.eu And Jenkins constantly builds the latest systemVM template for a full dot release and doesn't discriminate between minor dot releases

[GitHub] cloudstack pull request: CLOUDSTACK-6181: Allow RBD volumes to be ...

2015-05-26 Thread andrijapanic
Github user andrijapanic commented on the pull request: https://github.com/apache/cloudstack/pull/281#issuecomment-105579878 Daan, will test today/tomorow (Im in a little bit specific situation right now)... Thx for backporting this. --- If your project is set up for it, you can repl

[GitHub] cloudstack pull request: CLOUDSTACK-6181: Allow RBD volumes to be ...

2015-05-26 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/281#issuecomment-105575874 @andrijapanic do you still want this? I expect you to test it when we make an RC! --- If your project is set up for it, you can reply to this email and have yo

[GitHub] cloudstack pull request: CLOUDSTACK-8506 extra check on unknown vl...

2015-05-26 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/306 CLOUDSTACK-8506 extra check on unknown vlan @wilderrodrigues You can merge this pull request into a Git repository by running: $ git pull https://github.com/DaanHoogland/cloudstack ma

[GitHub] cloudstack pull request: 4.4.4 fixes

2015-05-26 Thread DaanHoogland
Github user DaanHoogland closed the pull request at: https://github.com/apache/cloudstack/pull/305 --- 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 featu

[GitHub] cloudstack pull request: CLOUDSTACK-8506

2015-05-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/303 --- 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: refresh browser - logged out from ACS ?

2015-05-26 Thread Andrija Panic
Thx Rafael, as usuall :) I remember there was some thread on this topic, but cant really find it... On 26 May 2015 at 17:14, Rafael Fonseca wrote: > Hi Andrija, > > I noticed the same is also happening on the 4.6.0-SNAPSHOT .. it's a bit > annoying. > > I'll have a closer look later today if i

Re: refresh browser - logged out from ACS ?

2015-05-26 Thread Rafael Fonseca
Hi Andrija, I noticed the same is also happening on the 4.6.0-SNAPSHOT .. it's a bit annoying. I'll have a closer look later today if i can find the time for it :) On Tue, May 26, 2015 at 4:11 PM, Andrija Panic wrote: > Hi guys, > > just wondering - when I refresh browser/UI I get logged out

[GitHub] cloudstack pull request: Build and create tomcat, fix several othe...

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/238#discussion_r31043937 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtConsoleProxyLoadCommandWrapper.java --- @@ -49,6 +49,11 @@ publ

Re: support for /31 Networks

2015-05-26 Thread Daan Hoogland
Devender, do you have a test possibility? Wilder made a fix for master which I backported to 4.4: https://github.com/DaanHoogland/cloudstack/tree/4.4 If you can please test it. Op ma 25 mei 2015 om 12:38 schreef Daan Hoogland : > Devender, You can backport it or wait for 4.4.4. There is still som

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11365325 Hi @rsafonseca, It's fixed, but I haven't created a PR yet. I want to run a huge bunch of tests against it. S

refresh browser - logged out from ACS ?

2015-05-26 Thread Andrija Panic
Hi guys, just wondering - when I refresh browser/UI I get logged out of ACS - 4.4.3 (testing with 4.5.1 in few minutes...). I remember there was some thread on this, but can't really find it anywhere This behaviour is not present in 4.3 and prior AFAIK. Any tips ? -- Andrija Panić

[GitHub] cloudstack pull request: 4.4.4 fixes

2015-05-26 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/305 4.4.4 fixes You can merge this pull request into a Git repository by running: $ git pull https://github.com/DaanHoogland/cloudstack 4.4 Alternatively you can review and apply these c

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Andrija Panic
For the time being, we are looking into OSS solutions for the storage, but will consider other stuff to. Thx for sharing On 26 May 2015 at 15:24, Erik Weber wrote: > If it's within your budget, we've had good results with NetApp and are > happy to recommend it. > I'm not responsible for configu

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Erik Weber
If it's within your budget, we've had good results with NetApp and are happy to recommend it. I'm not responsible for configuring it, so I cannot say anything about how we do that, but we're using it's clustering abilities and haven't had any downtime on it. I believe the guys at Shuberg Philis al

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Andrija Panic
Thx Erik, Interesting stuff :) We will test for now how Swift emulates S3 :) In general, we try to avoid single/multiply NFS boxes for SPOF and scaling reasons, as Secondary Storage, so we are yet to come with good solution, that works in ACS in nice fashion... On 26 May 2015 at 15:13, Erik Web

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Erik Weber
These are probably a bit early to use anywhere production, but they do look interesting: https://github.com/bouncestorage/swiftproxy https://github.com/bouncestorage/s3proxy -- Erik On Tue, May 26, 2015 at 3:09 PM, Pierre-Luc Dion wrote: > S3 seams the have different behavior, ex: download PC

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Pierre-Luc Dion
S3 seams the have different behavior, ex: download PCT status is display. I haven't extensively test but seams to work well, On Tue, May 26, 2015 at 9:05 AM, Andrija Panic wrote: > Guys, is using S3 for Secondary Storage any better supported in ACS in your > opinion? We plan to also try talkin

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Andrija Panic
Guys, is using S3 for Secondary Storage any better supported in ACS in your opinion? We plan to also try talking S3 to Swift, to see how that works... Thx On 26 May 2015 at 15:03, Erik Weber wrote: > On Tue, May 26, 2015 at 2:51 PM, Pierre-Luc Dion > wrote: > > > Hi Andrija, > > > > Swift inte

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Erik Weber
On Tue, May 26, 2015 at 2:51 PM, Pierre-Luc Dion wrote: > Hi Andrija, > > Swift integration in cloudstack + XenServer require interaction from > Xenserver to initiate connection to Swift proxies, this is made using > /etc/xapi.d/plugins/swiftxen which is a fairly old python script that only > su

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Andrija Panic
Hi Pierre, I understand the Xen limitations, we are here with KVM, and it seems most of the problems I described, are actually results of our modification to support v2 auth... My comments/answer down below :) On 26 May 2015 at 14:51, Pierre-Luc Dion wrote: > Hi Andrija, > > Swift integration

Re: Swift 2.x as Secondary Storage - auth v2 ?

2015-05-26 Thread Pierre-Luc Dion
Hi Andrija, Swift integration in cloudstack + XenServer require interaction from Xenserver to initiate connection to Swift proxies, this is made using /etc/xapi.d/plugins/swiftxen which is a fairly old python script that only support Swift api V1. I've tried to upgrade that script which seams to

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11362664 Cool, @rsafonseca ! I will post the results here. Thanks for the help. Cheers, Wilder

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11362623 Yep... I got the line (code) on it here. But It's a weird situation and I want to make sure that the agent bui

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11362581 I can't test ATM, but i will do so later when i get home :) --- If your project is set up for it, you can reply to this em

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11362538 Well.. it's actually this line, perhaps i was mistaken about line 72.. just got that from my fuzzy memory :)

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11362482 Build is done: http://jenkins.buildacloud.org/job/package-centos7-master/171/ I will try with that agent now.

[GitHub] cloudstack pull request: Pep8 changes that got lost with the stran...

2015-05-26 Thread isoutham
Github user isoutham commented on the pull request: https://github.com/apache/cloudstack/pull/304#issuecomment-105500276 Jenkins fails on --- 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

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11362169 Hi @rsafonseca Thanks for the logs... we are now on the same page: I also built the agent and tried (with Ma

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11362046 Just to be clear... NO error is logged in the agent logs when this occurs (with the current code). It just keeps trying to

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11361983 java.lang.NullPointerException at com.cloud.hypervisor.kvm.resource.wrapper.LibvirtStartCommandWrapper.execute(LibvirtS

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11361958 It's actually sending an UnsupportedAnswer.class object, that is why it's not StartAnswer.class. Putting some debu

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11361925 @wilderrodrigues Yes, i mean creating datacenter manually, since creating it in simulator with marvin shows no errors.

[GitHub] cloudstack pull request: CLOUDSTACK-8445: Keep only dvs tag for te...

2015-05-26 Thread gauravaradhye
Github user gauravaradhye closed the pull request at: https://github.com/apache/cloudstack/pull/300 --- 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 feat

[GitHub] cloudstack pull request: Pep8 changes that got lost with the stran...

2015-05-26 Thread isoutham
GitHub user isoutham opened a pull request: https://github.com/apache/cloudstack/pull/304 Pep8 changes that got lost with the strange double push request probl… pep8 changes and a couple of minor fixes to the test/systemvm/python unit tests. The unit tests are just a plac

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11360697 @rsafonseca I'm building the agent for centos7 now, it will take a while. In meantime, could you please send

Re: [bug]Preshared key not created when enable remote VPN

2015-05-26 Thread Andrija Panic
Ah, ok. Sorry for confision. Thanks Jayapal On 26 May 2015 at 11:43, Jayapal Reddy Uradi wrote: > Andrija, > > This commit is already there in master. > d81e831214cafc2c9f1e4f8e7de56fca65c2215f > > Thanks, > Jayapal > > On 26-May-2015, at 3:03 PM, Andrija Panic > wrote: > > > Jayapal, > > > >

[GitHub] cloudstack pull request: CLOUDSTACK-8506

2015-05-26 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/303#issuecomment-105469798 lgtm, the pull request builder is failing with a known issue. testing locally --- If your project is set up for it, you can reply to this email and have your r

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11360322 Hi @rsafonseca So you built the Agent. I will do the same and give it a try. Concerning the version

Re: [bug]Preshared key not created when enable remote VPN

2015-05-26 Thread Jayapal Reddy Uradi
Andrija, This commit is already there in master. d81e831214cafc2c9f1e4f8e7de56fca65c2215f Thanks, Jayapal On 26-May-2015, at 3:03 PM, Andrija Panic wrote: > Jayapal, > > did I understood correctly, that you just reverted it back so we can get > PSK in APU/UI? > > Thanks, > Andrija > > On 2

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11360242 Hi @wilderrodrigues Everything is built on the latest code in my testing, so agent is 4.6.0-SNAPSHOT as well.

Re: [bug]Preshared key not created when enable remote VPN

2015-05-26 Thread Andrija Panic
Jayapal, did I understood correctly, that you just reverted it back so we can get PSK in APU/UI? Thanks, Andrija On 26 May 2015 at 11:20, Jayapal Reddy Uradi wrote: > Hi Andrija, > > Listing PSK in API is removed for security reason and it got reverted > again. > This is commit revert id d81e8

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11360136 Hi @rsafonseca The snippet you pointed in the previous comment (i.e. try/catch) with the Exception being cau

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11359931 I did manual tests as well. Doing again now... ![image](https://cloud.githubusercontent.com/assets/5

Re: [bug]Preshared key not created when enable remote VPN

2015-05-26 Thread Jayapal Reddy Uradi
Hi Andrija, Listing PSK in API is removed for security reason and it got reverted again. This is commit revert id d81e831214cafc2c9f1e4f8e7de56fca65c2215f Thanks, Jayapal On 26-May-2015, at 2:45 PM, Andrija Panic wrote: > Hi Jayapal, > > "vpn_server_addr_id" "account_id" "network_id" "doma

RE: [GitHub] cloudstack pull request: CLOUDSTACK-8231: fixed merge conflicts wi...

2015-05-26 Thread Vadim Kimlaychuk
Rohit, I have merged my local branch with master and resolved conflicts. Pull request showed modifications that I have made precisely. But when I created the request it contains multiple files (not just those I have changed). Please review this. I think I have done something wrong agai

[GitHub] cloudstack pull request: CLOUDSTACK-8506

2015-05-26 Thread wilderrodrigues
GitHub user wilderrodrigues opened a pull request: https://github.com/apache/cloudstack/pull/303 CLOUDSTACK-8506 Removing the previous logic and adding a method to check if the CIDR is 31 bit prefixed Adding tests for the new method @DaanHoogland, could you have a look?

Re: [bug]Preshared key not created when enable remote VPN

2015-05-26 Thread Andrija Panic
Hi Jayapal, "vpn_server_addr_id" "account_id" "network_id" "domain_id" "local_ip" "ip_range" "ipsec_psk" "state" "id" "uuid" "vpc_id" "display" "16" "2" \N "1" "192.168.156.1" "192.168.156.2-192.168.156.50" "5bqKeKhqqWvHx8Re2hZVK7AC" "Running" "1" "b037d8b2-1ebf-4450-8c76-7bd9c73f0745" "7" "1" "13

Re: [bug]Preshared key not created when enable remote VPN

2015-05-26 Thread Jayapal Reddy Uradi
Hi Andrija, Is it not listing in the API or the PSK itself is not there DB ? Can you check the table once. SELECT * FROM cloud.remote_access_vpn; Thanks, Jayapal On 26-May-2015, at 2:35 PM, Andrija Panic wrote: > Hi, > > anybody familiar with some workarround/fix for this one: > https://issues

[GitHub] cloudstack pull request: CLOUDSTACK-8231: fixed merge conflicts wi...

2015-05-26 Thread vadimkim
GitHub user vadimkim opened a pull request: https://github.com/apache/cloudstack/pull/302 CLOUDSTACK-8231: fixed merge conflicts with previous patch We need to port this update to 4.5 branch as well. You can merge this pull request into a Git repository by running: $ git pull h

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11359702 This is where ANY exception gets thrown into an UnsupportedCommandAnswer, with no information about the actual exception be

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11359669 This happens in all my environments (Ubuntu, Fedora21, Centos6, Centos7) All marvin tests run fine in all environments.

[bug]Preshared key not created when enable remote VPN

2015-05-26 Thread Andrija Panic
Hi, anybody familiar with some workarround/fix for this one: https://issues.apache.org/jira/browse/CLOUDSTACK-8407 ? Thanks, -- Andrija Panić

[GitHub] cloudstack pull request:

2015-05-26 Thread wilderrodrigues
Github user wilderrodrigues commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11359616 @rsafonseca We running several Marvin tests (component/smoke) with KVm before sending PRs. We execute test_r

[GitHub] cloudstack pull request: CLOUDSTACK-8231: reverted 7ee9b10 commit ...

2015-05-26 Thread vadimkim
Github user vadimkim closed the pull request at: https://github.com/apache/cloudstack/pull/274 --- 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 i

[GitHub] cloudstack pull request:

2015-05-26 Thread rsafonseca
Github user rsafonseca commented on the pull request: https://github.com/apache/cloudstack/commit/0cdb4b610865fa5493cc79e9c9ef310bdf09ccaf#commitcomment-11359437 Hi @wilderrodrigues The refactoring of the Libvirt broke Libvirt functionality in runtime. Whenever spinning up a n

Re: Cloudstack 4.3.2 with KVM system template from shapeblue not supported by qemu version

2015-05-26 Thread Andrija Panic
Just convert the template (use some newer ubuntu version of qemu-img) to qcow2 version 2... They provide qcow2 version 3, you need version 2 for centos something like (but please google and confirm): qemu-img convert -f qcow2 /var/lib/libvirt/images/qcow2v3.img -O qcow2 -o compat=0.10 /var/li

[GitHub] cloudstack pull request: Disable enable zone pod cluster and host

2015-05-26 Thread pritisarap12
Github user pritisarap12 commented on the pull request: https://github.com/apache/cloudstack/pull/285#issuecomment-105429711 Done with review changes. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does n

[GitHub] cloudstack pull request: CLOUDSTACK-8308-Adding-automation-test-ca...

2015-05-26 Thread pritisarap12
GitHub user pritisarap12 opened a pull request: https://github.com/apache/cloudstack/pull/301 CLOUDSTACK-8308-Adding-automation-test-cases-for-VM/Volume-snapshot-h… …ardning: --Automating testcases for volume snapshot hardning testpath You can merge this pull reques

[GitHub] cloudstack pull request: CLOUDSTACK-6181: Allow RBD volumes to be ...

2015-05-26 Thread mlsorensen
Github user mlsorensen commented on the pull request: https://github.com/apache/cloudstack/pull/281#issuecomment-105419893 Just make sure someone has actually used it to resize a volume with data. On Mon, May 25, 2015 at 5:47 AM, Remi Bergsma wrote: > What do we w