[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9083: Add disk serial to...

2015-11-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1121#issuecomment-160559759 LGTM based on the code. I don't see any real issues. --- 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: [4.6] CLOUDSTACK-4787 - vmware diskcontro...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1132#issuecomment-160561489 @sateesh-chodapuneedi this is a real bug for many users as they are not able to use Windows 2012 templates with vmware/CloudStack, which is why we need the fix in

Re: What SDN are you using?

2015-11-30 Thread Nux!
Eric, I think OVS is implied when we talk VXLAN on KVM. Simon, thanks for your thoughts. It's what I was afraid of. Guess it's still early days for SDN. :-) Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "GoGoZoom PDX" > To

Re: What SDN are you using?

2015-11-30 Thread sebgoa
On Nov 29, 2015, at 8:19 PM, Simon Weller wrote: > Nux, > > The two most deployed SDN solutions are Vmware NSX-MH (Nicara) and Nuage VSP > (Alcatel-Lucent). Neither are cheap! > We've been evaluating whether we're going to run a POC on Nuage, as it does > have some very cool features includi

Re: [NOTICE] people.apache.org web space is moving to home.apache.org

2015-11-30 Thread sebgoa
Talluri, Can you answer Ian's comment below ? -sebastien On Nov 28, 2015, at 3:03 AM, Ian Duffy wrote: > What are the legal implications of any of hosting that > licensed/copyrighted/paid-for material if any? > > windows7.vhd > RHEL6-64bit.ova > RHEL764bitwithtools.vhd > 79211594-1d4a-4dee-ae

Re: 4.7 status update

2015-11-30 Thread sebgoa
On Nov 27, 2015, at 7:23 PM, Remi Bergsma wrote: > Hi all, > > The next version of Apache CloudStack will be built on top of 4.6 and > released from the stable master branch. In the past weeks we’ve added several > PRs on master with new features: > > * Upgrade to latest AWS S3 SDK > *

[GitHub] cloudstack pull request: CID-1338387: Deletion of method endPointS...

2015-11-30 Thread runseb
Github user runseb commented on the pull request: https://github.com/apache/cloudstack/pull/1124#issuecomment-160568739 @rafaelweingartner there has been lots of discussion on the list about testing. The TravisCI setup is supposed to do part of it, and we can also trigger the tests as

[GitHub] cloudstack pull request: [4.5] CLOUDSTACK-4787 - vmware diskcontro...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1131#issuecomment-160574800 @sateesh-chodapuneedi as mentioned on the other PR, the issue is that many users are not able to use Windows 2012 templates and we see this as a severe bug that ne

[GitHub] cloudstack pull request: [4.6] CLOUDSTACK-9083: Add disk serial to...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1121#issuecomment-160574861 Merging now that we're tests performed and 2LGTMs. --- 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: [4.6] CLOUDSTACK-9083: Add disk serial to...

2015-11-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1121 --- 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: [4.5] CLOUDSTACK-9083: Add disk serial to...

2015-11-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1120 --- 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: [4.6] CLOUDSTACK-9083: Add disk serial to...

2015-11-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1121#issuecomment-160576751 great @bhaisaab I'll create the merge forward --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] cloudstack pull request: Quota

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/768#issuecomment-160578342 @remibergsma LGTM, I've rebased the branch against latest master now. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack pull request: libvirt fix forward merge

2015-11-30 Thread DaanHoogland
GitHub user DaanHoogland opened a pull request: https://github.com/apache/cloudstack/pull/1141 libvirt fix forward merge @remibergsma @karuturi can you check You can merge this pull request into a Git repository by running: $ git pull https://github.com/DaanHoogland/cloudstack

[GitHub] cloudstack pull request: Quota

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/768#discussion_r46123680 --- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade452to460.java --- @@ -66,10 +66,25 @@ public boolean supportsRollingUpgrade() { @Overrid

[GitHub] cloudstack pull request: libvirt fix forward merge

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1141#issuecomment-160580647 LGTM --- 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 ena

Build failed in Jenkins: simulator-4.5-singlerun #281

2015-11-30 Thread jenkins
See -- Started by upstream project "build-4.5-simulator" build number 335 originally caused by: Started by upstream project "build-4.5" build number 543 originally caused by: Started by an

[GitHub] cloudstack pull request: Quota

2015-11-30 Thread abhinandanprateek
Github user abhinandanprateek commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/768#discussion_r46124731 --- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade452to460.java --- @@ -66,10 +66,25 @@ public boolean supportsRollingUpgrade() {

