<div dir="ltr"><div><div>If we're not doing another merge forward, I think we should do the cleanup soon.<br><br>The value of keeping that code around (to me) was this: you want to know when a Pulp2 code change occurred and that code had already been ported to Pulp3. That way you would know the Pulp2 changes had to also be ported to Pulp3. The problem is: that value would only be known at "merge forward time" via a merge conflicts. So if we're not merging forward again (for core), then we can't realize that benefit again like we were. This is why I think we're ready for cleanup soon.<br><br></div>Here is a cleanup task [0], in case others want to look/edit/comment on it.<br><br>[0]: <a href="https://pulp.plan.io/issues/2955">https://pulp.plan.io/issues/2955</a><br><br></div>-Brian<br><div><div><br><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Aug 3, 2017 at 8:33 AM, Michael Hrivnak <span dir="ltr"><<a href="mailto:mhrivnak@redhat.com" target="_blank">mhrivnak@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Tue, Aug 1, 2017 at 5:33 PM, Brian Bouterse <span dir="ltr"><<a href="mailto:bbouters@redhat.com" target="_blank">bbouters@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div>Originally 3.0-dev was branched from master, and we've merged master back into 3.0-dev twice, once at the end of 2016 and once in early 2017. I haven't heard about any plans to merge master -> 3.0-dev again. I want to check in if its time to declare that we aren't going to do that merge forward again.<br><br></div>I don't think there is much value in merging forward master -> 3.0-dev again because the 3.0 code is so different than the 2.y code on master that any diffs probably won't even make sense anymore. Also, the last merge forward was pretty darn painful and it took a while.<br><br></div>So with ^ in mind we could:<br><br></div>1) Decide that we don't need to merge forward master -> 3.0-dev again.<br></div></div></div></div></blockquote><div><br></div></span><div>I agree. We don't likely have much to gain by trying another merge, and it would definitely be painful. </div><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div></div><br>2) Clean up the 3.0-dev repo. Basically delete everything on 3.0-dev that isn't part of the Pulp3 codebase. We can hammer out exactly what those things are on an issue that isn't yet written.<br><br></div></div></div></blockquote><div><br></div></span><div>We were getting some value out of tracking what Pulp 2 code had been converted by removing it from the 3.0-dev branch as we went along. I'm not sure how well that's serving us though, and I'd be happy discontinuing that pattern. I figured at some point we would need a big purge, and doing that soon would be fine unless anyone finds the current pattern particularly helpful.</div></div><span class="HOEnZb"><font color="#888888"><br clear="all"><div><br></div>-- <br><div class="m_-5513509926693246213gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Michael</span> <span style="margin:0px!important;padding:0px!important">Hrivnak</span></p><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"></p><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><span style="margin:0px!important;padding:0px!important">Principal Software Engineer</span><span style="margin:0px!important;padding:0px!important">, <span style="margin:0px!important;padding:0px!important">RHCE</span></span> </span><span style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px"></span><br style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important"><p style="color:rgb(0,0,0);font-family:overpass-mono,monospace;font-size:10px;margin:0px!important;padding:0px!important">Red Hat</p></div></div>
</font></span></div></div>
</blockquote></div><br></div>