Re: Migrating CloudStack content from download.cloud.com

2016-05-13 Thread Indra Pramana
Dear all, Is there an alternative solution for now where I can download below URL? http://download.cloud.com/releases/2.2.0/eec2209b-9875-3c8d-92be-c001bd8a0faf.qcow2.bz2 It's the default CentOS 5.5 (64-bit) no GUI template for KVM. I checked and the above copy is not yet on cloudstack.apt-get.e

[GitHub] cloudstack pull request: [WIP][4.10+/long-term] Moving towards Jav...

2016-05-13 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1546#issuecomment-219203300 @jburwell Thanks, I'll split the changes into separate PR if everyone demands that. Otherwise each distinct pieces are implemented as separate commits that can be rev

[GitHub] cloudstack pull request: CLOUDSTACK-9377: Fix metrics pagesize iss...

2016-05-13 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1540#issuecomment-219203017 @swill done, closed and reopened to kick CIs --- 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: CLOUDSTACK-9377: Fix metrics pagesize iss...

2016-05-13 Thread rhtyd
GitHub user rhtyd reopened a pull request: https://github.com/apache/cloudstack/pull/1540 CLOUDSTACK-9377: Fix metrics pagesize issue Fixes listing of clusters and host to list all clusters/hosts by passing pagesize=-1. /cc @koushik-das @swill @DaanHoogland -- fixes the

[GitHub] cloudstack pull request: CLOUDSTACK-9377: Fix metrics pagesize iss...

2016-05-13 Thread rhtyd
Github user rhtyd closed the pull request at: https://github.com/apache/cloudstack/pull/1540 --- 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

RE: Migrating CloudStack content from download.cloud.com

2016-05-13 Thread Paul Angus
Raja, I've just picked up on this. There are a number of URLs which are hardcoded into the CloudStack database for the small built-in templates. These are in just about every copy of CloudStack out there. http://download.cloud.com/templates/builtin/f59f18fb-ae94-4f97-afd2-f84755767aca.vhd.bz2

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Ron Wheeler
https://jira.spring.io/browse/SPR-11656 has a pretty good description of spring 3.2.13 support for Java 8 They did add support for Java 8 bytecode. I am only at 3.2.4 and have no problems but we are pretty explicit about wiring and do not use autowiring. In the end, we will all end up moving

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Ron Wheeler
All my code is compiled with Java 8. I have not had any problem since I moved my stuff to Java 8 with Spring 3 but I may just be lucky. I am not sure how much of a change to code or configurations is required to upgrade to Spring 4 so it may not be a big deal to do both upgrades in one rele

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Ron Wheeler
Sorry. You misunderstood my comment. I was noting that users who want to run ACS which requires Java 8 could also run applications that can not run on Java 8 (although I have never heard of an application that would not run with the Java 8 JRE) by running both jre7 and jre8. Of course once y

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-13 Thread NuxRo
Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-219173152 It still doesn't look. I've tested this in a real setup and it seems to do what it's supposed to. Anything I can help with to get past these errors? --- If

Re: ACS PRs Status - 2016/05/12

2016-05-13 Thread Will Stevens
That is helpful, but I need to release an official ACS template with the release and it needs to be built from the RC candidate, so I need to figure out the process. *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com

Re: ACS PRs Status - 2016/05/12

2016-05-13 Thread John Burwell
Will, Rohit has built a new systemvm template that includes strongswan and quagga. He can point you to images, as well as, help with understanding the build/release process. Thanks, -John > Regards, John Burwell john.burw...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden,

[GitHub] cloudstack pull request: CLOUDSTACK-9136: remove ssh keypairs alon...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1212#issuecomment-219169448 Ok, what I will do is merge it into 4.6 and forward merge it to 4.7 and then run CI on it there. I also need to verify what is forward merged from 4.6 because I don'

[GitHub] cloudstack pull request: CLOUDSTACK-9136: remove ssh keypairs alon...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1212#issuecomment-219169003 That is fine, I don't mind doing that. Is that the process here? This is the first time I have handled a 4.6 PR. Will a 4.6 ever be released, or is that just not i

