)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"57133bad69cd00269a54b009eda46974bdd2a586","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Enhance nova-specs repo and webpage"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- Change from oslosphinx to openstackdocstheme"},{"line_number":10,"context_line":"- Switch to new PTI requirement for building docs"},{"line_number":11,"context_line":"- Remove \"Template\" section from index pages"},{"line_number":12,"context_line":"- Clean up"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Icc65afd5f6c0e22147b179c16d6a4fa5f47954ce"},{"line_number":15,"context_line":"Closes-Bug: #1755000"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"df7087c5_76a8248b","line":12,"range":{"start_line":9,"start_character":0,"end_line":12,"end_character":10},"updated":"2018-03-14 11:16:40.000000000","message":"I meant to split this into multiple commits, each focusing on another part. Maybe that\u0027s not necessary though. I\u0027ll let someone else weigh in","commit_id":"24fb925c8187035ba58fa0f1d29839355a48e8e1"}],"doc/requirements.txt":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"57133bad69cd00269a54b009eda46974bdd2a586","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"sphinx\u003e\u003d1.6.2 # BSD"},{"line_number":5,"context_line":"actdiag"},{"line_number":6,"context_line":"blockdiag"},{"line_number":7,"context_line":"nwdiag"},{"line_number":8,"context_line":"openstackdocstheme\u003e\u003d1.17.0 # Apache-2.0"},{"line_number":9,"context_line":"seqdiag"},{"line_number":10,"context_line":"sphinxcontrib-actdiag"}],"source_content_type":"text/plain","patch_set":2,"id":"df7087c5_b6bd5cd9","line":7,"range":{"start_line":5,"start_character":0,"end_line":7,"end_character":6},"updated":"2018-03-14 11:16:40.000000000","message":"Are these needed? I figured the sphinxcontrib-* packages would have installed them (or vice versa). Can you check?","commit_id":"24fb925c8187035ba58fa0f1d29839355a48e8e1"},{"author":{"_account_id":27153,"name":"Nguyen Hai","email":"nguyentrihai93@gmail.com","username":"nguyentrihai"},"change_message_id":"44774d941b3b1b0ca02885db772a1cdafa432bbc","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"sphinx\u003e\u003d1.6.2 # BSD"},{"line_number":5,"context_line":"actdiag"},{"line_number":6,"context_line":"blockdiag"},{"line_number":7,"context_line":"nwdiag"},{"line_number":8,"context_line":"openstackdocstheme\u003e\u003d1.17.0 # Apache-2.0"},{"line_number":9,"context_line":"seqdiag"},{"line_number":10,"context_line":"sphinxcontrib-actdiag"}],"source_content_type":"text/plain","patch_set":2,"id":"df7087c5_9c807748","line":7,"range":{"start_line":5,"start_character":0,"end_line":7,"end_character":6},"in_reply_to":"df7087c5_b6bd5cd9","updated":"2018-03-14 12:35:01.000000000","message":"We don\u0027t need them. I delete it.","commit_id":"24fb925c8187035ba58fa0f1d29839355a48e8e1"},{"author":{"_account_id":17190,"name":"Arvind Nadendla","email":"arvindn05@gmail.com","username":"arvindn05"},"change_message_id":"31e72d41ee8db944e854529d632e603108570201","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"sphinx\u003e\u003d1.6.2 # BSD"},{"line_number":5,"context_line":"openstackdocstheme\u003e\u003d1.17.0 # Apache-2.0"},{"line_number":6,"context_line":"sphinxcontrib-actdiag"},{"line_number":7,"context_line":"sphinxcontrib-blockdiag # BSD"},{"line_number":8,"context_line":"sphinxcontrib-nwdiag"},{"line_number":9,"context_line":"sphinxcontrib-seqdiag # BSD"}],"source_content_type":"text/plain","patch_set":8,"id":"bf659307_9b584b31","line":6,"updated":"2018-04-03 16:43:57.000000000","message":"should we specify license BSD for the actdiag and nwdiag packages as well?","commit_id":"d53adab76641bd5cf39e7493fdf2e1a0f4337939"},{"author":{"_account_id":27153,"name":"Nguyen Hai","email":"nguyentrihai93@gmail.com","username":"nguyentrihai"},"change_message_id":"35643a85d3e3a31ca79364c16abde11a8eec6a8b","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"sphinx\u003e\u003d1.6.2 # BSD"},{"line_number":5,"context_line":"openstackdocstheme\u003e\u003d1.17.0 # Apache-2.0"},{"line_number":6,"context_line":"sphinxcontrib-actdiag"},{"line_number":7,"context_line":"sphinxcontrib-blockdiag # BSD"},{"line_number":8,"context_line":"sphinxcontrib-nwdiag"},{"line_number":9,"context_line":"sphinxcontrib-seqdiag # BSD"}],"source_content_type":"text/plain","patch_set":8,"id":"bf659307_a601766c","line":6,"in_reply_to":"bf659307_9b584b31","updated":"2018-04-03 17:10:42.000000000","message":"Done","commit_id":"d53adab76641bd5cf39e7493fdf2e1a0f4337939"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8595adf36ac6051a0688aa5a757e9ce02a8c0cc1","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"oslosphinx\u003e\u003d4.7.0 # Apache-2.0"},{"line_number":5,"context_line":"sphinx\u003e\u003d1.6.2 # BSD"},{"line_number":6,"context_line":"yasfb\u003e\u003d0.5.1,!\u003d0.6.0"}],"source_content_type":"text/plain","patch_set":30,"id":"5f7c97a3_591e95bb","line":5,"updated":"2018-05-26 15:23:54.000000000","message":"Looks like there are a couple of sphinx versions that are blacklisted in g-r:\n\nhttps://github.com/openstack/requirements/blob/master/global-requirements.txt#L428\n\nShouldn\u0027t we also blacklist those here?\n\nBased on lower-constraints, it looks like we should depend on a minimum of 1.6.5:\n\nhttps://github.com/openstack/requirements/blob/master/lower-constraints.txt#L361\n\nWhere did the 1.6.2 minimum come from?","commit_id":"d2e6d73558e8e2cc34ddea8442e1bc0c5d4a31bf"}],"doc/source/conf.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5c54e2fe1031de6108b4bb8b7ed6b22843d407af","unresolved":false,"context_lines":[{"line_number":81,"context_line":"# Output file base name for HTML help builder."},{"line_number":82,"context_line":"htmlhelp_basename \u003d \u0027Nova-Specsdoc\u0027"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"# -- Options for Texinfo output ------------------------------------------------"},{"line_number":85,"context_line":""},{"line_number":86,"context_line":"# Grouping the document tree into Texinfo files. List of tuples"},{"line_number":87,"context_line":"# (source start file, target name, title, author,"},{"line_number":88,"context_line":"#  dir menu entry, description, category)"},{"line_number":89,"context_line":"texinfo_documents \u003d ["},{"line_number":90,"context_line":"  (\u0027index\u0027, \u0027Nova-specs\u0027, u\u0027Nova Specs\u0027,"},{"line_number":91,"context_line":"   u\u0027OpenStack Nova Team\u0027, \u0027nova-specs\u0027, \u0027Design specifications for the Nova project.\u0027,"},{"line_number":92,"context_line":"   \u0027Miscellaneous\u0027),"},{"line_number":93,"context_line":"]"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"# Documents to append as an appendix to all manuals."},{"line_number":96,"context_line":"#texinfo_appendices \u003d []"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"# If false, no module index is generated."},{"line_number":99,"context_line":"#texinfo_domain_indices \u003d True"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":"# How to display URL addresses: \u0027footnote\u0027, \u0027no\u0027, or \u0027inline\u0027."},{"line_number":102,"context_line":"#texinfo_show_urls \u003d \u0027footnote\u0027"},{"line_number":103,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"df7087c5_fad3d7d3","line":103,"range":{"start_line":84,"start_character":0,"end_line":103,"end_character":0},"updated":"2018-03-13 11:31:29.000000000","message":"We don\u0027t generate texinfo output so I\u0027m not sure why this is here","commit_id":"65ce0f7b67f646e07fae64bdd5c82588c572d5a3"}],"requirements.txt":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5c54e2fe1031de6108b4bb8b7ed6b22843d407af","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"pbr\u003e\u003d2.0.0,!\u003d2.1.0 # Apache-2.0"},{"line_number":5,"context_line":"testrepository\u003e\u003d0.0.18 # Apache-2.0/BSD"},{"line_number":6,"context_line":"testtools\u003e\u003d1.4.0 # MIT"},{"line_number":7,"context_line":"sphinx\u003e\u003d1.6.2 # BSD"}],"source_content_type":"text/plain","patch_set":1,"id":"df7087c5_7af82748","line":4,"range":{"start_line":4,"start_character":0,"end_line":4,"end_character":31},"updated":"2018-03-13 11:31:29.000000000","message":"We shouldn\u0027t be including this here (at least not in a versioned form)","commit_id":"65ce0f7b67f646e07fae64bdd5c82588c572d5a3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5c54e2fe1031de6108b4bb8b7ed6b22843d407af","unresolved":false,"context_lines":[{"line_number":4,"context_line":"pbr\u003e\u003d2.0.0,!\u003d2.1.0 # Apache-2.0"},{"line_number":5,"context_line":"testrepository\u003e\u003d0.0.18 # Apache-2.0/BSD"},{"line_number":6,"context_line":"testtools\u003e\u003d1.4.0 # MIT"},{"line_number":7,"context_line":"sphinx\u003e\u003d1.6.2 # BSD"},{"line_number":8,"context_line":"actdiag"},{"line_number":9,"context_line":"blockdiag"},{"line_number":10,"context_line":"nwdiag"},{"line_number":11,"context_line":"openstackdocstheme\u003e\u003d1.17.0 # Apache-2.0"},{"line_number":12,"context_line":"seqdiag"},{"line_number":13,"context_line":"sphinxcontrib-actdiag"},{"line_number":14,"context_line":"sphinxcontrib-blockdiag # BSD"},{"line_number":15,"context_line":"sphinxcontrib-nwdiag"},{"line_number":16,"context_line":"sphinxcontrib-seqdiag # BSD"}],"source_content_type":"text/plain","patch_set":1,"id":"df7087c5_3af70f62","line":16,"range":{"start_line":7,"start_character":0,"end_line":16,"end_character":27},"updated":"2018-03-13 11:31:29.000000000","message":"These should go into \u0027doc/requirements.txt\u0027 nowadays","commit_id":"65ce0f7b67f646e07fae64bdd5c82588c572d5a3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"72c6b9d4f093be4be4da6d4405da64abcd0cadeb","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"oslosphinx\u003e\u003d4.7.0 # Apache-2.0"},{"line_number":6,"context_line":"sphinx\u003e\u003d1.5.1,!\u003d1.6.1 # BSD"},{"line_number":7,"context_line":"testrepository\u003e\u003d0.0.18 # Apache-2.0/BSD"},{"line_number":8,"context_line":"testtools\u003e\u003d1.4.0 # MIT"},{"line_number":9,"context_line":"yasfb\u003e\u003d0.5.1,!\u003d0.6.0"}],"source_content_type":"text/plain","patch_set":16,"id":"9f6a8fd7_d4686888","side":"PARENT","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":22},"updated":"2018-04-18 13:18:02.000000000","message":"These should really be in testrequirements and this whole file removed, I\u0027d say","commit_id":"dad9782fc199a3181ef0d8b212f53f195c0ce657"}],"setup.cfg":[{"author":{"_account_id":27153,"name":"Nguyen Hai","email":"nguyentrihai93@gmail.com","username":"nguyentrihai"},"change_message_id":"ff4c6fcb7bf4f334a41831776339bc8fa1a1178f","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    License :: OSI Approved :: Apache Software License"},{"line_number":12,"context_line":"    Operating System :: POSIX :: Linux"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"[build_sphinx]"},{"line_number":15,"context_line":"builders \u003d html"},{"line_number":16,"context_line":"all_files \u003d 1"},{"line_number":17,"context_line":"build-dir \u003d doc/build"}],"source_content_type":"text/x-ttcn-cfg","patch_set":5,"id":"df7087c5_55d41fe9","side":"PARENT","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":14},"updated":"2018-03-22 17:16:39.000000000","message":"Remove the \u0027[build_sphinx]\u0027 section as described in:\nhttp://lists.openstack.org/pipermail/openstack-dev/2018-March/128594.html","commit_id":"3e21df3365910280d73651e790d023ae482c3cbd"}],"tox.ini":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"5c54e2fe1031de6108b4bb8b7ed6b22843d407af","unresolved":false,"context_lines":[{"line_number":1,"context_line":"[tox]"},{"line_number":2,"context_line":"# Hold back to 1.4, since that\u0027s what\u0027s in Fedora 20 repos"},{"line_number":3,"context_line":"# and we don\u0027t need anything newer for nova-specs tests"},{"line_number":4,"context_line":"minversion \u003d 2.0"},{"line_number":5,"context_line":"envlist \u003d docs,pep8"},{"line_number":6,"context_line":"skipsdist \u003d True"}],"source_content_type":"text/x-properties","patch_set":1,"id":"df7087c5_1a97abee","line":3,"range":{"start_line":2,"start_character":0,"end_line":3,"end_character":55},"updated":"2018-03-13 11:31:29.000000000","message":"This comment doesn\u0027t apply anymore","commit_id":"65ce0f7b67f646e07fae64bdd5c82588c572d5a3"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"72c6b9d4f093be4be4da6d4405da64abcd0cadeb","unresolved":false,"context_lines":[{"line_number":10,"context_line":"whitelist_externals \u003d find"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"[testenv:venv]"},{"line_number":13,"context_line":"commands \u003d {posargs}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[testenv:docs]"},{"line_number":16,"context_line":"deps \u003d -r{toxinidir}/doc/requirements.txt"}],"source_content_type":"text/x-properties","patch_set":16,"id":"9f6a8fd7_5412180b","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":20},"updated":"2018-04-18 13:18:02.000000000","message":"You need to add doc/requirements here, I\u0027d say (or do the below)","commit_id":"2eac3ef9f04a35fadb01168cf4e051f7db4d0cf8"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"72c6b9d4f093be4be4da6d4405da64abcd0cadeb","unresolved":false,"context_lines":[{"line_number":13,"context_line":"commands \u003d {posargs}"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"[testenv:docs]"},{"line_number":16,"context_line":"deps \u003d -r{toxinidir}/doc/requirements.txt"},{"line_number":17,"context_line":"commands \u003d sphinx-build -b html doc/source doc/build/html"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"[testenv:pep8]"}],"source_content_type":"text/x-properties","patch_set":16,"id":"9f6a8fd7_340f64e2","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":41},"updated":"2018-04-18 13:18:02.000000000","message":"I\u0027d move this to the top level \u0027[testenv] deps\u0027","commit_id":"2eac3ef9f04a35fadb01168cf4e051f7db4d0cf8"}]}