)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"316e26545ecad8360a2075137051bbae6c60988a","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Docs build: Add requirements.txt to deps"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Since we use usedevelop\u003dTrue, as a result neutron"},{"line_number":10,"context_line":"installation happens at \u0027develop-inst\u0027 phase and it installs"},{"line_number":11,"context_line":"requirements from requirements.txt without considering"},{"line_number":12,"context_line":"tox constraints, This can lead to issue where broken"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"86bc94d9_355b8164","line":9,"range":{"start_line":9,"start_character":13,"end_line":9,"end_character":23},"updated":"2022-04-28 15:00:37.000000000","message":"yeah, makes sense according to https://tox.wiki/en/latest/config.html","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"6f1a1b5f1137d1e5d940441f32cfd61f6aa8fc06","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cb13681e_71f9fe3a","updated":"2022-04-28 15:14:20.000000000","message":"This patch looks correct. It turned out with a release that introduced some fault for docs generation that on stable branches that package is not constrained, due to the reason described in commit message. So if we install neutron\u0027s requirements during installdeps phase then this failure will not happen.\n\nNote: this needs to be backported to stable branches.","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"61c4c4345c76348d5e52e55392c2dadeb396e17f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"46f6c3f9_c8e24f16","updated":"2022-04-29 02:44:49.000000000","message":"recheck functional","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"4a19ca8958ab61cbb88af6104668b052ac21dcda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"aa8c0cb2_5c1851c4","updated":"2022-04-28 23:19:29.000000000","message":"recheck functional timeout","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"7dde0bd60f71e36e2b4107476b2c82f0b1f35d02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"73bbcbb6_13a0b1f5","updated":"2022-04-28 15:28:26.000000000","message":"thanks","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"}],"tox.ini":[{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"382913651065fc93f88feeccd7fcd0f4aa13bd53","unresolved":true,"context_lines":[{"line_number":142,"context_line":"deps \u003d"},{"line_number":143,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":144,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":145,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":146,"context_line":"commands \u003d sphinx-build -W -b html doc/source doc/build/html"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"[testenv:pdf-docs]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"9e20e629_9b16a582","line":145,"range":{"start_line":145,"start_character":2,"end_line":145,"end_character":32},"updated":"2022-04-28 15:23:48.000000000","message":"We had a similar issue with neutron-lib fixed in  Iea61238f37fdf24c0264f96d104ee0b3b6aec8e2 (but it was dropped few months later in  I207be0b861c13bc6cf954b2247bb00546990bbf0)","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"9bdf3e4c6bb6340c3af12963ece719cbd0bd59c4","unresolved":true,"context_lines":[{"line_number":142,"context_line":"deps \u003d"},{"line_number":143,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":144,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":145,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":146,"context_line":"commands \u003d sphinx-build -W -b html doc/source doc/build/html"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"[testenv:pdf-docs]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"8ac45d5c_714b87af","line":145,"range":{"start_line":145,"start_character":2,"end_line":145,"end_character":32},"in_reply_to":"0a2b9e0b_a2e0d695","updated":"2022-04-29 06:49:06.000000000","message":"Ok pushed https://review.opendev.org/c/openstack/neutron/+/839859 to add NOTE, will squash it with other while backport.","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"4a19ca8958ab61cbb88af6104668b052ac21dcda","unresolved":true,"context_lines":[{"line_number":142,"context_line":"deps \u003d"},{"line_number":143,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":144,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":145,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":146,"context_line":"commands \u003d sphinx-build -W -b html doc/source doc/build/html"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"[testenv:pdf-docs]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"0a2b9e0b_a2e0d695","line":145,"range":{"start_line":145,"start_character":2,"end_line":145,"end_character":32},"in_reply_to":"54fb05ca_2c67cd92","updated":"2022-04-28 23:19:29.000000000","message":"Yes, let\u0027s unblock the gates but it sounds worth adding a \"please do not touch\" heads-up comment!","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"2522c8954c34a2d19fddf73c075c3053b911ce6a","unresolved":true,"context_lines":[{"line_number":142,"context_line":"deps \u003d"},{"line_number":143,"context_line":"  -c{env:TOX_CONSTRAINTS_FILE:https://releases.openstack.org/constraints/upper/master}"},{"line_number":144,"context_line":"  -r{toxinidir}/doc/requirements.txt"},{"line_number":145,"context_line":"  -r{toxinidir}/requirements.txt"},{"line_number":146,"context_line":"commands \u003d sphinx-build -W -b html doc/source doc/build/html"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"[testenv:pdf-docs]"}],"source_content_type":"text/x-properties","patch_set":1,"id":"54fb05ca_2c67cd92","line":145,"range":{"start_line":145,"start_character":2,"end_line":145,"end_character":32},"in_reply_to":"9e20e629_9b16a582","updated":"2022-04-28 15:36:01.000000000","message":"oh, that is fun :) maybe a comment would be good to avoid re-iterate this in the future :]","commit_id":"f15baf16dd18969efb94719798f5d4a731b0c4fd"}]}
