Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
Well, maybe, we are not even far of from it. call 'master' 'develop' and create a new 'master'. next make sure every body develops in a feature branch. also call x.y-forward hotfix-x.y. Don't bet your life this approach is going to save our world. What would really help is if everybody would really

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Mike Tutkowski
I see, Daan - thanks for the clarification. This is probably another good reason why we should seriously consider implementing the branching approach Sebastien recommended here: http://nvie.com/posts/a-successful-git-branching-model/ On Thu, Jul 17, 2014 at 10:06 PM, Daan Hoogland wrote: > Th

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
That is not the confusion Mike. The problem is that some changes that don't go into 4.4 keep causing conflicts. I made the mistake of adding the conflicting lines this time. this 4.4-forward branch is not suitable for providing cherry-picks for an RM because of this. I thin people should just branc

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Mike Tutkowski
Perhaps there is some confusion again as to the nature of the 4.4-forward branch. A while back, we agreed that changes put in here would not be cherry picked to 4.4 unless requested so by the developer and agreed to by the RM. Changes in 4.4-forward that do not go into 4.4 will at least go into 4

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
They keep coming in with cherry-picks that include this file. I will remove them. On Fri, Jul 18, 2014 at 12:22 AM, Nitin Mehta wrote: > Hi Daan - I am not sure I get your point here. These changes were put in > as I want them in 4.4.1, but were not critical enough to be put in 4.4. > > Thanks, >

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Nitin Mehta
Hi Daan - I am not sure I get your point here. These changes were put in as I want them in 4.4.1, but were not critical enough to be put in 4.4. Thanks, -Nitin On 17/07/14 2:58 PM, "Daan Hoogland" wrote: >sure? I saw that the last few lines where not in the last version. > >I'm not confortable

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
sure? I saw that the last few lines where not in the last version. I'm not confortable with this bit, it has been coming up a few time before already looks like some commit on 4.4-forward is trying to sneak it's way into the release: @@ -2439,4 +2474,16 @@ CONSTRAINT `fk_lb_healthcheck_policy_

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Amogh Vasekar
Seems good, looks like was an issue with a newline somewhere. But deploydb went fine on 4.4 Thanks, Amogh On 7/17/14 2:42 PM, "Daan Hoogland" wrote: >Amogh, I couldn't help myself. please have a look at the resulting >setup/db/db/schema-430to440.sql > >On Thu, Jul 17, 2014 at 11:35 PM, Daan Hoo

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
Amogh, I couldn't help myself. please have a look at the resulting setup/db/db/schema-430to440.sql On Thu, Jul 17, 2014 at 11:35 PM, Daan Hoogland wrote: > On Thu, Jul 17, 2014 at 11:19 PM, Amogh Vasekar > wrote: >> c8ca15b95a57a3d79b71c76c913e295f6490f05d > > Amogh, it has conflicts. I will hav

Re: [ACS 4.4] Cherry pick request

2014-07-17 Thread Daan Hoogland
On Thu, Jul 17, 2014 at 11:19 PM, Amogh Vasekar wrote: > c8ca15b95a57a3d79b71c76c913e295f6490f05d Amogh, it has conflicts. I will have a look at those in the morning -- Daan

[ACS 4.4] Cherry pick request

2014-07-17 Thread Amogh Vasekar
Hi Daan, Can you please cherry-pick c8ca15b95a57a3d79b71c76c913e295f6490f05d This is for CLOUDSTACK-6358. Thanks, Amogh

Re: [ACS 4.4] cherry-pick CLOUDSTACK-7047

2014-07-03 Thread Daan Hoogland
On Thu, Jul 3, 2014 at 3:27 AM, Prachi Damle wrote: ... > Commit# 40e28de7275804f4483868ba8a737878ff54fba8 You are intrroducing windows line endings! picking anyway. > Commit# f37b1524892ab805615dbff2c59e5a1d10c116d3 regards, -- Daan

[ACS 4.4] cherry-pick CLOUDSTACK-7047

2014-07-02 Thread Prachi Damle
Hi Daan, Please can you cherry-pick the following 2 commits to 4.4 CLOUDSTACK-7047 Commit# 40e28de7275804f4483868ba8a737878ff54fba8 Commit# f37b1524892ab805615dbff2c59e5a1d10c116d3 Thanks, Prachi

Re: [ACS 4.4] cherry-pick

2014-07-01 Thread Daan Hoogland
2 in On Tue, Jul 1, 2014 at 11:56 AM, Jayapal Reddy Uradi wrote: > Hi Daan, > > Please cherry-pick the following commits to 4.4 > > commit e336796cd5eae39bf7755d981426160a15bc6e9f > CLOUDSTACK-7028: Fixed adding route for additional public nic on fail over > > commit a69c1ee31d233d4a6f77d68fd

[ACS 4.4] cherry-pick

2014-07-01 Thread Jayapal Reddy Uradi
Hi Daan, Please cherry-pick the following commits to 4.4 commit e336796cd5eae39bf7755d981426160a15bc6e9f CLOUDSTACK-7028: Fixed adding route for additional public nic on fail over commit a69c1ee31d233d4a6f77d68fd70b9cfe3c85f89b CLOUDSTACK-7027: Removing left over static nat rule Thanks,

Re: [ACS-4.4] cherry-pick

2014-06-30 Thread Daan Hoogland
On Mon, Jun 30, 2014 at 1:21 PM, Jayapal Reddy Uradi wrote: > 004d616322ba55d83407545f5b10c95ce48f75b5 I missed this one but it is in now. -- Daan

[ACS-4.4] cherry-pick

2014-06-30 Thread Jayapal Reddy Uradi
Hi Daan, Can you please cherry pick the below commit to 4.4 commit 004d616322ba55d83407545f5b10c95ce48f75b5 CLOUDSTACK-6997: Increase proc ip_conntrack_max Thanks, Jayapal

Re: [ACS-4.4] cherry-pick

2014-06-24 Thread Daan Hoogland
On Tue, Jun 24, 2014 at 7:38 AM, Jayapal Reddy Uradi wrote: > a195e50ae7da4b4392844e2984823c1331ca1396 is in -- Daan

[ACS-4.4] cherry-pick

2014-06-23 Thread Jayapal Reddy Uradi
Hi Daan, Please cherry pick the following commit to 4.4 a195e50ae7da4b4392844e2984823c1331ca1396 Thanks, Jayapal

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-6935

2014-06-19 Thread Daan Hoogland
On Wed, Jun 18, 2014 at 10:35 PM, Yoshikazu Nojima wrote: > 45f0c7367680f4bfbcee470139b708d69322be78 is in -- Daan

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-6935

2014-06-18 Thread Yoshikazu Nojima
Daan, Could you cherry-pick 45f0c7367680f4bfbcee470139b708d69322be78 instead of cd414a0f56798ae801fc464be127e37daabef809? Regards, Noji 2014-06-18 11:59 GMT-06:00 Yoshikazu Nojima : > Daan, > > Could you cherry-pick the commit > "cd414a0f56798ae801fc464be127e37daabef809" into 4.4? > This is a fi

[ACS 4.4] cherry-pick request for CLOUDSTACK-6935

2014-06-18 Thread Yoshikazu Nojima
Daan, Could you cherry-pick the commit "cd414a0f56798ae801fc464be127e37daabef809" into 4.4? This is a fix to the bug CLOUDSTACK-6935. Bug description: Some storage pool filtering logic like hypervisor type check, storage type check for root volume and avoid list check is not enabled in ZoneWideSt

Re: [ACS 4.4] cherry-pick

2014-06-17 Thread Jayapal Reddy Uradi
Hi Daan, The conflicts resolved looks fine. You can ignore my patch. Thanks, Jayapal On 17-Jun-2014, at 5:50 PM, Daan Hoogland wrote: > I don't think I was clear. I don't need a new patch. I did resolve the > conflict. I need you to check if I did the right thing in branch 4.4 > in EventTypes

Re: [ACS 4.4] cherry-pick

2014-06-17 Thread Daan Hoogland
I don't think I was clear. I don't need a new patch. I did resolve the conflict. I need you to check if I did the right thing in branch 4.4 in EventTypes near line 550 your new patch doesn't apply to 4.4 or 4.4-forward > git am -s > ~/Downloads/0001-CLOUDSTACK-6922-Updated-events-for-firewall-in

Re: [ACS 4.4] cherry-pick

2014-06-17 Thread Jayapal Reddy Uradi
Hi Daan, Resolved the conflicts and uploaded patch in review board. https://reviews.apache.org/r/22679/ Thanks, Jayapal On 17-Jun-2014, at 4:52 PM, Daan Hoogland wrote: > On Tue, Jun 17, 2014 at 8:41 AM, Jayapal Reddy Uradi > wrote: >> 2214bd2502a34ee8e671eb6b4efe8cd26e29ac5a > > > I had c

Re: [ACS 4.4] cherry-pick

2014-06-17 Thread Daan Hoogland
On Tue, Jun 17, 2014 at 8:41 AM, Jayapal Reddy Uradi wrote: > 2214bd2502a34ee8e671eb6b4efe8cd26e29ac5a I had conflicts. Can you have a look near line 550? I resolved by changing IpAddress to PublicIpAddress. regards -- Daan

[ACS 4.4] cherry-pick

2014-06-17 Thread Jayapal Reddy Uradi
Hi Daan, Please cherry pick the following commit into 4.4 commit b0d726a872e2859a56ee677c15079cc3a59ab894 CLOUDSTACK-6923: updated listLBStickinessPolicies API to list using stickiness policy id Thanks, Jayapal

[ACS 4.4] cherry-pick

2014-06-16 Thread Jayapal Reddy Uradi
Hi Daan, Please cherry pick the following commit to 4.4 commit 2214bd2502a34ee8e671eb6b4efe8cd26e29ac5a CLOUDSTACK-6922: Updated events for firewall ingress and egress Thanks, Jayapal

Re: [ACS-4.4] Cherry-pick 8a9092c3cd4e3c034f9f4d0a7491875dc080e9dc

2014-06-12 Thread Nitin Mehta
Looks good Daan. Thanks. On 12/06/14 3:18 AM, "Daan Hoogland" wrote: >On Thu, Jun 12, 2014 at 12:49 AM, Nitin Mehta >wrote: >> 8a9092c3cd4e3c034f9f4d0a7491875dc080e9dc > > >is in after conflict reolution, please check. > >-- >Daan

Re: [ACS-4.4] Cherry-pick 8a9092c3cd4e3c034f9f4d0a7491875dc080e9dc

2014-06-12 Thread Daan Hoogland
On Thu, Jun 12, 2014 at 12:49 AM, Nitin Mehta wrote: > 8a9092c3cd4e3c034f9f4d0a7491875dc080e9dc is in after conflict reolution, please check. -- Daan

[ACS-4.4] Cherry-pick 8a9092c3cd4e3c034f9f4d0a7491875dc080e9dc

2014-06-11 Thread Nitin Mehta
Daan – Can you please cherry-pick 8a9092c3cd4e3c034f9f4d0a7491875dc080e9dc into 4.4 ? This is for CLOUDSTACK-6895 Thanks, -Nitin

Re: [ACS-4.4] Cherry-pick request

2014-06-11 Thread Daan Hoogland
On Wed, Jun 11, 2014 at 2:39 PM, Harikrishna Patnala wrote: > aa75b4388554a502b1073dd78050cd4b364a803e is in -- Daan

[ACS-4.4] Cherry-pick request

2014-06-11 Thread Harikrishna Patnala
Hi Daan, Can you please cherry-pick the following commit to 4.4 branch Commit aa75b4388554a502b1073dd78050cd4b364a803e in cloudstack's branch refs/heads/4.4-forward from Harikrishna Patnala [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=aa75b43 ] CLOUDSTACK-6869: SSH Public key co

Re: [ACS 4.4] Cherry-pick for CLOUDSTACK-6802 and CLOUDSTACK-6810

2014-05-30 Thread Daan Hoogland
On Fri, May 30, 2014 at 12:01 PM, Devdeep Singh wrote: > f5ce280da32c4e0837fa39651dc801ba3f906e41 is in -- Daan

[ACS 4.4] Cherry-pick for CLOUDSTACK-6802 and CLOUDSTACK-6810

2014-05-30 Thread Devdeep Singh
Hi Daan, Could you please cherry-pick commit f5ce280da32c4e0837fa39651dc801ba3f906e41 from 4.4-forward branch to 4.4 branch. commit f5ce280da32c4e0837fa39651dc801ba3f906e41 Author: Devdeep Singh Date: Tue May 20 11:40:51 2014 +0530 CLOUDSTACK-6810: Fix storage migration of a vm with volu

Re: [ACS 4.4] Cherry-pick for CLOUDSTACK-6783

2014-05-27 Thread Daan Hoogland
On Tue, May 27, 2014 at 4:23 PM, Wido den Hollander wrote: > 7415a27184c84bce23b1739a7c4d609e2c2e37db done -- Daan

[ACS 4.4] Cherry-pick for CLOUDSTACK-6783

2014-05-27 Thread Wido den Hollander
Hi Daan, Could you cherry-pick 7415a27184c84bce23b1739a7c4d609e2c2e37db from 4.4-forward to the 4.4 branch? This resolves CLOUDSTACK-6783 Thank you! Wido

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-5077

2014-05-05 Thread Daan Hoogland
thanks Bharat, It is in. I run git show on every commit that I then git cherry-pick so you don't need to send me a link. the full id will do. regards, Daan On Mon, May 5, 2014 at 12:27 PM, Bharat Kumar wrote: > Hi Daan, > > updating the link to diff. > https://git-wip-us.apache.org/repos/asf?p=

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-6463 and CLOUDSTACK-6466

2014-05-05 Thread Daan Hoogland
both in On Mon, May 5, 2014 at 11:56 AM, Harikrishna Patnala wrote: > Hi Daan, > > Could you cherry pick the following commits to 4.4 branch > > 1) Commit ce3074e9a23e71855798cb5decd01d7f9958f6c0 in cloudstack's branch > refs/heads/4.4-forward > [ https://git-wip-us.apache.org/repos/asf?p=clouds

Re: [ACS 4.4] cherry-pick request for CLOUDSTACK-5077

2014-05-05 Thread Bharat Kumar
Hi Daan, updating the link to diff. https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=99b4cf788eded0f6f155ff8cda51aeb2e505f764 Thanks Bharat. On 05-May-2014, at 3:36 pm, Bharat Kumar mailto:bharat.ku...@citrix.com>> wrote: Hi Daan, please cherry pick the following commit to

[ACS 4.4] cherry-pick request for CLOUDSTACK-5077

2014-05-05 Thread Bharat Kumar
Hi Daan, please cherry pick the following commit to 4.4 branch. commit 489bb0c7edf6ab011895d5f082328eb5fe48aac1 in cloudstack 4.4-forward branch. link to diff Cloudstack-5077: reserve cpu and memory only when vmware.reserve.cpu/mem are …

[ACS 4.4] cherry-pick request for CLOUDSTACK-6463 and CLOUDSTACK-6466

2014-05-05 Thread Harikrishna Patnala
Hi Daan, Could you cherry pick the following commits to 4.4 branch 1) Commit ce3074e9a23e71855798cb5decd01d7f9958f6c0 in cloudstack's branch refs/heads/4.4-forward [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=ce3074e ] CLOUDSTACK-6463: password is not set for VMs created from pas

Re: [ACS 4.4] Cherry-pick 3e7ea4e

2014-04-29 Thread Daan Hoogland
On Tue, Apr 29, 2014 at 8:07 AM, Nitin Mehta wrote: > 3e7ea4e H Nithin, It is in. I prefer full commit ids if you have them. thanks -- Daan

[ACS 4.4] Cherry-pick 3e7ea4e

2014-04-28 Thread Nitin Mehta
Hi Daan - Please cherry-pick the commit below. commit 3e7ea4e CLOUDSTACK-6530: Populate the first class entities in the context to be available for publishing more information for the event bus, checking the displayable property etc. Thanks, -Nitin