Re: ACS PRs Status - 2016/05/12

2016-05-13 Thread Will Stevens
Ok, thank you John. I need to sort out the process for testing and releasing a new system VM template. Another thing to do... *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Fri, May 13, 20

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-219168754 ok, thank you sir. 👍 --- 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 hav

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

2016-05-13 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-219168529 @swill I added a small fix for inside the SSVM. During some additional manual testing I found that the SSVM is picky in the order in the XML definition. My se

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1371#issuecomment-219168493 Please verify why the jobs are failing and re-push this PR to try to get it green. I need to verify the process to test new system templates and how to build and dis

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

2016-05-13 Thread syed
Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-219166142 @DaanHoogland sure ... doesn't hurt --- 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 d

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

2016-05-13 Thread syed
GitHub user syed reopened a pull request: https://github.com/apache/cloudstack/pull/1331 Fix Sync of template.properties in Swift When using Swift as a secondary storage, we create a template.properties file which stores the metadata about the template. Currently the data which is

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

2016-05-13 Thread syed
Github user syed closed the pull request at: https://github.com/apache/cloudstack/pull/1331 --- 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 e

[GitHub] cloudstack pull request: CLOUDSTACK-9136: remove ssh keypairs alon...

2016-05-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1212#issuecomment-219165095 @swill why not just test against 4.6 and then merge forward? if not, we need to rebase the branch. Pretty sure @ustcweizhou already did that in-house. --- If

Re: ACS PRs Status - 2016/05/12

2016-05-13 Thread John Burwell
Will, 1371 includes new Marvin test cases to verify the operation of OSPF. Bobby Stoyanov will be running these tests, as well as, manual rating of the UI features added to support it. Thanks, -John > Regards, John Burwell john.burw...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Cov

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

2016-05-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-219163616 @syed can you close and reopen to see if the error is persistent? --- If your project is set up for it, you can reply to this email and have your reply appear

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread John Burwell
Ron, According to Spring's documentation, Spring 3 only supports running byte code compiled for 1.7 on a Java8 VM [1]. Spring 4 is the first release to support byte code compiled to 1.8 running on a Java8 VM. Our experience with CloudStack 4.5 and master on Java8 has been that it runs, but

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

2016-05-13 Thread syed
Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-219163000 https://builds.apache.org/job/cloudstack-pr-analysis/1323/artifact/target/rat.txt here is the link to the file --- If your project is set up for it, you can reply

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

2016-05-13 Thread syed
Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-219162095 @swill I got the file from jenkins and it points to `utils/testsmallfileinactive` ... I have no idea where this file came from --- If your project is set up for it, y

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread John Burwell
Ron, Upon upgrade, we will start using features and runtime classes/methods introduced with Java8. Therefore, it will become mandatory upgrade from Java7 to Java8 when adopting the release with these changes. Thanks, -John > Regards, John Burwell john.burw...@shapeblue.com  www.shapeblue.c

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Wido den Hollander
> Op 13 mei 2016 om 22:51 schreef John Burwell : > > > All, > > Java8 is targeted for post 4.9. Oracle provides an official PPA for Java6, > Java7, Java8, and Java9 [1] that supports Ubuntu 14.04. It is important to > bear in mind that Java7 was EOL’ed on April 2015 [2]. In my opinion, the

[GitHub] cloudstack pull request: Taking fast and efficient volume snapshot...

2016-05-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1403#issuecomment-219161020 Then log the warning with just the message and add a debug statement with the stacktrace. logging stacktraces at a level even more strict then INFO does not co

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

2016-05-13 Thread syed
Github user syed commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-219160595 @swill looks like jenkins failed because of some licence issue in one file. I've checked all the files that I have changed and every one seems to have the correct lic