[GitHub] cloudstack pull request: Quota

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/768#discussion_r46124926 --- Diff: engine/schema/src/com/cloud/upgrade/dao/Upgrade452to460.java --- @@ -66,10 +66,25 @@ public boolean supportsRollingUpgrade() { @Overrid

Re: Package Repositories

2015-11-30 Thread sebgoa
Hi folks, we need to resolve this. 1-But I have to start with one comment: Apache open office releases binaries, users don't compile from source. So it is possible within ASF to officially release binaries. 2-We have several initiatives around repos, apt-get.eu (Wido), shapeblue repos, Nux mirr

[GitHub] cloudstack pull request: CLOUDSTACK-9077 Fix injectkeys.sh to work...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1109#issuecomment-160585862 @wido @remibergsma do we want this on 4.5? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160588176 @davidamorimfaria @remibergsma should we port this to 4.5? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub a

Re: What SDN are you using?

2015-11-30 Thread Vadim Kimlaychuk
I don't know if applicable, but I had very good impression from VyOS (http://vyos.net/wiki/Main_Page). Vadim. On 2015-11-29 20:56, Nux! wrote: Hello, So besides the folks using Nicira, can anyone recommend any other SDN thing or should I stay with v(x)lans? I'm kind of removed from this sid

Re: Package Repositories

2015-11-30 Thread Nux!
+1 your categories. Also +1 for the unified thing under cloudstack.apache.org domain. -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "sebgoa" > To: dev@cloudstack.apache.org > Sent: Monday, 30 November, 2015 10:08:59 > Subject: Re: P

Re: Package Repositories

2015-11-30 Thread Daan Hoogland
+1 all the way, sebastien On Mon, Nov 30, 2015 at 11:20 AM, Nux! wrote: > +1 your categories. > > Also +1 for the unified thing under cloudstack.apache.org domain. > > -- > Sent from the Delta quadrant using Borg technology! > > Nux! > www.nux.ro > > - Original Message - > > From: "sebgo

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160589376 @bhaisaab I think it is fine to do so, but centos7 support was still experimental. Not sure if we spring hopes there that are unjust. --- If your project is s

Re: What SDN are you using?

2015-11-30 Thread Frank Louwers
Hi Vadim, Are you using VyOS in a Cloudstack environment? > On 30 Nov 2015, at 11:20, Vadim Kimlaychuk wrote: > > I don't know if applicable, but I had very good impression from VyOS > (http://vyos.net/wiki/Main_Page). > > Vadim. > > On 2015-11-29 20:56, Nux! wrote: > >> Hello, >> So besid

RE: Package Repositories

2015-11-30 Thread Paul Angus
Sebastien, Can you confirm that users can expect to find the noredist build of the rpms and debs at apt-get.eu ? [ShapeBlue Ltd] Paul Angus VP Technology , ShapeBlue Ltd s: 02036170528 | t: @cloudyangus | m: +44 7711418784 e

Re: Package Repositories

2015-11-30 Thread Rohit Yadav
On 30-Nov-2015, at 3:38 PM, sebgoa mailto:run...@gmail.com>> wrote: - source - community repo - 3-rd party repo I am +1 with this, why ? -source is a no brainer - community repo (apt-get) because that's our defacto pkg repo even though we don't vote on packages. There was not vote to say thes

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160590038 @DaanHoogland I see, leaving it then. CentOS7 users should use 4.6/4.7 then. --- If your project is set up for it, you can reply to this email and have your reply

Build failed in Jenkins: simulator-4.5-singlerun #282

2015-11-30 Thread jenkins
See -- Started by upstream project "build-4.5-simulator" build number 336 originally caused by: Started by upstream project "build-4.5" build number 544 originally caused by: Started by an

[GitHub] cloudstack pull request: CLOUDSTACK-9092: L10n fix in "Add LDAP Ac...

2015-11-30 Thread milamberspace
Github user milamberspace commented on the pull request: https://github.com/apache/cloudstack/pull/1139#issuecomment-160591699 Not tested, but code is good. LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread davidamorimfaria
Github user davidamorimfaria commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160594088 @bhaisaab I see no harm in trying to package 4.5 with this spec. It should just work (read, services start) if all the files expected by the spec file are

[GitHub] cloudstack pull request: CLOUDSTACK-9095 : Hypervisor changes to s...

