)]}'
{"nova/policies/quota_class_sets.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"3c921c98bb4607c0b331c998dced6356f9f8f45c","unresolved":false,"context_lines":[{"line_number":24,"context_line":"quota_class_sets_policies \u003d ["},{"line_number":25,"context_line":"    policy.RuleDefault("},{"line_number":26,"context_line":"        name\u003dPOLICY_ROOT % \u0027show\u0027,"},{"line_number":27,"context_line":"        check_str\u003d\u0027is_admin:True or quota_class:%(quota_class)s\u0027),"},{"line_number":28,"context_line":"    policy.RuleDefault("},{"line_number":29,"context_line":"        name\u003dPOLICY_ROOT % \u0027discoverable\u0027,"},{"line_number":30,"context_line":"        check_str\u003dbase.RULE_ANY),"}],"source_content_type":"text/x-python","patch_set":9,"id":"3aaa91ec_5a9d3458","line":27,"range":{"start_line":27,"start_character":19,"end_line":27,"end_character":63},"updated":"2016-06-23 13:33:41.000000000","message":"nit: I wonder about adding this rule in base as well.","commit_id":"2327b6946d994aa0e7c9276c93ef085e4f8dcc49"},{"author":{"_account_id":8213,"name":"Claudiu Belu","email":"cbelu@cloudbasesolutions.com","username":"claudiub"},"change_message_id":"494039da3e1b2d74087bdb0a453c7c15707dd2e4","unresolved":false,"context_lines":[{"line_number":24,"context_line":"quota_class_sets_policies \u003d ["},{"line_number":25,"context_line":"    policy.RuleDefault("},{"line_number":26,"context_line":"        name\u003dPOLICY_ROOT % \u0027show\u0027,"},{"line_number":27,"context_line":"        check_str\u003d\u0027is_admin:True or quota_class:%(quota_class)s\u0027),"},{"line_number":28,"context_line":"    policy.RuleDefault("},{"line_number":29,"context_line":"        name\u003dPOLICY_ROOT % \u0027discoverable\u0027,"},{"line_number":30,"context_line":"        check_str\u003dbase.RULE_ANY),"}],"source_content_type":"text/x-python","patch_set":9,"id":"3aaa91ec_d08c6381","line":27,"range":{"start_line":27,"start_character":19,"end_line":27,"end_character":63},"in_reply_to":"3aaa91ec_5a9d3458","updated":"2016-06-23 13:53:31.000000000","message":"it makes a bit more sense for the keystone rules, as they seem more general, depending on the user_id and it is used multiple times, while this is used once, here.","commit_id":"2327b6946d994aa0e7c9276c93ef085e4f8dcc49"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"51abb383809db7cc2182a26584d77195552ff17f","unresolved":false,"context_lines":[{"line_number":24,"context_line":"quota_class_sets_policies \u003d ["},{"line_number":25,"context_line":"    policy.RuleDefault("},{"line_number":26,"context_line":"        name\u003dPOLICY_ROOT % \u0027show\u0027,"},{"line_number":27,"context_line":"        check_str\u003d\u0027is_admin:True or quota_class:%(quota_class)s\u0027),"},{"line_number":28,"context_line":"    policy.RuleDefault("},{"line_number":29,"context_line":"        name\u003dPOLICY_ROOT % \u0027discoverable\u0027,"},{"line_number":30,"context_line":"        check_str\u003dbase.RULE_ANY),"}],"source_content_type":"text/x-python","patch_set":9,"id":"3aaa91ec_10cb8b1f","line":27,"range":{"start_line":27,"start_character":19,"end_line":27,"end_character":63},"in_reply_to":"3aaa91ec_d08c6381","updated":"2016-06-23 14:03:08.000000000","message":"Yeah, very true. I just like the central list of all the \"default\" rules.","commit_id":"2327b6946d994aa0e7c9276c93ef085e4f8dcc49"}]}