<div dir="ltr"><div>Cursory search appears to indicate that these are part of some boilerplate configuration code. So I believe we should be fine if we remove it.<br></div><div>Still, I wonder why was the language set to `None` in the first place. <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 6, 2022 at 4:21 PM Takashi Kajinami <<a href="mailto:tkajinam@redhat.com">tkajinam@redhat.com</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"><div dir="ltr"><div>Thanks Brian,</div><div><br></div><div><br></div><div>Actually I hit this problem initially with heat-cfntools which is not using upper-constraints in doc build</div><div>but has language=None explicitly set, so what you mentioned makes quite clear sense.</div><div><br></div><div>Quick grep shows a few instances of the hard-coded language=None . These should be commented out</div><div>before we get the constraint bumped.</div><div><br></div><div>openstack/python-swiftclient/releasenotes/source/conf.py:language = None<br>openstack/blazar/api-ref/source/conf.py:language = None<br>openstack/heat-agents/doc/source/conf.py:language = None<br>openstack/heat-agents/releasenotes/source/conf.py:language = None<br>openstack/swift/releasenotes/source/conf.py:language = None<br>openstack/storlets/releasenotes/source/conf.py:language = None<br>openstack/placement/releasenotes/source/conf.py:language = None<br>openstack/osc-lib/releasenotes/source/conf.py:language = None<br>openstack/python-heatclient/releasenotes/source/conf.py:language = None<br>openstack/trove-dashboard/releasenotes/source/conf.py:language = None<br>openstack/virtualbmc/releasenotes/source/conf.py:language = None<br>openstack/heat-templates/doc/source/conf.py:language = None<br></div><div><br></div><div>Disclaimer: The above list might be missing some projects. <br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 6, 2022 at 10:19 PM Brian Rosmaita <<a href="mailto:rosmaita.fossdev@gmail.com" target="_blank">rosmaita.fossdev@gmail.com</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 6/6/22 12:27 AM, Takashi Kajinami wrote:<br>
[snip]<br>
> Updating conf.py would be still required so it's better to work on it <br>
> earlier<br>
> before the upper constraint is updated.<br>
<br>
I was just looking into this, and as long as your conf.py does not <br>
explicitly set<br>
   language = None<br>
the default language is set to 'en' internally by Sphinx in 5.0.0+ [0]. <br>
  So as long as you have the language setting either missing or <br>
commented out in your conf.py, you should not see this warning (which, <br>
when treated as an error, breaks the doc build).<br>
<br>
[0] <br>
<a href="https://github.com/sphinx-doc/sphinx/commit/e4e58a4f2791e528cdaa861b96636a1e37a558ba" rel="noreferrer" target="_blank">https://github.com/sphinx-doc/sphinx/commit/e4e58a4f2791e528cdaa861b96636a1e37a558ba</a><br>
<br>
> <br>
> On Mon, Jun 6, 2022 at 1:15 PM Takashi Kajinami <<a href="mailto:tkajinam@redhat.com" target="_blank">tkajinam@redhat.com</a> <br>
> <mailto:<a href="mailto:tkajinam@redhat.com" target="_blank">tkajinam@redhat.com</a>>> wrote:<br>
> <br>
>     Hi,<br>
> <br>
> <br>
>     It seems the openstack-tox-docs job is broken in some projects<br>
>     because of the following<br>
>     change made in Sphinx >= 5.0.0 .<br>
>     <a href="https://github.com/sphinx-doc/sphinx/commit/a3d09835522e6487cf93dcc871179db8d69e4180" rel="noreferrer" target="_blank">https://github.com/sphinx-doc/sphinx/commit/a3d09835522e6487cf93dcc871179db8d69e4180</a><br>
>     <<a href="https://github.com/sphinx-doc/sphinx/commit/a3d09835522e6487cf93dcc871179db8d69e4180" rel="noreferrer" target="_blank">https://github.com/sphinx-doc/sphinx/commit/a3d09835522e6487cf93dcc871179db8d69e4180</a>><br>
> <br>
>     We currently have Sphinx 4.5.0 in upper-constraints.txt but some<br>
>     projects (eg. Heat, Nova,<br>
>     Neutron) are using only their own docs-constraints.txt and pull the<br>
>     latest version which is 5.0.1 .<br>
> <br>
>     IMO we need to fix the problem by the following two steps, but I'd<br>
>     appreciate<br>
>     if anybody can share a better idea.<br>
> <br>
>       1. Update tox.ini to honor upper-constraints.txt. This is needed<br>
>     in master and<br>
>           stable/branches as well.<br>
> <br>
>       2. Update docs/source/conf.py to hard-code the language parameter<br>
>     to 'en'.<br>
>           This is required in master so that we can bump the upper<br>
>     version later.<br>
> <br>
>     Thank you,<br>
>     Takashi Kajinami<br>
> <br>
<br>
<br>
</blockquote></div>
</blockquote></div>