2015-11-30 Thread nlivens
GitHub user nlivens opened a pull request: https://github.com/apache/cloudstack/pull/1142 CLOUDSTACK-9095 : Hypervisor changes to support UserData for Nuage VSP You can merge this pull request into a Git repository by running: $ git pull https://github.com/nlivens/cloudstack m

Re: Package Repositories

2015-11-30 Thread sebgoa
On Nov 30, 2015, at 11:28 AM, Rohit Yadav wrote: > >> On 30-Nov-2015, at 3:38 PM, sebgoa wrote: >> >> - source >> - community repo >> - 3-rd party repo >> >> I am +1 with this, why ? >> >> -source is a no brainer >> - community repo (apt-get) because that's our defacto pkg repo even though

[GitHub] cloudstack pull request: CLOUDSTACK-9010: adjust packaging for cen...

2015-11-30 Thread bhaisaab
Github user bhaisaab commented on the pull request: https://github.com/apache/cloudstack/pull/1008#issuecomment-160599550 @davidamorimfaria thanks, I personally won't be able to fix/test this soonish so dropping this unless of course you or someone wants this in 4.5 and sends a PR.

Re: What SDN are you using?

2015-11-30 Thread Vadim Kimlaychuk
We have been using it at our DEV cloud around year ago. Now we are out or resources so dev- environment is destroyed. Vadim. On 2015-11-30 12:24, Frank Louwers wrote: Hi Vadim, Are you using VyOS in a Cloudstack environment? On 30 Nov 2015, at 11:20, Vadim Kimlaychuk wrote: I don't know i

[GitHub] cloudstack pull request: Removed the PlannerBase class because it ...

2015-11-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1108#issuecomment-160599664 Code seems good, waiting for @borisroman to come up with test results --- If your project is set up for it, you can reply to this email and have your reply appear on G

[GitHub] cloudstack pull request: CLOUDSTACK-9080: Resource limits for Prim...

2015-11-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1107#issuecomment-160600074 The fix seems good to me. I can't run the Marvin tests, but it looks good to me --- If your project is set up for it, you can reply to this email and have your reply

[poll] cloudstack exam

2015-11-30 Thread Giles Sirett
Quick poll: has anybody here taken the ACCEL cloudstack certification exam ? what did you think ? Too hard, too easy ? – about right ? Also, by way of reminder: if you use the code ACCELpromocodeASF when registering for the exam, 1/3 of the fee goes to the ACS project Kind Regards Giles Giles

Re: What SDN are you using?

2015-11-30 Thread Nux!
Hello Vadim, How exactly are you integrating VyOS in Cloudstack? We're also using it internally, but not sure how it would help us, especially with regards to SDN. Lucian -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Vadim Kimlayc

Re: What SDN are you using?

2015-11-30 Thread Vadim Kimlaychuk
We mostly use it as external (to Cloudstack) intelligent switch/router, but also installed it as guest VM within cloud to provide customized SDN functionality. Vadim. On 2015-11-30 13:10, Nux! wrote: Hello Vadim, How exactly are you integrating VyOS in Cloudstack? We're also using it inter

[GitHub] cloudstack pull request: CID-1338387: Deletion of method endPointS...

2015-11-30 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1124#issuecomment-160603505 I was thinking about that. Starting in January we will have some extra time, then we might have a look at that. --- If your project is set up for it, you

Re: Package Repositories

2015-11-30 Thread Rajani Karuturi
+1 on the categories. ~Rajani On Mon, Nov 30, 2015 at 3:38 PM, sebgoa wrote: > Hi folks, we need to resolve this. > > 1-But I have to start with one comment: > Apache open office releases binaries, users don't compile from source. So > it is possible within ASF to officially release binaries. >

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-11-30 Thread nvazquez
Github user nvazquez commented on the pull request: https://github.com/apache/cloudstack/pull/1094#issuecomment-160636372 @miguelaferreira thanks for reviewing! Sure, would be great to discuss them --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] cloudstack pull request: CLOUDSTACK-6276 Fixing affinity groups fo...

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1134#issuecomment-160638862 @DaanHoogland Ok, I understand what you mean. I do think that we need some clarifications on what the standards are (search vs list, service vs query vs dao, etc.) PS

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-11-30 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46148014 --- Diff: api/src/com/cloud/network/guru/NetworkGuruAdditionalFunctions.java --- @@ -0,0 +1,12 @@ +package com.cloud.network.guru; + +import

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-11-30 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46148337 --- Diff: setup/db/db/schema-460to470.sql --- @@ -18,3 +18,5 @@ --; -- Schema upgrade from 4.6.0 to 4.7.0; --; + +ALTER TABLE `clo

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-11-30 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46148362 --- Diff: setup/db/db/schema-461to470.sql --- @@ -18,3 +18,5 @@ --; -- Schema upgrade from 4.6.1 to 4.7.0; --; + +ALTER TABLE `clo

