<div dir="ltr"><div>Hey everyone,</div><div><br></div><div>During on the fuel meeting today we discussed the librarian changes and their status.</div><div>As part of this work, the wiki page was updated and a first attempt at migrating the</div><div>following modules has been completed pending merge:</div><div><br></div><div>stdlib </div><div>concat</div><div>inifile</div><div>ssh</div><div>ntp</div><div>apache</div><div>firewall</div><div>xinetd</div><div>cinder</div><div>apt*</div><div><br></div><div>It should be noted that apt is currently blocked by the lack of a mirror so while it has</div><div>been prepared, it should not be merged at this time.</div><div><br></div><div>As part of this migration we are doing two things. The first is an update to the build</div><div>process that is included as part of the initial librarian[0] patch.  The other patches</div><div>consist of the actual module code changes.  </div><div><br></div><div>Here is the list of the diffs for each change so that it can be reviewed and people can</div><div>raise concerns if there are any with this change. As part of the migration, I inspected</div><div>the code and file differences for each module to determine how much impact they might</div><div>have.  I chose the list of modules based on their minimal differences from the upstream</div><div>or if they already had our forked differences rolled into a newer version of the module.</div><div>For this list, I took the current stable iso (#110) and rebased the changes on top of this</div><div>to create a custom iso with just the librarian changes. We have kicked off a bvt_2 test for</div><div>the custom iso as well. From this iso I have extracted the fuel-library package from both</div><div>of these isos and exploded the fuel-library folder structure to do the diffs. </div><div><br></div><div>Code Changes:</div><div><br></div><div>For stdlib, the only differences are related to git, travis or fixtures[1].  There are no</div><div>puppet code changes as part of the librarian migration.</div><div><br></div><div>For concat, the only differences were a git folder and in a custom change to the spec tests[2].</div><div>The test difference[3], was a change we made because it was failing our syntax checker. </div><div>This change has been included in a newer version of concat (1.2.4) but are not necessary</div><div>when the module gets moved to be included via librarian.</div><div><br></div><div>For inifile, the only difference is the addition of git and metadata files[4].</div><div><br></div><div>For ssh, the only difference is a single line to have the config notify service[5]. This</div><div>difference is already covered by another file and is not needed[6].</div><div><br></div><div>For ntp, this change introduces more code changes[7] because we are updating the module</div><div>to the 4.0.0 version because of previous extending of functionality that is now covered by</div><div>4.0.0 vs 3.3.0[8]. The changes in our fork were upstreamed and are include in 4.0.0.</div><div><br></div><div>For apache, this change includes an upgrade from 1.2.0 to 1.3.0[9][10]. Our fork had a</div><div>customization made which was contributed upstream. (apache::mod::proxy_connect)</div><div><br></div><div>For firewall, this change also includes an upgrade from 1.0.2 to 1.2.0[11] as our fork had</div><div>mac supported added[12] in which is now covered upstream.</div><div><br></div><div>For xinetd, the only change was the addition of a .git folder and a .gitignore with librarian.</div><div><br></div><div>For cinder, the only change was the addition of .git, .gitignore, and .gitreview.</div><div><br></div><div>Once we can get the apt mirror created, the only change for that is also the addition of</div><div>.git.</div><div><br></div><div><br></div><div>If there are any of these upgrades/changes that we do not want to tackle right now, I can</div><div>adjust the review order such that it can be skipped for now.  Please take some time to</div><div>review these changes and raise concerns.  So far CI has been successful on all of these</div><div>changes, and bvt is currently running.</div><div><br></div><div>Also please take some time to review the changes themselves:</div><div><a href="https://review.openstack.org/#/q/status:open+project:stackforge/fuel-library+branch:master+topic:bp/fuel-puppet-librarian,n,z">https://review.openstack.org/#/q/status:open+project:stackforge/fuel-library+branch:master+topic:bp/fuel-puppet-librarian,n,z</a></div><div><br></div><div>Please raise any concerns as quickly as possible as this is the last call for objections</div><div>for these reviews.  This has been talked about extensively and these reviews have</div><div>been available for several weeks now.</div><div><br></div><div>Thanks,</div><div>-Alex</div><div><br></div><div><br></div><div>[0] <a href="https://review.openstack.org/#/c/202763/">https://review.openstack.org/#/c/202763/</a></div><div>[1] <a href="http://paste.openstack.org/show/406523/">http://paste.openstack.org/show/406523/</a></div><div>[2] <a href="http://paste.openstack.org/show/406524/">http://paste.openstack.org/show/406524/</a></div><div>[3] <a href="http://paste.openstack.org/show/406525/">http://paste.openstack.org/show/406525/</a></div><div>[4] <a href="http://paste.openstack.org/show/406526/">http://paste.openstack.org/show/406526/</a></div><div>[5] <a href="http://paste.openstack.org/show/406527/">http://paste.openstack.org/show/406527/</a></div><div>[6] <a href="https://github.com/saz/puppet-ssh/blob/v2.4.0/manifests/server/config.pp#L9">https://github.com/saz/puppet-ssh/blob/v2.4.0/manifests/server/config.pp#L9</a></div><div>[7] <a href="http://paste.openstack.org/show/406536/">http://paste.openstack.org/show/406536/</a></div><div>[8] <a href="https://github.com/puppetlabs/puppetlabs-ntp/compare/3.3.0...4.0.0">https://github.com/puppetlabs/puppetlabs-ntp/compare/3.3.0...4.0.0</a></div><div>[9] <a href="http://paste.openstack.org/show/406538/">http://paste.openstack.org/show/406538/</a></div><div>[10] <a href="https://github.com/puppetlabs/puppetlabs-apache/compare/1.2.0...1.3.0">https://github.com/puppetlabs/puppetlabs-apache/compare/1.2.0...1.3.0</a></div><div>[11] <a href="https://github.com/puppetlabs/puppetlabs-firewall/compare/1.0.2...1.2.0">https://github.com/puppetlabs/puppetlabs-firewall/compare/1.0.2...1.2.0</a></div><div>[12] <a href="https://review.openstack.org/#/c/92167/">https://review.openstack.org/#/c/92167/</a></div></div>