[GitHub] cloudstack pull request: CLOUDSTACK-9289:Automation for feature de...

2016-05-19 Thread nitt10prashant
Github user nitt10prashant commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-220521199 @swill i do not see any comment on parent ticket CLOUDSTACK-8676 , i have asked Sateesh to update the ticket , will update the PR as soon as i hear from

Re: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Ardhendu Shekhar
Congratulations, Will. Regards, Ardhendu Ardhendu Shekhar Senior Manager, Program Management | CloudPlatform | Accelerite, Hyderabad | Extn: 29370 | Desk: +91 40 6742 9370 | Cell: +91 988-500-2788 | ardhendu.shek...@accelerite.com | On 5/19/16, 12:26 PM, "Sebastien Goasguen"

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-19 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-220517071 ?OK, I can create one and open a PR up against it reverting those variable names. From: Anshul Gangwar

Re: Variable renaming in classes meant for Agents

2016-05-19 Thread Tutkowski, Mike
Thanks for sending out this e-mail, Anshul. This is a bit of a strange situation because we need to make sure people are either aware of the fact that properties in Command classes are serialized (and not change existing variable names) or come up with a less fragile way of choosing property

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-19 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-220516738 @mike-tutkowski No, I have not created ticket. You can create one. --- If your project is set up for it, you can reply to this email and have your reply appear