[GitHub] cloudstack pull request: Reimplement router.redundant.vrrp.interva...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1486#issuecomment-219160164 Ok, we need one more review on this. tag:needsreview tag:needreview --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] cloudstack pull request: Add perl-modules as install dependency fo...

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1495 --- 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: DAO: Hit the cache for entity flagged as ...

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1532 --- 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: Remove extraneous log directory and add c...

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1537 --- 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: DAO: Hit the cache for entity flagged as ...

2016-05-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1532#issuecomment-219158465 @swill you should always count the opinion of the RM as that of a real memeber of the community ;) --- If your project is set up for it, you can reply to this

[GitHub] cloudstack pull request: CPU socket count reporting correction

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1520 --- 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-9199: Fixed deployVirtualMachi...

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1280 --- 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: L10n update master 20160127

2016-05-13 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1376 --- 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

Re: ACS PRs Status - 2016/05/12

2016-05-13 Thread Will Stevens
Thank you, I added 1376 to my ready to merge list. I have 6 more ready to merge, so we should be down to 155 tonight... *Will STEVENS* Lead Developer *CloudOps* *| *Cloud Solutions Experts 420 rue Guy *|* Montreal *|* Quebec *|* H3J 1S6 w cloudops.com *|* tw @CloudOps_ On Fri, May 13, 2016 at 3

[GitHub] cloudstack pull request: Reimplement router.redundant.vrrp.interva...

2016-05-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1486#issuecomment-219157668 LGTM, tag:easypr --- 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

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Ron Wheeler
It would seem that a major version would be appropriate. I am not sure why someone could not use both Java 8 and Java 7. I have Java 3 versions 6, 7, and 8 available on my Windows 7 PC. I would have thought that Linux would be at least that flexible. Is it really not possible to run Java 8 on U

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread John Burwell
All, Java8 is targeted for post 4.9. Oracle provides an official PPA for Java6, Java7, Java8, and Java9 [1] that supports Ubuntu 14.04. It is important to bear in mind that Java7 was EOL’ed on April 2015 [2]. In my opinion, the recent history of Java vulnerabilities dictates that we cannot a

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

2016-05-13 Thread serg38
Github user serg38 commented on the pull request: https://github.com/apache/cloudstack/pull/1518#issuecomment-219154273 This doesn't seem to be related to this PR.Can you post managment-server.log extract around the failure time? --- If your project is set up for it, you can reply to

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Erik Weber
It only means that we are dropping support for newer ACS versions. It most likely won't get into 4.9, which means 14.04 will be supported a few more weeks/months. We could mention in the 4.9 RN that for the next release (call it 5.0 because of the change?) will only be supported by $xyz distributi

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

