[GitHub] cloudstack issue #1888: CLOUDSTACK-9710: Switch to JRE1.8

2017-02-07 Thread wido
Github user wido commented on the issue: https://github.com/apache/cloudstack/pull/1888 @swill: Yes we should! I think that was an oversight. Can you send a PR for this against master? --- If your project is set up for it, you can reply to this email and have your reply appear on

Re: FW: Google Summer of Code 2017 is coming

2017-02-07 Thread Daan Hoogland
Great Sachin, Do you have a subject in mind? I want to mentor but we may just find you an even better mentor based on your preferred subject ;) On Mon, Feb 6, 2017 at 3:00 PM, Ian Duffy wrote: > I believe Sachin (CCed) was showed interest in becoming a student a couple > of

Re: FW: Google Summer of Code 2017 is coming

2017-02-07 Thread sachin patil
Hello Daan, I had two particular projects in mind and there are pleny others as well. -- One is the ceph project @wido had posted here. -- The other is Ability to move VM to other project CLOUDSTACK-9737 ,this issue it still

Re: FW: Google Summer of Code 2017 is coming

2017-02-07 Thread Daan Hoogland
I have only just started to shift through the list of old wishes, Sachin. I am sure there are a lot there that , when not valid as such will provide plenty inspiration to think of new ones, Feel free to address Wido and Wei, though they are probably very busy. If you have an idea and want to

PRs for 4.10

2017-02-07 Thread Rajani Karuturi
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

[GitHub] cloudstack-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack-www/pull/34 Any further thoughts here guys? @terbolous, @swill, @pdion891 --- 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-www pull request #34: Remove link to “planet.apache.org” and ...

2017-02-07 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/34#discussion_r99876892 --- Diff: README.md --- @@ -15,14 +15,42 @@ Build the site with: To serve it locally: $ middleman server - -IMPORTANT: To

[GitHub] cloudstack-www pull request #34: Remove link to “planet.apache.org” and ...

2017-02-07 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/34#discussion_r99876990 --- Diff: README.md --- @@ -15,14 +15,42 @@ Build the site with: To serve it locally: $ middleman server - -IMPORTANT: To

[GitHub] cloudstack-www pull request #34: Remove link to “planet.apache.org” and ...

2017-02-07 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/34#discussion_r99877060 --- Diff: README.md --- @@ -15,14 +15,42 @@ Build the site with: To serve it locally: $ middleman server - -IMPORTANT: To

[GitHub] cloudstack-www pull request #34: Remove link to “planet.apache.org” and ...

2017-02-07 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/34#discussion_r99876616 --- Diff: README.md --- @@ -15,14 +15,42 @@ Build the site with: To serve it locally: $ middleman server - -IMPORTANT: To

[GitHub] cloudstack-www pull request #34: Remove link to “planet.apache.org” and ...

2017-02-07 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/34#discussion_r99875679 --- Diff: README.md --- @@ -15,14 +15,42 @@ Build the site with: To serve it locally: $ middleman server - -IMPORTANT: To

[GitHub] cloudstack-www pull request #34: Remove link to “planet.apache.org” and ...

2017-02-07 Thread swill
Github user swill commented on a diff in the pull request: https://github.com/apache/cloudstack-www/pull/34#discussion_r99876345 --- Diff: README.md --- @@ -15,14 +15,42 @@ Build the site with: To serve it locally: $ middleman server - -IMPORTANT: To

[GitHub] cloudstack-www issue #34: Remove link to “planet.apache.org” and update ...

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack-www/pull/34 Changes suggested by @swill applied. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1727 @koushik-das @rhtyd @rafaelweingartner Can you review changes @nvazquez put in place? CI tests are good. --- If your project is set up for it, you can reply to this email and have your reply

Re: PRs for 4.10

2017-02-07 Thread Sergey Levitskiy
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

[GitHub] cloudstack issue #1767: CLOUDSTACK-9457: Allow retrieval and modification of...

2017-02-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1767 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 enabled and wishes so, or if the

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1747 @nvazquez, @serg38, before checking code any deeper a have a more high-level question. This PR will change the response of an API method, right? It will remove some data that

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1727 Long time I do not review a PR here. I have to say, @nvazquez , great job!! @serg38 you read my mind!! I had just started reading @nvazquez PR description when you called me.

[GitHub] cloudstack issue #1767: CLOUDSTACK-9457: Allow retrieval and modification of...

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1767 I have just reviewed the PR. There are only small suggestions I made. BTW: from the Jira ticket https://issues.apache.org/jira/browse/CLOUDSTACK-9457, I ended up at

[GitHub] cloudstack issue #1727: CLOUDSTACK-9539: Support changing Service offering f...

2017-02-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1727 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 enabled and wishes so, or if the

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner This PR is two fold. 1. This is something that was done a long ago for hosts and was never implemented for storage. It transitions location of of storage tags over to

