You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Murali Reddy <Mu...@citrix.com> on 2013/04/01 09:00:07 UTC

[IP clearance] merge IP cleared features into master

I revoked the code for the features 'optional public IP assignment for EIP' [1] and 'network services in shared network' [2] features from master due to IP clearance issues. Both the features have passed IP clearance window [3][4]. Due to multiple refactors that happened at 4.1 code freeze time, patches are invalid now. I did not get time to work on new patch and test them out. I would like work (its almost rework now) on these features now and merge them back into master.

Let me know if there are any concerns.

[1] https://issues.apache.org/jira/browse/CLOUDSTACK-265
[2] https://issues.apache.org/jira/browse/CLOUDSTACK-312
[3] http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.mbox/%3CCAKprHVZyidEgdEU3pTb_DcTZjc5XyUtSOF_P+GexKzHNT_+tEw@mail.gmail.com%3E
[4] http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.mbox/%3CCAKprHVYp0XOZLWUj7yOmboi12A02c+wOe0F9oMKE0K2uUXsSFg@mail.gmail.com%3E

Thanks,
Murali

Re: [IP clearance] merge IP cleared features into master

Posted by Murali Reddy <Mu...@citrix.com>.
On 01/04/13 7:03 PM, "Chip Childers" <ch...@sungard.com> wrote:

>On Mon, Apr 01, 2013 at 07:00:07AM +0000, Murali Reddy wrote:
>> I revoked the code for the features 'optional public IP assignment for
>>EIP' [1] and 'network services in shared network' [2] features from
>>master due to IP clearance issues. Both the features have passed IP
>>clearance window [3][4]. Due to multiple refactors that happened at 4.1
>>code freeze time, patches are invalid now. I did not get time to work on
>>new patch and test them out. I would like work (its almost rework now)
>>on these features now and merge them back into master.
>> 
>> Let me know if there are any concerns.
>
>+1
>
>I'd suggest creating a feature branch, and then requesting a merge once
>you have things sorted out (WRT the refactoring that forces changes to
>the previously approved patches).

Ok, will go through the regular process.

>
>> 
>> [1] https://issues.apache.org/jira/browse/CLOUDSTACK-265
>> [2] https://issues.apache.org/jira/browse/CLOUDSTACK-312
>> [3] 
>>http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.
>>mbox/%3CCAKprHVZyidEgdEU3pTb_DcTZjc5XyUtSOF_P+GexKzHNT_+tEw@mail.gmail.co
>>m%3E
>> [4] 
>>http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.
>>mbox/%3CCAKprHVYp0XOZLWUj7yOmboi12A02c+wOe0F9oMKE0K2uUXsSFg@mail.gmail.co
>>m%3E
>> 
>> Thanks,
>> Murali
>

Re: [IP clearance] merge IP cleared features into master

Posted by Chip Childers <ch...@sungard.com>.
On Mon, Apr 01, 2013 at 07:00:07AM +0000, Murali Reddy wrote:
> I revoked the code for the features 'optional public IP assignment for EIP' [1] and 'network services in shared network' [2] features from master due to IP clearance issues. Both the features have passed IP clearance window [3][4]. Due to multiple refactors that happened at 4.1 code freeze time, patches are invalid now. I did not get time to work on new patch and test them out. I would like work (its almost rework now) on these features now and merge them back into master.
> 
> Let me know if there are any concerns.

+1

I'd suggest creating a feature branch, and then requesting a merge once
you have things sorted out (WRT the refactoring that forces changes to
the previously approved patches).

> 
> [1] https://issues.apache.org/jira/browse/CLOUDSTACK-265
> [2] https://issues.apache.org/jira/browse/CLOUDSTACK-312
> [3] http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.mbox/%3CCAKprHVZyidEgdEU3pTb_DcTZjc5XyUtSOF_P+GexKzHNT_+tEw@mail.gmail.com%3E
> [4] http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.mbox/%3CCAKprHVYp0XOZLWUj7yOmboi12A02c+wOe0F9oMKE0K2uUXsSFg@mail.gmail.com%3E
> 
> Thanks,
> Murali

Re: [IP clearance] merge IP cleared features into master

Posted by Chiradeep Vittal <Ch...@citrix.com>.
+1

On 4/1/13 12:00 AM, "Murali Reddy" <Mu...@citrix.com> wrote:

>I revoked the code for the features 'optional public IP assignment for
>EIP' [1] and 'network services in shared network' [2] features from
>master due to IP clearance issues. Both the features have passed IP
>clearance window [3][4]. Due to multiple refactors that happened at 4.1
>code freeze time, patches are invalid now. I did not get time to work on
>new patch and test them out. I would like work (its almost rework now) on
>these features now and merge them back into master.
>
>Let me know if there are any concerns.
>
>[1] https://issues.apache.org/jira/browse/CLOUDSTACK-265
>[2] https://issues.apache.org/jira/browse/CLOUDSTACK-312
>[3] 
>http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.m
>box/%3CCAKprHVZyidEgdEU3pTb_DcTZjc5XyUtSOF_P+GexKzHNT_+tEw@mail.gmail.com%
>3E
>[4] 
>http://mail-archives.apache.org/mod_mbox/incubator-cloudstack-dev/201301.m
>box/%3CCAKprHVYp0XOZLWUj7yOmboi12A02c+wOe0F9oMKE0K2uUXsSFg@mail.gmail.com%
>3E
>
>Thanks,
>Murali