pbr incompatible with sphinx 1.3b1

Added pbr since it looks like we need to change how we call sphinx.

In https://review.openstack.org/127754 I fix the install_command in the oslotest directory.

https://review.openstack.org/#/c/127712/ pins the requirement for now.

Was this actually impacting other projects, or just oslotest?

According to logstash data this error was only ever encountered twice in the past week, both times on oslotest change 120316, roughly 19 hours apart. http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwibGluZSA3MDUsIGluIF9zcGhpbnhfcnVuXCIgQU5EIHRhZ3M6XCJjb25zb2xlXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjYwNDgwMCIsImdyYXBobW9kZSI6ImNvdW50IiwidGltZSI6eyJ1c2VyX2ludGVydmFsIjowfSwic3RhbXAiOjE0MTMwNDA5MzE5Mjd9

With a current version of tox, running 'tox -e pep8' in an up to date checkout of openstacl/oslotest's master branch reveals that cmdargs did 'pip install --pre -r requirements.txt -r test-requirements.txt' which would explain why oslotest tox runs are ending up with a beta of Sphinx (normally we don't use --pre). This seems to be because oslosphinx doesn't override install_command in its tox.ini.

I missed that my https://review.openstack.org/127794 change is actually the same as the one Doug already proposed in https://review.openstack.org/127754 and so have abandoned it.

I've proposed an optional revert of the unnecessary openstack/requirements change in https://review.openstack.org/127803 but testing indicates that at least pip does not factor != prerelease versions into its determination of whether it should consider prereleases sufficient to satisfy positive inclusive requirements, so while unnecessary the requirements pin is at least safe. As it's already merged and caused requirements update proposals to go out to all projects, I'm abandoning the revert in an attempt not to generate further unnecessary churn on those reviews.

Fix proposed to branch: master
Review: https://review.openstack.org/187672

Reviewed: https://review.openstack.org/187672
Committed: https://git.openstack.org/cgit/openstack-dev/pbr/commit/?id=1dfe9ef348c777bef67b2c2b1d35e50ebc720333
Submitter: Jenkins
Branch: master

commit 1dfe9ef348c777bef67b2c2b1d35e50ebc720333
Author: Thomas Bechtold <email address hidden>
Date: Tue Jun 2 18:40:38 2015 +0200

    Remove sphinx_config.init_values() manual call

    The function signature for Sphinx's config.init_values() changed since
    version 1.3. That leads to:

    TypeError: init_values() takes exactly 2 arguments (1 given)

    But the function is already correctly called from
    application.Sphinx()'s constructor. So use the configuration object
    from the application instead of creating an own config object.

    Change-Id: I343c26560bfe5116d5348b50b1890b3442ed845a
    Closes-Bug: #1379998

This commit was reverted and is still in review. See https://review.openstack.org/#/c/193462/ . Please set the status back to "In Progress".

Changed pbr status to confirmed.

This is also blocking compilation of the virtualenvwrapper package on, at least, FreeBSD:

22:06:15 (504) % sudo make
===> Building for py27-virtualenvwrapper-4.3.2
running build_sphinx
[pbr] Generating AUTHORS
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "setup.py", line 7, in <module>
    pbr=True,
  File "/usr/local/lib/python2.7/distutils/core.py", line 151, in setup
    dist.run_commands()
  File "/usr/local/lib/python2.7/distutils/dist.py", line 953, in run_commands
    self.run_command(cmd)
  File "/usr/local/lib/python2.7/distutils/dist.py", line 972, in run_command
    cmd_obj.run()
  File "/usr/local/lib/python2.7/site-packages/pbr/builddoc.py", line 189, in run
    self._sphinx_run()
  File "/usr/local/lib/python2.7/site-packages/pbr/builddoc.py", line 133, in _sphinx_run
    sphinx_config.init_values()
TypeError: init_values() takes exactly 2 arguments (1 given)
*** [do-build] Error code 1

Stop in /usr/ports/devel/py-virtualenvwrapper.
*** [stage] Error code 1

Stop in /usr/ports/devel/py-virtualenvwrapper.

Change abandoned by Thomas Bechtold (<email address hidden>) on branch: master
Review: https://review.openstack.org/193462
Reason: Fixed by http://git.openstack.org/cgit/openstack-dev/pbr/commit/?id=e19572e2d10107ad316d81f90a4310268faac387 .

But without tests and warnings disabled. Anyway - at least newer Sphinx versions are working now.

Fix proposed to branch: master
Review: https://review.openstack.org/243065

Reviewed: https://review.openstack.org/243065
Committed: https://git.openstack.org/cgit/openstack-dev/pbr/commit/?id=a5d46d5ce7b690997fee7601d459231e05b965dd
Submitter: Jenkins
Branch: master

commit a5d46d5ce7b690997fee7601d459231e05b965dd
Author: Ian Y. Choi <email address hidden>
Date: Mon Nov 9 21:14:47 2015 +0900

    More support Sphinx >=1.3b1 and <1.3.1

    Although https://review.openstack.org/#/c/171309/
    says Sphinx >=1.3, the actual implementation does not
    support Sphinx >=1.3b1 and 1.3.1.
    I have seen Sphinx release notes and source codes for
    those versions, and found that up to 1.2.3 used
    init_values(self) and Sphinx from 1.3b1 used
    init_values(self, warn).

    So to change the code from >=1.3.1 to >1.2.3, it can
    more support Sphinx >=1.3b1 and <1.3.1.

    Change-Id: Ia73be8111d3af7f72aaf640e4c7c519a5d0b08c5
    Partial-Bug: #1379998
    Related-Bug: #1496882

IMO, https://review.openstack.org/#/c/171309/ already fixed pbr incompatiblity with sphinx 1.3b1, and
the mentioned review https://review.openstack.org/#/c/193462/ was abandoned because of https://review.openstack.org/#/c/171309/ .

Since pbr 1.8.1 is already released and included that fix, I think this bug does not affect anymore.

Just note that my partial bug fix is for more support on Sphinx >=1.3b1 and <1.3.1.

Please change the bug status if those fixes did not solve this bug.

Related fix proposed to branch: master
Review: https://review.openstack.org/413795

Reviewed: https://review.openstack.org/413795
Committed: https://git.openstack.org/cgit/openstack-dev/pbr/commit/?id=9fd7aa2cc7fe50f68bd9c86c3db7a8f7ae710c05
Submitter: Jenkins
Branch: master

commit 9fd7aa2cc7fe50f68bd9c86c3db7a8f7ae710c05
Author: Dirk Mueller <email address hidden>
Date: Wed Dec 21 23:29:52 2016 +0100

    Don't raise exception on missing man pages

    The revert in Ia6cfbfe5b10a5b714fbb9f21ca61380aaf231638 actually
    broke Sphinx 1.3.x support again. Try to fix it for real this
    time by avoiding an exception on missing man_pages.

    NOTE(dmllr): don't change dict while iterating over it, hopefully
    this fixes the gating failure with python 3.5.x

    Change-Id: I52d45fa0a0d42de690d3a492068f7bb03483a224
    Related-Bug: 1379998