)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"change_message_id":"c21668ae97243a304b496533ea408c4ce1e8fc04","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c7dbb4c7_eabc3ae9","updated":"2026-07-30 17:38:51.000000000","message":"looks good, thanks Doug!","commit_id":"36952b46e803c005d352a897942157a13b7e910f"}],"watcherclient/v1/audit_template_shell.py":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"c254c0fce5795a40ed72230a07ce198c5edba898","unresolved":true,"context_lines":[{"line_number":266,"context_line":""},{"line_number":267,"context_line":"        api_ver \u003d self.app_args.os_infra_optim_api_version"},{"line_number":268,"context_line":"        if api_versioning.allow_audit_template_default_parameters(api_ver):"},{"line_number":269,"context_line":"            field_list.append(\u0027default_parameters\u0027)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        fields \u003d dict((k, v) for (k, v) in vars(parsed_args).items()"},{"line_number":272,"context_line":"                      if k in field_list and v is not None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"0c1273f6_98601426","line":269,"updated":"2026-07-24 15:59:45.000000000","message":"Currently, if -p is used but api used is \u003c 1.7, the default_parameter is silently ignored and not used when creating the audittemplate. Is this the expected behavior? I\u0027d say if a user uses --default-parameter but uses api version \u003c 1.7 it should fail or at least give some warning message\n\n```\n$ openstack optimize audittemplate create -s dummy -p para1\u003d5 dummy3 dummy --insecure\n+--------------------+--------------------------------------+\n| Field              | Value                                |\n+--------------------+--------------------------------------+\n| UUID               | eddaaaad-9d4e-46cb-8c00-b48d2090a2ac |\n| Created At         | 2026-07-24T15:54:16.552868+00:00     |\n| Updated At         | None                                 |\n| Deleted At         | None                                 |\n| Description        | None                                 |\n| Name               | dummy3                               |\n| Goal               | dummy                                |\n| Strategy           | dummy                                |\n| Audit Scope        | []                                   |\n| Default Parameters |                                      |\n+--------------------+--------------------------------------+\n```","commit_id":"c1667999af37ab178e52f828673c64e498dd180f"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"6e348e3c6eadd5b1736a104559b3bdc78a9e399d","unresolved":true,"context_lines":[{"line_number":266,"context_line":""},{"line_number":267,"context_line":"        api_ver \u003d self.app_args.os_infra_optim_api_version"},{"line_number":268,"context_line":"        if api_versioning.allow_audit_template_default_parameters(api_ver):"},{"line_number":269,"context_line":"            field_list.append(\u0027default_parameters\u0027)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        fields \u003d dict((k, v) for (k, v) in vars(parsed_args).items()"},{"line_number":272,"context_line":"                      if k in field_list and v is not None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"68d89376_76f7672a","line":269,"in_reply_to":"0c1273f6_98601426","updated":"2026-07-28 19:27:46.000000000","message":"I think that improves yeah, i can add this check in the next PS, thanks Alfredo","commit_id":"c1667999af37ab178e52f828673c64e498dd180f"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"44ebaa973e4a8aaae7420e25e448485c3d7fc90f","unresolved":false,"context_lines":[{"line_number":266,"context_line":""},{"line_number":267,"context_line":"        api_ver \u003d self.app_args.os_infra_optim_api_version"},{"line_number":268,"context_line":"        if api_versioning.allow_audit_template_default_parameters(api_ver):"},{"line_number":269,"context_line":"            field_list.append(\u0027default_parameters\u0027)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        fields \u003d dict((k, v) for (k, v) in vars(parsed_args).items()"},{"line_number":272,"context_line":"                      if k in field_list and v is not None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a36048c_92af1401","line":269,"in_reply_to":"14f0cf7c_3e106ec0","updated":"2026-07-29 07:04:06.000000000","message":"Thanks!","commit_id":"c1667999af37ab178e52f828673c64e498dd180f"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"a785fc6e82ead0e32c1c7f40ab1ff0475ff74459","unresolved":true,"context_lines":[{"line_number":266,"context_line":""},{"line_number":267,"context_line":"        api_ver \u003d self.app_args.os_infra_optim_api_version"},{"line_number":268,"context_line":"        if api_versioning.allow_audit_template_default_parameters(api_ver):"},{"line_number":269,"context_line":"            field_list.append(\u0027default_parameters\u0027)"},{"line_number":270,"context_line":""},{"line_number":271,"context_line":"        fields \u003d dict((k, v) for (k, v) in vars(parsed_args).items()"},{"line_number":272,"context_line":"                      if k in field_list and v is not None)"}],"source_content_type":"text/x-python","patch_set":2,"id":"14f0cf7c_3e106ec0","line":269,"in_reply_to":"68d89376_76f7672a","updated":"2026-07-28 19:52:10.000000000","message":"Added and validated in my env. Thanks!","commit_id":"c1667999af37ab178e52f828673c64e498dd180f"}]}