<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 19, 2014 at 11:41 AM, Joe Gordon <span dir="ltr"><<a href="mailto:joe.gordon0@gmail.com" target="_blank">joe.gordon0@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Wed, Feb 19, 2014 at 7:06 AM, Doug Hellmann<br>

<<a href="mailto:doug.hellmann@dreamhost.com">doug.hellmann@dreamhost.com</a>> wrote:<br>
><br>
><br>
><br>
> On Tue, Feb 18, 2014 at 9:03 PM, Joe Gordon <<a href="mailto:joe.gordon0@gmail.com">joe.gordon0@gmail.com</a>> wrote:<br>
>><br>
>> On Wed, Feb 12, 2014 at 12:16 PM, Doug Hellmann<br>
>> <<a href="mailto:doug.hellmann@dreamhost.com">doug.hellmann@dreamhost.com</a>> wrote:<br>
>> > If you have a change in your project that is blocked waiting for a patch<br>
>> > to<br>
>> > land in oslo (in the incubator, or any of the libraries we manage)<br>
>> > *please*<br>
>> > either open a blueprint or mark the associated bug as also affecting the<br>
>> > relevant oslo project, then let me know about it so I can put it on our<br>
>> > review priority list. We have a lot going on in oslo right now, but will<br>
>> > do<br>
>> > our best to prioritize reviews that affect features landing in other<br>
>> > projects -- if you let us know about them.<br>
>><br>
>><br>
>> While I don't think this is what you meant when you said let oslo help<br>
>> you, I do have a request:<br>
>><br>
>> While trying to do a basic oslo-incubator update ('./update.sh<br>
>> --nodeps --modules fixture --base nova --dest-dir ../nova')  I hit a<br>
>> bug <a href="https://bugs.launchpad.net/oslo/+bug/1281860" target="_blank">https://bugs.launchpad.net/oslo/+bug/1281860</a><br>
>><br>
>> Due to the nature of oslo-incubator (it may break at any time) it is<br>
>> hard for downstream projects (nova, cinder etc.) to keep there<br>
>> oslo-incubator copies up to date, so when someone wants to sync across<br>
>> a new change they have to deal with many unrelated changes, some of<br>
>> which may break things. For example<br>
>><br>
>> oslo-incubator$ ./update.sh --config-file<br>
>> ../cinder/openstack-common.conf --base cinder --dest-dir ../cinder<br>
>> cinder$ git diff --stat HEAD<br>
>> 52 files changed, 3568 insertions(+), 961 deletions(-)<br>
>><br>
>><br>
>> I would like to propose making the oslo team responsible for syncing<br>
>> across oslo-incubator code, they know the code base best and can fix<br>
>> things when they break.  This doesn't mean no one else can use<br>
>> update.sh it just means that the oslo team would make sure that syncs<br>
>> are done in a timely fashion, so the diffs don't get too big.<br>
><br>
><br>
> The intent has always been for the person making a change in the incubator<br>
> to be responsible for updating consuming projects (whether that person is an<br>
> oslo core reviewer or not). That hasn't always happened -- some changes are<br>
> only synced into the project where the change was wanted, some changes<br>
> aren't synced at all.<br>
<br>
</div></div>I had no idea that is how it supposed to work today. That means for<br>
every patch I make to oslo-incubator I should be making 10+ other<br>
patches as well, that is a lot of overhead.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">It is. It's a bad idea to have that many projects copying code from the incubator, and it was never the plan that the incubator would be used that way. Making syncing easier isn't going to solve the problem. It only perpetuates the situation where syncing is needed at all.</div>
<div class="gmail_default" style="font-size:small"><br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="">
> Although it doesn't look like it from the outside, we have made significant<br>
> progress in creating the tools and processes we need to move code out of the<br>
> incubator into libraries. I expect to accelerate those moves late in this<br>
> cycle and early in the next so the libraries can be adopted early in the<br>
> next cycle.<br>
<br>
</div>Does this mean, there are plans to stop doing code syncs via<br>
oslo-incubator all together?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">No, although I hope to restore it to its original purpose. As I said on IRC, oslo is meant to be a place to collaborate to evolve an API and create libraries. Somehow the perception has become that the incubator is a place to dump code for someone else to maintain.</div>
</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
I am really happy to see the accelerated move to libraries, but that<br>
is not an immediate solution, we still have to deal with icehouse. It<br>
sounds like we both agree the status quo is broken and the long term<br>
solution is move code out of incubator. We need a short term solution<br>
as well, having cinder 4k lines off of oslo or nova being almost 10k<br>
lines off is unacceptable.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">So far, every discussion on the subject eventually comes back to how big the required patches are, though, and no one has come up with any proposals for addressing that. I don't have a solution either, but am open to suggestions. I do know that whatever solution we come up with will require work from both sides of each sync -- oslo and the receiving project.</div>
<div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">In the mean time, I have been focusing on addressing the underlying issue by moving code into libraries. In past releases, we graduated the config library and the messaging library, each with a certain amount of pain. That pain taught us valuable lessons, which we have applied when creating processes and tools to make things go more smoothly for other libraries. We are verifying them now with oslo.vmware and oslo.test, and when we have the kinks worked out we will move on to other libraries [1].</div>
<div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Doug</div><br></div><div><div class="gmail_default" style="font-size:small">[1] <a href="https://wiki.openstack.org/wiki/Oslo/GraduationStatus">https://wiki.openstack.org/wiki/Oslo/GraduationStatus</a></div>
<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class=""><div class="h5"><br>
><br>
> Doug<br>
><br>
> _______________________________________________<br>
> OpenStack-dev mailing list<br>
> <a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>