[DISCUSS] PR testing process

2016-03-28 Thread Sanjeev Neelarapu
Hi, Currently CI is picking the code from the PR branch, which may or may not be rebased with latest master. This is causing test failures even though they were fixed in latest master. e.g.: test_vpc_site2site_vpn. Error Message local variable 'vm1' referenced before assignment. There were few

Re: [DISCUSS] PR testing process

2016-03-28 Thread Remi Bergsma
Hi Sanjeev, I’d propose taking the latest master, ensure it is up2date, make a temp branch then merge the PR to be tested on that. This ensures you test the result of the merge, like it will happen om master. For the merging, use the same script we use when we merge for real: https://github.com

Re: [DISCUSS] PR testing process

2016-03-28 Thread Bharat Kumar
Hi Sanjeev, Thanks for bringing this up. This is happening because the PR code and the master have deviated since the pr has been created(assuming that PR was rebased with master at the time of creation) and we start testing it. Ideally we should start testing as soon as the pr is created but

Re: [DISCUSS] PR testing process

2016-03-28 Thread Remi Bergsma
To be more exact: Merge to the base branch the PR was created against. Could be master, could be 4.7 and such. The code snipped I linked to also does that. Sent from my iPhone > On 28 Mar 2016, at 09:13, Remi Bergsma wrote: > > Hi Sanjeev, > > I’d propose taking the latest master, ensure it

Re: [DISCUSS] PR testing process

2016-03-28 Thread Remi Bergsma
Indeed. If a merge fails then you'll not be able to merge it to master or another release branch later on anyway so the author must rebase against the base branch first. No need spending test cycles on that. Most likely github already reports that PR as 'unstable' so you could also check for t

RE: Migrating CloudStack content from download.cloud.com

2016-03-28 Thread Paul Angus
If someone can figure out a community/ASF acceptable way of the ACS community owning a suitable vendor-neutral domain name, within the community we have enough resources to physically host it (including the space donated by BT via ShapeBlue). Regards, Paul Angus paul.an...@shapeblue.com  ww

Re: [DISCUSS] Request for comments: Dynamic Role Based API Access Checker for CloudStack

2016-03-28 Thread Koushik Das
Thanks Rohit, for the replies. Added some more comments based on the replies. -Koushik From: Rohit Yadav Sent: Friday, March 25, 2016 6:42 PM To: dev@cloudstack.apache.org; us...@cloudstack.apache.org Subject: RE: [DISCUSS] Request for comments: Dynamic R

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-28 Thread sanju1010
GitHub user sanju1010 opened a pull request: https://github.com/apache/cloudstack/pull/1455 [CLOUDSTACK-9328]: Fix vlan issues from test suite test_privategw_acl.py in BVT Please refer to CLOUDSTACK-9328 for the details. Test Results: == test_01_vpc_privateg

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-28 Thread shwetaag
Github user shwetaag commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1455#discussion_r57562054 --- Diff: test/integration/smoke/test_privategw_acl.py --- @@ -287,6 +298,16 @@ def performVPCTests(self, vpc_off, restart_with_cleanup = False):

[GitHub] cloudstack pull request: [CLOUDSTACK-9328]: Fix vlan issues from t...

2016-03-28 Thread sanju1010
Github user sanju1010 commented on the pull request: https://github.com/apache/cloudstack/pull/1455#issuecomment-202342204 Made changes as per Shwetaag's comments. --- 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 p

[GitHub] cloudstack pull request: Map LDAP group to Cloudstack account

2016-03-28 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1285#issuecomment-202347507 ### ACS CI BVT Run **Sumarry:** Build Number 139 Hypervisor xenserver NetworkType Advanced Passed=104 Failed=1 Skipped=4

Introduction

2016-03-28 Thread Boris Stoyanov
Hi CloudStack, My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m based in Sofia, Bulgaria. I will be taking the role of Software Engineer in Test, and as you may have guessed I’ll mostly focus on testing CloudStack. I have about 10 years of experience in testing, whic

[GitHub] cloudstack pull request: maven: Upgrade dependency versions

2016-03-28 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/1397#issuecomment-202369219 +1 to all the Travis changes based on code review and verification of travis tests. I would have put those changes in a separate PR however. --- If your project is

Re: Introduction

2016-03-28 Thread Nux!
Hi Boris, Welcome aboard. :-) Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Boris Stoyanov" > To: dev@cloudstack.apache.org > Sent: Monday, 28 March, 2016 13:21:14 > Subject: Introduction > Hi CloudStack, > > My name is B

Re: Introduction

2016-03-28 Thread Simon Weller
Welcome Boris. - Si From: Boris Stoyanov Sent: Monday, March 28, 2016 7:21 AM To: dev@cloudstack.apache.org Subject: Introduction Hi CloudStack, My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m based in Sofia, Bulgaria. I wil

Re: Introduction

