<div dir="ltr">Okay, folks, we had a short meeting to synchronize our vision of how it should happen.<div><br></div><div>We will start by merging least-invasive modules like stdlib today and then continue doing merges one by one in discrete manner and revert things immediately if something goes wrong.</div><div><br></div><div>So there is a list of action items:</div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>Alex Schultz will send a schedule of which modules will be merged on which week and ensure that core reviewers know which commits they should merge when either by keeping W-1 on particular commits or by sharing the schedule in commit message so that noone can forget about it or maybe some other conveinent method that he can invent. </div></blockquote><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div>I will remove my -2 for the inital librarian commit.</div></blockquote><div><br></div>Thanks everyone for the collaboration and not calling me a selfish lunatic :-)<br><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 31, 2015 at 6:29 PM, Mike Scherbakov <span dir="ltr"><<a href="mailto:mscherbakov@mirantis.com" target="_blank">mscherbakov@mirantis.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">Vladimir,<div>can you please elaborate on "such invasive changes"?</div><div><br></div><div>There was a plan developed, including risk mitigation, etc. - like do 'grep -r' to check, and revert the change all together right away if we see regression. So far, everyone was aligned with the plan. It was discussed yesterday during IRC meeting [1]. Again, no one had objections.</div><div><br></div><div>Please provide your concerns, explain your opinion in more details. I'd like other core reviewers to jump in here and reply. If you need details of the approach, please jump in a call with Alex Schultz.</div><div><br></div><div>Thank you,</div><div><br></div><div>[1] <a href="http://eavesdrop.openstack.org/meetings/fuel/2015/fuel.2015-07-30-16.00.html" target="_blank">http://eavesdrop.openstack.org/meetings/fuel/2015/fuel.2015-07-30-16.00.html</a></div><div><div class="h5"><div dir="ltr"><div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 31, 2015 at 5:52 AM Vladimir Kuklin <<a href="mailto:vkuklin@mirantis.com" target="_blank">vkuklin@mirantis.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Folks<div><br></div><div>I do actively support our initiative to use librarian to be as close as possible to upstream, but let's not merge such invasive changes until we announce Hard Code Freeze and create stable/7.0 branch. So far I put my -2 onto the first commit in the chain. Let's get through 7.0 and then land this code in the master as early as possible after HCF.</div><div><br></div></div><div class="gmail_extra"></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 31, 2015 at 3:21 PM, Aleksandra Fedorova <span dir="ltr"><<a href="mailto:afedorova@mirantis.com" target="_blank">afedorova@mirantis.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>> So far CI has been successful on all of these<br>
changes, and bvt is currently running.<br>
<br>
</span>Small update - BVT test passed.<br>
<div><div><br>
On Fri, Jul 31, 2015 at 6:27 AM, Alex Schultz <<a href="mailto:aschultz@mirantis.com" target="_blank">aschultz@mirantis.com</a>> wrote:<br>
> Hey everyone,<br>
><br>
> During on the fuel meeting today we discussed the librarian changes and<br>
> their status.<br>
> As part of this work, the wiki page was updated and a first attempt at<br>
> migrating the<br>
> following modules has been completed pending merge:<br>
><br>
> stdlib<br>
> concat<br>
> inifile<br>
> ssh<br>
> ntp<br>
> apache<br>
> firewall<br>
> xinetd<br>
> cinder<br>
> apt*<br>
><br>
> It should be noted that apt is currently blocked by the lack of a mirror so<br>
> while it has<br>
> been prepared, it should not be merged at this time.<br>
><br>
> As part of this migration we are doing two things. The first is an update to<br>
> the build<br>
> process that is included as part of the initial librarian[0] patch.  The<br>
> other patches<br>
> consist of the actual module code changes.<br>
><br>
> Here is the list of the diffs for each change so that it can be reviewed and<br>
> people can<br>
> raise concerns if there are any with this change. As part of the migration,<br>
> I inspected<br>
> the code and file differences for each module to determine how much impact<br>
> they might<br>
> have.  I chose the list of modules based on their minimal differences from<br>
> the upstream<br>
> or if they already had our forked differences rolled into a newer version of<br>
> the module.<br>
> For this list, I took the current stable iso (#110) and rebased the changes<br>
> on top of this<br>
> to create a custom iso with just the librarian changes. We have kicked off a<br>
> bvt_2 test for<br>
> the custom iso as well. From this iso I have extracted the fuel-library<br>
> package from both<br>
> of these isos and exploded the fuel-library folder structure to do the<br>
> diffs.<br>
><br>
> Code Changes:<br>
><br>
> For stdlib, the only differences are related to git, travis or fixtures[1].<br>
> There are no<br>
> puppet code changes as part of the librarian migration.<br>
><br>
> For concat, the only differences were a git folder and in a custom change to<br>
> the spec tests[2].<br>
> The test difference[3], was a change we made because it was failing our<br>
> syntax checker.<br>
> This change has been included in a newer version of concat (1.2.4) but are<br>
> not necessary<br>
> when the module gets moved to be included via librarian.<br>
><br>
> For inifile, the only difference is the addition of git and metadata<br>
> files[4].<br>
><br>
> For ssh, the only difference is a single line to have the config notify<br>
> service[5]. This<br>
> difference is already covered by another file and is not needed[6].<br>
><br>
> For ntp, this change introduces more code changes[7] because we are updating<br>
> the module<br>
> to the 4.0.0 version because of previous extending of functionality that is<br>
> now covered by<br>
> 4.0.0 vs 3.3.0[8]. The changes in our fork were upstreamed and are include<br>
> in 4.0.0.<br>
><br>
> For apache, this change includes an upgrade from 1.2.0 to 1.3.0[9][10]. Our<br>
> fork had a<br>
> customization made which was contributed upstream.<br>
> (apache::mod::proxy_connect)<br>
><br>
> For firewall, this change also includes an upgrade from 1.0.2 to 1.2.0[11]<br>
> as our fork had<br>
> mac supported added[12] in which is now covered upstream.<br>
><br>
> For xinetd, the only change was the addition of a .git folder and a<br>
> .gitignore with librarian.<br>
><br>
> For cinder, the only change was the addition of .git, .gitignore, and<br>
> .gitreview.<br>
><br>
> Once we can get the apt mirror created, the only change for that is also the<br>
> addition of<br>
> .git.<br>
><br>
><br>
> If there are any of these upgrades/changes that we do not want to tackle<br>
> right now, I can<br>
> adjust the review order such that it can be skipped for now.  Please take<br>
> some time to<br>
> review these changes and raise concerns.  So far CI has been successful on<br>
> all of these<br>
> changes, and bvt is currently running.<br>
><br>
> Also please take some time to review the changes themselves:<br>
> <a href="https://review.openstack.org/#/q/status:open+project:stackforge/fuel-library+branch:master+topic:bp/fuel-puppet-librarian,n,z" rel="noreferrer" target="_blank">https://review.openstack.org/#/q/status:open+project:stackforge/fuel-library+branch:master+topic:bp/fuel-puppet-librarian,n,z</a><br>
><br>
> Please raise any concerns as quickly as possible as this is the last call<br>
> for objections<br>
> for these reviews.  This has been talked about extensively and these reviews<br>
> have<br>
> been available for several weeks now.<br>
><br>
> Thanks,<br>
> -Alex<br>
><br>
><br>
> [0] <a href="https://review.openstack.org/#/c/202763/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/202763/</a><br>
> [1] <a href="http://paste.openstack.org/show/406523/" rel="noreferrer" target="_blank">http://paste.openstack.org/show/406523/</a><br>
> [2] <a href="http://paste.openstack.org/show/406524/" rel="noreferrer" target="_blank">http://paste.openstack.org/show/406524/</a><br>
> [3] <a href="http://paste.openstack.org/show/406525/" rel="noreferrer" target="_blank">http://paste.openstack.org/show/406525/</a><br>
> [4] <a href="http://paste.openstack.org/show/406526/" rel="noreferrer" target="_blank">http://paste.openstack.org/show/406526/</a><br>
> [5] <a href="http://paste.openstack.org/show/406527/" rel="noreferrer" target="_blank">http://paste.openstack.org/show/406527/</a><br>
> [6]<br>
> <a href="https://github.com/saz/puppet-ssh/blob/v2.4.0/manifests/server/config.pp#L9" rel="noreferrer" target="_blank">https://github.com/saz/puppet-ssh/blob/v2.4.0/manifests/server/config.pp#L9</a><br>
> [7] <a href="http://paste.openstack.org/show/406536/" rel="noreferrer" target="_blank">http://paste.openstack.org/show/406536/</a><br>
> [8] <a href="https://github.com/puppetlabs/puppetlabs-ntp/compare/3.3.0...4.0.0" rel="noreferrer" target="_blank">https://github.com/puppetlabs/puppetlabs-ntp/compare/3.3.0...4.0.0</a><br>
> [9] <a href="http://paste.openstack.org/show/406538/" rel="noreferrer" target="_blank">http://paste.openstack.org/show/406538/</a><br>
> [10] <a href="https://github.com/puppetlabs/puppetlabs-apache/compare/1.2.0...1.3.0" rel="noreferrer" target="_blank">https://github.com/puppetlabs/puppetlabs-apache/compare/1.2.0...1.3.0</a><br>
> [11] <a href="https://github.com/puppetlabs/puppetlabs-firewall/compare/1.0.2...1.2.0" rel="noreferrer" target="_blank">https://github.com/puppetlabs/puppetlabs-firewall/compare/1.0.2...1.2.0</a><br>
> [12] <a href="https://review.openstack.org/#/c/92167/" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/92167/</a><br>
><br>
</div></div>> __________________________________________________________________________<br>
> OpenStack Development Mailing List (not for usage questions)<br>
> Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
> <a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
><br>
<span><font color="#888888"><br>
<br>
<br>
--<br>
Aleksandra Fedorova<br>
Fuel CI Engineer<br>
bookwar<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div></div><div class="gmail_extra">-- <br><div><div dir="ltr"><div><div dir="ltr">Yours Faithfully,<br>Vladimir Kuklin,<br>Fuel Library Tech Lead,<br>Mirantis, Inc.<br>+7 (495) 640-49-04<br>+7 (926) 702-39-68<br>Skype kuklinvv<br>35bk3, Vorontsovskaya Str.<br>Moscow, Russia,<br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.com</a><br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.ru</a><br><a href="mailto:vkuklin@mirantis.com" target="_blank">vkuklin@mirantis.com</a></div></div></div></div>
</div>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div></div></div></div></div></div><span class="HOEnZb"><font color="#888888"><div dir="ltr">-- <br></div><div dir="ltr">Mike Scherbakov<br>#mihgen</div>
</font></span><br>__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr">Yours Faithfully,<br>Vladimir Kuklin,<br>Fuel Library Tech Lead,<br>Mirantis, Inc.<br>+7 (495) 640-49-04<br>+7 (926) 702-39-68<br>Skype kuklinvv<br>35bk3, Vorontsovskaya Str.<br>Moscow, Russia,<br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.com</a><br><a href="http://www.mirantis.ru/" target="_blank">www.mirantis.ru</a><br><a href="mailto:vkuklin@mirantis.com" target="_blank">vkuklin@mirantis.com</a></div></div></div></div>
</div>