)]}'
{"specs/oslo.policy/policy-yaml.rst":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"71d7a055b4d47d26d2d6625ecd0e5ad50ae8aeff","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Support policy files in YAML"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"`bp example \u003chttps://blueprints.launchpad.net/oslo.policy/+spec/policy-yaml\u003e`_"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"JSON is great for a wire format, but awful for a configuration file."},{"line_number":14,"context_line":"oslo.policy can easily also support a safe subset of YAML."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a57f1b5_63cd3984","line":11,"range":{"start_line":11,"start_character":4,"end_line":11,"end_character":11},"updated":"2016-02-12 20:37:30.000000000","message":"change to policy-yaml","commit_id":"70b0481a3262790100ffc90a5b3b7220cde6cde2"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"32a92e6d84541f69d6667f7bb6a073a90d3bb625","unresolved":false,"context_lines":[{"line_number":8,"context_line":"Support policy files in YAML"},{"line_number":9,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"`bp example \u003chttps://blueprints.launchpad.net/oslo.policy/+spec/policy-yaml\u003e`_"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"JSON is great for a wire format, but awful for a configuration file."},{"line_number":14,"context_line":"oslo.policy can easily also support a safe subset of YAML."}],"source_content_type":"text/x-rst","patch_set":1,"id":"dae33548_31b1ae90","line":11,"range":{"start_line":11,"start_character":4,"end_line":11,"end_character":11},"in_reply_to":"3a57f1b5_63cd3984","updated":"2016-02-15 17:24:35.000000000","message":"Done","commit_id":"70b0481a3262790100ffc90a5b3b7220cde6cde2"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"6f590b628efb8fc422501ac081154bcc249c2294","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"oslo.policy defines a `policy_file` config option which defaults to"},{"line_number":54,"context_line":"``policy.json``. This should change to default to ``policy.yaml``, but also"},{"line_number":55,"context_line":"look for ``policy.json`` if `policy.yaml` doesn\u0027t exist."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Files to change:"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a57f1b5_439a7dba","line":55,"updated":"2016-02-12 20:29:40.000000000","message":"I understand this is easier (don\u0027t have to update all projects to say policy.json explicitly), but it just seems wrong to have the code looking for something other than what the conf option actually says to look for. Maybe this could be temporary... give the various projects some time to adjust and then go back to only looking for what is specified?","commit_id":"70b0481a3262790100ffc90a5b3b7220cde6cde2"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"dffbe8d23e1645638d799b5b72ad15206bd627e8","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"oslo.policy defines a `policy_file` config option which defaults to"},{"line_number":54,"context_line":"``policy.json``. This should change to default to ``policy.yaml``, but also"},{"line_number":55,"context_line":"look for ``policy.json`` if `policy.yaml` doesn\u0027t exist."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Files to change:"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"dae33548_11d90af4","line":55,"in_reply_to":"3a57f1b5_439a7dba","updated":"2016-02-15 17:24:27.000000000","message":"Done","commit_id":"70b0481a3262790100ffc90a5b3b7220cde6cde2"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"71d7a055b4d47d26d2d6625ecd0e5ad50ae8aeff","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"oslo.policy defines a `policy_file` config option which defaults to"},{"line_number":54,"context_line":"``policy.json``. This should change to default to ``policy.yaml``, but also"},{"line_number":55,"context_line":"look for ``policy.json`` if `policy.yaml` doesn\u0027t exist."},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Files to change:"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a57f1b5_a3b251e3","line":55,"in_reply_to":"3a57f1b5_439a7dba","updated":"2016-02-12 20:37:30.000000000","message":"For the help text, I\u0027ll probably wind up having no default value and describe the default behavior in the text. I\u0027ll update this text to try to make it clearer.","commit_id":"70b0481a3262790100ffc90a5b3b7220cde6cde2"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"14cd994db5130883d04fa6263561e76dc240aab7","unresolved":false,"context_lines":[{"line_number":32,"context_line":"for my custom roles and rules."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"As a developer, I should be able to provide a sample policy file that\u0027s"},{"line_number":35,"context_line":"self-describing. As in, add comments to the sample policy file."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_145b2bf7","line":35,"range":{"start_line":35,"start_character":62,"end_line":35,"end_character":63},"updated":"2016-02-23 19:24:54.000000000","message":"Are there any other use cases beside being able to add comments?","commit_id":"1a30175b8afd7d0c4ed06abb69ecd8cf38451ec0"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"14cd994db5130883d04fa6263561e76dc240aab7","unresolved":false,"context_lines":[{"line_number":39,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Rather than parse the policy file using the JSON parser, oslo.policy will"},{"line_number":42,"context_line":"parse it using the YAML parser (provided by PyYAML). Since JSON is a subset of"},{"line_number":43,"context_line":"YAML existing JSON files will continue to work. Projects and deployers can"},{"line_number":44,"context_line":"switch to simplified YAML and document using comments if they want."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"`oslo.policy` will be changed to use PyYAML\u0027s `safe_load()` to parse the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_b4535fe1","line":43,"range":{"start_line":42,"start_character":53,"end_line":43,"end_character":47},"updated":"2016-02-23 19:24:54.000000000","message":"This is statement may not be 100% true. Technically, JSON is supposed to be a subset of YAML, but there are a few incompatibilities.","commit_id":"1a30175b8afd7d0c4ed06abb69ecd8cf38451ec0"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"a71899fbfc6fef7f4fc7847c949512d3925a1404","unresolved":false,"context_lines":[{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Rather than parse the policy file using the JSON parser, oslo.policy will"},{"line_number":42,"context_line":"parse it using the YAML parser (provided by PyYAML). Since JSON is a subset of"},{"line_number":43,"context_line":"YAML existing JSON files will continue to work. Projects and deployers can"},{"line_number":44,"context_line":"switch to simplified YAML and document using comments if they want."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"`oslo.policy` will be changed to use PyYAML\u0027s `safe_load()` to parse the"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dae33548_a1d59f1c","line":43,"range":{"start_line":43,"start_character":0,"end_line":43,"end_character":4},"updated":"2016-02-18 22:50:27.000000000","message":"add a comma after YAML","commit_id":"1a30175b8afd7d0c4ed06abb69ecd8cf38451ec0"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"14cd994db5130883d04fa6263561e76dc240aab7","unresolved":false,"context_lines":[{"line_number":44,"context_line":"switch to simplified YAML and document using comments if they want."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"`oslo.policy` will be changed to use PyYAML\u0027s `safe_load()` to parse the"},{"line_number":47,"context_line":"policy file rather than the JSON parser ( `jsonutils.loads()` )."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"For some reason the name of the method to read the policy file includes the"},{"line_number":50,"context_line":"format. This was incorrect to begin with and is made even less accurate since"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_74fb97c1","line":47,"range":{"start_line":47,"start_character":62,"end_line":47,"end_character":64},"updated":"2016-02-23 19:24:54.000000000","message":"can we load based on file extension?","commit_id":"1a30175b8afd7d0c4ed06abb69ecd8cf38451ec0"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"14cd994db5130883d04fa6263561e76dc240aab7","unresolved":false,"context_lines":[{"line_number":99,"context_line":"representations for the same result I assume it takes more work to parse it."},{"line_number":100,"context_line":"The policy file is read when the server starts and also whenever the file"},{"line_number":101,"context_line":"changes (it used to be read on every request, but that\u0027s been changed to check"},{"line_number":102,"context_line":"the modification time), so I don\u0027t think this is going to be noticable."},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"Other Deployer Impact"},{"line_number":105,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bae84128_54cdb383","line":102,"updated":"2016-02-23 19:24:54.000000000","message":"Policy file can be reloaded dynamically. But yeah, they are highly static so that should be a problem.","commit_id":"1a30175b8afd7d0c4ed06abb69ecd8cf38451ec0"}]}