)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5f7865a81f05d9bed59b3b9543e603e568c33c61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"b5829f21_3c95828b","updated":"2023-06-02 10:37:36.000000000","message":"few nits which I wanted to clarify/check. If those will be replied I can +2 and approve this patch immediately 😊","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"7541dc2fbbbcc2fd4bb9f0ef6149f11566b44b13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"fb11ae27_1fdda1fd","updated":"2023-06-03 17:13:38.000000000","message":"recheck neutron-functional-with-uwsgi","commit_id":"a1ca3a7650f6e6b0070f73fde9a4272b6ccc78ec"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"4c5349944536b18ca29163e2e459a18e9b933704","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"3658f6bd_403d06c2","updated":"2023-06-12 16:07:39.000000000","message":"recheck functional","commit_id":"7da72b7f2d009e46807cc2c4fb30cc96f3693344"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"6e95cd60171bcf89cc800246b79a7afc8e35ff3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"f3c6f6c2_45510f57","updated":"2023-06-15 20:00:02.000000000","message":"recheck functional","commit_id":"7da72b7f2d009e46807cc2c4fb30cc96f3693344"}],"api-ref/source/v2/parameters.yaml":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5f7865a81f05d9bed59b3b9543e603e568c33c61","unresolved":true,"context_lines":[{"line_number":6325,"context_line":"security_group_rule-is-default:"},{"line_number":6326,"context_line":"  description: |"},{"line_number":6327,"context_line":"    Indicates if the security group rule belongs to the default security"},{"line_number":6328,"context_line":"    group of this project or not."},{"line_number":6329,"context_line":"  in: body"},{"line_number":6330,"context_line":"  required: true"},{"line_number":6331,"context_line":"  type: boolean"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"57b877d0_5f7a08f2","line":6328,"range":{"start_line":6328,"start_character":13,"end_line":6328,"end_character":17},"updated":"2023-06-02 10:37:36.000000000","message":"nit: I\u0027m not english speaker but shouldn\u0027t be \"the project\" here?","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"af159a3c8b55f1b037985163ea4a7418b224a4ba","unresolved":false,"context_lines":[{"line_number":6325,"context_line":"security_group_rule-is-default:"},{"line_number":6326,"context_line":"  description: |"},{"line_number":6327,"context_line":"    Indicates if the security group rule belongs to the default security"},{"line_number":6328,"context_line":"    group of this project or not."},{"line_number":6329,"context_line":"  in: body"},{"line_number":6330,"context_line":"  required: true"},{"line_number":6331,"context_line":"  type: boolean"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"8a063742_6e089380","line":6328,"range":{"start_line":6328,"start_character":13,"end_line":6328,"end_character":17},"in_reply_to":"57b877d0_5f7a08f2","updated":"2023-06-02 11:34:30.000000000","message":"Done","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"}],"api-ref/source/v2/security-group-rules.inc":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5f7865a81f05d9bed59b3b9543e603e568c33c61","unresolved":true,"context_lines":[{"line_number":83,"context_line":"   - revision_number: revision_number"},{"line_number":84,"context_line":"   - id: security_group_rule-id"},{"line_number":85,"context_line":"   - description: description"},{"line_number":86,"context_line":"   - is_default: security_group_rule-is-default"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Response Example"},{"line_number":89,"context_line":"----------------"}],"source_content_type":"text/x-c++src","patch_set":4,"id":"479facb5_99ddcc96","line":86,"updated":"2023-06-02 10:37:36.000000000","message":"I\u0027m not sure about this naming. IMHO \"is_default\" can be somehow confusing as it may suggest that rule is actually somehow default rule. Maybe it could be something like \"default_security_group\" or something similar to clearly show that this rule belongs to the \"Default\" security group.","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"af159a3c8b55f1b037985163ea4a7418b224a4ba","unresolved":false,"context_lines":[{"line_number":83,"context_line":"   - revision_number: revision_number"},{"line_number":84,"context_line":"   - id: security_group_rule-id"},{"line_number":85,"context_line":"   - description: description"},{"line_number":86,"context_line":"   - is_default: security_group_rule-is-default"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Response Example"},{"line_number":89,"context_line":"----------------"}],"source_content_type":"text/x-c++src","patch_set":4,"id":"1ae25f21_72950ac4","line":86,"in_reply_to":"479facb5_99ddcc96","updated":"2023-06-02 11:34:30.000000000","message":"Right but I don\u0027t want to write here the SG ID, so I prefer to change this to \"belongs_to_default_sg\"","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"}],"neutron_lib/api/definitions/security_groups_rules_is_default.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5f7865a81f05d9bed59b3b9543e603e568c33c61","unresolved":true,"context_lines":[{"line_number":16,"context_line":"ALIAS \u003d \u0027security-groups-rules-is-default\u0027"},{"line_number":17,"context_line":"IS_SHIM_EXTENSION \u003d False"},{"line_number":18,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":19,"context_line":"NAME \u003d \u0027Security group rule belong to the default security group\u0027"},{"line_number":20,"context_line":"DESCRIPTION \u003d (\u0027Flag to determine if the security group rule belongs to the \u0027"},{"line_number":21,"context_line":"               \u0027project default security group\u0027)"},{"line_number":22,"context_line":"UPDATED_TIMESTAMP \u003d \u00272023-05-23T10:00:00-00:00\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"0cf44b60_743c0c7a","line":19,"range":{"start_line":19,"start_character":28,"end_line":19,"end_character":34},"updated":"2023-06-02 10:37:36.000000000","message":"belongs ?","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5f7865a81f05d9bed59b3b9543e603e568c33c61","unresolved":true,"context_lines":[{"line_number":16,"context_line":"ALIAS \u003d \u0027security-groups-rules-is-default\u0027"},{"line_number":17,"context_line":"IS_SHIM_EXTENSION \u003d False"},{"line_number":18,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":19,"context_line":"NAME \u003d \u0027Security group rule belong to the default security group\u0027"},{"line_number":20,"context_line":"DESCRIPTION \u003d (\u0027Flag to determine if the security group rule belongs to the \u0027"},{"line_number":21,"context_line":"               \u0027project default security group\u0027)"},{"line_number":22,"context_line":"UPDATED_TIMESTAMP \u003d \u00272023-05-23T10:00:00-00:00\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"7ed720bb_3226ddb0","line":19,"range":{"start_line":19,"start_character":42,"end_line":19,"end_character":49},"updated":"2023-06-02 10:37:36.000000000","message":"maybe \"to the project\u0027s default security group\"","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"af159a3c8b55f1b037985163ea4a7418b224a4ba","unresolved":false,"context_lines":[{"line_number":16,"context_line":"ALIAS \u003d \u0027security-groups-rules-is-default\u0027"},{"line_number":17,"context_line":"IS_SHIM_EXTENSION \u003d False"},{"line_number":18,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":19,"context_line":"NAME \u003d \u0027Security group rule belong to the default security group\u0027"},{"line_number":20,"context_line":"DESCRIPTION \u003d (\u0027Flag to determine if the security group rule belongs to the \u0027"},{"line_number":21,"context_line":"               \u0027project default security group\u0027)"},{"line_number":22,"context_line":"UPDATED_TIMESTAMP \u003d \u00272023-05-23T10:00:00-00:00\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"de18e2ce_3940dbcc","line":19,"range":{"start_line":19,"start_character":28,"end_line":19,"end_character":34},"in_reply_to":"0cf44b60_743c0c7a","updated":"2023-06-02 11:34:30.000000000","message":"Done","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"af159a3c8b55f1b037985163ea4a7418b224a4ba","unresolved":false,"context_lines":[{"line_number":16,"context_line":"ALIAS \u003d \u0027security-groups-rules-is-default\u0027"},{"line_number":17,"context_line":"IS_SHIM_EXTENSION \u003d False"},{"line_number":18,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":19,"context_line":"NAME \u003d \u0027Security group rule belong to the default security group\u0027"},{"line_number":20,"context_line":"DESCRIPTION \u003d (\u0027Flag to determine if the security group rule belongs to the \u0027"},{"line_number":21,"context_line":"               \u0027project default security group\u0027)"},{"line_number":22,"context_line":"UPDATED_TIMESTAMP \u003d \u00272023-05-23T10:00:00-00:00\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"a6fe28b9_1b176888","line":19,"range":{"start_line":19,"start_character":42,"end_line":19,"end_character":49},"in_reply_to":"7ed720bb_3226ddb0","updated":"2023-06-02 11:34:30.000000000","message":"Exactly, we need to make this distinction.","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5f7865a81f05d9bed59b3b9543e603e568c33c61","unresolved":true,"context_lines":[{"line_number":18,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":19,"context_line":"NAME \u003d \u0027Security group rule belong to the default security group\u0027"},{"line_number":20,"context_line":"DESCRIPTION \u003d (\u0027Flag to determine if the security group rule belongs to the \u0027"},{"line_number":21,"context_line":"               \u0027project default security group\u0027)"},{"line_number":22,"context_line":"UPDATED_TIMESTAMP \u003d \u00272023-05-23T10:00:00-00:00\u0027"},{"line_number":23,"context_line":"IS_DEFAULT \u003d \u0027is_default\u0027"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3477347c_7bf5a76e","line":21,"updated":"2023-06-02 10:37:36.000000000","message":"project\u0027s default ?","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"af159a3c8b55f1b037985163ea4a7418b224a4ba","unresolved":false,"context_lines":[{"line_number":18,"context_line":"IS_STANDARD_ATTR_EXTENSION \u003d False"},{"line_number":19,"context_line":"NAME \u003d \u0027Security group rule belong to the default security group\u0027"},{"line_number":20,"context_line":"DESCRIPTION \u003d (\u0027Flag to determine if the security group rule belongs to the \u0027"},{"line_number":21,"context_line":"               \u0027project default security group\u0027)"},{"line_number":22,"context_line":"UPDATED_TIMESTAMP \u003d \u00272023-05-23T10:00:00-00:00\u0027"},{"line_number":23,"context_line":"IS_DEFAULT \u003d \u0027is_default\u0027"},{"line_number":24,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"58564541_5ed97936","line":21,"in_reply_to":"3477347c_7bf5a76e","updated":"2023-06-02 11:34:30.000000000","message":"Done","commit_id":"951e559257b6c0d4fa50c343abd4f6aefbeea623"}]}