Re: [DISCUSS] PR testing process

2016-03-30 Thread Bharat Kumar
Hi Sanjeev, working other piece currently, I have not stared on it. Thanks, Bharat. > On 31-Mar-2016, at 10:35 AM, Sanjeev Neelarapu > wrote: > > Hi Bharat, > > Just wanted to check if the CI is still getting the code from the PR branch > or from the latest

[GitHub] cloudstack pull request: CLOUDSTACK-9175: [VMware DRS] Adding new ...

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

RE: [DISCUSS] PR testing process

2016-03-30 Thread Sanjeev Neelarapu
Hi Bharat, Just wanted to check if the CI is still getting the code from the PR branch or from the latest master? Best Regards, Sanjeev N Chief Product Engineer, Accelerite Off: +91 40 6722 9368 | EMail: sanjeev.neelar...@accelerite.com -Original Message- From: Remi Bergsma

[GitHub] cloudstack pull request: Removed unused methods from EventBus inte...

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

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

2016-03-30 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/1425#issuecomment-203654592 @nvazquez, Nice changes, great job refactoring your implementation. I found no more troubles in this PR, then LGTM. Ty. --- If your project is set

Re: [ANNOUNCE] Registration open for CloudStack Collaboration Conference North America 2016

2016-03-30 Thread Pierre-Luc Dion
Minor clarification, Has you will see on the event site, it's from June 1st to 3rd. Since we are expecting people traveling on June 1st we were planning to startup the event on the Wednesday (June 1st) late afternoon. It's still not clear if we would initiate the hackathon, or perform a round

[GitHub] cloudstack pull request: CLOUDSTACK-9322: Support for Internal LB ...

2016-03-30 Thread pedro-martins
Github user pedro-martins commented on the pull request: https://github.com/apache/cloudstack/pull/1452#issuecomment-203581569 @nlivens great! And sorry for the delay =) I saw that you used the String.format as I suggested, but and about the using of CollectionUtils.isEmpty() in

[ANNOUNCE] Registration open for CloudStack Collaboration Conference North America 2016

2016-03-30 Thread Pierre-Luc Dion
All, We are pleased to announce that the registration for the CloudStack Collaboration Conference North America 2016 is open. We would like to invite you all to Montreal, Canada on June 2 and 3 to participate in this conference. Registrations are now available, so head over to the evenbrite (

[GitHub] cloudstack pull request: Removed unnecessary code from getGuestOsT...

2016-03-30 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1262#issuecomment-203569921 @cristofolini, I can give an LGTM for this PR. Thanks for the tests and the removal of unused code. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: Removed unnecessary code from getGuestOsT...

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

[GitHub] cloudstack pull request: Removed unused code from com.cloud.api.Ap...

2016-03-30 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1263#issuecomment-203492750 @DaanHoogland @rafaelweingartner well noted, I have to remove those ".class" files; @pdube I am going to verify this checkstyle error. Thanks. ---

[GitHub] cloudstack pull request: Remove classes with no references

2016-03-30 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1453#issuecomment-203484371 Thank you @pdube! --- 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

Re: Migrating CloudStack content from download.cloud.com

2016-03-30 Thread sebgoa
> On Mar 30, 2016, at 7:36 AM, Raja Pullela wrote: > > Just to summarize... and to close this thread for now, > > 1) short term - we will move the content to cloudstack.apt-get.eu > 2) long term - can someone from PMC or Cloudstack Apache Infra help find out >

[GitHub] cloudstack pull request: Remove classes with no references

2016-03-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1453#issuecomment-203469470 I built, ran the unit tests and ran the server with the simulator. Thanks for cleaning up! LGTM [INFO]

[GitHub] cloudstack pull request: writeIfNotHere requires an array of strin...

2016-03-30 Thread hogniRunar
Github user hogniRunar commented on the pull request: https://github.com/apache/cloudstack/pull/1456#issuecomment-203448721 To reproduce it you just need to set up a basic baremetal zone, start an instance and on the bootserver for baremetal you can look at

[GitHub] cloudstack pull request: writeIfNotHere requires an array of strin...

2016-03-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1456#issuecomment-203437320 Hi @hogniRunar, The code change LGTM. I verified for references to the function, but did not test it. Could you create a JIRA ticket for it please? Could you

[GitHub] cloudstack pull request: Removed unused code from com.cloud.api.Ap...

2016-03-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1263#issuecomment-203434250 Hi @GabrielBrascher Travis noted: Failed during checkstyle execution: There are 1 checkstyle errors. Have you run *mvn validate* on your code?

[GitHub] cloudstack pull request: Removed unused code from com.cloud.api.Ap...

2016-03-30 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1263#issuecomment-203406148 @GabrielBrascher, it seems that everything is ok with this PR (giving CI), but to push it forward with reviews, can you remove the .class from your

[GitHub] cloudstack pull request: Removed unused code from com.cloud.api.Ap...

2016-03-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1263#issuecomment-203396908 @GabrielBrascher you have .class files in your commit according to github. Can you check? And please repush anyway? thanks --- If your project is

[QUESTION] baremetal router

2016-03-30 Thread Daan Hoogland
People, in the following code the baremetal-vr.py configuration script is started unconditionally, this means it will be started on ssvm and cpvm as well. but also on virtual routervms; CMDLINE=$(cat /var/cache/cloud/cmdline) router=$(echo "$CMDLINE" | grep -o type=router) vpcrouter=$(echo

[GitHub] cloudstack pull request: Removed unused code from com.cloud.api.Ap...

2016-03-30 Thread bvbharatk
Github user bvbharatk commented on the pull request: https://github.com/apache/cloudstack/pull/1263#issuecomment-203392098 ### ACS CI BVT Run **Sumarry:** Build Number 147 Hypervisor xenserver NetworkType Advanced Passed=106 Failed=0 Skipped=4

RE: Migrating CloudStack content from download.cloud.com

2016-03-30 Thread Raja Pullela
Abhi, they all moved to support.accelerite.com and are accessible there. Best, Raja Senior Manager, Product Development Accelerite, 2055, Laurelwood Road, Santa Clara, CA 95054, USA Phone: 1-408-216-7010, www.accelerite.com, @accelerite -Original Message- From: Abhinandan Prateek

[GitHub] cloudstack pull request: writeIfNotHere requires an array of strin...

2016-03-30 Thread hogniRunar
GitHub user hogniRunar opened a pull request: https://github.com/apache/cloudstack/pull/1456 writeIfNotHere requires an array of strings, not a string You can merge this pull request into a Git repository by running: $ git pull https://github.com/greenqloud/cloudstack

Re: Migrating CloudStack content from download.cloud.com

2016-03-30 Thread Abhinandan Prateek
Raja, There are many trouble shooting articles on support.citrix.com, they have all been removed. Will you know where they are going to be moved. On 30/03/16, 11:06 AM, "Raja Pullela" wrote: >Just to summarize... and to close this thread for now, > >1)