<div dir="ltr">So, trying to summarize this thread...<div><br></div><div>1) For each "screen" (step) in debconf, we shall provide a description of what it accomplishes in the associated configuration file(s).</div>
<div><br></div><div>2) For each patch potentially affecting Debian, include Tom Goirand in the review process.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 3, 2014 at 12:32 PM, Andreas Jaeger <span dir="ltr"><<a href="mailto:aj@suse.com" target="_blank">aj@suse.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 06/03/2014 10:22 AM, Thomas Goirand wrote:<br>
> On 06/03/2014 02:47 AM, Andreas Jaeger wrote:<br>
>> I've took the time to review the distribution specific parts in a bit<br>
>> more detail.<br>
>><br>
>> Looking at the Image Service example, it's basically really only this<br>
>> section which is completely different:<br>
>><br>
>> On 06/02/2014 05:46 PM, Thomas Goirand wrote:<br>
>>> [...]<br>
>>> Another example. See this (from the Ubuntu guide):<br>
>>><br>
>>> <a href="http://docs.openstack.org/icehouse/install-guide/install/apt/content/glance-install.html" target="_blank">http://docs.openstack.org/icehouse/install-guide/install/apt/content/glance-install.html</a><br>

>>><br>
>>> and now see this (from the Debian guide):<br>
>>><br>
>>> <a href="http://docs.openstack.org/icehouse/install-guide/install/apt-debian/content/glance-install.html" target="_blank">http://docs.openstack.org/icehouse/install-guide/install/apt-debian/content/glance-install.html</a><br>

>><br>
>> And this is different from all other distributions, Debian is the odd<br>
>> one out.<br>
><br>
> Yeah!<br>
<br>
</div>;)<br>
<div class=""><br>
>> Looking at the description for me it's not so much that Debian<br>
>> is using a different tool - it's a complete different way of describing<br>
>> it. While for the non-Debian distros, we explicitly document which<br>
>> variables to set, I see here "Respond to prompts" without any guidance<br>
>> on what to answer - and that means to me what you document is a<br>
>> different kind of guide then what we describe. Looking at the<br>
>> screenshot, the non-Debian guides would write something like:<br>
>> "Respond the pipeline question with 'keystone'".<br>
>><br>
>> And that's for me the question - are we working on the same kind of guide?<br>
><br>
> I agree with this. I believe there should be more explanation of what<br>
> directive is edited by responding to the prompts. If you let me, I'll<br>
> add that in the Debian doc.<br>
<br>
</div>Great, will review happily ;)<br>
<div class=""><br>
>> Thomas, btw. I think part of this is also triggered by comments like you<br>
>> do in <a href="https://review.openstack.org/#/c/97156/" target="_blank">https://review.openstack.org/#/c/97156/</a>. I do have added you as<br>
>> reviewer in the past to several patches and sometimes didn't get a<br>
>> review in a timely manner. We work on a best effort base and many of us<br>
>> on a volunteer base - like you - and while I understand your some of<br>
>> your frustration, such comments might lead some to not review your<br>
>> patches at all,<br>
>><br>
>> Andreas<br>
><br>
> Andreas, I'm sorry if you feel bad about the patch comment. I thought<br>
> what happened was a mistake, and lots of what I did for Havana went<br>
> away. It as a bit frustrating for me, which lead to that text. Though:<br>
><br>
> "It'd be nice if I didn't have to do such work again in the future (eg:<br>
> if reviewers took care of not having these get in again)."<br>
><br>
> is IMO written in a nice way. It's "it'd be nice if", not "WTF did the<br>
> reviewer"... So please don't take it as badness, it's just a bit of<br>
> frustration on my side.<br>
<br>
</div>Ok, understood.<br>
<div class=""><br>
> Also, you must know that I appreciated a lot all the time you spent<br>
> reviewing my patches. Which is why I was so happy to see you in Atlanta.<br>
> I felt regret I didn't spend enough time with you and had no opportunity<br>
> to drink a beer with you to say thank you. I hope we'll met again in<br>
> Paris, and this time, I'll make sure to spend enough time with the doc<br>
> team! :)<br>
<br>
</div>Yeah, Atlanta was too short in some ways, I would have loved to chat a<br>
bit more with you as well.<br>
<div class=""><br>
> Last thing: I think I did a few review. If I missed some, then please<br>
> forgive me. Please add me again on reviews, and I'll do my best to find<br>
> the time to do the work.<br>
<br>
</div>Ok, will try to add you again to reviews if I think those affect Debian,<br>
<div class="im HOEnZb"><br>
Andreas<br>
--<br>
 Andreas Jaeger aj@{<a href="http://suse.com" target="_blank">suse.com</a>,<a href="http://opensuse.org" target="_blank">opensuse.org</a>} Twitter/Identica: jaegerandi<br>
  SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany<br>
   GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)<br>
    GPG fingerprint = 93A3 365E CE47 B889 DF7F  FED1 389A 563C C272 A126<br>
<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Openstack-docs mailing list<br>
<a href="mailto:Openstack-docs@lists.openstack.org">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>