[GitHub] cloudstack pull request: CLOUDSTACK-9349: Enable root disk detach ...

2016-04-23 Thread dmabry
Github user dmabry commented on the pull request: https://github.com/apache/cloudstack/pull/1500#issuecomment-213853837 I made all requested changes. I'm going to run a full bubble CI test against this PR and I will post the results. --- If your project is set up for it, you can

[GitHub] cloudstack pull request: [Bug-Id: CLOUDSTACK-9306]Replace testdata...

2016-04-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1436#issuecomment-213841496 Please review the details of the CI run. I am using a KVM [bubble](https://github.com/MissionCriticalCloud/bubble-toolkit) to test this and I have not tried to

[GitHub] cloudstack pull request: [Bug-Id: CLOUDSTACK-9306]Replace testdata...

2016-04-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1436#issuecomment-213841009 ### CI RESULTS ``` Tests Run: 9 Skipped: 0 Failed: 0 Errors: 2 ``` **Summary of the problem(s):** ```

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-04-23 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-213825554 regression suite passed: [1516.results.network.txt](https://github.com/apache/cloudstack/files/233136/1516.results.network.txt)

[GitHub] cloudstack pull request: When no zone name is available display a ...

2016-04-23 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1477#issuecomment-213822378 LGTM. Do you think we should translate this? --- 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: Multithreaded Builds

2016-04-23 Thread Will Stevens
I will buy you a beer. :P On Apr 23, 2016 10:08 AM, "Paul Angus" wrote: > I'll collect that $5 when i come over Will.  > > > > From: Daan Hoogland > Sent: 23 Apr 2016 15:02 > To: dev > Subject: Re: Multithreaded Builds > > No Will, it was to

Re: Multithreaded Builds

2016-04-23 Thread Will Stevens
Extra hardware is always welcome. :) Now to automate a bubble run from end to end based on a queue of PRs (which I definitely have). Last night I built a marvin result parser that integrates with upr to automate posting back bubble results automatically. I have a couple loose ends to finish up,

Re: Multithreaded Builds

2016-04-23 Thread Paul Angus
I'll collect that $5 when i come over Will.  From: Daan Hoogland Sent: 23 Apr 2016 15:02 To: dev Subject: Re: Multithreaded Builds No Will, it was to prove you need extra hardware! On Sat, Apr 23, 2016 at 3:55 PM, Will Stevens wrote: >

Re: Multithreaded Builds

2016-04-23 Thread Daan Hoogland
No Will, it was to prove you need extra hardware! On Sat, Apr 23, 2016 at 3:55 PM, Will Stevens wrote: > Haha. Awesome. :) > > This was more to validate that it was OK that I configure parallelism in my > CI environment. > On Apr 23, 2016 9:34 AM, "Ian Rae"

Re: Multithreaded Builds

2016-04-23 Thread Will Stevens
Haha. Awesome. :) This was more to validate that it was OK that I configure parallelism in my CI environment. On Apr 23, 2016 9:34 AM, "Ian Rae" wrote: > Ok ok, point made, we can buy you new hardware Will ;) > > On Saturday, 23 April 2016, Will Stevens

Re: [GitHub] cloudstack pull request: Cloudstack-8961: Changes related to the U...

2016-04-23 Thread Marcus Jorge
Fx"dxxxffzxzsd Em 21 de abr de 2016 07:10, "jayapalu" escreveu: > Github user jayapalu commented on the pull request: > > https://github.com/apache/cloudstack/pull/1255#issuecomment-212842613 > > @kansal Can you please complete the localisation changes. > > > --- >

Re: CaaS in CloudStack with Kubernetes

2016-04-23 Thread Ian Rae
Great stuff Seb, Skippbox and ShapeBlue! Look forward to tire kicking... On Friday, 22 April 2016, Sebastien Goasguen wrote: > Hi folks, > > I have been quite silent this past few weeks because aside from voting in > a new VP (soon to be announced), I have been hard at work

Re: Multithreaded Builds

2016-04-23 Thread Ian Rae
Ok ok, point made, we can buy you new hardware Will ;) On Saturday, 23 April 2016, Will Stevens wrote: > Thanks for the reports guys. :) > On Apr 23, 2016 7:22 AM, "Paul Angus" > wrote: > > > My local builds (Centos6 on VMware

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-23 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-213728252 Let's leave this open for now. I want to understand what is needed to fix this project wide. --- If your project is set up for it, you can reply to this email and

RE: Here goes nothing... :)

