)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"82157f16149a93bc7a29237f00f3f05d54088ca4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"78935548_7eeb3abd","updated":"2026-06-03 15:35:56.000000000","message":"Thanks for the clarification","commit_id":"6c813fa50bd41232b61dbd7f3eeda6e95e74ea07"}],"openstackclient/network/v2/fwaas/group.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"25cc0bc5ddeed3b7ebafe51d2785cd52cc7caade","unresolved":true,"context_lines":[{"line_number":332,"context_line":"            metavar\u003d\u0027\u003cfirewall-group\u003e\u0027,"},{"line_number":333,"context_line":"            help\u003d_(\u0027Firewall group to update (name or ID)\u0027),"},{"line_number":334,"context_line":"        )"},{"line_number":335,"context_line":"        parser.add_argument(\u0027--name\u0027, help\u003d_(\u0027Name for the firewall group\u0027))"},{"line_number":336,"context_line":"        parser.add_argument("},{"line_number":337,"context_line":"            \u0027--port\u0027,"},{"line_number":338,"context_line":"            metavar\u003d\u0027\u003cport\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f30b30c_760d5ef4","line":335,"updated":"2026-06-03 14:07:51.000000000","message":"Can this be a separate change? I assume you\u0027re trying to add support for renaming firewall groups?","commit_id":"6c813fa50bd41232b61dbd7f3eeda6e95e74ea07"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ce1b92d528de54f89b37fabde9162abc02585d69","unresolved":false,"context_lines":[{"line_number":332,"context_line":"            metavar\u003d\u0027\u003cfirewall-group\u003e\u0027,"},{"line_number":333,"context_line":"            help\u003d_(\u0027Firewall group to update (name or ID)\u0027),"},{"line_number":334,"context_line":"        )"},{"line_number":335,"context_line":"        parser.add_argument(\u0027--name\u0027, help\u003d_(\u0027Name for the firewall group\u0027))"},{"line_number":336,"context_line":"        parser.add_argument("},{"line_number":337,"context_line":"            \u0027--port\u0027,"},{"line_number":338,"context_line":"            metavar\u003d\u0027\u003cport\u003e\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"af7da7b0_51489cbd","line":335,"in_reply_to":"5f30b30c_760d5ef4","updated":"2026-06-03 14:31:07.000000000","message":"Actually this is not new addition. Previously it was in the _get_common_parser() function and was also in the SetFirewallGroup command. Now it is moved to be defined here explicitly as for Set command it still makes sense to have it that way.","commit_id":"6c813fa50bd41232b61dbd7f3eeda6e95e74ea07"}],"openstackclient/network/v2/fwaas/rule.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"25cc0bc5ddeed3b7ebafe51d2785cd52cc7caade","unresolved":true,"context_lines":[{"line_number":445,"context_line":"            \u0027firewall_rule\u0027,"},{"line_number":446,"context_line":"            metavar\u003d\u0027\u003cfirewall-rule\u003e\u0027,"},{"line_number":447,"context_line":"            help\u003d_(\u0027Firewall rule to set (name or ID)\u0027),"},{"line_number":448,"context_line":"        )"},{"line_number":449,"context_line":"        parser.add_argument("},{"line_number":450,"context_line":"            \u0027--name\u0027,"},{"line_number":451,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":452,"context_line":"            help\u003d_(\u0027Name of the firewall rule\u0027),"},{"line_number":453,"context_line":"        )"},{"line_number":454,"context_line":"        return parser"},{"line_number":455,"context_line":""},{"line_number":456,"context_line":"    def take_action(self, parsed_args: argparse.Namespace) -\u003e None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"4009d645_fb49bdfb","line":453,"range":{"start_line":448,"start_character":9,"end_line":453,"end_character":9},"updated":"2026-06-03 14:07:51.000000000","message":"Ditto. Also, perhaps `s/Name/New name/`?","commit_id":"6c813fa50bd41232b61dbd7f3eeda6e95e74ea07"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"ce1b92d528de54f89b37fabde9162abc02585d69","unresolved":false,"context_lines":[{"line_number":445,"context_line":"            \u0027firewall_rule\u0027,"},{"line_number":446,"context_line":"            metavar\u003d\u0027\u003cfirewall-rule\u003e\u0027,"},{"line_number":447,"context_line":"            help\u003d_(\u0027Firewall rule to set (name or ID)\u0027),"},{"line_number":448,"context_line":"        )"},{"line_number":449,"context_line":"        parser.add_argument("},{"line_number":450,"context_line":"            \u0027--name\u0027,"},{"line_number":451,"context_line":"            metavar\u003d\u0027\u003cname\u003e\u0027,"},{"line_number":452,"context_line":"            help\u003d_(\u0027Name of the firewall rule\u0027),"},{"line_number":453,"context_line":"        )"},{"line_number":454,"context_line":"        return parser"},{"line_number":455,"context_line":""},{"line_number":456,"context_line":"    def take_action(self, parsed_args: argparse.Namespace) -\u003e None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"0e5c8fe9_0e4742fe","line":453,"range":{"start_line":448,"start_character":9,"end_line":453,"end_character":9},"in_reply_to":"4009d645_fb49bdfb","updated":"2026-06-03 14:31:07.000000000","message":"The same as for group - this is not new, it was previously added through the _get_common_parser() function","commit_id":"6c813fa50bd41232b61dbd7f3eeda6e95e74ea07"}]}