2016-03-28 Thread Erik Weber
Welcome :-) Den mandag 28. mars 2016 skrev Boris Stoyanov følgende: > Hi CloudStack, > > My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. > I’m based in Sofia, Bulgaria. I will be taking the role of Software > Engineer in Test, and as you may have guessed I’ll mostly focus

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-28 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57573444 --- Diff: core/src/com/cloud/agent/api/storage/PrimaryStorageDownloadCommand.java --- @@ -87,4 +88,12 @@ public String getPrimaryStorageUrl() {

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-28 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57573475 --- Diff: core/src/com/cloud/agent/api/storage/CopyVolumeCommand.java --- @@ -75,4 +76,12 @@ public boolean toSecondaryStorage() { public String

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-28 Thread jburwell
Github user jburwell commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57573431 --- Diff: core/src/com/cloud/agent/api/storage/ListTemplateCommand.java --- @@ -43,6 +49,10 @@ public DataStoreTO getDataStore() { return st

[GitHub] cloudstack pull request: OSPF: adding dynamically routing capabili...

2016-03-28 Thread agneya2001
Github user agneya2001 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1371#discussion_r57577607 --- Diff: core/src/com/cloud/agent/api/storage/CopyVolumeCommand.java --- @@ -75,4 +76,12 @@ public boolean toSecondaryStorage() { public Stri

[GitHub] cloudstack pull request: CLOUDSTACK-9200: Fixed failed to delete s...

2016-03-28 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1282#issuecomment-202495572 ### ACS CI BVT Run **Sumarry:** Build Number 140 Hypervisor xenserver NetworkType Advanced Passed=104 Failed=1 Skipped=4

[GitHub] cloudstack pull request: CLOUDSTACK-9298: Improve performance of r...

2016-03-28 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1425#issuecomment-202498197 Thanks @rafaelweingartner! @pedro-martins could you review the last changes? --- If your project is set up for it, you can reply to this email and have your re

Re: Introduction

2016-03-28 Thread Tutkowski, Mike
Welcome aboard, Boris! > On Mar 28, 2016, at 6:21 AM, Boris Stoyanov > wrote: > > Hi CloudStack, > > My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m > based in Sofia, Bulgaria. I will be taking the role of Software Engineer in > Test, and as you may have guessed

Re: Travis build and tests

2016-03-28 Thread Rohit Yadav
> Regards, Rohit Yadav rohit.ya...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue On 26-Mar-2016, at 9:56 PM, Sebastien Goasguen wrote: > > Rohit, > > Thank you, thank you so very very much for this ! :) Thanks Sebastien, we all need this >

[GitHub] cloudstack pull request: Fix Sync of template.properties in Swift

2016-03-28 Thread kollyma
Github user kollyma commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-202547429 @syed: thanks for the feedback. Here a more detailed description, based on your comments. Setup: CS4.8, nfs secondary storage, kvm with local storage ```

Re: Introduction

2016-03-28 Thread Ahmad Emneina
Welcome Bobby! Ahmad E > On Mar 28, 2016, at 5:21 AM, Boris Stoyanov > wrote: > > Hi CloudStack, > > My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m > based in Sofia, Bulgaria. I will be taking the role of Software Engineer in > Test, and as you may have guessed

[SSL CERTS] Importing ROOT and INTERMEDIATE certs for SSVM

2016-03-28 Thread ilya
I'm having difficulty getting ROOT and INTERMEDIATE certificates to show up in SSVM java keystore. I've followed the procedure on http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.8/systemvm.html?highlight=pkcs and https://cwiki.apache.org/confluence/display/CLOUDSTACK/P

Re: Introduction

2016-03-28 Thread ilya
Hi Boris Welcome! On 3/28/16 5:21 AM, Boris Stoyanov wrote: > Hi CloudStack, > > My name is Boris Stoyanov (Bobby) and today is my first day @ShapeBlue. I’m > based in Sofia, Bulgaria. I will be taking the role of Software Engineer in > Test, and as you may have guessed I’ll mostly focus on t

[GitHub] cloudstack pull request: CLOUDSTACK-9199: Fixed deployVirtualMachi...

2016-03-28 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1280#issuecomment-202622942 ### ACS CI BVT Run **Sumarry:** Build Number 141 Hypervisor xenserver NetworkType Advanced Passed=101 Failed=5 Skipped=4

[GitHub] cloudstack-cloudmonkey pull request: Build a CI environment in ord...

2016-03-28 Thread imduffy15
Github user imduffy15 closed the pull request at: https://github.com/apache/cloudstack-cloudmonkey/pull/2 --- 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 th

[GitHub] cloudstack pull request: CLOUDSTACK-9198: Virtual router gets depl...

2016-03-28 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1278#issuecomment-202714007 ### ACS CI BVT Run **Sumarry:** Build Number 142 Hypervisor xenserver NetworkType Advanced Passed=99 Failed=3 Skipped=4