Re: PRs for 4.10

2017-02-07 Thread Will Stevens
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" wrote: > @milamber I will soon merge the localization PRs > >

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread rafaelweingartner
Github user rafaelweingartner commented on the issue: https://github.com/apache/cloudstack/pull/1747 @serg38 got it. Thanks for the explanation --- 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

Re: PRs for 4.10

2017-02-07 Thread Rajani Karuturi
@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

[GitHub] cloudstack issue #1747: CLOUDSTACK-9574: Redesign storage views

2017-02-07 Thread serg38
Github user serg38 commented on the issue: https://github.com/apache/cloudstack/pull/1747 @rafaelweingartner Can you review this 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 your project does not have this

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-07 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1741 Packaging result: ✔centos6 ✔centos7 ✔debian. JID-474 --- 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

Re: PRs for 4.10

2017-02-07 Thread Daan Hoogland
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 wrote: > I would like to get 1741(strongswan) in, but I need to get Trillian to run >

Re:PRs for 4.10

2017-02-07 Thread Haijiao
Expect the following PRs to be merged into 4.10, thanks ! PR #1929CLOUDSTACK-9770: fix missing ip routes in VR, 2 LGTM ready PR #1915CLOUDSTACK-9746 system-vm: logrotate config causes critical failures

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-07 Thread DaanHoogland
Github user DaanHoogland commented on the issue: https://github.com/apache/cloudstack/pull/1741 @blueorangutan package --- 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

[GitHub] cloudstack issue #1741: Updated StrongSwan VPN Implementation

2017-02-07 Thread blueorangutan
Github user blueorangutan commented on the issue: https://github.com/apache/cloudstack/pull/1741 @DaanHoogland a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress. --- If your project is set up for it, you can reply to this email and have your

[GitHub] cloudstack pull request #1900: CLOUDSTACK-8862: Introduced new state attachi...

2017-02-07 Thread anshul1886
GitHub user anshul1886 reopened a pull request: https://github.com/apache/cloudstack/pull/1900 CLOUDSTACK-8862: Introduced new state attaching for volume. This will… … make sure that other attach operation on same volume will fail gracefully without calling access calls for

[GitHub] cloudstack pull request #1900: CLOUDSTACK-8862: Introduced new state attachi...

2017-02-07 Thread anshul1886
Github user anshul1886 closed the pull request at: https://github.com/apache/cloudstack/pull/1900 --- 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

Re: PRs for 4.10

2017-02-07 Thread Wido den Hollander
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 het > volgende geschreven: > > Hi all, > If you know of any PRs which meet the below criteria, please mail them

[GitHub] cloudstack pull request #1767: CLOUDSTACK-9457: Allow retrieval and modifica...

2017-02-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1767 --- 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

[GitHub] cloudstack pull request #1905: CLOUDSTACK-9738: [Vmware] Optimize vm expunge...

2017-02-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1905 --- 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

[GitHub] cloudstack issue #1905: CLOUDSTACK-9738: [Vmware] Optimize vm expunge proces...

2017-02-07 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1905 Thanks for the clarification. Its a nice enhancement. LGTM for the changes. Marvin tests are always welcome :) --- If your project is set up for it, you can reply to this email and have

[GitHub] cloudstack issue #1644: Honors the snapshot.backup.rightafter configuration ...

2017-02-07 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1644 A separate PR for master is not required. #1697 will be forward merged. --- 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 issue #1697: CLOUDSTACK-4858 Honors the snapshot.backup.rightafte...

2017-02-07 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1697 merging this to 4.9+ branches --- 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

[GitHub] cloudstack-docs-admin pull request #41: 50% of the doc strings are translate...

2017-02-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack-docs-admin/pull/41 --- 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

[GitHub] cloudstack issue #1892: CLOUDSTACK-9731: Hardcoded label appears on the Add ...

2017-02-07 Thread sureshanaparti
Github user sureshanaparti commented on the issue: https://github.com/apache/cloudstack/pull/1892 @koushik-das, UI screenshots of hardcoded label on the Add zone wizard and the UI fix for it are attached. Please check.

[GitHub] cloudstack issue #1892: CLOUDSTACK-9731: Hardcoded label appears on the Add ...

2017-02-07 Thread karuturi
Github user karuturi commented on the issue: https://github.com/apache/cloudstack/pull/1892 trivial. merged --- 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

[GitHub] cloudstack pull request #1892: CLOUDSTACK-9731: Hardcoded label appears on t...

2017-02-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/cloudstack/pull/1892 --- 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

Re: PRs for 4.10

2017-02-07 Thread Milamber
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.

Re: PRs for 4.10

2017-02-07 Thread Will Stevens
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" wrote: > >

Re: PRs for 4.10

2017-02-07 Thread Milamber
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