Re: Package Repositories

2015-11-30 Thread Pierre-Luc Dion
Hi all, So just to make this email smaller, it will be strait... All discussions around repos is not a rant on Shapeblue where you guys are doing a super job on maintaining the community and the project. This as to do only with installation instructions of an opensource project where repos from i

New Defects reported by Coverity Scan for cloudstack

2015-11-30 Thread scan-admin
Hi, Please find the latest report on new defect(s) introduced to cloudstack found with Coverity Scan. 2 new defect(s) introduced to cloudstack found with Coverity Scan. 4 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(

Re: What SDN are you using?

2015-11-30 Thread Simon Weller
> We have the native KVM VXLAN implementation running in our lab and it seems > to work very well. We went 100% >10/40GB using open platform switches and > Cumulus linux. >Good to know. On the subject of VXLAN on KVM, note that is does not use Openvswitch. Openvswitch doesn't support multica

Re: What SDN are you using?

2015-11-30 Thread Nux!
Simon, To clarify, we're talking about this implementation, right? https://issues.apache.org/jira/browse/CLOUDSTACK-2328 -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Simon Weller" > To: dev@cloudstack.apache.org, "Kuangching Wang"

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

2015-11-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1054#issuecomment-160648931 @borisroman @DaanHoogland Anybody else? :) --- 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 proj

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

2015-11-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/985#issuecomment-160649027 Still working on this one. However, I need to get stuff into libvirt-java upstream. Working on that here: https://github.com/wido/libvirt-java/commits/qemu-guest-comman

Re: Package Repositories

2015-11-30 Thread Wido den Hollander
On 30-11-15 15:29, Pierre-Luc Dion wrote: > Hi all, > > So just to make this email smaller, it will be strait... > > All discussions around repos is not a rant on Shapeblue where you guys are > doing a super job on maintaining the community and the project. This as to > do only with installatio

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

2015-11-30 Thread NuxRo
Github user NuxRo commented on the pull request: https://github.com/apache/cloudstack/pull/985#issuecomment-160654716 Ok, godspeed. With a bit of luck perhaps we'll see this somewhere in 4.7.x? --- If your project is set up for it, you can reply to this email and have your reply

Re: What SDN are you using?

2015-11-30 Thread Simon Weller
> >From: Nux! >Sent: Monday, November 30, 2015 8:39 AM >To: dev@cloudstack.apache.org >Cc: Kuangching Wang; us...@cloudstack.apache.org >Subject: Re: What SDN are you using? > >Simon, > >To clarify, we're talking about this implementation, right? >https://is

[GitHub] cloudstack pull request: Show Agent name in Ping response

2015-11-30 Thread wido
GitHub user wido opened a pull request: https://github.com/apache/cloudstack/pull/1143 Show Agent name in Ping response Before: DEBUG [c.c.a.m.AgentManagerImpl] (AgentManager-Handler-4:null) Ping from 66 After: DEBUG [c.c.a.m.AgentManagerImpl] (AgentManage

Re: Package Repositories

2015-11-30 Thread Daan Hoogland
On Mon, Nov 30, 2015 at 3:38 PM, Wido den Hollander wrote: > > Paul, as far as releases I know, package on cloudstack.apt-get.eu > include > > noredis libraries. > > > > The debs don't I think. I never checked the RPMs. ​Any RPMs I uploaded do. Those are mostly the 4.4 ones (not sure but I thin

Re: Package Repositories

2015-11-30 Thread Nux!
BTW, if these bits are "no redistribute", why are we distributing them? -- Sent from the Delta quadrant using Borg technology! Nux! www.nux.ro - Original Message - > From: "Daan Hoogland" > To: "dev" > Sent: Monday, 30 November, 2015 15:28:24 > Subject: Re: Package Repositories > On M

[GitHub] cloudstack pull request: CLOUDSTACK-9074: Support shared networkin...

2015-11-30 Thread nvazquez
Github user nvazquez commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1094#discussion_r46159749 --- Diff: api/src/com/cloud/network/Networks.java --- @@ -251,6 +252,10 @@ public static URI fromString(String candidate) { if (com

[GitHub] cloudstack pull request: Show Agent name in Ping response

2015-11-30 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1143#issuecomment-160670527 LGTM --- 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

[GitHub] cloudstack pull request: Show Agent name in Ping response

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1143#issuecomment-160674157 LGTM, but consider using the default log4j formatter. ex: `logger.debug("Ping from {} ({})", hostid, hostname);` and extracting the message to a constant ---

[GitHub] cloudstack pull request: Show Agent name in Ping response

2015-11-30 Thread wido
Github user wido commented on the pull request: https://github.com/apache/cloudstack/pull/1143#issuecomment-160679884 @pdube Thanks! I didn't know that existed :) --- 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

Re: Package Repositories

2015-11-30 Thread sebgoa
On Nov 30, 2015, at 4:39 PM, Nux! wrote: > BTW, if these bits are "no redistribute", why are we distributing them? This is really a case of evolution and expectation. When Wido started this, these pkgs were not sanctioned by the *project*, hence non-official release pkgs and not voted on. Th

Re: Package Repositories

2015-11-30 Thread sebgoa
On Nov 30, 2015, at 11:28 AM, Paul Angus wrote: > Sebastien, > > Can you confirm that users can expect to find the noredist build of the rpms > and debs at apt-get.eu ? > I don't know What I understood from Rohit, is that the "shapeblue" repo also contains pkgs that are hot-fixed and have

[GitHub] cloudstack pull request: Show Agent name in Ping response

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1143#issuecomment-160684591 @wido :) --- 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 en

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

2015-11-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-160704405 there is a merge conflict @rodrigo93 . can you resolve it please? --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

2015-11-30 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-160705037 @DaanHoogland , where did you see the merge conflict Daan ? I did not find it. --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack pull request: Add marvin test to verify if DomainAdmin ...

2015-11-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1122#issuecomment-160711475 LGTM, the test runs fine here: ``` Testcreation, adding and removing tag on public IP address ... === TestName: test_24_public_IP_tag | Status : SUC

[GitHub] cloudstack pull request: CLOUDSTACK-8988: Removed unused adapters ...

2015-11-30 Thread rodrigo93
Github user rodrigo93 commented on the pull request: https://github.com/apache/cloudstack/pull/943#issuecomment-160714102 @DaanHoogland Can you tell us where the conflict is? It seems ok to me. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] cloudstack pull request: Fix secondary storage not working with sw...

2015-11-30 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/1133#issuecomment-160715727 LGTM, quickly tested, can create vm, templates and snapshots. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] cloudstack pull request: Fix secondary storage not working with sw...

2015-11-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1133 --- 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: Update L10N resource files with 4.6 strin...

2015-11-30 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/1138#issuecomment-160719956 LGTM, seams to fix linguistic issues, at least solving few for French. --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-11-30 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1138#discussion_r46184167 --- Diff: client/WEB-INF/classes/resources/messages_fr_FR.properties --- @@ -1739,7 +1740,7 @@ message.action.enable.nexusVswitch=Confirmer l\\'activation

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1138#issuecomment-160720589 Suggested a small french change, but still LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If you

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1138#issuecomment-160721861 @remibergsma Fine with me --- 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: Update L10N resource files with 4.6 strin...

2015-11-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1138#issuecomment-160721455 Thanks for the reviews, I'll merge it to 4.6 like this. The improvement should go via Transiflex website and ends up in a new PR when @milamberspace submits the

[GitHub] cloudstack pull request: Update L10N resource files with 4.6 strin...

2015-11-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1138 --- 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: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-11-30 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1144 [4.6.1] CLOUDSTACK-9022: keep Destroyed volumes for sometime for now, the Destroyed volumes will be expunged in Storage cleanup thread, no matter when they are destroyed. In Expunging v

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-11-30 Thread ustcweizhou
GitHub user ustcweizhou opened a pull request: https://github.com/apache/cloudstack/pull/1145 [4.6.1] CLOUDSTACK-9022: keep Destroyed volumes for sometime for now, the Destroyed volumes will be expunged in Storage cleanup thread, no matter when they are destroyed. In Expunging v

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1144#issuecomment-160739865 Hey @ustcweizhou, could you create the PR towards 4.6 please, it is hard to determine exactly what you changed. Thanks --- If your project is set up for it, you can

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread remibergsma
GitHub user remibergsma opened a pull request: https://github.com/apache/cloudstack/pull/1146 make sure all files are updates with new version During the previous release I had to edit these manually. This will fix that so all versions get updated. You can merge this pull request i

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-11-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1144#issuecomment-160756847 @ustcweizhou This is a mistake I guess, PR against the wrong branch? --- If your project is set up for it, you can reply to this email and have your reply appea

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread DaanHoogland
Github user DaanHoogland commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1146#discussion_r46201120 --- Diff: tools/build/build_asf.sh --- @@ -101,6 +101,11 @@ perl -pi -e "s/-SNAPSHOT//" build/replace.properties perl -pi -e "s/-SNAPSHOT//" ser

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-11-30 Thread ustcweizhou
Github user ustcweizhou closed the pull request at: https://github.com/apache/cloudstack/pull/1144 --- 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 featu

[GitHub] cloudstack pull request: [4.6.1] CLOUDSTACK-9022: keep Destroyed v...

2015-11-30 Thread ustcweizhou
Github user ustcweizhou commented on the pull request: https://github.com/apache/cloudstack/pull/1144#issuecomment-160758013 @pdube @remibergsma I will recreate a new PR. --- 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: make sure all files are updates with new ...

2015-11-30 Thread DaanHoogland
Github user DaanHoogland commented on the pull request: https://github.com/apache/cloudstack/pull/1146#issuecomment-160761783 Looks good to me. The version-branch should have gone in like this from the get go. --- If your project is set up for it, you can reply to this email and have

Re: Package Repositories

2015-11-30 Thread John Burwell
Sebastian, As I mentioned in my initial message, the public ShapeBlue repositories [1] are noredist builds of release baselines with no additional patches or changes. My understanding is that the project cannot officially distribute the noredist plugins due to inability of the community to dis

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1146#discussion_r46204918 --- Diff: tools/build/build_asf.sh --- @@ -101,6 +101,11 @@ perl -pi -e "s/-SNAPSHOT//" build/replace.properties perl -pi -e "s/-SNAPSHOT//" services/c

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread pdube
Github user pdube commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1146#discussion_r46204974 --- Diff: tools/build/build_asf.sh --- @@ -123,11 +128,7 @@ git clean -f #create a RC branch RELEASE_BRANCH="RC"`date +%Y%m%dT%H%M` ---

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1146#issuecomment-160764367 The build script now makes a branch that makes sense: `4.6.1-RC20151130T2206` instead of `4.6-RC20151130T2206`. Also the versions are updated properly.

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread remibergsma
Github user remibergsma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1146#discussion_r46205215 --- Diff: tools/build/build_asf.sh --- @@ -101,6 +101,11 @@ perl -pi -e "s/-SNAPSHOT//" build/replace.properties perl -pi -e "s/-SNAPSHOT//" serv

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread remibergsma
Github user remibergsma commented on a diff in the pull request: https://github.com/apache/cloudstack/pull/1146#discussion_r46205478 --- Diff: tools/build/build_asf.sh --- @@ -123,11 +128,7 @@ git clean -f #create a RC branch RELEASE_BRANCH="RC"`date +%Y%m%dT%H%M`

[GitHub] cloudstack pull request:

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/commit/7d7420fd1e5908e1bc4ee04a314241212f7a6481#commitcomment-14693674 +1 --- 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 proj

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread pdube
Github user pdube commented on the pull request: https://github.com/apache/cloudstack/pull/1146#issuecomment-160765582 Ok then, LGTM --- 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 featu

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread rafaelweingartner
Github user rafaelweingartner commented on the pull request: https://github.com/apache/cloudstack/pull/1146#issuecomment-160765816 It LGTM, I would just recommend using “sed” in combination with “find” to look up for every file that we want to change the version number, in

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread remibergsma
Github user remibergsma commented on the pull request: https://github.com/apache/cloudstack/pull/1146#issuecomment-160766514 @rafaelweingartner I agree and I think this needs some TLC indeed. Investigate why it doesn't work with mvn and then do one global search/replace for the remain

[GitHub] cloudstack pull request: make sure all files are updates with new ...

2015-11-30 Thread pdion891
Github user pdion891 commented on the pull request: https://github.com/apache/cloudstack/pull/1146#issuecomment-160766945 LGTM --- 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 ena

  1   2   >