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

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

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

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

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

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

Re: Introduction

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

[SSL CERTS] Importing ROOT and INTERMEDIATE certs for SSVM

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

Re: Introduction

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 >

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

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: Travis build and tests

> 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,

Re: Introduction

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 >

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

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

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

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: OSPF: adding dynamically routing capabili...

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

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

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

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

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

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

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() {

Re: Introduction

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

Re: Introduction

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

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

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

Introduction

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,

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

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

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

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

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

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...

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: ==

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

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]

RE: Migrating CloudStack content from download.cloud.com

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 

Re: [DISCUSS] PR testing process

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

Re: [DISCUSS] PR testing process

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

Re: [DISCUSS] PR testing process

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

Re: [DISCUSS] PR testing process

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:

[DISCUSS] PR testing process

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

[GitHub] cloudstack pull request: CLOUDSTACK-9323: Fix cancel host maintena...

Github user abhinandanprateek commented on the pull request: https://github.com/apache/cloudstack/pull/1454#issuecomment-202256571 Marvin test output: root@ccp:~/cloudstack(host-maint)# ./host_maint.sh ++ date + echo Mon Mar 28 11:47:45 IST 2016 Mon Mar 28