You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Rajani Karuturi <ra...@apache.org> on 2017/02/07 13:07:59 UTC
PRs for 4.10
Hi all,
If you know of any PRs which meet the below criteria, please mail them to
me.
1. No open questions.
2. Has two LGTMs
3. Has CI test results.
~Rajani
http://cloudplatform.accelerite.com
Re: PRs for 4.10
Posted by Will Stevens <wi...@gmail.com>.
I got my Jenkins fixed today. I will run tests against my bubble env
tonight and we can go with that if Trillian can not do a systemvm template
from the PR for this one.
On Feb 7, 2017 7:16 PM, "Rajani Karuturi" <ra...@apache.org> wrote:
> @milamber I will soon merge the localization PRs
>
> @will Yes. that would be great addition to 4.10. Its pending from
> a long time.
>
> @sergey 1913 has open questions. Can you answer them please? rest
> I will take a look.
>
> PS: based on the pattern(17*[0,2,4,8,]7) I also checked 1707. I
> will add it to my PR list :)
>
> Thanks,
>
> ~ Rajani
>
> http://cloudplatform.accelerite.com/
>
> On February 8, 2017 at 2:23 AM, Sergey Levitskiy
> (Sergey.Levitskiy@autodesk.com) wrote:
>
> I would like to include :
> PR1727 CLOUDSTACK-9539: Support changing Service offering for
> instance with VM Snapshots
> PR1747 CLOUDSTACK-9574: Redesign storage views
> PR1767 CLOUDSTACK-9457: Allow retrieval and modification of VM
> and template details via API and UI
> They have all tests green and just need another LGTM
>
> 2 optimization PRs will be nice to add as well:
> PR1905 CLOUDSTACK-9738: [Vmware] Optimize vm expunge process for
> instances with vm snapshots
> PR1913 CLOUDSTACK-9752: [Vmware] Optimization of volume
> attachment to vm
Re: PRs for 4.10
Posted by Rajani Karuturi <ra...@apache.org>.
@milamber I will soon merge the localization PRs
@will Yes. that would be great addition to 4.10. Its pending from
a long time.
@sergey 1913 has open questions. Can you answer them please? rest
I will take a look.
PS: based on the pattern(17*[0,2,4,8,]7) I also checked 1707. I
will add it to my PR list :)
Thanks,
~ Rajani
http://cloudplatform.accelerite.com/
On February 8, 2017 at 2:23 AM, Sergey Levitskiy
(Sergey.Levitskiy@autodesk.com) wrote:
I would like to include :
PR1727 CLOUDSTACK-9539: Support changing Service offering for
instance with VM Snapshots
PR1747 CLOUDSTACK-9574: Redesign storage views
PR1767 CLOUDSTACK-9457: Allow retrieval and modification of VM
and template details via API and UI
They have all tests green and just need another LGTM
2 optimization PRs will be nice to add as well:
PR1905 CLOUDSTACK-9738: [Vmware] Optimize vm expunge process for
instances with vm snapshots
PR1913 CLOUDSTACK-9752: [Vmware] Optimization of volume
attachment to vm
Re: PRs for 4.10
Posted by Sergey Levitskiy <Se...@autodesk.com>.
I would like to include :
PR1727 CLOUDSTACK-9539: Support changing Service offering for instance with VM Snapshots
PR1747 CLOUDSTACK-9574: Redesign storage views
PR1767 CLOUDSTACK-9457: Allow retrieval and modification of VM and template details via API and UI
They have all tests green and just need another LGTM
2 optimization PRs will be nice to add as well:
PR1905 CLOUDSTACK-9738: [Vmware] Optimize vm expunge process for instances with vm snapshots
PR1913 CLOUDSTACK-9752: [Vmware] Optimization of volume attachment to vm
Re: PRs for 4.10
Posted by Milamber <mi...@apache.org>.
No 2 LGTMs, but this PR only updated the localization files for 4.10
(green on CI)
https://github.com/apache/cloudstack/pull/1914
On 07/02/2017 13:07, Rajani Karuturi wrote:
> Hi all,
> If you know of any PRs which meet the below criteria, please mail them to
> me.
> 1. No open questions.
> 2. Has two LGTMs
> 3. Has CI test results.
>
> ~Rajani
> http://cloudplatform.accelerite.com
>
RE: PRs for 4.10
Posted by Daan Hoogland <da...@shapeblue.com>.
Rajani, I found 844 browsing through old wishes and I think it should go in. testing is underway but I don't know about the review and new test code yet.
daan.hoogland@shapeblue.com
www.shapeblue.com
53 Chandos Place, Covent Garden, Utrecht Utrecht 3531 VENetherlands
@shapeblue
-----Original Message-----
From: Wido den Hollander [mailto:wido@widodh.nl]
Sent: woensdag 8 februari 2017 07:28
To: dev@cloudstack.apache.org
Subject: Re: PRs for 4.10
A small bugfix for IPv6 in Basic Networking: https://github.com/apache/cloudstack/pull/1927
Would be great!
Wido
> Op 7 feb. 2017 om 14:08 heeft Rajani Karuturi <ra...@apache.org> het volgende geschreven:
>
> Hi all,
> If you know of any PRs which meet the below criteria, please mail them
> to me.
> 1. No open questions.
> 2. Has two LGTMs
> 3. Has CI test results.
>
> ~Rajani
> http://cloudplatform.accelerite.com
Re: PRs for 4.10
Posted by Wido den Hollander <wi...@widodh.nl>.
A small bugfix for IPv6 in Basic Networking: https://github.com/apache/cloudstack/pull/1927
Would be great!
Wido
> Op 7 feb. 2017 om 14:08 heeft Rajani Karuturi <ra...@apache.org> het volgende geschreven:
>
> Hi all,
> If you know of any PRs which meet the below criteria, please mail them to
> me.
> 1. No open questions.
> 2. Has two LGTMs
> 3. Has CI test results.
>
> ~Rajani
> http://cloudplatform.accelerite.com
Re:PRs for 4.10
Posted by Haijiao <18...@163.com>.
Expect the following PRs to be merged into 4.10, thanks !
PR #1929 CLOUDSTACK-9770: fix missing ip routes in VR, 2 LGTM ready
PR #1915 CLOUDSTACK-9746 system-vm: logrotate config causes critical failures 1 LGTM ready, just need CI test
PR #1727 CLOUDSTACK-9539: Support changing Service offering for instance with VM Snapshots 2 LGTM ready
PR #1903 CLOUDSTACK-9356: FIX Cannot add users in VPC VPN waiting for review
PR #1829 CLOUDSTACK-9363: Fix HVM VM restart bug in XenServer waiting for review
PR #1849 CLOUDSTACK-9690: Scale CentOS7 VM fails with error waiting for review
Regards,
在2017年02月07 21时07分, "Rajani Karuturi"<ra...@apache.org>写道:
Hi all,
If you know of any PRs which meet the below criteria, please mail them to
me.
1. No open questions.
2. Has two LGTMs
3. Has CI test results.
~Rajani
http://cloudplatform.accelerite.com
Re: PRs for 4.10
Posted by Rajani Karuturi <ra...@apache.org>.
please use the criteria mentioned at http://acspr.ena.net/help.html to tag
PRs as merge ready.
~Rajani
http://cloudplatform.accelerite.com/
On Thu, Feb 23, 2017 at 4:24 AM, Sergey Levitskiy <
Sergey.Levitskiy@autodesk.com> wrote:
> I would like to get this added
> https://github.com/apache/cloudstack/pull/1935
>
>
> On 2/22/17, 2:33 PM, "David Mabry" <dm...@ena.com> wrote:
>
> I would like to see the following PRs merged:
>
> https://github.com/apache/cloudstack/pull/1915
> https://github.com/apache/cloudstack/pull/1954 <- This one could be
> backported to 4.8/4.9
>
> Collectively, these solve an issue that causes the VPC router to fail
> due to a full /var/log FS. We’ve tested these extensively in the lab and
> confirmed that it does solve the problem.
>
> Thanks,
> David Mabry
>
> On 2/22/17, 2:00 PM, "williamstevens@gmail.com on behalf of Will
> Stevens" <williamstevens@gmail.com on behalf of wstevens@cloudops.com>
> wrote:
>
> I would like to get this fix in:
> https://github.com/apache/cloudstack/pull/1907
>
> This fix is really important because it causes routing issues when
> the IP,
> which was not cleaned up, is later used on a different VR.
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 22, 2017 at 5:29 AM, Rohit Yadav <
> rohit.yadav@shapeblue.com>
> wrote:
>
> > Rajani and all,
> >
> >
> > Please consider following PRs for review and merge:
> >
> >
> > https://github.com/apache/cloudstack/pull/1829 (XenServer HVM
> VM attached
> > disk limit bugfix, enough LGTMs and test results)
> >
> > https://github.com/apache/cloudstack/pull/1941 (same as above
> with
> > additional fixes, lacks review lgtms)
> > https://github.com/apache/cloudstack/pull/1896 (translation
> updates)
> >
> > https://github.com/apache/cloudstack/pull/1770 (template size
> issue
> > bugfix)
> >
> > https://github.com/apache/cloudstack/pull/1950 (Ubuntu 16.04
> packaging
> > tomcat6/7 support)
> >
> > https://github.com/apache/cloudstack/pull/1951 (xenserver7
> capability fix
> > and upgrade path from 4.9.2->4.9.3)
> >
> > https://github.com/apache/cloudstack/pull/1944 (metrics view
> and infra
> > tab/UI performance improvement)
> > https://github.com/apache/cloudstack/pull/1903 (cannot add
> users to vpc
> > vpn)
> >
> > https://github.com/apache/cloudstack/pull/1945
> >
> > <https://github.com/apache/cloudstack/pull/1945>https://
> > github.com/apache/cloudstack/pull/1946
> >
> > <https://github.com/apache/cloudstack/pull/1946>https://
> > github.com/apache/cloudstack/pull/1947
> > https://github.com/apache/cloudstack/pull/1856
> > https://github.com/apache/cloudstack/pull/1212
> >
> >
> >
> > Regards.
> >
> > ________________________________
> > From: Rajani Karuturi <ra...@apache.org>
> > Sent: 22 February 2017 14:41:42
> > To: dev@cloudstack.apache.org
> > Subject: Re: PRs for 4.10
> >
> > 6 days to go for the first RC.
> > Please get the required lgtms and tests ready for the PRs you
> like to see
> > in 4.10.
> > I will try to merge all the PRs that meet the criteria.
> >
> > Thanks,
> > ~Rajani
> >
> > On 19 Feb 2017 3:11 p.m., "Rajani Karuturi" <ra...@apache.org>
> wrote:
> >
> > > noted.
> > >
> > >
> > > ~ Rajani
> > >
> > > http://cloudplatform.accelerite.com/
> > >
> > >
> > > On February 17, 2017 at 6:03 PM, Frank Maximus (
> > > frank.maximus@nuagenetworks.net) wrote:
> > >
> > > I have a couple of bugfixes on previous version outstanding,
> > > which I would like to have merged to 4.10,
> > > both still requiring review:
> > > on 4.8: PR#1912 <https://github.com/apache/
> cloudstack/pull/1912>: which
> > > fixes password service running on internal lb vms, making it
> impossible
> > to
> > > do loadbalancing on port 8080
> > > on 4.9: PR#1925 <https://github.com/apache/
> cloudstack/pull/1925>: Minor
> > > plugin fix
> > >
> > > Kind Regards,
> > > Frank
> > >
> > > On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed <
> sahmed@cloudops.com> wrote:
> > >
> > > I'd like to include https://github.com/apache/
> cloudstack/pull/1928 to
> > 4.10
> > > as well. This is a simple fix that adds hypervisor
> capabilities for
> > > XenServer 7
> > >
> > > Thanks,
> > > -Syed
> > >
> > > On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <
> wstevens@cloudops.com>
> > > wrote:
> > >
> > > Not sure, I will see if I can find some time tomorrow to look
> at this.
> > > Thanks...
> > >
> > > *Will STEVENS*
> > > Lead Developer
> > >
> > > <https://goo.gl/NYZ8KK>
> > >
> > > On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <
> rajani@apache.org>
> > > wrote:
> > >
> > > Thanks Will. I will take a look at this today and merge.
> > >
> > > Is it possible to add a systemvmtemplate job on
> > > builds.apachecloudstack.net (https://builds.cloudstack.org/)
> > > similar to
> > > http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
> > >
> > > Thanks,
> > >
> > > ~ Rajani
> > >
> > > http://cloudplatform.accelerite.com/
> > >
> > > On February 13, 2017 at 10:46 PM, Will Stevens
> > > (wstevens@cloudops.com) wrote:
> > >
> > > At this point, I think 1741 is ready to merge. I think we need
> > > one more
> > > LGTM though...
> > >
> > > *Will STEVENS*
> > > Lead Developer
> > >
> > > <https://goo.gl/NYZ8KK>
> > >
> > > On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> > > <ws...@cloudops.com> wrote:
> > >
> > > @Daan: In order for the tests to pass with 1741, the tests have
> > > to be run
> > > with the SystemVM from the PR being used.
> > >
> > > *Will STEVENS*
> > > Lead Developer
> > >
> > > <https://goo.gl/NYZ8KK>
> > >
> > > On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> > > <da...@gmail.com>
> > > wrote:
> > >
> > > About 1741, I started the packaging, Will but saw Boris making
> a
> > > remark about the system VMs. I'll look into that and start the
> > > tests.
> > >
> > > On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> > > <wi...@gmail.com>
> > > wrote:
> > >
> > > I would like to get 1741(strongswan) in, but I need to get
> > > Trillian to
> > >
> > > run
> > >
> > > against it with a system vm built from my pr. I have done a CI
> in
> > > bubble
> > > and it came back clean, but would like confirmation through
> > > Trillian as
> > > well.
> > >
> > > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org>
> wrote:
> > >
> > > And this PR for 4.9 but need to be foward to 4.10 too.
> > >
> > > https://github.com/apache/cloudstack/pull/1929
> > >
> > > 2 LGTM
> > > CI green
> > >
> > > On 07/02/2017 13:07, Rajani Karuturi wrote:
> > >
> > > Hi all,
> > > If you know of any PRs which meet the below criteria, please
> > > mail
> > >
> > > them to
> > >
> > > me.
> > > 1. No open questions.
> > > 2. Has two LGTMs
> > > 3. Has CI test results.
> > >
> > > ~Rajani
> > > http://cloudplatform.accelerite.com
> > >
> > > --
> > > Daan
> > >
> > >
> >
> > rohit.yadav@shapeblue.com
> > www.shapeblue.com
> > 53 Chandos Place, Covent Garden, London WC2N 4HSUK
> > @shapeblue
> >
> >
> >
> >
>
>
>
>
>
Re: PRs for 4.10
Posted by Sergey Levitskiy <Se...@autodesk.com>.
I would like to get this added
https://github.com/apache/cloudstack/pull/1935
On 2/22/17, 2:33 PM, "David Mabry" <dm...@ena.com> wrote:
I would like to see the following PRs merged:
https://github.com/apache/cloudstack/pull/1915
https://github.com/apache/cloudstack/pull/1954 <- This one could be backported to 4.8/4.9
Collectively, these solve an issue that causes the VPC router to fail due to a full /var/log FS. We’ve tested these extensively in the lab and confirmed that it does solve the problem.
Thanks,
David Mabry
On 2/22/17, 2:00 PM, "williamstevens@gmail.com on behalf of Will Stevens" <williamstevens@gmail.com on behalf of wstevens@cloudops.com> wrote:
I would like to get this fix in:
https://github.com/apache/cloudstack/pull/1907
This fix is really important because it causes routing issues when the IP,
which was not cleaned up, is later used on a different VR.
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 22, 2017 at 5:29 AM, Rohit Yadav <ro...@shapeblue.com>
wrote:
> Rajani and all,
>
>
> Please consider following PRs for review and merge:
>
>
> https://github.com/apache/cloudstack/pull/1829 (XenServer HVM VM attached
> disk limit bugfix, enough LGTMs and test results)
>
> https://github.com/apache/cloudstack/pull/1941 (same as above with
> additional fixes, lacks review lgtms)
> https://github.com/apache/cloudstack/pull/1896 (translation updates)
>
> https://github.com/apache/cloudstack/pull/1770 (template size issue
> bugfix)
>
> https://github.com/apache/cloudstack/pull/1950 (Ubuntu 16.04 packaging
> tomcat6/7 support)
>
> https://github.com/apache/cloudstack/pull/1951 (xenserver7 capability fix
> and upgrade path from 4.9.2->4.9.3)
>
> https://github.com/apache/cloudstack/pull/1944 (metrics view and infra
> tab/UI performance improvement)
> https://github.com/apache/cloudstack/pull/1903 (cannot add users to vpc
> vpn)
>
> https://github.com/apache/cloudstack/pull/1945
>
> <https://github.com/apache/cloudstack/pull/1945>https://
> github.com/apache/cloudstack/pull/1946
>
> <https://github.com/apache/cloudstack/pull/1946>https://
> github.com/apache/cloudstack/pull/1947
> https://github.com/apache/cloudstack/pull/1856
> https://github.com/apache/cloudstack/pull/1212
>
>
>
> Regards.
>
> ________________________________
> From: Rajani Karuturi <ra...@apache.org>
> Sent: 22 February 2017 14:41:42
> To: dev@cloudstack.apache.org
> Subject: Re: PRs for 4.10
>
> 6 days to go for the first RC.
> Please get the required lgtms and tests ready for the PRs you like to see
> in 4.10.
> I will try to merge all the PRs that meet the criteria.
>
> Thanks,
> ~Rajani
>
> On 19 Feb 2017 3:11 p.m., "Rajani Karuturi" <ra...@apache.org> wrote:
>
> > noted.
> >
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> >
> > On February 17, 2017 at 6:03 PM, Frank Maximus (
> > frank.maximus@nuagenetworks.net) wrote:
> >
> > I have a couple of bugfixes on previous version outstanding,
> > which I would like to have merged to 4.10,
> > both still requiring review:
> > on 4.8: PR#1912 <https://github.com/apache/cloudstack/pull/1912>: which
> > fixes password service running on internal lb vms, making it impossible
> to
> > do loadbalancing on port 8080
> > on 4.9: PR#1925 <https://github.com/apache/cloudstack/pull/1925>: Minor
> > plugin fix
> >
> > Kind Regards,
> > Frank
> >
> > On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed <sa...@cloudops.com> wrote:
> >
> > I'd like to include https://github.com/apache/cloudstack/pull/1928 to
> 4.10
> > as well. This is a simple fix that adds hypervisor capabilities for
> > XenServer 7
> >
> > Thanks,
> > -Syed
> >
> > On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <ws...@cloudops.com>
> > wrote:
> >
> > Not sure, I will see if I can find some time tomorrow to look at this.
> > Thanks...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org>
> > wrote:
> >
> > Thanks Will. I will take a look at this today and merge.
> >
> > Is it possible to add a systemvmtemplate job on
> > builds.apachecloudstack.net (https://builds.cloudstack.org/)
> > similar to
> > http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
> >
> > Thanks,
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> > On February 13, 2017 at 10:46 PM, Will Stevens
> > (wstevens@cloudops.com) wrote:
> >
> > At this point, I think 1741 is ready to merge. I think we need
> > one more
> > LGTM though...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> > <ws...@cloudops.com> wrote:
> >
> > @Daan: In order for the tests to pass with 1741, the tests have
> > to be run
> > with the SystemVM from the PR being used.
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> > <da...@gmail.com>
> > wrote:
> >
> > About 1741, I started the packaging, Will but saw Boris making a
> > remark about the system VMs. I'll look into that and start the
> > tests.
> >
> > On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> > <wi...@gmail.com>
> > wrote:
> >
> > I would like to get 1741(strongswan) in, but I need to get
> > Trillian to
> >
> > run
> >
> > against it with a system vm built from my pr. I have done a CI in
> > bubble
> > and it came back clean, but would like confirmation through
> > Trillian as
> > well.
> >
> > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
> >
> > And this PR for 4.9 but need to be foward to 4.10 too.
> >
> > https://github.com/apache/cloudstack/pull/1929
> >
> > 2 LGTM
> > CI green
> >
> > On 07/02/2017 13:07, Rajani Karuturi wrote:
> >
> > Hi all,
> > If you know of any PRs which meet the below criteria, please
> > mail
> >
> > them to
> >
> > me.
> > 1. No open questions.
> > 2. Has two LGTMs
> > 3. Has CI test results.
> >
> > ~Rajani
> > http://cloudplatform.accelerite.com
> >
> > --
> > Daan
> >
> >
>
> rohit.yadav@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London WC2N 4HSUK
> @shapeblue
>
>
>
>
Re: PRs for 4.10
Posted by David Mabry <dm...@ena.com>.
I would like to see the following PRs merged:
https://github.com/apache/cloudstack/pull/1915
https://github.com/apache/cloudstack/pull/1954 <- This one could be backported to 4.8/4.9
Collectively, these solve an issue that causes the VPC router to fail due to a full /var/log FS. We’ve tested these extensively in the lab and confirmed that it does solve the problem.
Thanks,
David Mabry
On 2/22/17, 2:00 PM, "williamstevens@gmail.com on behalf of Will Stevens" <williamstevens@gmail.com on behalf of wstevens@cloudops.com> wrote:
I would like to get this fix in:
https://github.com/apache/cloudstack/pull/1907
This fix is really important because it causes routing issues when the IP,
which was not cleaned up, is later used on a different VR.
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 22, 2017 at 5:29 AM, Rohit Yadav <ro...@shapeblue.com>
wrote:
> Rajani and all,
>
>
> Please consider following PRs for review and merge:
>
>
> https://github.com/apache/cloudstack/pull/1829 (XenServer HVM VM attached
> disk limit bugfix, enough LGTMs and test results)
>
> https://github.com/apache/cloudstack/pull/1941 (same as above with
> additional fixes, lacks review lgtms)
> https://github.com/apache/cloudstack/pull/1896 (translation updates)
>
> https://github.com/apache/cloudstack/pull/1770 (template size issue
> bugfix)
>
> https://github.com/apache/cloudstack/pull/1950 (Ubuntu 16.04 packaging
> tomcat6/7 support)
>
> https://github.com/apache/cloudstack/pull/1951 (xenserver7 capability fix
> and upgrade path from 4.9.2->4.9.3)
>
> https://github.com/apache/cloudstack/pull/1944 (metrics view and infra
> tab/UI performance improvement)
> https://github.com/apache/cloudstack/pull/1903 (cannot add users to vpc
> vpn)
>
> https://github.com/apache/cloudstack/pull/1945
>
> <https://github.com/apache/cloudstack/pull/1945>https://
> github.com/apache/cloudstack/pull/1946
>
> <https://github.com/apache/cloudstack/pull/1946>https://
> github.com/apache/cloudstack/pull/1947
> https://github.com/apache/cloudstack/pull/1856
> https://github.com/apache/cloudstack/pull/1212
>
>
>
> Regards.
>
> ________________________________
> From: Rajani Karuturi <ra...@apache.org>
> Sent: 22 February 2017 14:41:42
> To: dev@cloudstack.apache.org
> Subject: Re: PRs for 4.10
>
> 6 days to go for the first RC.
> Please get the required lgtms and tests ready for the PRs you like to see
> in 4.10.
> I will try to merge all the PRs that meet the criteria.
>
> Thanks,
> ~Rajani
>
> On 19 Feb 2017 3:11 p.m., "Rajani Karuturi" <ra...@apache.org> wrote:
>
> > noted.
> >
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> >
> > On February 17, 2017 at 6:03 PM, Frank Maximus (
> > frank.maximus@nuagenetworks.net) wrote:
> >
> > I have a couple of bugfixes on previous version outstanding,
> > which I would like to have merged to 4.10,
> > both still requiring review:
> > on 4.8: PR#1912 <https://github.com/apache/cloudstack/pull/1912>: which
> > fixes password service running on internal lb vms, making it impossible
> to
> > do loadbalancing on port 8080
> > on 4.9: PR#1925 <https://github.com/apache/cloudstack/pull/1925>: Minor
> > plugin fix
> >
> > Kind Regards,
> > Frank
> >
> > On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed <sa...@cloudops.com> wrote:
> >
> > I'd like to include https://github.com/apache/cloudstack/pull/1928 to
> 4.10
> > as well. This is a simple fix that adds hypervisor capabilities for
> > XenServer 7
> >
> > Thanks,
> > -Syed
> >
> > On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <ws...@cloudops.com>
> > wrote:
> >
> > Not sure, I will see if I can find some time tomorrow to look at this.
> > Thanks...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org>
> > wrote:
> >
> > Thanks Will. I will take a look at this today and merge.
> >
> > Is it possible to add a systemvmtemplate job on
> > builds.apachecloudstack.net (https://builds.cloudstack.org/)
> > similar to
> > http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
> >
> > Thanks,
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> > On February 13, 2017 at 10:46 PM, Will Stevens
> > (wstevens@cloudops.com) wrote:
> >
> > At this point, I think 1741 is ready to merge. I think we need
> > one more
> > LGTM though...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> > <ws...@cloudops.com> wrote:
> >
> > @Daan: In order for the tests to pass with 1741, the tests have
> > to be run
> > with the SystemVM from the PR being used.
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> > <da...@gmail.com>
> > wrote:
> >
> > About 1741, I started the packaging, Will but saw Boris making a
> > remark about the system VMs. I'll look into that and start the
> > tests.
> >
> > On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> > <wi...@gmail.com>
> > wrote:
> >
> > I would like to get 1741(strongswan) in, but I need to get
> > Trillian to
> >
> > run
> >
> > against it with a system vm built from my pr. I have done a CI in
> > bubble
> > and it came back clean, but would like confirmation through
> > Trillian as
> > well.
> >
> > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
> >
> > And this PR for 4.9 but need to be foward to 4.10 too.
> >
> > https://github.com/apache/cloudstack/pull/1929
> >
> > 2 LGTM
> > CI green
> >
> > On 07/02/2017 13:07, Rajani Karuturi wrote:
> >
> > Hi all,
> > If you know of any PRs which meet the below criteria, please
> > mail
> >
> > them to
> >
> > me.
> > 1. No open questions.
> > 2. Has two LGTMs
> > 3. Has CI test results.
> >
> > ~Rajani
> > http://cloudplatform.accelerite.com
> >
> > --
> > Daan
> >
> >
>
> rohit.yadav@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London WC2N 4HSUK
> @shapeblue
>
>
>
>
Re: PRs for 4.10
Posted by Will Stevens <ws...@cloudops.com>.
I would like to get this fix in:
https://github.com/apache/cloudstack/pull/1907
This fix is really important because it causes routing issues when the IP,
which was not cleaned up, is later used on a different VR.
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 22, 2017 at 5:29 AM, Rohit Yadav <ro...@shapeblue.com>
wrote:
> Rajani and all,
>
>
> Please consider following PRs for review and merge:
>
>
> https://github.com/apache/cloudstack/pull/1829 (XenServer HVM VM attached
> disk limit bugfix, enough LGTMs and test results)
>
> https://github.com/apache/cloudstack/pull/1941 (same as above with
> additional fixes, lacks review lgtms)
> https://github.com/apache/cloudstack/pull/1896 (translation updates)
>
> https://github.com/apache/cloudstack/pull/1770 (template size issue
> bugfix)
>
> https://github.com/apache/cloudstack/pull/1950 (Ubuntu 16.04 packaging
> tomcat6/7 support)
>
> https://github.com/apache/cloudstack/pull/1951 (xenserver7 capability fix
> and upgrade path from 4.9.2->4.9.3)
>
> https://github.com/apache/cloudstack/pull/1944 (metrics view and infra
> tab/UI performance improvement)
> https://github.com/apache/cloudstack/pull/1903 (cannot add users to vpc
> vpn)
>
> https://github.com/apache/cloudstack/pull/1945
>
> <https://github.com/apache/cloudstack/pull/1945>https://
> github.com/apache/cloudstack/pull/1946
>
> <https://github.com/apache/cloudstack/pull/1946>https://
> github.com/apache/cloudstack/pull/1947
> https://github.com/apache/cloudstack/pull/1856
> https://github.com/apache/cloudstack/pull/1212
>
>
>
> Regards.
>
> ________________________________
> From: Rajani Karuturi <ra...@apache.org>
> Sent: 22 February 2017 14:41:42
> To: dev@cloudstack.apache.org
> Subject: Re: PRs for 4.10
>
> 6 days to go for the first RC.
> Please get the required lgtms and tests ready for the PRs you like to see
> in 4.10.
> I will try to merge all the PRs that meet the criteria.
>
> Thanks,
> ~Rajani
>
> On 19 Feb 2017 3:11 p.m., "Rajani Karuturi" <ra...@apache.org> wrote:
>
> > noted.
> >
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> >
> > On February 17, 2017 at 6:03 PM, Frank Maximus (
> > frank.maximus@nuagenetworks.net) wrote:
> >
> > I have a couple of bugfixes on previous version outstanding,
> > which I would like to have merged to 4.10,
> > both still requiring review:
> > on 4.8: PR#1912 <https://github.com/apache/cloudstack/pull/1912>: which
> > fixes password service running on internal lb vms, making it impossible
> to
> > do loadbalancing on port 8080
> > on 4.9: PR#1925 <https://github.com/apache/cloudstack/pull/1925>: Minor
> > plugin fix
> >
> > Kind Regards,
> > Frank
> >
> > On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed <sa...@cloudops.com> wrote:
> >
> > I'd like to include https://github.com/apache/cloudstack/pull/1928 to
> 4.10
> > as well. This is a simple fix that adds hypervisor capabilities for
> > XenServer 7
> >
> > Thanks,
> > -Syed
> >
> > On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <ws...@cloudops.com>
> > wrote:
> >
> > Not sure, I will see if I can find some time tomorrow to look at this.
> > Thanks...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org>
> > wrote:
> >
> > Thanks Will. I will take a look at this today and merge.
> >
> > Is it possible to add a systemvmtemplate job on
> > builds.apachecloudstack.net (https://builds.cloudstack.org/)
> > similar to
> > http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
> >
> > Thanks,
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> > On February 13, 2017 at 10:46 PM, Will Stevens
> > (wstevens@cloudops.com) wrote:
> >
> > At this point, I think 1741 is ready to merge. I think we need
> > one more
> > LGTM though...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> > <ws...@cloudops.com> wrote:
> >
> > @Daan: In order for the tests to pass with 1741, the tests have
> > to be run
> > with the SystemVM from the PR being used.
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> > <da...@gmail.com>
> > wrote:
> >
> > About 1741, I started the packaging, Will but saw Boris making a
> > remark about the system VMs. I'll look into that and start the
> > tests.
> >
> > On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> > <wi...@gmail.com>
> > wrote:
> >
> > I would like to get 1741(strongswan) in, but I need to get
> > Trillian to
> >
> > run
> >
> > against it with a system vm built from my pr. I have done a CI in
> > bubble
> > and it came back clean, but would like confirmation through
> > Trillian as
> > well.
> >
> > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
> >
> > And this PR for 4.9 but need to be foward to 4.10 too.
> >
> > https://github.com/apache/cloudstack/pull/1929
> >
> > 2 LGTM
> > CI green
> >
> > On 07/02/2017 13:07, Rajani Karuturi wrote:
> >
> > Hi all,
> > If you know of any PRs which meet the below criteria, please
> > mail
> >
> > them to
> >
> > me.
> > 1. No open questions.
> > 2. Has two LGTMs
> > 3. Has CI test results.
> >
> > ~Rajani
> > http://cloudplatform.accelerite.com
> >
> > --
> > Daan
> >
> >
>
> rohit.yadav@shapeblue.com
> www.shapeblue.com
> 53 Chandos Place, Covent Garden, London WC2N 4HSUK
> @shapeblue
>
>
>
>
Re: PRs for 4.10
Posted by Rohit Yadav <ro...@shapeblue.com>.
Rajani and all,
Please consider following PRs for review and merge:
https://github.com/apache/cloudstack/pull/1829 (XenServer HVM VM attached disk limit bugfix, enough LGTMs and test results)
https://github.com/apache/cloudstack/pull/1941 (same as above with additional fixes, lacks review lgtms)
https://github.com/apache/cloudstack/pull/1896 (translation updates)
https://github.com/apache/cloudstack/pull/1770 (template size issue bugfix)
https://github.com/apache/cloudstack/pull/1950 (Ubuntu 16.04 packaging tomcat6/7 support)
https://github.com/apache/cloudstack/pull/1951 (xenserver7 capability fix and upgrade path from 4.9.2->4.9.3)
https://github.com/apache/cloudstack/pull/1944 (metrics view and infra tab/UI performance improvement)
https://github.com/apache/cloudstack/pull/1903 (cannot add users to vpc vpn)
https://github.com/apache/cloudstack/pull/1945
<https://github.com/apache/cloudstack/pull/1945>https://github.com/apache/cloudstack/pull/1946
<https://github.com/apache/cloudstack/pull/1946>https://github.com/apache/cloudstack/pull/1947
https://github.com/apache/cloudstack/pull/1856
https://github.com/apache/cloudstack/pull/1212
Regards.
________________________________
From: Rajani Karuturi <ra...@apache.org>
Sent: 22 February 2017 14:41:42
To: dev@cloudstack.apache.org
Subject: Re: PRs for 4.10
6 days to go for the first RC.
Please get the required lgtms and tests ready for the PRs you like to see
in 4.10.
I will try to merge all the PRs that meet the criteria.
Thanks,
~Rajani
On 19 Feb 2017 3:11 p.m., "Rajani Karuturi" <ra...@apache.org> wrote:
> noted.
>
>
> ~ Rajani
>
> http://cloudplatform.accelerite.com/
>
>
> On February 17, 2017 at 6:03 PM, Frank Maximus (
> frank.maximus@nuagenetworks.net) wrote:
>
> I have a couple of bugfixes on previous version outstanding,
> which I would like to have merged to 4.10,
> both still requiring review:
> on 4.8: PR#1912 <https://github.com/apache/cloudstack/pull/1912>: which
> fixes password service running on internal lb vms, making it impossible to
> do loadbalancing on port 8080
> on 4.9: PR#1925 <https://github.com/apache/cloudstack/pull/1925>: Minor
> plugin fix
>
> Kind Regards,
> Frank
>
> On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed <sa...@cloudops.com> wrote:
>
> I'd like to include https://github.com/apache/cloudstack/pull/1928 to 4.10
> as well. This is a simple fix that adds hypervisor capabilities for
> XenServer 7
>
> Thanks,
> -Syed
>
> On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <ws...@cloudops.com>
> wrote:
>
> Not sure, I will see if I can find some time tomorrow to look at this.
> Thanks...
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org>
> wrote:
>
> Thanks Will. I will take a look at this today and merge.
>
> Is it possible to add a systemvmtemplate job on
> builds.apachecloudstack.net (https://builds.cloudstack.org/)
> similar to
> http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
>
> Thanks,
>
> ~ Rajani
>
> http://cloudplatform.accelerite.com/
>
> On February 13, 2017 at 10:46 PM, Will Stevens
> (wstevens@cloudops.com) wrote:
>
> At this point, I think 1741 is ready to merge. I think we need
> one more
> LGTM though...
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> <ws...@cloudops.com> wrote:
>
> @Daan: In order for the tests to pass with 1741, the tests have
> to be run
> with the SystemVM from the PR being used.
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> <da...@gmail.com>
> wrote:
>
> About 1741, I started the packaging, Will but saw Boris making a
> remark about the system VMs. I'll look into that and start the
> tests.
>
> On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> <wi...@gmail.com>
> wrote:
>
> I would like to get 1741(strongswan) in, but I need to get
> Trillian to
>
> run
>
> against it with a system vm built from my pr. I have done a CI in
> bubble
> and it came back clean, but would like confirmation through
> Trillian as
> well.
>
> On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
>
> And this PR for 4.9 but need to be foward to 4.10 too.
>
> https://github.com/apache/cloudstack/pull/1929
>
> 2 LGTM
> CI green
>
> On 07/02/2017 13:07, Rajani Karuturi wrote:
>
> Hi all,
> If you know of any PRs which meet the below criteria, please
> mail
>
> them to
>
> me.
> 1. No open questions.
> 2. Has two LGTMs
> 3. Has CI test results.
>
> ~Rajani
> http://cloudplatform.accelerite.com
>
> --
> Daan
>
>
rohit.yadav@shapeblue.com
www.shapeblue.com
53 Chandos Place, Covent Garden, London WC2N 4HSUK
@shapeblue
Re: PRs for 4.10
Posted by Rajani Karuturi <ra...@apache.org>.
6 days to go for the first RC.
Please get the required lgtms and tests ready for the PRs you like to see
in 4.10.
I will try to merge all the PRs that meet the criteria.
Thanks,
~Rajani
On 19 Feb 2017 3:11 p.m., "Rajani Karuturi" <ra...@apache.org> wrote:
> noted.
>
>
> ~ Rajani
>
> http://cloudplatform.accelerite.com/
>
>
> On February 17, 2017 at 6:03 PM, Frank Maximus (
> frank.maximus@nuagenetworks.net) wrote:
>
> I have a couple of bugfixes on previous version outstanding,
> which I would like to have merged to 4.10,
> both still requiring review:
> on 4.8: PR#1912 <https://github.com/apache/cloudstack/pull/1912>: which
> fixes password service running on internal lb vms, making it impossible to
> do loadbalancing on port 8080
> on 4.9: PR#1925 <https://github.com/apache/cloudstack/pull/1925>: Minor
> plugin fix
>
> Kind Regards,
> Frank
>
> On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed <sa...@cloudops.com> wrote:
>
> I'd like to include https://github.com/apache/cloudstack/pull/1928 to 4.10
> as well. This is a simple fix that adds hypervisor capabilities for
> XenServer 7
>
> Thanks,
> -Syed
>
> On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <ws...@cloudops.com>
> wrote:
>
> Not sure, I will see if I can find some time tomorrow to look at this.
> Thanks...
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org>
> wrote:
>
> Thanks Will. I will take a look at this today and merge.
>
> Is it possible to add a systemvmtemplate job on
> builds.apachecloudstack.net (https://builds.cloudstack.org/)
> similar to
> http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
>
> Thanks,
>
> ~ Rajani
>
> http://cloudplatform.accelerite.com/
>
> On February 13, 2017 at 10:46 PM, Will Stevens
> (wstevens@cloudops.com) wrote:
>
> At this point, I think 1741 is ready to merge. I think we need
> one more
> LGTM though...
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> <ws...@cloudops.com> wrote:
>
> @Daan: In order for the tests to pass with 1741, the tests have
> to be run
> with the SystemVM from the PR being used.
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> <da...@gmail.com>
> wrote:
>
> About 1741, I started the packaging, Will but saw Boris making a
> remark about the system VMs. I'll look into that and start the
> tests.
>
> On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> <wi...@gmail.com>
> wrote:
>
> I would like to get 1741(strongswan) in, but I need to get
> Trillian to
>
> run
>
> against it with a system vm built from my pr. I have done a CI in
> bubble
> and it came back clean, but would like confirmation through
> Trillian as
> well.
>
> On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
>
> And this PR for 4.9 but need to be foward to 4.10 too.
>
> https://github.com/apache/cloudstack/pull/1929
>
> 2 LGTM
> CI green
>
> On 07/02/2017 13:07, Rajani Karuturi wrote:
>
> Hi all,
> If you know of any PRs which meet the below criteria, please
> mail
>
> them to
>
> me.
> 1. No open questions.
> 2. Has two LGTMs
> 3. Has CI test results.
>
> ~Rajani
> http://cloudplatform.accelerite.com
>
> --
> Daan
>
>
Re: PRs for 4.10
Posted by Rajani Karuturi <ra...@apache.org>.
noted.
~ Rajani
http://cloudplatform.accelerite.com/
On February 17, 2017 at 6:03 PM, Frank Maximus
(frank.maximus@nuagenetworks.net) wrote:
I have a couple of bugfixes on previous version outstanding,
which I would like to have merged to 4.10,
both still requiring review:
on 4.8: PR#1912
<https://github.com/apache/cloudstack/pull/1912>: which
fixes password service running on internal lb vms, making it
impossible to
do loadbalancing on port 8080
on 4.9: PR#1925
<https://github.com/apache/cloudstack/pull/1925>: Minor
plugin fix
Kind Regards,
Frank
On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed
<sa...@cloudops.com> wrote:
I'd like to include
https://github.com/apache/cloudstack/pull/1928 to 4.10
as well. This is a simple fix that adds hypervisor capabilities
for
XenServer 7
Thanks,
-Syed
On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens
<ws...@cloudops.com>
wrote:
Not sure, I will see if I can find some time tomorrow to look at
this.
Thanks...
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi
<ra...@apache.org>
wrote:
Thanks Will. I will take a look at this today and merge.
Is it possible to add a systemvmtemplate job on
builds.apachecloudstack.net (https://builds.cloudstack.org/)
similar to
http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
Thanks,
~ Rajani
http://cloudplatform.accelerite.com/
On February 13, 2017 at 10:46 PM, Will Stevens
(wstevens@cloudops.com) wrote:
At this point, I think 1741 is ready to merge. I think we need
one more
LGTM though...
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
<ws...@cloudops.com> wrote:
@Daan: In order for the tests to pass with 1741, the tests have
to be run
with the SystemVM from the PR being used.
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
<da...@gmail.com>
wrote:
About 1741, I started the packaging, Will but saw Boris making a
remark about the system VMs. I'll look into that and start the
tests.
On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
<wi...@gmail.com>
wrote:
I would like to get 1741(strongswan) in, but I need to get
Trillian to
run
against it with a system vm built from my pr. I have done a CI
in
bubble
and it came back clean, but would like confirmation through
Trillian as
well.
On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
And this PR for 4.9 but need to be foward to 4.10 too.
https://github.com/apache/cloudstack/pull/1929
2 LGTM
CI green
On 07/02/2017 13:07, Rajani Karuturi wrote:
Hi all,
If you know of any PRs which meet the below criteria, please
mail
them to
me.
1. No open questions.
2. Has two LGTMs
3. Has CI test results.
~Rajani
http://cloudplatform.accelerite.com
--
Daan
Re: PRs for 4.10
Posted by Frank Maximus <fr...@nuagenetworks.net>.
I have a couple of bugfixes on previous version outstanding,
which I would like to have merged to 4.10,
both still requiring review:
on 4.8: PR#1912 <https://github.com/apache/cloudstack/pull/1912>: which
fixes password service running on internal lb vms, making it impossible to
do loadbalancing on port 8080
on 4.9: PR#1925 <https://github.com/apache/cloudstack/pull/1925>: Minor
plugin fix
Kind Regards,
Frank
On Tue, Feb 14, 2017 at 10:56 PM Syed Ahmed <sa...@cloudops.com> wrote:
I'd like to include https://github.com/apache/cloudstack/pull/1928 to 4.10
as well. This is a simple fix that adds hypervisor capabilities for
XenServer 7
Thanks,
-Syed
On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <ws...@cloudops.com>
wrote:
> Not sure, I will see if I can find some time tomorrow to look at this.
> Thanks...
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org>
> wrote:
>
> > Thanks Will. I will take a look at this today and merge.
> >
> > Is it possible to add a systemvmtemplate job on
> > builds.apachecloudstack.net (https://builds.cloudstack.org/)
> > similar to
> > http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
> >
> > Thanks,
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> > On February 13, 2017 at 10:46 PM, Will Stevens
> > (wstevens@cloudops.com) wrote:
> >
> > At this point, I think 1741 is ready to merge. I think we need
> > one more
> > LGTM though...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> > <ws...@cloudops.com> wrote:
> >
> > @Daan: In order for the tests to pass with 1741, the tests have
> > to be run
> > with the SystemVM from the PR being used.
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> > <da...@gmail.com>
> > wrote:
> >
> > About 1741, I started the packaging, Will but saw Boris making a
> > remark about the system VMs. I'll look into that and start the
> > tests.
> >
> > On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> > <wi...@gmail.com>
> > wrote:
> >
> > I would like to get 1741(strongswan) in, but I need to get
> > Trillian to
> >
> > run
> >
> > against it with a system vm built from my pr. I have done a CI in
> > bubble
> > and it came back clean, but would like confirmation through
> > Trillian as
> > well.
> >
> > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
> >
> > And this PR for 4.9 but need to be foward to 4.10 too.
> >
> > https://github.com/apache/cloudstack/pull/1929
> >
> > 2 LGTM
> > CI green
> >
> > On 07/02/2017 13:07, Rajani Karuturi wrote:
> >
> > Hi all,
> > If you know of any PRs which meet the below criteria, please
> > mail
> >
> > them to
> >
> > me.
> > 1. No open questions.
> > 2. Has two LGTMs
> > 3. Has CI test results.
> >
> > ~Rajani
> > http://cloudplatform.accelerite.com
> >
> > --
> > Daan
> >
>
Re: PRs for 4.10
Posted by Syed Ahmed <sa...@cloudops.com>.
I'd like to include https://github.com/apache/cloudstack/pull/1928 to 4.10
as well. This is a simple fix that adds hypervisor capabilities for
XenServer 7
Thanks,
-Syed
On Tue, Feb 14, 2017 at 12:06 AM, Will Stevens <ws...@cloudops.com>
wrote:
> Not sure, I will see if I can find some time tomorrow to look at this.
> Thanks...
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org>
> wrote:
>
> > Thanks Will. I will take a look at this today and merge.
> >
> > Is it possible to add a systemvmtemplate job on
> > builds.apachecloudstack.net (https://builds.cloudstack.org/)
> > similar to
> > http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
> >
> > Thanks,
> >
> > ~ Rajani
> >
> > http://cloudplatform.accelerite.com/
> >
> > On February 13, 2017 at 10:46 PM, Will Stevens
> > (wstevens@cloudops.com) wrote:
> >
> > At this point, I think 1741 is ready to merge. I think we need
> > one more
> > LGTM though...
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> > <ws...@cloudops.com> wrote:
> >
> > @Daan: In order for the tests to pass with 1741, the tests have
> > to be run
> > with the SystemVM from the PR being used.
> >
> > *Will STEVENS*
> > Lead Developer
> >
> > <https://goo.gl/NYZ8KK>
> >
> > On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> > <da...@gmail.com>
> > wrote:
> >
> > About 1741, I started the packaging, Will but saw Boris making a
> > remark about the system VMs. I'll look into that and start the
> > tests.
> >
> > On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> > <wi...@gmail.com>
> > wrote:
> >
> > I would like to get 1741(strongswan) in, but I need to get
> > Trillian to
> >
> > run
> >
> > against it with a system vm built from my pr. I have done a CI in
> > bubble
> > and it came back clean, but would like confirmation through
> > Trillian as
> > well.
> >
> > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
> >
> > And this PR for 4.9 but need to be foward to 4.10 too.
> >
> > https://github.com/apache/cloudstack/pull/1929
> >
> > 2 LGTM
> > CI green
> >
> > On 07/02/2017 13:07, Rajani Karuturi wrote:
> >
> > Hi all,
> > If you know of any PRs which meet the below criteria, please
> > mail
> >
> > them to
> >
> > me.
> > 1. No open questions.
> > 2. Has two LGTMs
> > 3. Has CI test results.
> >
> > ~Rajani
> > http://cloudplatform.accelerite.com
> >
> > --
> > Daan
> >
>
Re: PRs for 4.10
Posted by Will Stevens <ws...@cloudops.com>.
Not sure, I will see if I can find some time tomorrow to look at this.
Thanks...
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Mon, Feb 13, 2017 at 11:58 PM, Rajani Karuturi <ra...@apache.org> wrote:
> Thanks Will. I will take a look at this today and merge.
>
> Is it possible to add a systemvmtemplate job on
> builds.apachecloudstack.net (https://builds.cloudstack.org/)
> similar to
> http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
>
> Thanks,
>
> ~ Rajani
>
> http://cloudplatform.accelerite.com/
>
> On February 13, 2017 at 10:46 PM, Will Stevens
> (wstevens@cloudops.com) wrote:
>
> At this point, I think 1741 is ready to merge. I think we need
> one more
> LGTM though...
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
> <ws...@cloudops.com> wrote:
>
> @Daan: In order for the tests to pass with 1741, the tests have
> to be run
> with the SystemVM from the PR being used.
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
> <da...@gmail.com>
> wrote:
>
> About 1741, I started the packaging, Will but saw Boris making a
> remark about the system VMs. I'll look into that and start the
> tests.
>
> On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
> <wi...@gmail.com>
> wrote:
>
> I would like to get 1741(strongswan) in, but I need to get
> Trillian to
>
> run
>
> against it with a system vm built from my pr. I have done a CI in
> bubble
> and it came back clean, but would like confirmation through
> Trillian as
> well.
>
> On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
>
> And this PR for 4.9 but need to be foward to 4.10 too.
>
> https://github.com/apache/cloudstack/pull/1929
>
> 2 LGTM
> CI green
>
> On 07/02/2017 13:07, Rajani Karuturi wrote:
>
> Hi all,
> If you know of any PRs which meet the below criteria, please
> mail
>
> them to
>
> me.
> 1. No open questions.
> 2. Has two LGTMs
> 3. Has CI test results.
>
> ~Rajani
> http://cloudplatform.accelerite.com
>
> --
> Daan
>
Re: PRs for 4.10
Posted by Rajani Karuturi <ra...@apache.org>.
Thanks Will. I will take a look at this today and merge.
Is it possible to add a systemvmtemplate job on
builds.apachecloudstack.net (https://builds.cloudstack.org/)
similar to
http://jenkins.buildacloud.org/job/build-systemvm64-master/ ?
Thanks,
~ Rajani
http://cloudplatform.accelerite.com/
On February 13, 2017 at 10:46 PM, Will Stevens
(wstevens@cloudops.com) wrote:
At this point, I think 1741 is ready to merge. I think we need
one more
LGTM though...
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens
<ws...@cloudops.com> wrote:
@Daan: In order for the tests to pass with 1741, the tests have
to be run
with the SystemVM from the PR being used.
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland
<da...@gmail.com>
wrote:
About 1741, I started the packaging, Will but saw Boris making a
remark about the system VMs. I'll look into that and start the
tests.
On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens
<wi...@gmail.com>
wrote:
I would like to get 1741(strongswan) in, but I need to get
Trillian to
run
against it with a system vm built from my pr. I have done a CI in
bubble
and it came back clean, but would like confirmation through
Trillian as
well.
On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
And this PR for 4.9 but need to be foward to 4.10 too.
https://github.com/apache/cloudstack/pull/1929
2 LGTM
CI green
On 07/02/2017 13:07, Rajani Karuturi wrote:
Hi all,
If you know of any PRs which meet the below criteria, please
mail
them to
me.
1. No open questions.
2. Has two LGTMs
3. Has CI test results.
~Rajani
http://cloudplatform.accelerite.com
--
Daan
Re: PRs for 4.10
Posted by Will Stevens <ws...@cloudops.com>.
At this point, I think 1741 is ready to merge. I think we need one more
LGTM though...
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 8, 2017 at 11:32 AM, Will Stevens <ws...@cloudops.com> wrote:
> @Daan: In order for the tests to pass with 1741, the tests have to be run
> with the SystemVM from the PR being used.
>
> *Will STEVENS*
> Lead Developer
>
> <https://goo.gl/NYZ8KK>
>
> On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland <da...@gmail.com>
> wrote:
>
>> About 1741, I started the packaging, Will but saw Boris making a
>> remark about the system VMs. I'll look into that and start the tests.
>>
>> On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens <wi...@gmail.com>
>> wrote:
>> > I would like to get 1741(strongswan) in, but I need to get Trillian to
>> run
>> > against it with a system vm built from my pr. I have done a CI in bubble
>> > and it came back clean, but would like confirmation through Trillian as
>> > well.
>> >
>> > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
>> >
>> >>
>> >> And this PR for 4.9 but need to be foward to 4.10 too.
>> >>
>> >> https://github.com/apache/cloudstack/pull/1929
>> >>
>> >> 2 LGTM
>> >> CI green
>> >>
>> >>
>> >> On 07/02/2017 13:07, Rajani Karuturi wrote:
>> >>
>> >>> Hi all,
>> >>> If you know of any PRs which meet the below criteria, please mail
>> them to
>> >>> me.
>> >>> 1. No open questions.
>> >>> 2. Has two LGTMs
>> >>> 3. Has CI test results.
>> >>>
>> >>> ~Rajani
>> >>> http://cloudplatform.accelerite.com
>> >>>
>> >>>
>> >>
>>
>>
>>
>> --
>> Daan
>>
>
>
Re: PRs for 4.10
Posted by Will Stevens <ws...@cloudops.com>.
@Daan: In order for the tests to pass with 1741, the tests have to be run
with the SystemVM from the PR being used.
*Will STEVENS*
Lead Developer
<https://goo.gl/NYZ8KK>
On Wed, Feb 8, 2017 at 2:46 AM, Daan Hoogland <da...@gmail.com>
wrote:
> About 1741, I started the packaging, Will but saw Boris making a
> remark about the system VMs. I'll look into that and start the tests.
>
> On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens <wi...@gmail.com>
> wrote:
> > I would like to get 1741(strongswan) in, but I need to get Trillian to
> run
> > against it with a system vm built from my pr. I have done a CI in bubble
> > and it came back clean, but would like confirmation through Trillian as
> > well.
> >
> > On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
> >
> >>
> >> And this PR for 4.9 but need to be foward to 4.10 too.
> >>
> >> https://github.com/apache/cloudstack/pull/1929
> >>
> >> 2 LGTM
> >> CI green
> >>
> >>
> >> On 07/02/2017 13:07, Rajani Karuturi wrote:
> >>
> >>> Hi all,
> >>> If you know of any PRs which meet the below criteria, please mail them
> to
> >>> me.
> >>> 1. No open questions.
> >>> 2. Has two LGTMs
> >>> 3. Has CI test results.
> >>>
> >>> ~Rajani
> >>> http://cloudplatform.accelerite.com
> >>>
> >>>
> >>
>
>
>
> --
> Daan
>
Re: PRs for 4.10
Posted by Daan Hoogland <da...@gmail.com>.
About 1741, I started the packaging, Will but saw Boris making a
remark about the system VMs. I'll look into that and start the tests.
On Tue, Feb 7, 2017 at 8:19 PM, Will Stevens <wi...@gmail.com> wrote:
> I would like to get 1741(strongswan) in, but I need to get Trillian to run
> against it with a system vm built from my pr. I have done a CI in bubble
> and it came back clean, but would like confirmation through Trillian as
> well.
>
> On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
>
>>
>> And this PR for 4.9 but need to be foward to 4.10 too.
>>
>> https://github.com/apache/cloudstack/pull/1929
>>
>> 2 LGTM
>> CI green
>>
>>
>> On 07/02/2017 13:07, Rajani Karuturi wrote:
>>
>>> Hi all,
>>> If you know of any PRs which meet the below criteria, please mail them to
>>> me.
>>> 1. No open questions.
>>> 2. Has two LGTMs
>>> 3. Has CI test results.
>>>
>>> ~Rajani
>>> http://cloudplatform.accelerite.com
>>>
>>>
>>
--
Daan
Re: PRs for 4.10
Posted by Will Stevens <wi...@gmail.com>.
I would like to get 1741(strongswan) in, but I need to get Trillian to run
against it with a system vm built from my pr. I have done a CI in bubble
and it came back clean, but would like confirmation through Trillian as
well.
On Feb 7, 2017 2:17 PM, "Milamber" <mi...@apache.org> wrote:
>
> And this PR for 4.9 but need to be foward to 4.10 too.
>
> https://github.com/apache/cloudstack/pull/1929
>
> 2 LGTM
> CI green
>
>
> On 07/02/2017 13:07, Rajani Karuturi wrote:
>
>> Hi all,
>> If you know of any PRs which meet the below criteria, please mail them to
>> me.
>> 1. No open questions.
>> 2. Has two LGTMs
>> 3. Has CI test results.
>>
>> ~Rajani
>> http://cloudplatform.accelerite.com
>>
>>
>
Re: PRs for 4.10
Posted by Milamber <mi...@apache.org>.
And this PR for 4.9 but need to be foward to 4.10 too.
https://github.com/apache/cloudstack/pull/1929
2 LGTM
CI green
On 07/02/2017 13:07, Rajani Karuturi wrote:
> Hi all,
> If you know of any PRs which meet the below criteria, please mail them to
> me.
> 1. No open questions.
> 2. Has two LGTMs
> 3. Has CI test results.
>
> ~Rajani
> http://cloudplatform.accelerite.com
>