<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 29, 2021 at 8:09 PM Jeremy Stanley <<a href="mailto:fungi@yuggoth.org">fungi@yuggoth.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 2021-03-29 19:47:24 -0400 (-0400), Peter Matulis wrote:<br>
> I changed the testenv to 'pdf-docs' and the build is still being skipped.<br>
> <br>
> Do I need to submit a PR to have this [1] set to 'false'?<br>
> <br>
> [1]:<br>
> <a href="https://opendev.org/openstack/openstack-zuul-jobs/src/commit/01746b6df094c25f0cd67690b44adca0fb4ee1fd/zuul.d/jobs.yaml#L970" rel="noreferrer" target="_blank">https://opendev.org/openstack/openstack-zuul-jobs/src/commit/01746b6df094c25f0cd67690b44adca0fb4ee1fd/zuul.d/jobs.yaml#L970</a><br>
[...]<br>
<br>
Oh, yep that'll need to be adjusted or overridden as well. I see<br>
that <a href="https://review.opendev.org/678077" rel="noreferrer" target="_blank">https://review.opendev.org/678077</a> explicitly chose not to do<br>
PDF builds for deploy guides for the original PDF docs<br>
implementation a couple of years ago. Unfortunately the commit<br>
message doesn't say why, but maybe this is a good opportunity to<br>
start.</blockquote><div><br></div><div>Any other thoughts before I propose a change to the below?</div><div><br></div><div><a href="https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L970">https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/zuul.d/jobs.yaml#L970</a><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">However, many (most?) API projects seem to include their<br>
deployment guides in their software Git repos, so switching this on<br>
for everyone might break their deploy guide builds. If we combine it<br>
with an expectation for a deploy-guide-specific PDF building tox<br>
testenv like you had previously, then it would get safely skipped by<br>
any projects without that testenv defined.<br>
-- <br>
Jeremy Stanley<br>
</blockquote></div></div>