)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"0d62017e348f91583fa2e9099ad7cbe543e9a051","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"6a4eabfd_960ce35e","updated":"2023-10-27 16:09:40.000000000","message":"Just for reference mail thread for this:\nhttps://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/7INVOBFU62CO2ELGNQGYY7DJ4F4UM4PX/","commit_id":"842f2f8e6cbd39c4eefddc9e1b74c0f7f8ea55ff"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"56cf79114404c30e5ed187ef2999a4e012d596ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"c5c62662_e9eea232","updated":"2023-10-27 16:06:43.000000000","message":"good clarification, thanks","commit_id":"842f2f8e6cbd39c4eefddc9e1b74c0f7f8ea55ff"}],"doc/source/install/compute-install-option1-obs.rst":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"22d63f2e103f2c04fe550c3d23ceba88c1b234e5","unresolved":true,"context_lines":[{"line_number":13,"context_line":"  complete the following actions:"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"  * In the ``[ovs]`` section, map the provider virtual network to the"},{"line_number":16,"context_line":"    provider physical network interface:"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"    .. path /etc/neutron/plugins/ml2/openvswitch_agent.ini"},{"line_number":19,"context_line":"    .. code-block:: ini"}],"source_content_type":"text/x-rst","patch_set":2,"id":"b62a7e63_3db26852","line":16,"range":{"start_line":16,"start_character":13,"end_line":16,"end_character":29},"updated":"2023-10-26 10:33:41.000000000","message":"This should also be \"bridge\" then, right?","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"fa89edf5e7af959e3d3c0f274b961f3a28d59642","unresolved":false,"context_lines":[{"line_number":13,"context_line":"  complete the following actions:"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"  * In the ``[ovs]`` section, map the provider virtual network to the"},{"line_number":16,"context_line":"    provider physical network interface:"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"    .. path /etc/neutron/plugins/ml2/openvswitch_agent.ini"},{"line_number":19,"context_line":"    .. code-block:: ini"}],"source_content_type":"text/x-rst","patch_set":2,"id":"11c94983_de77ee63","line":16,"range":{"start_line":16,"start_character":13,"end_line":16,"end_character":29},"in_reply_to":"b62a7e63_3db26852","updated":"2023-10-26 11:05:14.000000000","message":"Done","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"22d63f2e103f2c04fe550c3d23ceba88c1b234e5","unresolved":true,"context_lines":[{"line_number":24,"context_line":"    .. end"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the underlying"},{"line_number":27,"context_line":"    provider physical network interface. See :doc:`environment-networking-obs`"},{"line_number":28,"context_line":"    and :doc:`../admin/deploy-ovs-provider` for more information."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  * In the ``[securitygroup]`` section, enable security groups and"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7257deb1_67681f5e","line":27,"range":{"start_line":27,"start_character":13,"end_line":27,"end_character":29},"updated":"2023-10-26 10:33:41.000000000","message":"and this?","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"fa89edf5e7af959e3d3c0f274b961f3a28d59642","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    .. end"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the underlying"},{"line_number":27,"context_line":"    provider physical network interface. See :doc:`environment-networking-obs`"},{"line_number":28,"context_line":"    and :doc:`../admin/deploy-ovs-provider` for more information."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  * In the ``[securitygroup]`` section, enable security groups and"}],"source_content_type":"text/x-rst","patch_set":2,"id":"71e7bdeb_4dd062c7","line":27,"range":{"start_line":27,"start_character":13,"end_line":27,"end_character":29},"in_reply_to":"7257deb1_67681f5e","updated":"2023-10-26 11:05:14.000000000","message":"Done","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"22d63f2e103f2c04fe550c3d23ceba88c1b234e5","unresolved":true,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the underlying"},{"line_number":27,"context_line":"    provider physical network interface. See :doc:`environment-networking-obs`"},{"line_number":28,"context_line":"    and :doc:`../admin/deploy-ovs-provider` for more information."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  * In the ``[securitygroup]`` section, enable security groups and"},{"line_number":31,"context_line":"    configure the Open vSwitch native or the hybrid iptables firewall driver:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"54049f58_94bce299","line":28,"updated":"2023-10-26 10:33:41.000000000","message":"Since this is expected to be a complete step-by-step guide, I think you should explicitly add the `add-br` and `add-port` commands here.","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"fa89edf5e7af959e3d3c0f274b961f3a28d59642","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the underlying"},{"line_number":27,"context_line":"    provider physical network interface. See :doc:`environment-networking-obs`"},{"line_number":28,"context_line":"    and :doc:`../admin/deploy-ovs-provider` for more information."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  * In the ``[securitygroup]`` section, enable security groups and"},{"line_number":31,"context_line":"    configure the Open vSwitch native or the hybrid iptables firewall driver:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"78c01ea5_3f76d898","line":28,"in_reply_to":"54049f58_94bce299","updated":"2023-10-26 11:05:14.000000000","message":"This guide is just about configuration of the neutron agent. Info about creation of bridges and ports is already in the ../admin/deploy-ovs-provider doc which is linked here.","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"3871aad4fd976ff643a67eb5e2feac8b1ef8fb66","unresolved":true,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the underlying"},{"line_number":27,"context_line":"    provider physical network interface. See :doc:`environment-networking-obs`"},{"line_number":28,"context_line":"    and :doc:`../admin/deploy-ovs-provider` for more information."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  * In the ``[securitygroup]`` section, enable security groups and"},{"line_number":31,"context_line":"    configure the Open vSwitch native or the hybrid iptables firewall driver:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"b4d6721f_9d748c3a","line":28,"in_reply_to":"78c01ea5_3f76d898","updated":"2023-10-26 11:10:12.000000000","message":"IMO the install guide is a dedicated document that should be able to be used standalone. You can link to the admin guide for reference or more details, but it should be possible to create a working deployment by just following the install guide step by step.","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"36b765825d2c42b88b32594530abd2c334503d63","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the underlying"},{"line_number":27,"context_line":"    provider physical network interface. See :doc:`environment-networking-obs`"},{"line_number":28,"context_line":"    and :doc:`../admin/deploy-ovs-provider` for more information."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"  * In the ``[securitygroup]`` section, enable security groups and"},{"line_number":31,"context_line":"    configure the Open vSwitch native or the hybrid iptables firewall driver:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"5adab417_b5f23f34","line":28,"in_reply_to":"b4d6721f_9d748c3a","updated":"2023-10-27 14:08:54.000000000","message":"Done","commit_id":"54b59dfca008c3846a0b6a4f4d9118dac3cf2e45"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"d18b0b0b668f7883aa4aeff2297cfdad177d1efb","unresolved":true,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    .. code-block:: bash"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"       $ ovs-vsctl add-br $PROVIDER_BRIDGE_NAME"},{"line_number":37,"context_line":"       $ ovs-vsctl add-port $PROVIDER_BRIDGE_NAME $PROVIDER_INTERFACE_NAME"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    .. end"}],"source_content_type":"text/x-rst","patch_set":5,"id":"85cdb8b8_525d7526","line":36,"range":{"start_line":36,"start_character":7,"end_line":36,"end_character":8},"updated":"2023-10-27 15:39:07.000000000","message":"would be good to change it to the root prompt \u0027#\u0027 like is done at other places.","commit_id":"a45b7026d1a57b4e6c6b1bcbe78d82c62e56c79e"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"a51cab0c03a2e8dc27847e702b3259c70df33126","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    .. code-block:: bash"},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"       $ ovs-vsctl add-br $PROVIDER_BRIDGE_NAME"},{"line_number":37,"context_line":"       $ ovs-vsctl add-port $PROVIDER_BRIDGE_NAME $PROVIDER_INTERFACE_NAME"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"    .. end"}],"source_content_type":"text/x-rst","patch_set":5,"id":"a54ec344_3d5264ec","line":36,"range":{"start_line":36,"start_character":7,"end_line":36,"end_character":8},"in_reply_to":"85cdb8b8_525d7526","updated":"2023-10-27 15:57:55.000000000","message":"Done","commit_id":"a45b7026d1a57b4e6c6b1bcbe78d82c62e56c79e"}],"doc/source/install/compute-install-option1-rdo.rst":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"ab6ee194d7c6498be4340d991dcddefb6841fef7","unresolved":true,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    .. end"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the bridge connected to"},{"line_number":27,"context_line":"    the underlying provider physical bridge."},{"line_number":28,"context_line":"    See :doc:`environment-networking-rdo` and :doc:`../admin/deploy-ovs-provider`"},{"line_number":29,"context_line":"    for more information."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"fd3c84c8_a4243791","line":27,"range":{"start_line":26,"start_character":58,"end_line":27,"end_character":43},"updated":"2023-10-27 14:22:55.000000000","message":"The only thing I noticed is that this sentence is different from all the other files, the others say:\n\n\"... name of the underlying provider physical bridge.\"\n\nI think we should make them all the same.","commit_id":"57db4d97dceef8fc606d8dc4ab6a82b6e972b656"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"07709ef50bfad1c322b0c30a4f49b5fdb0f8ff5a","unresolved":false,"context_lines":[{"line_number":23,"context_line":""},{"line_number":24,"context_line":"    .. end"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    Replace ``PROVIDER_BRIDGE_NAME`` with the name of the bridge connected to"},{"line_number":27,"context_line":"    the underlying provider physical bridge."},{"line_number":28,"context_line":"    See :doc:`environment-networking-rdo` and :doc:`../admin/deploy-ovs-provider`"},{"line_number":29,"context_line":"    for more information."},{"line_number":30,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"2f01c61e_c82ac5f9","line":27,"range":{"start_line":26,"start_character":58,"end_line":27,"end_character":43},"in_reply_to":"fd3c84c8_a4243791","updated":"2023-10-27 14:31:44.000000000","message":"Done","commit_id":"57db4d97dceef8fc606d8dc4ab6a82b6e972b656"}]}
