Open Stack

2017-11-16 19:33 GMT+09:00 Andreas Jaeger <aj at suse.com>:
> On 2017-11-16 11:27, Akihiro Motoki wrote:
>> 2017-11-16 18:59 GMT+09:00 Andreas Jaeger <aj at suse.com>:
>>> On 2017-11-15 19:07, Doug Hellmann wrote:
>>>> [...]
>>>> Someone needs to figure that out. Unfortunately, I don't have the
>>>> bandwidth right now. Maybe you, or someone else from one of the affected projects, can work on it?
>>>>
>>>> Two options have been discussed so far:
>>>>
>>>> 1. Create special jobs for neutron and horizon projects that install
>>>>    neutron or horizon, like we do for the release jobs.
>>>>
>>>> 2. Redefine the release notes job so that it doesn't use tox
>>>>    but installs only the pieces it needs to run a sphinx build. The
>>>>    CTI is already defined in a way to support that [1].
>>>>
>>>> My current preference is for option 2. There may be other options
>>>> that we haven't explored, though.
>>> I gave it a try in https://review.openstack.org/520362. Note that this
>>> is only the building part, publishing part is separate - let's first
>>> iterate on this one,
>> I noticed most projects require to install themselves now to set
>> 'version' and 'release' variables in releasenotes/source/conf.py.
>> For example, http://git.openstack.org/cgit/openstack/nova/tree/releasenotes/source/conf.py#n51
>> I think we cannot build release notes without installing a project itself.
>
> That's really sad ;(
>
> But why would we need to set version and release for the releasenotes at
> all? We only have one - unversioned - document.
>
> I suggest to just get rid of it,

Totally agree.

I think when we migrated to openstckdocstheme version info was shown
in the rendered text
so we started to use version info in conf.py. Some projects started it
and most projects followed it.
After migrating openstckdocstheme there is no visible version info in
the rendered relnotes,
so I think we can drop it.

Looking at  the openstackdocstheme document [1], PDF generation
expects 'release' variable
but AFAIK we only generate HTML version of release notes and I think
we can drop 'release'
and 'version' variables from conf.py.

[1] https://docs.openstack.org/openstackdocstheme/latest/