<div dir="ltr"><div>We had originally planned for the FFEs for both fuel-openstack-tasks[1] and fuel-remove-conflict-openstack to [2] to close on 3/20, This would have placed them before changes that conflict with fuel-refactor-osnailyfacter-for-puppet-master-compatibility [3].</div><div><br></div><div>[1] <a href="http://lists.openstack.org/pipermail/openstack-dev/2016-March/088297.html">http://lists.openstack.org/pipermail/openstack-dev/2016-March/088297.html</a></div><div>[2] <a href="http://lists.openstack.org/pipermail/openstack-dev/2016-March/088298.html">http://lists.openstack.org/pipermail/openstack-dev/2016-March/088298.html</a></div><div>[3] <a href="http://lists.openstack.org/pipermail/openstack-dev/2016-March/089028.html">http://lists.openstack.org/pipermail/openstack-dev/2016-March/089028.html</a></div><div><br></div><div>However we found this morning that the changes from [2], and more of issue [1] will result in further issues such as [4], where as the task files move, any task that explicitly relied on it, now no longer is in the same path.</div><div><br></div><div>[4] <a href="https://review.openstack.org/#/c/295170/">https://review.openstack.org/#/c/295170/</a></div><div><br></div><div>Due to this newly identified issue with backwards comparability. It appears that [4] shows that we have plugins using interfaces that we don't have formal coverage for so If we introduce this set of changes, we will cause breakage for plugins that use fuel's current tasks.</div><div><br></div><div>From a deprecation standpoint we don't have a way to deal with this, unless  fuel-openstack-tasks [1] lands after fuel-refactor-osnailyfacter-for-puppet-master-compatibility [3]. In this case we can take advantage of the class include stubs, leaving a copy in the old location (osnailyfacter/modular/roles/compute.pp) pointing to the new include location (include openstack_tasks::roles::compute) and adding a warning for deprecation. The tasks includes in the new location openstack_tasks/examples/roles/compute.pp would simply include the updated class location w/o the warning.</div><div><br></div><div>This would take care of [1] and it's review [5]</div><div><br></div><div>[5] <a href="https://review.openstack.org/283332">https://review.openstack.org/283332</a></div><div><br></div><div>This still leaves [2] un-addressed, we still have 3 open CR for it:</div><div><br></div><div>[6] Compute <a href="https://review.openstack.org/285567">https://review.openstack.org/285567</a></div><div>[7] Cinder <a href="https://review.openstack.org/294736">https://review.openstack.org/294736</a></div><div>[8] Swift <a href="https://review.openstack.org/294979">https://review.openstack.org/294979</a></div><div><br></div><div>Compute [6] is in good shape, while Cinder [7] and Swift [8] are not. For these do we want to continue to land them, if so what do we want to do about the now deprecated openstack:: tasks? We could leave them in place with a warning since we would not be using them</div><div><br></div></div><div dir="ltr">-- <br></div><div dir="ltr"><p dir="ltr">--</p><p dir="ltr"><span style="font-size:13.1999998092651px">Andrew Woodward</span></p><p dir="ltr"><span style="font-size:13.1999998092651px">Mirantis</span></p><p dir="ltr"><span style="font-size:13.1999998092651px">Fuel Community Ambassador</span></p><p dir="ltr"><span style="font-size:13.1999998092651px">Ceph Community</span></p>
</div>