)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"44ab6b5c715959f3965c08d4759c4858756a2ac0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6725b16c_8cc4bccf","updated":"2022-08-01 07:13:36.000000000","message":"Please add \u0027DB Impact\u0027 and \u0027REST API Impact\u0027 sections.","commit_id":"5b80f9266879159b5b85d757993409402ff76b68"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"d024d8f5d9dece879de276287ce6611f37007d8e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5424d5af_10f39fb6","updated":"2022-08-02 13:04:52.000000000","message":"Oops! I\u0027ve corrected it to `firewall_groups` -- Thanks!","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"ee6ccb48b1c5f122bf7fe1fc9921bfc9ab7b3e4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"47046e30_7aa5519b","updated":"2022-08-23 13:23:14.000000000","message":"Addressing comments... Thanks!","commit_id":"1df76e6d9e4fb8e0104d964541f2544ffdf6308d"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"c5377ade864d7a89c7bc9c47dc77721feab509c9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"b48b266b_5596b0f2","updated":"2022-08-23 23:37:28.000000000","message":"Still you have one formatting issue. Other than that, it looks good to me","commit_id":"1df76e6d9e4fb8e0104d964541f2544ffdf6308d"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"264e8c3812bf64e300c4a15e3f0343baa7449014","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"b40407c2_66c8d709","updated":"2022-08-23 23:44:38.000000000","message":"Uploaded a change to the last table formatting","commit_id":"940bbd8c7db45054f9269bffc950527e557932a2"}],"specs/zed/fwaas-group-ordering.rst":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"44ab6b5c715959f3965c08d4759c4858756a2ac0","unresolved":true,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Similar to `firewall_policy_rule_associations_v2`, the"},{"line_number":27,"context_line":"`firewall_group_port_associations_v2` table should have a required"},{"line_number":28,"context_line":"`position` column to maintain the order in which `firewall groups` are"},{"line_number":29,"context_line":"applied to ports."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"In addition, modification of this ordering should be limited by user role."}],"source_content_type":"text/x-rst","patch_set":1,"id":"31fd9bf2_9e1e89d6","line":28,"range":{"start_line":28,"start_character":1,"end_line":28,"end_character":9},"updated":"2022-08-01 07:13:36.000000000","message":"Will it be required from user input, or auto incremented?","commit_id":"5b80f9266879159b5b85d757993409402ff76b68"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"c3a5ad7d9bed7087a3366eb4f9d4b32e8fbcb0d0","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Similar to `firewall_policy_rule_associations_v2`, the"},{"line_number":27,"context_line":"`firewall_group_port_associations_v2` table should have a required"},{"line_number":28,"context_line":"`position` column to maintain the order in which `firewall groups` are"},{"line_number":29,"context_line":"applied to ports."},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"In addition, modification of this ordering should be limited by user role."}],"source_content_type":"text/x-rst","patch_set":1,"id":"bce9f338_b03afda0","line":28,"range":{"start_line":28,"start_character":1,"end_line":28,"end_character":9},"in_reply_to":"31fd9bf2_9e1e89d6","updated":"2022-08-01 18:12:25.000000000","message":"Auto incremented by default, but can be specified via user input. I\u0027ve added this to the spec.","commit_id":"5b80f9266879159b5b85d757993409402ff76b68"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"8281b1b75def5bbecd2f2d23f2974e9da9b087b6","unresolved":true,"context_lines":[{"line_number":72,"context_line":"REST API Impact"},{"line_number":73,"context_line":"---------------"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"`PUT` and `POST` types for `/v2.0/fw/address_groups` will be updated to support the addition"},{"line_number":76,"context_line":"of `position` and `tier.`"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"1. Response bodies should include the new fields."}],"source_content_type":"text/x-rst","patch_set":5,"id":"43082c6d_8243a525","line":75,"range":{"start_line":75,"start_character":37,"end_line":75,"end_character":51},"updated":"2022-08-02 06:26:08.000000000","message":"not \u0027address_groups\u0027 I guess","commit_id":"699bcee7a06c62120f538bf0e820a2baf7555e44"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"08a3f3cf2a6b565abe757d55cb6b3c16cbcf81a3","unresolved":false,"context_lines":[{"line_number":72,"context_line":"REST API Impact"},{"line_number":73,"context_line":"---------------"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"`PUT` and `POST` types for `/v2.0/fw/address_groups` will be updated to support the addition"},{"line_number":76,"context_line":"of `position` and `tier.`"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"1. Response bodies should include the new fields."}],"source_content_type":"text/x-rst","patch_set":5,"id":"68b51bd7_38fe09b7","line":75,"range":{"start_line":75,"start_character":37,"end_line":75,"end_character":51},"in_reply_to":"43082c6d_8243a525","updated":"2022-08-02 13:05:36.000000000","message":"Fixed","commit_id":"699bcee7a06c62120f538bf0e820a2baf7555e44"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"841068ef331678dfc4c36156dd2a9439bab508a0","unresolved":true,"context_lines":[{"line_number":18,"context_line":"Problem Description"},{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Refer to the linked RFE."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Proposed Change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"72ab2342_1e405c9c","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":24},"updated":"2022-08-19 21:13:51.000000000","message":"The linked RFE contains a lot of stuff. Would you please state explicitly the problem this spec addresses?","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"ee6ccb48b1c5f122bf7fe1fc9921bfc9ab7b3e4c","unresolved":false,"context_lines":[{"line_number":18,"context_line":"Problem Description"},{"line_number":19,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"Refer to the linked RFE."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"Proposed Change"},{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":6,"id":"394af720_c14d2e77","line":21,"range":{"start_line":21,"start_character":0,"end_line":21,"end_character":24},"in_reply_to":"72ab2342_1e405c9c","updated":"2022-08-23 13:23:14.000000000","message":"I have added to the problem description section as requested.","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"841068ef331678dfc4c36156dd2a9439bab508a0","unresolved":true,"context_lines":[{"line_number":34,"context_line":"In iptables, this is typically referred to as `HEAD` and `TAIL` rules. All `HEAD`"},{"line_number":35,"context_line":"groups should be applied first, in order. All `TAIL` groups should be applied last,"},{"line_number":36,"context_line":"in order. All other groups would be applied in between, again, in order. Only"},{"line_number":37,"context_line":"openstack administrators should have access to the `HEAD` and `TAIL` tiers."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Ex."},{"line_number":40,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"f35b7c2d_4f6c0586","line":37,"range":{"start_line":37,"start_character":10,"end_line":37,"end_character":25},"updated":"2022-08-19 21:13:51.000000000","message":"Does this mean users with the \u0027admin\u0027 role? This aspect of OpenStack has become pretty complicated in recent cycles","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"ee6ccb48b1c5f122bf7fe1fc9921bfc9ab7b3e4c","unresolved":false,"context_lines":[{"line_number":34,"context_line":"In iptables, this is typically referred to as `HEAD` and `TAIL` rules. All `HEAD`"},{"line_number":35,"context_line":"groups should be applied first, in order. All `TAIL` groups should be applied last,"},{"line_number":36,"context_line":"in order. All other groups would be applied in between, again, in order. Only"},{"line_number":37,"context_line":"openstack administrators should have access to the `HEAD` and `TAIL` tiers."},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"Ex."},{"line_number":40,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"ea6cfe10_07db6875","line":37,"range":{"start_line":37,"start_character":10,"end_line":37,"end_character":25},"in_reply_to":"f35b7c2d_4f6c0586","updated":"2022-08-23 13:23:14.000000000","message":"Yes, I updated the text to better state this. I\u0027m open to suggestions here. The intention is that there can be firewall rules that cannot be modified/removed by a tenant. In other words, an Openstack administrator should be able to enforce a set of rules for all tenants, without fear that a tenant could circumvent that security.","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"841068ef331678dfc4c36156dd2a9439bab508a0","unresolved":true,"context_lines":[{"line_number":135,"context_line":"Existing entries should be assigned consecutive `position` numbers starting at 1, and the"},{"line_number":136,"context_line":"default `tier` value of `null.`"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"**Firewall Group Port associations**"},{"line_number":139,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":140,"context_line":"| Attribute | Type    | Req | CRUD     | Description                                                |"},{"line_number":141,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":142,"context_line":"| position  | integer | Y   | CRU      | Position at which this firewall group is evaluated         |"},{"line_number":143,"context_line":"+--------------------------------------+--------------------------------------+----------+----------+"},{"line_number":144,"context_line":"| tier      | String  | Y   | CRU      | Tier at which this firewall group exists (HEAD, TAIL, null)|"},{"line_number":145,"context_line":"|           |         |     |          | Default: null                                              |"},{"line_number":146,"context_line":"+---------------------------------------------------------------------------------------------------+"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"References"},{"line_number":149,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"7f60817a_8ffc024d","line":147,"range":{"start_line":138,"start_character":0,"end_line":147,"end_character":0},"updated":"2022-08-19 21:13:51.000000000","message":"The rendition of this in the final document is unreadable: https://8c06a6cede9420c6761b-340bd32e45a22d1b24bb452a4a4f9f06.ssl.cf5.rackcdn.com/851607/6/check/openstack-tox-docs/baa2987/docs/specs/zed/fwaas-group-ordering.html#data-model-impact/ I suggest you place this inside a code-block directive. For example, look how the output of the openstack command in this section\n\nCreate a routed provider network\n~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n\nThe following steps create a routed provider network with two segments. Each\nsegment contains one IPv4 subnet and one IPv6 subnet.\n\n#. Source the administrative project credentials.\n#. Create a VLAN provider network which includes a default segment. In this\n   example, the network uses the ``provider1`` physical network with VLAN ID\n   2016.\n\n   .. code-block:: console\n\n      $ openstack network create --share --provider-physical-network provider1 \\\n        --provider-network-type vlan --provider-segment 2016 multisegment1\n      +---------------------------+--------------------------------------+\n      | Field                     | Value                                |\n      +---------------------------+--------------------------------------+\n      | admin_state_up            | UP                                   |\n      | id                        | 6ab19caa-dda9-4b3d-abc4-5b8f435b98d9 |\n      | ipv4_address_scope        | None                                 |\n      | ipv6_address_scope        | None                                 |\n      | l2_adjacency              | True                                 |\n      | mtu                       | 1500                                 |\n      | name                      | multisegment1                        |\n      | port_security_enabled     | True                                 |\n      | provider:network_type     | vlan                                 |\n      | provider:physical_network | provider1                            |\n      | provider:segmentation_id  | 2016                                 |\n      | revision_number           | 1                                    |\n      | router:external           | Internal                             |\n      | shared                    | True                                 |\n      | status                    | ACTIVE                               |\n      | subnets                   |                                      |\n      | tags                      | []                                   |\n      +---------------------------+--------------------------------------+\n\n\nis rendered here: https://github.com/openstack/neutron/blob/master/doc/source/admin/config-routed-networks.rst#create-a-routed-provider-network","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"7df59030f459451084881a9083f04cb30960e444","unresolved":true,"context_lines":[{"line_number":135,"context_line":"Existing entries should be assigned consecutive `position` numbers starting at 1, and the"},{"line_number":136,"context_line":"default `tier` value of `null.`"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"**Firewall Group Port associations**"},{"line_number":139,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":140,"context_line":"| Attribute | Type    | Req | CRUD     | Description                                                |"},{"line_number":141,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":142,"context_line":"| position  | integer | Y   | CRU      | Position at which this firewall group is evaluated         |"},{"line_number":143,"context_line":"+--------------------------------------+--------------------------------------+----------+----------+"},{"line_number":144,"context_line":"| tier      | String  | Y   | CRU      | Tier at which this firewall group exists (HEAD, TAIL, null)|"},{"line_number":145,"context_line":"|           |         |     |          | Default: null                                              |"},{"line_number":146,"context_line":"+---------------------------------------------------------------------------------------------------+"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"References"},{"line_number":149,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"caf6eea0_1ce563e0","line":147,"range":{"start_line":138,"start_character":0,"end_line":147,"end_character":0},"in_reply_to":"7f60817a_8ffc024d","updated":"2022-08-22 14:44:15.000000000","message":"I think that just having empty line between L138 and 139 will fix it and it will look like table from L41-55 which is rendered fine: https://8c06a6cede9420c6761b-340bd32e45a22d1b24bb452a4a4f9f06.ssl.cf5.rackcdn.com/851607/6/check/openstack-tox-docs/baa2987/docs/specs/zed/fwaas-group-ordering.html#proposed-change","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"ee6ccb48b1c5f122bf7fe1fc9921bfc9ab7b3e4c","unresolved":false,"context_lines":[{"line_number":135,"context_line":"Existing entries should be assigned consecutive `position` numbers starting at 1, and the"},{"line_number":136,"context_line":"default `tier` value of `null.`"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"**Firewall Group Port associations**"},{"line_number":139,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":140,"context_line":"| Attribute | Type    | Req | CRUD     | Description                                                |"},{"line_number":141,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":142,"context_line":"| position  | integer | Y   | CRU      | Position at which this firewall group is evaluated         |"},{"line_number":143,"context_line":"+--------------------------------------+--------------------------------------+----------+----------+"},{"line_number":144,"context_line":"| tier      | String  | Y   | CRU      | Tier at which this firewall group exists (HEAD, TAIL, null)|"},{"line_number":145,"context_line":"|           |         |     |          | Default: null                                              |"},{"line_number":146,"context_line":"+---------------------------------------------------------------------------------------------------+"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"References"},{"line_number":149,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":150,"context_line":""}],"source_content_type":"text/x-rst","patch_set":6,"id":"1c040f80_c9b16271","line":147,"range":{"start_line":138,"start_character":0,"end_line":147,"end_character":0},"in_reply_to":"caf6eea0_1ce563e0","updated":"2022-08-23 13:23:14.000000000","message":"I fixed the formatting so it displays correctly now.","commit_id":"53e0bbaaffa257991dd82523496ed252463e4f66"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"c5377ade864d7a89c7bc9c47dc77721feab509c9","unresolved":true,"context_lines":[{"line_number":147,"context_line":"Existing entries should be assigned consecutive `position` numbers starting at 1, and the"},{"line_number":148,"context_line":"default `tier` value of `null.`"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"**Firewall Group Port associations**"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"+-----------+---------+-----+----------+------------------------------------------------------------+"},{"line_number":153,"context_line":"| Attribute | Type    | Req | CRUD     | Description                                                |"},{"line_number":154,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":155,"context_line":"| position  | integer | Y   | CRU      | Position at which this firewall group is evaluated         |"},{"line_number":156,"context_line":"+--------------------------------------+--------------------------------------+----------+----------+"},{"line_number":157,"context_line":"| tier      | String  | Y   | CRU      | Tier at which this firewall group exists (HEAD, TAIL, null)|"},{"line_number":158,"context_line":"|           |         |     |          | Default: null                                              |"},{"line_number":159,"context_line":"+-----------+---------+-----+----------+------------------------------------------------------------+"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"References"},{"line_number":162,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":8,"id":"d442c7fe_a65fb897","line":159,"range":{"start_line":150,"start_character":0,"end_line":159,"end_character":101},"updated":"2022-08-23 23:37:28.000000000","message":"I don\u0027t think this is what you want for the table: https://357e216ac78de18f4c7d-f42d955827477dc68a274454ee4340d5.ssl.cf5.rackcdn.com/851607/8/check/openstack-tox-docs/cb895e3/docs/specs/zed/fwaas-group-ordering.html#data-model-impact. It is better but still poorly formatted. Unlike the two tables above, columns don\u0027t align with your headings.\n\nI suspect the problem is line 158. You want to put \u0027Default: null\u0027 at the end of line 157.\n\nOr you might want to do something like this: https://docs.openstack.org/neutron/latest/admin/config-qos.html#supported-qos-rule-types. The source for these tables is here: https://raw.githubusercontent.com/openstack/neutron/master/doc/source/admin/config-qos.rst","commit_id":"1df76e6d9e4fb8e0104d964541f2544ffdf6308d"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"8c9efd072d98fd0e194b8c46493dafe54dcbf6ac","unresolved":false,"context_lines":[{"line_number":147,"context_line":"Existing entries should be assigned consecutive `position` numbers starting at 1, and the"},{"line_number":148,"context_line":"default `tier` value of `null.`"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"**Firewall Group Port associations**"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"+-----------+---------+-----+----------+------------------------------------------------------------+"},{"line_number":153,"context_line":"| Attribute | Type    | Req | CRUD     | Description                                                |"},{"line_number":154,"context_line":"+--------------------------------------+------------------------------------------------------------+"},{"line_number":155,"context_line":"| position  | integer | Y   | CRU      | Position at which this firewall group is evaluated         |"},{"line_number":156,"context_line":"+--------------------------------------+--------------------------------------+----------+----------+"},{"line_number":157,"context_line":"| tier      | String  | Y   | CRU      | Tier at which this firewall group exists (HEAD, TAIL, null)|"},{"line_number":158,"context_line":"|           |         |     |          | Default: null                                              |"},{"line_number":159,"context_line":"+-----------+---------+-----+----------+------------------------------------------------------------+"},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"References"},{"line_number":162,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":8,"id":"31215e5f_6c114296","line":159,"range":{"start_line":150,"start_character":0,"end_line":159,"end_character":101},"in_reply_to":"d442c7fe_a65fb897","updated":"2022-08-24 13:39:28.000000000","message":"Thanks for fixing this up!","commit_id":"1df76e6d9e4fb8e0104d964541f2544ffdf6308d"},{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"a02cbeb20b1b8fa492c1f7f98f1543273eeafebb","unresolved":true,"context_lines":[{"line_number":149,"context_line":""},{"line_number":150,"context_line":"**Firewall Group Port associations**"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"+-----------+---------+-----+----------+---------------------------------------------------------------------------+"},{"line_number":153,"context_line":"| Attribute | Type    | Req | CRUD     | Description                                                               |"},{"line_number":154,"context_line":"+-----------+---------+-----+----------+---------------------------------------------------------------------------+"},{"line_number":155,"context_line":"| position  | integer | Y   | CRU      | Position at which this firewall group is evaluated                        |"}],"source_content_type":"text/x-rst","patch_set":10,"id":"d7f7da09_392d760c","line":152,"updated":"2022-08-24 02:12:36.000000000","message":"Yeah: https://24c188d3a6791b5c5cba-1ec32409cb7076383d725ae618c678a6.ssl.cf2.rackcdn.com/851607/10/check/openstack-tox-docs/8aa1c35/docs/specs/zed/fwaas-group-ordering.html#data-model-impact","commit_id":"5ed9f7a835253cc2c653abcc500112c1089d7ddd"},{"author":{"_account_id":33525,"name":"Anthony","email":"atimmins@datto.com","username":"atimmins"},"change_message_id":"1ca32dfa31812abdbdba40dfc30dbf58aac34f68","unresolved":false,"context_lines":[{"line_number":149,"context_line":""},{"line_number":150,"context_line":"**Firewall Group Port associations**"},{"line_number":151,"context_line":""},{"line_number":152,"context_line":"+-----------+---------+-----+----------+---------------------------------------------------------------------------+"},{"line_number":153,"context_line":"| Attribute | Type    | Req | CRUD     | Description                                                               |"},{"line_number":154,"context_line":"+-----------+---------+-----+----------+---------------------------------------------------------------------------+"},{"line_number":155,"context_line":"| position  | integer | Y   | CRU      | Position at which this firewall group is evaluated                        |"}],"source_content_type":"text/x-rst","patch_set":10,"id":"a40d2de6_c381524d","line":152,"in_reply_to":"d7f7da09_392d760c","updated":"2022-08-24 13:39:52.000000000","message":"Done","commit_id":"5ed9f7a835253cc2c653abcc500112c1089d7ddd"}]}