)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"1691bd7bd289a7bde5d71a4767594b0439be29a1","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"UpgradeImpact"},{"line_number":27,"context_line":"DocImpact"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Change-Id: I1c5cb7834c2cf3295e10bad7fd07cfacb8e4ac50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1ac06dbe_85a02ba7","line":28,"updated":"2016-08-25 04:40:21.000000000","message":"Lite-spec: https://review.openstack.org/360220","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"116e0b2a9d8a85e9b778a0ced14a022891a90b46","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"UpgradeImpact"},{"line_number":27,"context_line":"DocImpact"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Change-Id: I1c5cb7834c2cf3295e10bad7fd07cfacb8e4ac50"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa7ab95a_e25810fd","line":28,"in_reply_to":"1ac06dbe_85a02ba7","updated":"2016-08-27 03:56:30.000000000","message":"Done","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"}],"glance/common/config.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"029f7eae6547d0db5cf182665d25820b990ff126","unresolved":false,"context_lines":[{"line_number":119,"context_line":"    # related rules set to admin only once this option is finally removed."},{"line_number":120,"context_line":"    cfg.BoolOpt(\u0027show_multiple_locations\u0027,"},{"line_number":121,"context_line":"                default\u003dFalse, deprecated_for_removal\u003dTrue,"},{"line_number":122,"context_line":"                help\u003d_(\u0027Whether to include the backend image locations \u0027"},{"line_number":123,"context_line":"                       \u0027in image properties. \u0027"},{"line_number":124,"context_line":"                       \u0027For example, if using the file system store a URL of \u0027"},{"line_number":125,"context_line":"                       \u0027\"file:///path/to/image\" will be returned to the user \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_0627e812","line":122,"updated":"2016-08-26 14:54:48.000000000","message":"Maybe add:\n\n    deprecated_reason\u003d_(\"\"\"\\\n    This option will be removed in the Ocata release because the same\n    functionality can be achieved with greater granularity by using\n    policies.  Please see the Newton release notes for more\n    information.\\\n    \"\"\"),\n    deprecated_since\u003d\u0027Newton\u0027","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"69a8ee6a4b4f3df4074525332c19b55a3ed271ec","unresolved":false,"context_lines":[{"line_number":119,"context_line":"    # related rules set to admin only once this option is finally removed."},{"line_number":120,"context_line":"    cfg.BoolOpt(\u0027show_multiple_locations\u0027,"},{"line_number":121,"context_line":"                default\u003dFalse, deprecated_for_removal\u003dTrue,"},{"line_number":122,"context_line":"                help\u003d_(\u0027Whether to include the backend image locations \u0027"},{"line_number":123,"context_line":"                       \u0027in image properties. \u0027"},{"line_number":124,"context_line":"                       \u0027For example, if using the file system store a URL of \u0027"},{"line_number":125,"context_line":"                       \u0027\"file:///path/to/image\" will be returned to the user \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_22c5988d","line":122,"in_reply_to":"fa7ab95a_0627e812","updated":"2016-08-27 03:37:37.000000000","message":"+1","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"116e0b2a9d8a85e9b778a0ced14a022891a90b46","unresolved":false,"context_lines":[{"line_number":119,"context_line":"    # related rules set to admin only once this option is finally removed."},{"line_number":120,"context_line":"    cfg.BoolOpt(\u0027show_multiple_locations\u0027,"},{"line_number":121,"context_line":"                default\u003dFalse, deprecated_for_removal\u003dTrue,"},{"line_number":122,"context_line":"                help\u003d_(\u0027Whether to include the backend image locations \u0027"},{"line_number":123,"context_line":"                       \u0027in image properties. \u0027"},{"line_number":124,"context_line":"                       \u0027For example, if using the file system store a URL of \u0027"},{"line_number":125,"context_line":"                       \u0027\"file:///path/to/image\" will be returned to the user \u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa7ab95a_2221184e","line":122,"in_reply_to":"fa7ab95a_0627e812","updated":"2016-08-27 03:56:30.000000000","message":"Done","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"}],"releasenotes/notes/deprecate-show-multiple-location-9890a1e961def2f6.yaml":[{"author":{"_account_id":7179,"name":"Brad Pokorny","email":"brad_pokorny@symantec.com","username":"bpokorny"},"change_message_id":"be0a652ab6dd496fccfaa06d551b481aae4f8025","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"prelude: \u003e"},{"line_number":3,"context_line":"    Deprecate the `show_multiple_locations` option in favor of a RBAC control"},{"line_number":4,"context_line":"    over locations. Maintaining two different ways to configure, enable and/or"},{"line_number":5,"context_line":"    disable a feature is painful for developers and operators. Given that we"},{"line_number":6,"context_line":"    have a more granular way to manage this feature and that it\u0027s the preferred"},{"line_number":7,"context_line":"    one given the provided ability to chose what each role can/cannot do, this"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7ac969e8_29f8697c","line":4,"updated":"2016-08-12 21:32:02.000000000","message":"Just to confirm, is the way to control this via RBAC policies with the set_image_location rule in policy.json?","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"92f334f730f9d1242dc6282fa5b072494f8a5cf2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"prelude: \u003e"},{"line_number":3,"context_line":"    Deprecate the `show_multiple_locations` option in favor of a RBAC control"},{"line_number":4,"context_line":"    over locations. Maintaining two different ways to configure, enable and/or"},{"line_number":5,"context_line":"    disable a feature is painful for developers and operators. Given that we"},{"line_number":6,"context_line":"    have a more granular way to manage this feature and that it\u0027s the preferred"},{"line_number":7,"context_line":"    one given the provided ability to chose what each role can/cannot do, this"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7ac969e8_8f035d0d","line":4,"in_reply_to":"7ac969e8_29f8697c","updated":"2016-08-12 22:20:01.000000000","message":"Brad, thanks for your question. We will post the confirmation and update the release notes accordingly as we decide to move forward here.","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"1691bd7bd289a7bde5d71a4767594b0439be29a1","unresolved":false,"context_lines":[{"line_number":8,"context_line":"    patch proposes to remove the global config option."},{"line_number":9,"context_line":"upgrade:"},{"line_number":10,"context_line":"  - For the newton release, this option will still be honored. However, it\u0027s"},{"line_number":11,"context_line":"    important to update policy.json files for glance-api nodes."},{"line_number":12,"context_line":"  - Future releases will ignore this option and just follow the policy rules."},{"line_number":13,"context_line":"    It\u0027s recommended that this option is disabled for public endpoints and"},{"line_number":14,"context_line":"    enabled internally for service-to-service communication."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1ac06dbe_c211795f","line":11,"updated":"2016-08-25 04:40:21.000000000","message":"Can you add a line here with a link to the git tree policy file?\n\nSpecifically we need to mention, the 3 policies admins need to consider as they wish to start using them:\n\ndelete_image_location, get_image_location, set_image_location","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"116e0b2a9d8a85e9b778a0ced14a022891a90b46","unresolved":false,"context_lines":[{"line_number":8,"context_line":"    patch proposes to remove the global config option."},{"line_number":9,"context_line":"upgrade:"},{"line_number":10,"context_line":"  - For the newton release, this option will still be honored. However, it\u0027s"},{"line_number":11,"context_line":"    important to update policy.json files for glance-api nodes."},{"line_number":12,"context_line":"  - Future releases will ignore this option and just follow the policy rules."},{"line_number":13,"context_line":"    It\u0027s recommended that this option is disabled for public endpoints and"},{"line_number":14,"context_line":"    enabled internally for service-to-service communication."}],"source_content_type":"text/x-yaml","patch_set":4,"id":"fa7ab95a_e299f06f","line":11,"in_reply_to":"1ac06dbe_c211795f","updated":"2016-08-27 03:56:30.000000000","message":"I ignored the link to policy in git tree but added the 3 policies in consideration.","commit_id":"22007969f89e74a84ad58e2c9a2a8c9d2401714b"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"4a22fb85dc0394add0364fd25f0733e2e3c8b5f4","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    that we have a more granular way to manage this feature"},{"line_number":8,"context_line":"    and that it is the preferred one given the provided"},{"line_number":9,"context_line":"    ability to chose what each role can/cannot do, this"},{"line_number":10,"context_line":"    patch proposes to remove the global config option."},{"line_number":11,"context_line":"upgrade:"},{"line_number":12,"context_line":"  - For the Newton release, this option will still be"},{"line_number":13,"context_line":"    honored. However, it is important to update"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"fa7ab95a_6ae4cb3a","line":10,"updated":"2016-08-28 03:41:56.000000000","message":"Mentioning what a patch proposes to do here doesn\u0027t seem appropriate for release notes (presumably the patch was approved and merged as part of the release).  I think the prelude should be a bit shorter and not so apologetic/justificatory, for example:\n\nDeprecate the ``show_multiple_locations`` option in favor of the existing RBAC control over locations.  Maintaining two different ways to configure, enable and/or disable a feature is painful for developers and operators, so the less granular means of controlling this feature will be eliminated in the Ocata release.","commit_id":"bb0545b18fdee0ab13507d482fa32a78ff834ff8"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"569481596e0522870e5b77472d5286ed6a8274d8","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    that we have a more granular way to manage this feature"},{"line_number":8,"context_line":"    and that it is the preferred one given the provided"},{"line_number":9,"context_line":"    ability to chose what each role can/cannot do, this"},{"line_number":10,"context_line":"    patch proposes to remove the global config option."},{"line_number":11,"context_line":"upgrade:"},{"line_number":12,"context_line":"  - For the Newton release, this option will still be"},{"line_number":13,"context_line":"    honored. However, it is important to update"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"fa7ab95a_be7eee16","line":10,"in_reply_to":"fa7ab95a_6ae4cb3a","updated":"2016-08-29 14:01:32.000000000","message":"Used the advice to change a few details. Please see the next PS.","commit_id":"bb0545b18fdee0ab13507d482fa32a78ff834ff8"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3ce7c803933db67e3bc7fb9e99da663b18ebb6ed","unresolved":false,"context_lines":[{"line_number":12,"context_line":"upgrade:"},{"line_number":13,"context_line":"  - For the Newton release, this option will still be"},{"line_number":14,"context_line":"    honored. However, it is important to update"},{"line_number":15,"context_line":"    policy.json file for glance-api nodes. In particular,"},{"line_number":16,"context_line":"    please consider updating the policies"},{"line_number":17,"context_line":"    ``delete_image_location``, ``get_image_location`` and"},{"line_number":18,"context_line":"    ``set_image_location`` as per your requirements. As this"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa7ab95a_7d2d42fd","line":15,"range":{"start_line":15,"start_character":4,"end_line":15,"end_character":15},"updated":"2016-08-29 16:16:32.000000000","message":"nit: surround with `` for consistency","commit_id":"c53fcf8d87f8a32264d292e5899dd1d2a7f8b41e"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"bac86f085266c95206cdb9cf52603f76a2557633","unresolved":false,"context_lines":[{"line_number":12,"context_line":"upgrade:"},{"line_number":13,"context_line":"  - For the Newton release, this option will still be"},{"line_number":14,"context_line":"    honored. However, it is important to update"},{"line_number":15,"context_line":"    policy.json file for glance-api nodes. In particular,"},{"line_number":16,"context_line":"    please consider updating the policies"},{"line_number":17,"context_line":"    ``delete_image_location``, ``get_image_location`` and"},{"line_number":18,"context_line":"    ``set_image_location`` as per your requirements. As this"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa7ab95a_3105a113","line":15,"range":{"start_line":15,"start_character":4,"end_line":15,"end_character":15},"in_reply_to":"fa7ab95a_7d2d42fd","updated":"2016-08-29 17:06:38.000000000","message":"Done","commit_id":"c53fcf8d87f8a32264d292e5899dd1d2a7f8b41e"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3ce7c803933db67e3bc7fb9e99da663b18ebb6ed","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    of your cloud."},{"line_number":22,"context_line":"  - Future releases will ignore this option and just"},{"line_number":23,"context_line":"    follow the policy rules. It is recommended that this"},{"line_number":24,"context_line":"    option is disabled for public endpoints and is being"},{"line_number":25,"context_line":"    used internally for service-to-service communication."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa7ab95a_9dbda6b9","line":24,"range":{"start_line":24,"start_character":51,"end_line":24,"end_character":56},"updated":"2016-08-29 16:16:32.000000000","message":"nit: s/being/only/","commit_id":"c53fcf8d87f8a32264d292e5899dd1d2a7f8b41e"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"bac86f085266c95206cdb9cf52603f76a2557633","unresolved":false,"context_lines":[{"line_number":21,"context_line":"    of your cloud."},{"line_number":22,"context_line":"  - Future releases will ignore this option and just"},{"line_number":23,"context_line":"    follow the policy rules. It is recommended that this"},{"line_number":24,"context_line":"    option is disabled for public endpoints and is being"},{"line_number":25,"context_line":"    used internally for service-to-service communication."}],"source_content_type":"text/x-yaml","patch_set":8,"id":"fa7ab95a_5102250c","line":24,"range":{"start_line":24,"start_character":51,"end_line":24,"end_character":56},"in_reply_to":"fa7ab95a_9dbda6b9","updated":"2016-08-29 17:06:38.000000000","message":"Done","commit_id":"c53fcf8d87f8a32264d292e5899dd1d2a7f8b41e"}]}