2016-04-23 Thread Will Stevens
I plan to do quite a bit of work on this front over the next couple weeks in prep for the RC next month. I am also trying to push the idea of distributed CI and I feel that stabilizing and improving the state of our tests is a big piece of that. On Apr 23, 2016 3:17 AM, "Paul Angus"

RE: Multithreaded Builds

2016-04-23 Thread Will Stevens
Thanks for the reports guys. :) On Apr 23, 2016 7:22 AM, "Paul Angus" wrote: > My local builds (Centos6 on VMware Workstation) run happily using 4 cores. > The speed itself is down to my overclocked processor, but still, it runs > stably on 4 cores running at 25% over

RE: Multithreaded Builds

2016-04-23 Thread Paul Angus
My local builds (Centos6 on VMware Workstation) run happily using 4 cores. The speed itself is down to my overclocked processor, but still, it runs stably on 4 cores running at 25% over there stock speed. mvn -T 4.0C -Dnoredist install [INFO] BUILD SUCCESS [INFO]

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-04-23 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r60826448 --- Diff: engine/schema/src/com/cloud/capacity/dao/CapacityDaoImpl.java --- @@ -962,35 +962,58 @@ public boolean removeBy(Short capacityType, Long

[GitHub] cloudstack pull request: CLOUDSTACK-9366: Capacity of one zone-wid...

2016-04-23 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r60826329 --- Diff: engine/schema/src/com/cloud/capacity/dao/CapacityDaoImpl.java --- @@ -962,35 +962,58 @@ public boolean removeBy(Short capacityType, Long

Re: CaaS in CloudStack with Kubernetes

2016-04-23 Thread Daan Hoogland
Nice site, Go skipbox On Sat, Apr 23, 2016 at 11:19 AM, Madan Ganesh Velayudham < madangan...@actonmagic.com> wrote: > +1 Sebastien; This is a great add to CloudStack. > > Congratulations to Skippbox and Shapeblue for this milestone. > > Would love to give it a real try to embrace it in AoC. > >

[GitHub] cloudstack pull request: CLOUDSTACK-9299: Out-of-band Management f...

2016-04-23 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1502#issuecomment-213702911 I added a few comments, but before I continue: - Why assume a IP-Address and not a hostname OR IP? I know enough IPMIs why are available through a DNS hostname

[GitHub] cloudstack pull request: CLOUDSTACK-9299: Out-of-band Management f...

2016-04-23 Thread wido
Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1502#discussion_r60825823 --- Diff: api/src/org/apache/cloudstack/api/command/admin/outofbandmanagement/ConfigureOutOfBandManagementCmd.java --- @@ -0,0 +1,127 @@ +// Licensed

[GitHub] cloudstack pull request: CLOUDSTACK-9299: Out-of-band Management f...

2016-04-23 Thread wido
Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1502#discussion_r60825819 --- Diff: api/src/org/apache/cloudstack/api/command/admin/outofbandmanagement/ConfigureOutOfBandManagementCmd.java --- @@ -0,0 +1,127 @@ +// Licensed

Re: CaaS in CloudStack with Kubernetes

2016-04-23 Thread Wido den Hollander
> Op 22 april 2016 om 18:02 schreef Sebastien Goasguen : > > > Hi folks, > > I have been quite silent this past few weeks because aside from voting in a > new VP (soon to be announced), I have been hard at work for my new endeavors > with Skippbox. > > I am quite excited

[GitHub] cloudstack pull request: CLOUDSTACK-8818: Use MySQL native connect...

2016-04-23 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-213701028 Ok, leave this PR open or closed? Keep in mind, the API of both packages is rather similar. Just the connection creating is different. Atferwards it is

[GitHub] cloudstack pull request: CLOUDSTACK-8562: Dynamic Role-Based API C...

2016-04-23 Thread koushik-das
Github user koushik-das commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-213693730 I did some basic tests yesterday. Some comments based on that. Will do some more testing next week. - I see that commands.properties is removed altogether

[GitHub] cloudstack pull request: BUG-ID:CLOUDSTACK-9331:added code in marv...

2016-04-23 Thread cloudsadhu
Github user cloudsadhu commented on the pull request: https://github.com/apache/cloudstack/pull/1458#issuecomment-213686569 Thanks Swill and now All the checks are passed.Please merge this PR. --- If your project is set up for it, you can reply to this email and have your

RE: Here goes nothing... :)

2016-04-23 Thread Paul Angus
Hi Will, I little while back I created a matrix to attempt to keep track of the tests and their status. Various other projects have stopped me getting to the stage where I could start putting anything in... https://cwiki.apache.org/confluence/display/CLOUDSTACK/Marvin+Test+Matrix Kind