)]}'
{"nova/policies/services.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"d2a57cff9d3331bccee7b1f25bba6b077b2e89ea","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    base.create_rule_default("},{"line_number":27,"context_line":"        BASE_POLICY_NAME,"},{"line_number":28,"context_line":"        base.RULE_ADMIN_API,"},{"line_number":29,"context_line":"        \"\"\"Lists all running Compute services in a region, enables"},{"line_number":30,"context_line":"or disables scheduling for a Compute service, logs disabled Compute service"},{"line_number":31,"context_line":"information, and deletes a Compute service.\"\"\","},{"line_number":32,"context_line":"        ["},{"line_number":33,"context_line":"            {"},{"line_number":34,"context_line":"                \u0027method\u0027: \u0027GET\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffe62b97_e574ac6a","line":31,"range":{"start_line":29,"start_character":11,"end_line":31,"end_character":43},"updated":"2017-03-27 11:16:53.000000000","message":"This is taken from the api-ref, I quite like keeping the original text.","commit_id":"8930cb271c49e0f0917c76c6b9f21cd70ff8de62"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"3839ba7efe4d56970005da909246b390766d8573","unresolved":false,"context_lines":[{"line_number":26,"context_line":"    base.create_rule_default("},{"line_number":27,"context_line":"        BASE_POLICY_NAME,"},{"line_number":28,"context_line":"        base.RULE_ADMIN_API,"},{"line_number":29,"context_line":"        \"\"\"Lists all running Compute services in a region, enables"},{"line_number":30,"context_line":"or disables scheduling for a Compute service, logs disabled Compute service"},{"line_number":31,"context_line":"information, and deletes a Compute service.\"\"\","},{"line_number":32,"context_line":"        ["},{"line_number":33,"context_line":"            {"},{"line_number":34,"context_line":"                \u0027method\u0027: \u0027GET\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffe62b97_0a71dfe0","line":31,"range":{"start_line":29,"start_character":59,"end_line":31,"end_character":11},"updated":"2017-03-27 11:12:53.000000000","message":"maybe change tihs to following?\n\nEnable scheduling for a service, Disable scheduling for a service with an optional log.","commit_id":"8930cb271c49e0f0917c76c6b9f21cd70ff8de62"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"3839ba7efe4d56970005da909246b390766d8573","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            {"},{"line_number":50,"context_line":"                \u0027method\u0027: \u0027DELETE\u0027,"},{"line_number":51,"context_line":"                \u0027path\u0027: \u0027/os-services/{service_id}\u0027"},{"line_number":52,"context_line":"            }"},{"line_number":53,"context_line":"        ]),"},{"line_number":54,"context_line":"    policy.RuleDefault("},{"line_number":55,"context_line":"        name\u003dPOLICY_ROOT % \u0027discoverable\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffe62b97_052670b8","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":13},"updated":"2017-03-27 11:12:53.000000000","message":"sorry if I missed some talk ,seems force-down is not mentioned here, we plan to add it later with mircoversion info?","commit_id":"8930cb271c49e0f0917c76c6b9f21cd70ff8de62"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"d2a57cff9d3331bccee7b1f25bba6b077b2e89ea","unresolved":false,"context_lines":[{"line_number":49,"context_line":"            {"},{"line_number":50,"context_line":"                \u0027method\u0027: \u0027DELETE\u0027,"},{"line_number":51,"context_line":"                \u0027path\u0027: \u0027/os-services/{service_id}\u0027"},{"line_number":52,"context_line":"            }"},{"line_number":53,"context_line":"        ]),"},{"line_number":54,"context_line":"    policy.RuleDefault("},{"line_number":55,"context_line":"        name\u003dPOLICY_ROOT % \u0027discoverable\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"ffe62b97_459fd8b3","line":52,"range":{"start_line":52,"start_character":12,"end_line":52,"end_character":13},"in_reply_to":"ffe62b97_052670b8","updated":"2017-03-27 11:16:53.000000000","message":"No need for micro version into, but it should be added here I think.","commit_id":"8930cb271c49e0f0917c76c6b9f21cd70ff8de62"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"6a0883d5b0015fc55f5de3bcde35f1ede424993a","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        \"\"\"Lists all running Compute services in a region, enables"},{"line_number":30,"context_line":"or disables scheduling for a Compute service, logs disabled Compute service"},{"line_number":31,"context_line":"information, set or unset forced_down flag for the compute service and"},{"line_number":32,"context_line":"deletes a Compute service.\"\"\","},{"line_number":33,"context_line":"        ["},{"line_number":34,"context_line":"            {"},{"line_number":35,"context_line":"                \u0027method\u0027: \u0027GET\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ffe62b97_ca9fde4d","line":32,"range":{"start_line":32,"start_character":18,"end_line":32,"end_character":25},"updated":"2017-03-30 06:57:17.000000000","message":"maybe missed part of context, but other patch do this one by one? so for each method / path, we create a default rule?","commit_id":"e128d0a42a6957fbbdbf5885793bfc32d4d9ccc8"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9ab95cceda91ccaf9c046dcff92dc0a297fa6ffb","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        \"\"\"Lists all running Compute services in a region, enables"},{"line_number":30,"context_line":"or disables scheduling for a Compute service, logs disabled Compute service"},{"line_number":31,"context_line":"information, set or unset forced_down flag for the compute service and"},{"line_number":32,"context_line":"deletes a Compute service.\"\"\","},{"line_number":33,"context_line":"        ["},{"line_number":34,"context_line":"            {"},{"line_number":35,"context_line":"                \u0027method\u0027: \u0027GET\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ffe62b97_9688dc22","line":32,"range":{"start_line":32,"start_character":18,"end_line":32,"end_character":25},"in_reply_to":"ffe62b97_4a7e2ee4","updated":"2017-03-30 10:05:38.000000000","message":"Yep, this correctly reflects what the code does.\n\nBut this renders poorly:\nhttp://docs-draft.openstack.org/82/449782/2/check/gate-nova-docs-ubuntu-xenial/cbd81f6//doc/build/html/sample_policy.html","commit_id":"e128d0a42a6957fbbdbf5885793bfc32d4d9ccc8"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"97378d88282be9f67ecba5b600a1a2ecf0fc11d7","unresolved":false,"context_lines":[{"line_number":29,"context_line":"        \"\"\"Lists all running Compute services in a region, enables"},{"line_number":30,"context_line":"or disables scheduling for a Compute service, logs disabled Compute service"},{"line_number":31,"context_line":"information, set or unset forced_down flag for the compute service and"},{"line_number":32,"context_line":"deletes a Compute service.\"\"\","},{"line_number":33,"context_line":"        ["},{"line_number":34,"context_line":"            {"},{"line_number":35,"context_line":"                \u0027method\u0027: \u0027GET\u0027,"}],"source_content_type":"text/x-python","patch_set":2,"id":"ffe62b97_4a7e2ee4","line":32,"range":{"start_line":32,"start_character":18,"end_line":32,"end_character":25},"in_reply_to":"ffe62b97_ca9fde4d","updated":"2017-03-30 07:04:22.000000000","message":"This due to we use single rule for all the APIs.","commit_id":"e128d0a42a6957fbbdbf5885793bfc32d4d9ccc8"}]}
