)]}'
{"doc/source/admin/deploy-lb.rst":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"8a3692e018979620c6bf6b70e0e4c27c5c9e7afe","unresolved":true,"context_lines":[{"line_number":23,"context_line":"errors. To use Linux Bridge mechanism driver in newer operating systems that"},{"line_number":24,"context_line":"use ``nftables`` by default, it is needed to switch back to the legacy tool:"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    .. code-block:: console"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"        $ /usr/bin/update-alternatives --set ebtables /usr/sbin/ebtables-legacy"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"2aed4eab_5088ceee","line":26,"updated":"2021-04-07 13:06:14.000000000","message":"I think this might be indented too much?  Resultant doc seems to have a grey bar in front of the console output.  Possibly just three spaces would be better?  I didn\u0027t run tox -e docs to check.\n\nhttps://d6719958a62f91f6448f-3b6731d84f2c2d3abd7d6f092a1e1022.ssl.cf1.rackcdn.com/785137/1/check/openstack-tox-docs/bf8bcf5/docs/admin/deploy-lb.html","commit_id":"8500b496c727e3b4f6c2e9c26295083371c6a1a1"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"9615151242133bc702032e45af336ae0ea65999e","unresolved":true,"context_lines":[{"line_number":23,"context_line":"errors. To use Linux Bridge mechanism driver in newer operating systems that"},{"line_number":24,"context_line":"use ``nftables`` by default, it is needed to switch back to the legacy tool:"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    .. code-block:: console"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"        $ /usr/bin/update-alternatives --set ebtables /usr/sbin/ebtables-legacy"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"87281cdf_0b82677d","line":26,"in_reply_to":"2aed4eab_5088ceee","updated":"2021-04-08 09:15:11.000000000","message":"Right. I think I don\u0027t need to indent the code block. Let me check that.","commit_id":"8500b496c727e3b4f6c2e9c26295083371c6a1a1"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"188ab1f7ae73e222bf7a6ae8924c9e6973a3f808","unresolved":true,"context_lines":[{"line_number":44,"context_line":"     transition to this new framework. Those binaries replicate the same"},{"line_number":45,"context_line":"     commands as the legacy one but using the new framework. The binaries"},{"line_number":46,"context_line":"     have the same name ended in ``-nft``."},{"line_number":47,"context_line":"   * The new ``ntftables`` framework using the new API. All Netfilter"},{"line_number":48,"context_line":"     operations are executed using this new API and one single binary, ``nft``."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"   Currently we support the first two options. The migration (total or partial)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"bb3d544f_f7f54f15","line":47,"range":{"start_line":47,"start_character":15,"end_line":47,"end_character":24},"updated":"2021-04-08 16:16:09.000000000","message":"s/nftables","commit_id":"7e4aa09c0051dd5dc9f4ea8c966f3ffac4b5b4b4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8a72df71d8e2033fe3221a2e12e44f4bb11ae125","unresolved":false,"context_lines":[{"line_number":44,"context_line":"     transition to this new framework. Those binaries replicate the same"},{"line_number":45,"context_line":"     commands as the legacy one but using the new framework. The binaries"},{"line_number":46,"context_line":"     have the same name ended in ``-nft``."},{"line_number":47,"context_line":"   * The new ``ntftables`` framework using the new API. All Netfilter"},{"line_number":48,"context_line":"     operations are executed using this new API and one single binary, ``nft``."},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"   Currently we support the first two options. The migration (total or partial)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"8c14bfa3_e13cc4e6","line":47,"range":{"start_line":47,"start_character":15,"end_line":47,"end_character":24},"in_reply_to":"bb3d544f_f7f54f15","updated":"2021-04-09 08:20:46.000000000","message":"Done","commit_id":"7e4aa09c0051dd5dc9f4ea8c966f3ffac4b5b4b4"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"188ab1f7ae73e222bf7a6ae8924c9e6973a3f808","unresolved":true,"context_lines":[{"line_number":66,"context_line":"The ``ipset`` tool is not compatible with ``nftables``. To disable it,"},{"line_number":67,"context_line":"``enable_ipset`` must be set to ``False``."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":".. path /etc/neutron/plugins/ml2/ml2_conf.ini"},{"line_number":70,"context_line":".. code-block:: ini"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"   [securitygroup]"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7563358e_14b2871a","line":69,"updated":"2021-04-08 16:16:09.000000000","message":"I don\u0027t see this info in the doc, so the user might not know the file to edit.","commit_id":"7e4aa09c0051dd5dc9f4ea8c966f3ffac4b5b4b4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c83196d3656f4f01a108fba0460d84a99570dc90","unresolved":true,"context_lines":[{"line_number":66,"context_line":"The ``ipset`` tool is not compatible with ``nftables``. To disable it,"},{"line_number":67,"context_line":"``enable_ipset`` must be set to ``False``."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":".. path /etc/neutron/plugins/ml2/ml2_conf.ini"},{"line_number":70,"context_line":".. code-block:: ini"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"   [securitygroup]"}],"source_content_type":"text/x-rst","patch_set":3,"id":"f3935e71_44b18ac7","line":69,"in_reply_to":"7563358e_14b2871a","updated":"2021-04-09 07:51:54.000000000","message":"Strangely this \".. path \" is not visible in the rendered version, see this one for example: https://opendev.org/openstack/neutron/raw/branch/master/doc/source/install/controller-install-option2-ubuntu.rst\n\nI tried with .. file but that is the same (not rendered)","commit_id":"7e4aa09c0051dd5dc9f4ea8c966f3ffac4b5b4b4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8a72df71d8e2033fe3221a2e12e44f4bb11ae125","unresolved":true,"context_lines":[{"line_number":66,"context_line":"The ``ipset`` tool is not compatible with ``nftables``. To disable it,"},{"line_number":67,"context_line":"``enable_ipset`` must be set to ``False``."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":".. path /etc/neutron/plugins/ml2/ml2_conf.ini"},{"line_number":70,"context_line":".. code-block:: ini"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"   [securitygroup]"}],"source_content_type":"text/x-rst","patch_set":3,"id":"4e4a1369_620857cd","line":69,"in_reply_to":"f3935e71_44b18ac7","updated":"2021-04-09 08:20:46.000000000","message":"Actually I don\u0027t see any reference to this \"path\" directive in the rst documentation.\n\nI\u0027ll add the name in the doc text (same as in other documents) and I\u0027ll keep this no-op reference there.","commit_id":"7e4aa09c0051dd5dc9f4ea8c966f3ffac4b5b4b4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8a505e7fd1f8c9100671444cfe008ae522e9372b","unresolved":true,"context_lines":[{"line_number":72,"context_line":"   [securitygroup]"},{"line_number":73,"context_line":"   # ..."},{"line_number":74,"context_line":"   enable_ipset \u003d False"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":".. toctree::"},{"line_number":78,"context_line":"   :maxdepth: 2"}],"source_content_type":"text/x-rst","patch_set":3,"id":"f7a1c787_c5325858","line":75,"updated":"2021-04-08 13:18:19.000000000","message":"nit: .. end\nto tell the truth I can\u0027t see the difference in the rendered html, but seen on other pages","commit_id":"7e4aa09c0051dd5dc9f4ea8c966f3ffac4b5b4b4"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"8a72df71d8e2033fe3221a2e12e44f4bb11ae125","unresolved":true,"context_lines":[{"line_number":72,"context_line":"   [securitygroup]"},{"line_number":73,"context_line":"   # ..."},{"line_number":74,"context_line":"   enable_ipset \u003d False"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":".. toctree::"},{"line_number":78,"context_line":"   :maxdepth: 2"}],"source_content_type":"text/x-rst","patch_set":3,"id":"2fc2aca1_6d30cd8b","line":75,"in_reply_to":"f7a1c787_c5325858","updated":"2021-04-09 08:20:46.000000000","message":"I\u0027ve seen it too, but there is not reference to this directive in the documentation.\n\nhttps://developer.lsst.io/v/gpdf-patch-2/docs/rst_styleguide.html#source-code","commit_id":"7e4aa09c0051dd5dc9f4ea8c966f3ffac4b5b4b4"}]}