2016-05-13 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-219152627 @swill Yes, running this in production on our cloud I just started an Instance and the Agent (backport to 4.8, clean merge) produced this XML: ```

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1331#issuecomment-219150983 @syed failed again. :( --- 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

[GitHub] cloudstack pull request: Taking fast and efficient volume snapshot...

2016-05-13 Thread jburwell
Github user jburwell commented on the pull request: https://github.com/apache/cloudstack/pull/1403#issuecomment-219148309 @DaanHoogland in my experience, stack traces are a critical piece of information for operationally debugging CloudStack. Unfortunately, our logging lacks the clar

[GitHub] cloudstack pull request: Taking fast and efficient volume snapshot...

2016-05-13 Thread mike-tutkowski
Github user mike-tutkowski commented on the pull request: https://github.com/apache/cloudstack/pull/1403#issuecomment-219146275 @jburwell @DaanHoogland I have a solution for quickly looking up if a cluster supports resigning that I think we'll all be happy with. Upon a host co

Re: ACS PRs Status - 2016/05/12

2016-05-13 Thread Milamber
- 1376 - master (ready, pending 1515 conflict) 1376 is ready to merge to master branch (Jenkins and Travis are green). 1515 is already merged on 4.8 On 12/05/2016 22:02, Will Stevens wrote: ACS PRs - 1537 - master (ready, pending Jenkins) - 1514 - 4.7 (pending Jenkins) -

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1280#issuecomment-219141565 I think we are ok on this one. thanks @rhtyd. I will merge this one... --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] cloudstack pull request: Make the generated json files unique to p...

2016-05-13 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1470#issuecomment-219137807 LGTM based on the manual testing above. --- 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 pr

[GitHub] cloudstack pull request: CLOUDSTACK-9162: Unable to add VPN user v...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1241#issuecomment-219137690 @kansal can you close and reopen for do a force push to try to get Jenkins green. I am not sure what to do about this PR because I don't know that I have a way to ve

[GitHub] cloudstack pull request: Make the generated json files unique to p...

2016-05-13 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1470#issuecomment-219137482 Kill the Master VR: ![int_before_fo](https://cloud.githubusercontent.com/assets/17278194/15259402/b4659c14-1916-11e6-8169-b397f102.png) ![ki

[GitHub] cloudstack pull request: Make the generated json files unique to p...

2016-05-13 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1470#issuecomment-219137269 Add a new ACL, add a rule to it and apply it to a Tier: ![acs_before](https://cloud.githubusercontent.com/assets/17278194/15259376/94178760-1916-11e6-845d

[GitHub] cloudstack pull request: Make the generated json files unique to p...

2016-05-13 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1470#issuecomment-219137332 Add a private gw: ![add_pg](https://cloud.githubusercontent.com/assets/17278194/15259388/a3d1da2a-1916-11e6-8077-d9980fc8641a.png) --- If your project

[GitHub] cloudstack pull request: Make the generated json files unique to p...

2016-05-13 Thread kiwiflyer
Github user kiwiflyer commented on the pull request: https://github.com/apache/cloudstack/pull/1470#issuecomment-219137126 Hot add static route: ![hot_add_static_route](https://cloud.githubusercontent.com/assets/17278194/15259361/77dad958-1916-11e6-904c-d0ae35b8554a.png) --

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1518#issuecomment-219137064 I have tried to CI this PR a couple times. I get the following during Deploy DC. ``` Deploy DC Started Exception Occurred :['Traceback (most rec

[GitHub] cloudstack pull request: CLOUDSTACK-9136: remove ssh keypairs alon...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1212#issuecomment-219134524 @rhtyd I can run CI on this, but it is based against 4.6. I am not sure what to do about that. Should @ustcweizhou at least rebase against 4.7 so we can get it in a

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Wido den Hollander
I am in favor of Java 8, but it means we would drop Ubuntu 14.04 support. That would make life a bit easier though, since Ubuntu 16.04 has systemd and 14.04 still has sysvinit and that make packaging a problem. > Op 13 mei 2016 om 19:10 schreef Rohit Yadav : > > > All, > > > I've started som

[GitHub] cloudstack pull request: CLOUDSTACK-8970 Centos 6.{1,2,3,4,5} gues...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/956#issuecomment-219130130 @SudharmaJain can you re-push or close and reopen this PR to kick off Jenkins again? @rhtyd and @nvazquez I know you both use VMware. I don't have a way to v

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/669#issuecomment-219129109 Thanks @anshul1886. Keep in mind that the freeze date for 4.9 is Monday next week. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1503#issuecomment-219126476 I am running my CI against it again to make sure this does not pop again. If it doesn't then I will merge it right away as Jenkins and Travis are green. Thx... --

Re: Organizing CCCNA16 Hackathon

2016-05-13 Thread Paul Angus
I'm keen to sit in on the Ci/CD work, but also the documentation work. (re-writing the concepts section is 'onging') Kind regards, Paul Angus Regards, Paul Angus paul.an...@shapeblue.com  www.shapeblue.com 53 Chandos Place, Covent Garden, London WC2N 4HSUK @shapeblue

[GitHub] cloudstack pull request: Add perl-modules as install dependency fo...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1495#issuecomment-219123470 I think we should merge this one at this point. @sverrirab my understanding is that once #1533 is merged, this will be required. If that is the case, when this one

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-13 Thread olivierlemasle
Github user olivierlemasle commented on the pull request: https://github.com/apache/cloudstack/pull/1503#issuecomment-219121751 In the three tracebacks, the error is located in https://github.com/apache/cloudstack/blob/821b2da1267c486bb1f5c9e90d3d1969b6fe947f/tools/marvin/marvin/lib/ba

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-13 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-219121539 I like the first two. The third should read something like: CLOUDSTACK-8303 allow multiple ranges & finite lease time the 4th and 5th I really need t

[GitHub] cloudstack pull request: Remove extraneous log directory and add c...

2016-05-13 Thread PaulAngus
Github user PaulAngus commented on the pull request: https://github.com/apache/cloudstack/pull/1537#issuecomment-219119839 Thanks Will. Kind regards, Paul Angus Regards Paul Angus VP Technology www.shapeblue.com From: Will Steven

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-13 Thread DaanHoogland
GitHub user DaanHoogland reopened a pull request: https://github.com/apache/cloudstack/pull/1297 CLOUDSTACK-9203 Implement security group move on updateVM API call cherry-picked from a exoscale internal fix Conflicts: api/src/org/apache/cloudstack/api/command/user/

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-13 Thread DaanHoogland
Github user DaanHoogland closed the pull request at: https://github.com/apache/cloudstack/pull/1297 --- 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 feat

Re: [DISCUSS] Moving to Java8

2016-05-13 Thread Ron Wheeler
I am not sure what you are using in Spring but I have been using Spring 3.x.x with Java 8 for some time. I can believe that Spring 4 is "better" than Spring 3 but you might see if you can separate these two technology upgrades just to keep it simple. There is enough in Java 8 to make a pretty b

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-219109520 Holy errors batman!!! It looks like it started like this: ``` [INFO] --- cobertura-maven-plugin:2.6:instrument (default-cli) @ cloud-server

[GitHub] cloudstack pull request: CPU socket count reporting correction

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1520#issuecomment-219105793 Awesome, thanks guys. I will merge 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 pr

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1545#issuecomment-219104928 Ya, I like this. I think this is a nice advancement. If we are going to try to get this into 4.9, I will need some help getting this moving forward cause we are pre

[DISCUSS] Moving to Java8

2016-05-13 Thread Rohit Yadav
All, I've started some work [pr] that aims to adapt CloudStack to recent changes in the environments and the ecosystem: - Java8, Spring 4.x - SystemD - MariaDB and MySQL - Reduce distro provided package dependency - Packaging, CI and testing These are long term goals but I've identified

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-13 Thread insom
Github user insom commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-219101084 @swill Hmm. Now it's a _different_ Cobertura error --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] cloudstack pull request: Kvm plugin restructure

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1521#issuecomment-219101062 We have merge conflicts on this one. As soon as the merge conflicts are sorted out, I will run CI against it and hopefully we can get some code reviews quickly so we

[GitHub] cloudstack pull request: CLOUDSTACK-9203 Implement security group ...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1297#issuecomment-219100429 This one errored again in both Jenkins and Travis. Not sure if that signifies an actual problem or if things are just acting up. I fix to some of the issues in Jenk

[GitHub] cloudstack pull request: Fix Nio/CPU issue and CI failures

2016-05-13 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1543#discussion_r63215625 --- Diff: test/integration/component/test_acl_listvolume.py --- @@ -357,9 +357,9 @@ def tearDownClass(cls): cls.apiclient = super(TestVolumeLis

[GitHub] cloudstack pull request: Fix Nio/CPU issue and CI failures

2016-05-13 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1543#discussion_r63215404 --- Diff: test/integration/component/test_acl_listvolume.py --- @@ -357,9 +357,9 @@ def tearDownClass(cls): cls.apiclient = super(TestVolumeLis

[GitHub] cloudstack pull request: CLOUDSTACK-8830 - [Vmware] VM snapshot fa...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/798#issuecomment-219096839 @maneesha-p can you rebase and push so we can get this PR green? I would love some code review as well. Thx... --- If your project is set up for it, you can reply t

[GitHub] cloudstack pull request: Fix Nio/CPU issue and CI failures

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1543#issuecomment-219095904 Thanks for the effort stabilizing Jenkins and Travis the last few days. This has been super helpful as we get close to the freeze. Very much appreciated. Most of t

[GitHub] cloudstack pull request: Fix Nio/CPU issue and CI failures

2016-05-13 Thread rhtyd
Github user rhtyd commented on the pull request: https://github.com/apache/cloudstack/pull/1543#issuecomment-219095201 Thanks @swill --- 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 feat

[GitHub] cloudstack pull request: Fixes for VirtualRouters in Basic Network...

2016-05-13 Thread The-Loeki
Github user The-Loeki commented on the pull request: https://github.com/apache/cloudstack/pull/1547#issuecomment-219094194 Like this then :dart: --- 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 ha

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1518#issuecomment-219093988 I will run this through my KVM CI right now... --- 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: DAO: Hit the cache for entity flagged as ...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1532#issuecomment-219093207 I am counting my own code review on this one given the scope of the change and the fact that a review is the only thing holding this up. I will merge this one... -

[GitHub] cloudstack-docs-admin pull request: CLOUDSTACK-9299: doc for out-o...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack-docs-admin/pull/38#issuecomment-219092042 Thanks for taking care of this @rhtyd. 👍 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-219091474 @insom yes, I tend to agree with you. the main reason I highlighted it was because the error was network related and the code changed details related to an interface

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-219088252 This one is coming back with a clean bill of health. I think we are ready on this one. Can I get some code review? Thanks... --- If your project is set up for it,

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1511#issuecomment-219088009 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 9h 21m 07s ``` **Associ

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-219087700 The CI does not cover the actual code, but instead makes sure nothing else is broken by these changes. The test results provided by @nlivens show the tests for the a

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

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1494#issuecomment-219086756 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 0 Errors: 0 Duration: 9h 17m 55s ``` **Associ

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-13 Thread insom
Github user insom commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-219085734 @swill I've closed and re-opened. This change is to a shell script only and the failures are within Marvin, so I'm pretty confident that they are unrelated to my prop

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-13 Thread insom
Github user insom closed the pull request at: https://github.com/apache/cloudstack/pull/1513 --- 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-9362: Skip VXLANs when rewriti...

2016-05-13 Thread insom
GitHub user insom reopened a pull request: https://github.com/apache/cloudstack/pull/1513 CLOUDSTACK-9362: Skip VXLANs when rewriting the bridge name for migrations (4.8-2) From the [JIRA issue](https://issues.apache.org/jira/browse/CLOUDSTACK-9362): > https://github.com/

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-219084969 @insom, @dmabry and @kiwiflyer, the CI failure seems like it could be related to this change, but I can't be sure. Can you guys provide some guidance since all of yo

[GitHub] cloudstack pull request: CLOUDSTACK-9362: Skip VXLANs when rewriti...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1513#issuecomment-219084090 ### CI RESULTS ``` Tests Run: 85 Skipped: 0 Failed: 1 Errors: 0 Duration: 9h 01m 07s ``` **Summary of the p

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1503#issuecomment-219083712 I have never seen these test failures before, so I believe they are somehow related to this change. @olivierlemasle can you review the CI results and post your feedb

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

2016-05-13 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1518#issuecomment-219083470 @koushik-das I only use Vmware hypervisor so I couldn't test it for Xen or KVM. However, I reviewed code and found `copySnapshotToTemplateFromNfsToNfsXenserver` me

[GitHub] cloudstack pull request: CLOUDSTACK-9358: StringIndexOutOfBoundsEx...

2016-05-13 Thread swill
Github user swill commented on the pull request: https://github.com/apache/cloudstack/pull/1503#issuecomment-219082695 ### CI RESULTS ``` Tests Run: 82 Skipped: 0 Failed: 0 Errors: 3 Duration: 8h 28m 52s ``` **Summary of the p

  1   2   >