Re: [GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-19 Thread Tutkowski, Mike
I see that you just sent an e-mail to @dev about this - thanks! From: mike-tutkowski Sent: Thursday, May 19, 2016 10:46 PM To: dev@cloudstack.apache.org Subject: [GitHub] cloudstack pull request: Notify listeners when a host has been

Variable renaming in classes meant for Agents

2016-05-19 Thread Anshul Gangwar
Hi, We should not allow renaming of variables in classes which ends with Command and TO. As these objects are meant to be consumed by Agents. Agents may not be written in java so relying on these variable names to get the info. One such example is Hyper-V agent. Hyper-V support is currently

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-19 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-220516362 Yes, I can send an e-mail out in a bit. In the short term (at least for 4.9), I can just revert the changed names in those Command classes.

Regarding files generated by WSDL

2016-05-19 Thread sangeeta lal
Dear CloudStack Team, I am Sangeeta, a PhD scholar from India. I am working in the area of automated software engineering and my aim is to analyse logging statements. While analysing the CloudStack project,* I found may files having following statement written in them*: */*This file was

[GitHub] cloudstack pull request: CLOUDSTACK-9289:Automation for feature de...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1417#issuecomment-220515109 @nitt10prashant any update on this one? I am very close to freezing, so I figured I would check in. Cheers... --- If your project is set up for it, you can reply

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

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1436#issuecomment-220514893 @sanju1010 ping... --- 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: Notify listeners when a host has been add...

2016-05-19 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-220514841 @mike-tutkowski then we can raise this in dev list. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] cloudstack pull request: CLOUDSTACK-6975: Prevent dnsmasq from sta...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1514#issuecomment-220514409 @dsclose can you please rebase and repush this one to kick off jenkins and travis again. We have fixes in both travis and jenkins so we should be able to get this

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-19 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-220514243 I wonder how many people are actually aware of that. :-) Sent from my iPhone On May 19, 2016, at 9:04 PM, Anshul Gangwar

[GitHub] cloudstack pull request: Undetected bug correct and refactor of th...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1499#issuecomment-220513799 @alexandrelimassantana this one has merge conflicts. Can you rebase and fix the conflicts? Thanks... --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CLOUDSTACK-8906: /var/log/cloud/ doesn't ...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/883#issuecomment-220513693 Can someone with a xenserver setup validate this one? @bvbharat can you queue this one up for us? Thanks... --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Bug-ID: CLOUDSTACK-8870: Skip external de...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/846#issuecomment-220513570 @kishankavala please rebase this PR as we have merge conflicts. Thanks... --- If your project is set up for it, you can reply to this email and have your reply

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

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-220512874 ### CI RESULTS ``` Tests Run: 84 Skipped: 0 Failed: 0 Errors: 2 Duration: 8h 36m 08s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add channel to Instances...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-220512533 These failures are not related to this PR, but if you have ideas why they are happening I would be interested. I will wait till @wido has a chance to work on this

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add channel to Instances...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-220512458 ### CI RESULTS ``` Tests Run: 78 Skipped: 0 Failed: 0 Errors: 4 Duration: 7h 47m 49s ``` **Summary of the

[GitHub] cloudstack pull request: Notify listeners when a host has been add...

2016-05-19 Thread anshul1886
Github user anshul1886 commented on the pull request: https://github.com/apache/cloudstack/pull/816#issuecomment-220512436 @mike-tutkowski I believe class name ending in Command signifies that they are meant for agent and should be treated like API params. --- If your project is set

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-220512305 I don't think these issues are related to this PR, but if you have any insight as to why they might be happening I would be interested. I will run this CI again to

[GitHub] cloudstack pull request: CLOUDSTACK-9180: Optimize concurrent VM d...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1251#issuecomment-220512147 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 1 Errors: 4 Duration: 9h 31m 01s ``` **Summary of the

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220512003 These errors are not related to this PR, but if you know where these errors are coming from I would be interested in some input. :) --- If your project is set up

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220511935 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 3 Duration: 8h 29m 09s ``` **Summary of the

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220511825 I am going to run the test again because it was an intermittent issue, so a couple tests is not a bad idea. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220511743 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 8h 51m 55s ```

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63988259 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59 @@ +/*

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63988285 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -1298,6 +1304,27 @@ private static DiskTO

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread anshul1886
Github user anshul1886 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63988200 --- Diff: plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java --- @@ -594,6 +597,25 @@ private

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1410#issuecomment-220505362 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 4h 18m 28s ```

Re: Refactoring CitrixResourceBase

2016-05-19 Thread Rafael Weingärtner
Hi Syed, That is a great idea; however, it is a very hard task. The idea of Tim is great; actually, we already have some sort of hierarchy that is used in “CitrixResourceBase.java”. I would suggest you first removing the unused code, unused variable, and duplicate methods; that would be one PR.

Re: Migrating CloudStack content from download.cloud.com

2016-05-19 Thread Daan Hoogland
sounds good, so if we put both sections together the worry is over. acs is a complex system and requires a lot of reading so let''s not put it in a different section. would be duplicate code anyhow. On Thu, May 19, 2016 at 11:56 PM, Chiradeep Vittal wrote: > Well, you have

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-220464828 Really like the functionality so I think we must get this in. I have some remarks about the code that I stand by even though they are not entirely to blame on

Re: Migrating CloudStack content from download.cloud.com

2016-05-19 Thread Chiradeep Vittal
Well, you have to define "worry" :) I'm proposing an almost identical step to http://docs.cloudstack.apache.org/projects/cloudstack-installation/en/4.6/management-server/#prepare-the-system-vm-template If they can do that, then it is not a stretch to assume they can download the default template

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63961096 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/wrapper/xenbase/CitrixCopyFileInVmCommandWrapper.java --- @@

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63960701 --- Diff: plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java --- @@ -5402,4 +5402,20 @@ public

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63960622 --- Diff: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java --- @@ -1298,6 +1304,27 @@ private static DiskTO

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63959786 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59 @@ +/*

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63959351 --- Diff: plugins/hypervisors/hyperv/src/com/cloud/hypervisor/hyperv/resource/HypervDirectConnectResource.java --- @@ -594,6 +597,25 @@ private

Re: Migrating CloudStack content from download.cloud.com

2016-05-19 Thread Will Stevens
I definitely agree with this. I don't think people doing a PoC should have to worry about getting a template installed on first setup. I think that is a basic feature if we want platform adoption. Getting the first boot process from download to first VM launched should be as smooth as possible.

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/669#discussion_r63959496 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/wrapper/LibvirtCopyFileInVmCommandWrapper.java --- @@ -0,0 +1,59 @@ +/*

Re: Migrating CloudStack content from download.cloud.com

2016-05-19 Thread Daan Hoogland
Chiradeep, It makes sense not to have this old default very insecure guest template still be downloaded but it would be nice to have some kind of meta-data pointing to the latest version of an acceptable template (symlink) for trial installs so PoCcers are swiftly on their way. Would you agree?

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220458013 thanks @KrisSterckx, and to make it even more very clear: basically you will only need to install acs and nuage and configure them correctly. neat. you guys

[GitHub] cloudstack pull request: CLOUDSTACK-9368: Fix for Support configur...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1518#discussion_r63955721 --- Diff: engine/storage/src/org/apache/cloudstack/storage/image/NfsImageStoreDriverImpl.java --- @@ -0,0 +1,30 @@ +package

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-220447959 I think with @milamberspace's review, we are only missing one code review to get this one in. Thanks everyone... --- If your project is set up for it, you can reply

[GitHub] cloudstack pull request: Made the adding new keyboard language sup...

2016-05-19 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-220446031 LGTM. Tested manually with a real test deployment of CS49, with my French azerty keyboard: works fine. --- If your project is set up for it, you

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add channel to Instances...

2016-05-19 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-220440726 @swill I will work on the changes tomorrow morning (my time, CET). Shouldn't be very hard to do. --- If your project is set up for it, you can reply to this email

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-220433360 @swill I must have made a booboo spliting the style parts from the test upgrades. I will look at it over the weekend. --- If your project is set up for it,

Re: Refactoring CitrixResourceBase

2016-05-19 Thread Daan Hoogland
Syed, gogogo. actually it has shrunk to 5k lines since 2012 ;) I like your initiative and initial direction. A lot of small steps to improve the blob have been taken and I would sugest to keep going in small steps. On Thu, May 19, 2016 at 9:44 PM, Tim Mackey wrote: > +1 > >

[GitHub] cloudstack pull request: 4.9/master bountycastle changes

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-220431955 This is failing to build for me. ``` Running org.apache.cloudstack.affinity.AffinityApiUnitTest Tests run: 7, Failures: 0, Errors: 0, Skipped: 0, Time

Re: Refactoring CitrixResourceBase

2016-05-19 Thread Tim Mackey
+1 When I went through this last time, not only was it hard to understand the flows, but the XenServer version management was a pain. Would suggest creating a base class which always works (i.e. is independent of XenServer version) for core functions. Then add in that which exists for a specific

[GitHub] cloudstack pull request: CLOUDSTACK-9368: Fix for Support configur...

2016-05-19 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1518#issuecomment-220427628 @koushik-das I pushed new changes including an intermediate class as you suggested @swill hopefully CI passes now --- If your project is set up for it, you

Refactoring CitrixResourceBase

2016-05-19 Thread Syed Mushtaq
Hi All, I would like to refactor CitrixResourceBase class which is responsible for communicating with Xenserver. It has grown too long (>5K lines) and has absolutely no testing. In my first pass I want to separate out the functionality buy the subsystem it targets (compute, storage, network etc)

[ANNOUNCE] GitHub Changes

2016-05-19 Thread Will Stevens
Hey Everyone, As Seb alluded to in the VP announcement thread, we have some reasonably big news to report. As many of you know, we have been working with the ASF to find ways to better utilize the features of Github. Github has been working very well as a medium for managing pull requests and

Re: Migrating CloudStack content from download.cloud.com

2016-05-19 Thread Chiradeep Vittal
Pierre, I meant the auto-download of default templates, not systemvm. Note that the initial systemvm template has to be hand-downloaded anyway. If you are a single hypervisor cloud, then that is all that is needed. On Thu, May 19, 2016 at 11:08 AM, Pierre-Luc Dion wrote: > I

Re: Migrating CloudStack content from download.cloud.com

2016-05-19 Thread Pierre-Luc Dion
I would be -1 about disabling the autodownload of the systemvm template, has this should be simplify deployment of a cloudstack system. although, the download URL should be a configurable value, maybe in a property file or thru the API somehow or via a pre-install command. I really like

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

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-220397187 @rhtyd thanks, just wanted to make sure we extracted anything that was useful from @anshul1886's experience. Thanks for working this out guys... --- If your

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1410#issuecomment-220396410 I will kick off another CI run just to verify again. Thanks for the update on this @ustcweizhou. 👍 --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220395388 I already started another run against this PR, so that should also validate nothing was accidentally broken. --- If your project is set up for it, you can reply to

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-19 Thread KrisSterckx
Github user KrisSterckx commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220394412 Hi @DaanHoogland The architecture is `cloudstack ---cmp--> nuage-vsp-acs-client ---cfg--> Nuage VSP platform` with -

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

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1516#issuecomment-220393200 Thank you for the clean up. It looks much better now. I will rerun CI on this... --- If your project is set up for it, you can reply to this email and have your

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

2016-05-19 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-220389940 @swill the problem was environment dependent where an unclean environment was used to build/run management server, it is recommended to git clean -fdx one's local

Re: jenkins pr check

2016-05-19 Thread Will Stevens
Ya, we will see if people actually do kick off new jenkins runs. I will follow up with a few people to ask them to as I know their PRs are running into some of the problems we have discussed. *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|*

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

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1489#issuecomment-220385286 Thanks guys for getting that sorted out. Is the problems that @anshul1886 ran into something that others could potentially run into? Would it be useful to document

RE: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Sateesh Chodapuneedi
Congrats, Will! Wish you the best. Regards, Sateesh Chodapuneedi Chief Product Engineer, CloudPlatform Development, Accelerite. Off: +91 80 6772 1329 | EMail: sateesh.chodapune...@accelerite.com > -Original Message- > From: Sebastien Goasguen [mailto:run...@gmail.com] > Sent: Thursday,

[GitHub] cloudstack pull request: CLOUDSTACK-6928: fix issue disk I/O throt...

2016-05-19 Thread GabrielBrascher
Github user GabrielBrascher commented on the pull request: https://github.com/apache/cloudstack/pull/1410#issuecomment-220382990 Thanks for the update @ustcweizhou, now I can't find anything to :-1: this code. Based on code review and the CI result, the code LGTM. --- If your

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220380639 @swill thanks, though the latter changes are all packaging related. I've built rpm/deb repository to show packaging works:

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-19 Thread rhtyd
GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/1549 CLOUDSTACK-9348: NioConnection improvements Reopened PR with squashed changes for a re-review and testing after https://github.com/apache/cloudstack/pull/1493 and sub-sequent PRs got reverted

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-19 Thread rhtyd
Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1549 --- 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-9348: NioConnection improvemen...

2016-05-19 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220380081 @swill yes, this PR includes all the reverted changes and the addHost fix. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220379062 This was run prior to your changes. I will need to rerun to incorporate the recent changes... --- If your project is set up for it, you can reply to this email and

[GitHub] cloudstack pull request: dynamic-roles: packaging improvements

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1551#issuecomment-220378822 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 4h 15m 16s ```

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-19 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220375273 So @KrisSterckx what are the copile, install, configuration and runtime dependencies that users of your plugin will encounter? What do they need to buy

Re: jenkins pr check

2016-05-19 Thread Daan Hoogland
Maybe I should have formulated my request A little less open minded :) Hope it did help. As I choose one of two possible solutions let me know if I should try the other one. On Thu, May 19, 2016 at 5:25 PM, Will Stevens wrote: > Haha. Too bad you can't batch this email

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add channel to Instances...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-220365886 Sorry, I need to queue this one up... --- 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

[GitHub] cloudstack pull request: CLOUDSTACK-9348: NioConnection improvemen...

2016-05-19 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1549#issuecomment-220362988 I will get this run through CI. Just to be clear, this PR includes all of the changes that were included in 1543, 1538, 1534, 1493 as well as a fix for the

Re: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Gabriel Beims Bräscher
Thank you for all the effort Sebastien. Congrats Will!! Let’s keep up this great work. Cheers, Gabriel. 2016-05-19 12:17 GMT-03:00 Will Stevens : > Thank you all for the well wishes. :) > > Thank you Seb for all the hard work you have put into this project. You >

Re: jenkins pr check

2016-05-19 Thread Will Stevens
Haha. Too bad you can't batch this email to a smaller groups. Daan just crashed the Jenkins... :P Seriously, thank you for your help on this. I think this will make a big difference going forward. *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal

Re: How to fix Jenkins git workspace issues

2016-05-19 Thread Will Stevens
Thanks guys. I think this will make a big difference.  *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Thu, May 19, 2016 at 5:01 AM, Daan Hoogland wrote: > On

Re: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Will Stevens
Thank you all for the well wishes. :) Thank you Seb for all the hard work you have put into this project. You have enabled us to make larger strides forward and we all really appreciate that. Thanks...  On Thu, May 19, 2016 at 10:56 AM, Giles Sirett wrote: >

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

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63895181 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF)

RE: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Giles Sirett
Sebastien - thank you for your drive, dedication and hard work over the last year Will - many congratulations - keep up the good work ! Kind Regards Giles giles.sir...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue -Original Message-

RE: Migrating CloudStack content from download.cloud.com

2016-05-19 Thread Giles Sirett
Erik I agree on the existing users - not the end of the world. We could even release a tiny patch to update the DB records The piece that we need to be really careful with is the "I've just downloaded cloudstack thing for the first time" scenario But - there seems to be plenty of good ideas

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

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63894357 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF)

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

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63893826 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF)

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

2016-05-19 Thread sudhansu7
Github user sudhansu7 commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1516#discussion_r63893791 --- Diff: test/integration/component/maint/test_capacity_host_delete.py --- @@ -0,0 +1,195 @@ +# Licensed to the Apache Software Foundation (ASF)

[GitHub] cloudstack pull request: add DHCP lease folders for Ubuntu

2016-05-19 Thread syed
Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1552#issuecomment-220349481 Can cofirm the dhcp folder on Ubuntu Trusty LTS (14.04) LGTM. ``` $ lsb_release -a No LSB modules are available. Distributor ID: Ubuntu

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add channel to Instances...

2016-05-19 Thread wido
Github user wido commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1545#discussion_r63889814 --- Diff: plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java --- @@ -1978,11 +1978,16 @@ So if getMinSpeed()

[GitHub] cloudstack pull request: CLOUDSTACK-8715: Add channel to Instances...

2016-05-19 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-220342553 Thanks for the feedback, I will work on that tomorrow. @rhtyd I tested this on both Ubuntu and CentOS systems. The main issue with Ubuntu is AppArmor

Re: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Ahmad Emneina
Awesome stuff Will! Congrats. Ahmad E > On May 18, 2016, at 11:56 PM, Sebastien Goasguen wrote: > > Morning Everyone, > > Yesterday at the ASF board meeting, the board passed the resolution making > Will Stevens the new Vice President of the Apache CloudStack project. > >

Re: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Nicolás Vázquez
Congratulations Will! 2016-05-19 11:21 GMT-03:00 Rafael Weingärtner : > Thanks for all the hard work you have been doing. > Congratulations Will. > > > On Thu, May 19, 2016 at 11:11 AM, Suresh Sadhu < > suresh.sa...@accelerite.com> > wrote: > > > Congratulations

Re: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Rafael Weingärtner
Thanks for all the hard work you have been doing. Congratulations Will. On Thu, May 19, 2016 at 11:11 AM, Suresh Sadhu wrote: > Congratulations Will!! > > Regards > Sadhu > > -Original Message- > From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] > Sent:

RE: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Suresh Sadhu
Congratulations Will!! Regards Sadhu -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Thursday, May 19, 2016 2:55 PM To: dev Cc: us...@cloudstack.apache.org; market...@cloudstack.apache.org Subject: Re: [ANNOUNCE] Will Stevens as

RE: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Paul Angus
Thank you Sebastien and congratulations Will. Kind regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue -Original Message- From: Syed Mushtaq [mailto:syed1.mush...@gmail.com] Sent: 19 May 2016 14:25 To:

[GitHub] cloudstack pull request: Remodeling of Nuage VSP Plugin + CLOUDSTA...

2016-05-19 Thread KrisSterckx
Github user KrisSterckx commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-220326489 @DaanHoogland to your question : We are embarking full CloudStack build automation as that gives us cloudstack-nuagevsp-plugin <> nuagevsp-client-jar

Re: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Syed Mushtaq
Congratulations Will! Super excited to be working with you :) -Syed On Thu, May 19, 2016 at 5:24 AM, Daan Hoogland wrote: > thanks Sebastien, > > looking forward to a workshop on your new endevours and please have a > thought for poor Will once in a while. > > On Thu,

RE: [ANNOUNCE] Will Stevens as new Apache CloudStack VP

2016-05-19 Thread Sanjeev Neelarapu
Congratulations Will!! Best Regards, Sanjeev N Chief Product Engineer, Accelerite Off: +91 40 6722 9368 | EMail: sanjeev.neelar...@accelerite.com -Original Message- From: Daan Hoogland [mailto:daan.hoogl...@gmail.com] Sent: Thursday, May 19, 2016 2:55 PM To: dev

Re: MySQL : No suitable driver found for jdbc:mysql

2016-05-19 Thread Nick LIVENS
When I add the mysql-connector-java.jar to /usr/lib/jvm/java/jre/lib/ext without touching the CLASSPATH in /etc/sysconfig/cloudstack-management it's working fine as well.. So either something is broken in the tomcat class loading mechanism, either it's related to some environment specifics of

Re: [Discuss] CloudStack documentation

2016-05-19 Thread Ron Wheeler
Identify where most issues are raised in the ML and fix the docs to reduce the confusion. I suspect that Networking is the source of most problems but perhaps other have a better sense of this. Ron On 19/05/2016 8:29 AM, Ron Wheeler wrote: Removal of duplicate information as part of item 2 .

  1   2   >