)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"c084ede15e93cf7b82bdeea56693b18d9d450f92","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add policy config for get_flavor_service_profile operation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Policy for this API call was commented out but it seems that"},{"line_number":10,"context_line":"it is needed to fix Parole test failure, see related bug description"},{"line_number":11,"context_line":"for details."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4a0e97845a40888f7f2c7267d9adb7be663ee380"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_8dab0974","line":10,"range":{"start_line":10,"start_character":20,"end_line":10,"end_character":26},"updated":"2019-09-16 16:32:13.000000000","message":"nit: Patrole","commit_id":"02adea047be5fc48af96211306464234520f4133"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"135e1c864c1ce698761697921b26d1b7f894e265","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Add policy config for get_flavor_service_profile operation"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Policy for this API call was commented out but it seems that"},{"line_number":10,"context_line":"it is needed to fix Parole test failure, see related bug description"},{"line_number":11,"context_line":"for details."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I4a0e97845a40888f7f2c7267d9adb7be663ee380"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_3a3a3ee7","line":10,"range":{"start_line":10,"start_character":20,"end_line":10,"end_character":26},"in_reply_to":"3fa7e38b_8dab0974","updated":"2019-09-17 07:09:10.000000000","message":"Done","commit_id":"02adea047be5fc48af96211306464234520f4133"}],"neutron/conf/policies/flavor.py":[{"author":{"_account_id":4694,"name":"Miguel Lavalle","email":"miguel@mlavalle.com","username":"minsel"},"change_message_id":"468b212e120760ecc28b2ea81f7aba85aac59ab4","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    policy.RuleDefault("},{"line_number":126,"context_line":"        \u0027get_flavor_service_profile\u0027,"},{"line_number":127,"context_line":"        base.RULE_ANY,"},{"line_number":128,"context_line":"        \u0027Get a flavor associate with a given service profiles\u0027,"},{"line_number":129,"context_line":"    ),"},{"line_number":130,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":131,"context_line":"        \u0027create_flavor_service_profile\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_2d9d3559","line":128,"range":{"start_line":128,"start_character":30,"end_line":128,"end_character":31},"updated":"2019-09-16 16:31:02.000000000","message":"nit: ed","commit_id":"02adea047be5fc48af96211306464234520f4133"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"135e1c864c1ce698761697921b26d1b7f894e265","unresolved":false,"context_lines":[{"line_number":125,"context_line":"    policy.RuleDefault("},{"line_number":126,"context_line":"        \u0027get_flavor_service_profile\u0027,"},{"line_number":127,"context_line":"        base.RULE_ANY,"},{"line_number":128,"context_line":"        \u0027Get a flavor associate with a given service profiles\u0027,"},{"line_number":129,"context_line":"    ),"},{"line_number":130,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":131,"context_line":"        \u0027create_flavor_service_profile\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_9a3572fa","line":128,"range":{"start_line":128,"start_character":30,"end_line":128,"end_character":31},"in_reply_to":"3fa7e38b_2d9d3559","updated":"2019-09-17 07:09:10.000000000","message":"Done","commit_id":"02adea047be5fc48af96211306464234520f4133"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"c0f6890dfe340e41647ecdc726a244b9e2841256","unresolved":false,"context_lines":[{"line_number":126,"context_line":"        \u0027get_flavor_service_profile\u0027,"},{"line_number":127,"context_line":"        base.RULE_ANY,"},{"line_number":128,"context_line":"        \u0027Get a flavor associated with a given service profiles\u0027,"},{"line_number":129,"context_line":"    ),"},{"line_number":130,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":131,"context_line":"        \u0027create_flavor_service_profile\u0027,"},{"line_number":132,"context_line":"        base.RULE_ADMIN_ONLY,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_a401f0b1","line":129,"updated":"2019-10-07 08:03:17.000000000","message":"It is better to use DocumentedRuleDefault. This rule is not referred by other rules, so it is not apparent where this rule is used.\n\nIn my understanding, it is now referred only in the DELETE operation (as there is no corresponding GET operations like flavors/\u003cid\u003e/service_profiles/).\n\nHow about adding the method and path information and also describing the situation in the description at L.128?","commit_id":"b149eb7e03f7b5e56969e79728c18e6a23431fec"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"5d585524fdadec335b389a86a114765687313744","unresolved":false,"context_lines":[{"line_number":126,"context_line":"        \u0027get_flavor_service_profile\u0027,"},{"line_number":127,"context_line":"        base.RULE_ANY,"},{"line_number":128,"context_line":"        \u0027Get a flavor associated with a given service profiles\u0027,"},{"line_number":129,"context_line":"    ),"},{"line_number":130,"context_line":"    policy.DocumentedRuleDefault("},{"line_number":131,"context_line":"        \u0027create_flavor_service_profile\u0027,"},{"line_number":132,"context_line":"        base.RULE_ADMIN_ONLY,"}],"source_content_type":"text/x-python","patch_set":2,"id":"3fa7e38b_7af7cf5a","line":129,"in_reply_to":"3fa7e38b_a401f0b1","updated":"2019-10-10 10:59:14.000000000","message":"Done","commit_id":"b149eb7e03f7b5e56969e79728c18e6a23431fec"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"77c1c1a9b34e7b7f98f8524bbc7854d1148e608b","unresolved":false,"context_lines":[{"line_number":130,"context_line":"        \u0027This rule is currently referred only in the DELETE \u0027"},{"line_number":131,"context_line":"        \u0027of flavor_service_profile. \u0027,"},{"line_number":132,"context_line":"        ["},{"line_number":133,"context_line":"            {"},{"line_number":134,"context_line":"                \u0027method\u0027: \u0027GET\u0027,"},{"line_number":135,"context_line":"                \u0027path\u0027: ASSOC_COLLECTION_PATH,"},{"line_number":136,"context_line":"            },"},{"line_number":137,"context_line":"            {"},{"line_number":138,"context_line":"                \u0027method\u0027: \u0027GET\u0027,"},{"line_number":139,"context_line":"                \u0027path\u0027: ASSOC_RESOURCE_PATH,"},{"line_number":140,"context_line":"            },"},{"line_number":141,"context_line":"        ]"},{"line_number":142,"context_line":"    ),"},{"line_number":143,"context_line":"    policy.DocumentedRuleDefault("}],"source_content_type":"text/x-python","patch_set":3,"id":"3fa7e38b_0dda244a","line":140,"range":{"start_line":133,"start_character":12,"end_line":140,"end_character":14},"updated":"2019-10-10 15:32:41.000000000","message":"There is no corresponding API as mentioned in the description above. How about DELETE method here like L.159-162 below?\nOr, RuleDefault now works as you provide sufficient information.\n\nIn my local build of the docs, RuleDefault looks better. Sorry that I commented the different thing from my previous comment on RuleDefault vs DocumentedRuleDefault.","commit_id":"ca4d7103642e65bc1c414a479f5013ab6b7a59c9"}]}