<div dir="ltr"><div><div><div><div>Hey guys,<br><br></div>I am really happy to announce that we have the Icehouse branch now,<br></div>so I guess this conversation is not exactly required anymore. I would<br>still like to wait for a day before I do some required cleanups and the<br>publish process inside openstack-manuals to maintain the sanity and<br></div>the repository should be clean and ready to merge the Juno patches by<br>tomorrow morning.<br><br></div>Regards,<br>Pranav<br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jan 27, 2015 at 11:46 AM, Roger Luethi <span dir="ltr"><<a href="mailto:rl@patchworkscience.org" target="_blank">rl@patchworkscience.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On Mon, 26 Jan 2015 19:01:20 +0000, Matjaz Pancur wrote:<br>
> This “wait-for-icehouse-branching” delay is way too long. We were talking<br>
> on today’s IRC meeting that we should just start reviewing/merging and<br>
> not wait on the branching process anymore, since it can be done on an<br>
> arbitrary commit anyway.<br>
<br>
</span>There is one catch, though: after we merge the first Juno-only patch, we<br>
can no longer easily merge patches destined for both Icehouse and Juno (we<br>
would have to backport them). Therefore, it would be good to know if the<br>
patches under review are Juno-only or also good for Icehouse.<br>
<br>
FWIW, my patches are all for Icehouse and Juno, and I will not submit<br>
Juno-only patches until a) the Juno branch is cut or b) a patch that is<br>
clearly Juno-only has been merged.<br>
<span><br>
> Here is the list of patches we should review/merge to get the ball rolling:<br>
><br>
> * all of the Sean’s XML-to-RST series:<br>
<br>
</span>I reordered them and added a note on what they do (from the commit message)<br>
and the files they create:<br>
<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/141226/" target="_blank">https://review.openstack.org/#/c/141226/</a><br>
<br>
[creates directory structure]<br>
creates landslide structure for the guides<br>
-------------------------------------------------<br>
<br>
This landslide structure brings a whole lot of redundancy: identical<br>
files in four copies. As long as we only have the associate-guide, we<br>
only need the structure for one guide, so we could merge a patch limited<br>
to that. Once we have a second guide, we need a better structure with<br>
less redundancy.<br>
<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/142259/" target="_blank">https://review.openstack.org/#/c/142259/</a><br>
<br>
01-getting-started.rst<br>
convert associate guide chapter 1 XML to RST<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/142184/" target="_blank">https://review.openstack.org/#/c/142184/</a><br>
<br>
02-associate-getting-started-quiz.rst<br>
convert associate guide chapter 2 XML to RST<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/141236/" target="_blank">https://review.openstack.org/#/c/141236/</a><br>
<br>
03-controller-node.rst<br>
convert associate guide chapter 3 XML to RST<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/142290/" target="_blank">https://review.openstack.org/#/c/142290/</a><br>
<br>
04-associate-controller-node-quiz.rst<br>
convert associate guide chapter 4 XML to RST<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/142333/" target="_blank">https://review.openstack.org/#/c/142333/</a><br>
<br>
06-associate-compute-node-quiz.rst<br>
convert associate guide chapter 6 XML to RST<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/142344/" target="_blank">https://review.openstack.org/#/c/142344/</a><br>
<br>
07-associate-network-node.rst<br>
convert associate guide chapter 7 XML to RST<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/142339/" target="_blank">https://review.openstack.org/#/c/142339/</a><br>
<br>
08-associate-network-node-quiz.rst<br>
convert associate guide chapter 8 XML to RST<br>
-------------------------------------------------<br>
> <a href="https://review.openstack.org/#/c/142350/" target="_blank">https://review.openstack.org/#/c/142350/</a><br>
<br>
09-object-storage-node.rst<br>
convert associate guide chapter 9 XML to RST<br>
-------------------------------------------------<br>
<br>
Looks like chapter 5 is missing. Also, the file names are not consistent<br>
(e.g. 03-controller-node.rst vs. 07-associate-network-node.rst).<br>
<br>
IMO each conversion patch should remove the XML file it replaces; we<br>
want to reduce redundancy, not add to it.<br>
<br>
RST files in OpenStack repos are line-wrapped at 80 characters, but these<br>
converted files are not.<br>
<br>
The converted files are supposed to be slides (similar to the<br>
upstream-training slides), but they are not. They look like books split<br>
into slide-sized bits (and even then, several slides have content<br>
overflowing). The proposed 01-getting-started.rst is 50 KB! This would not<br>
give us any of the benefits we were hoping for when we decided to replace<br>
books with slides.<br>
<span><font color="#888888"><br>
Roger<br>
</font></span><div><div><br>
_______________________________________________<br>
OpenStack-docs mailing list<br>
<a href="mailto:OpenStack-docs@lists.openstack.org" target="_blank">OpenStack-docs@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs</a><br>
</div></div></blockquote></div><br></div></div>