)]}'
{"specs/kilo/replace-snet-config-with-endpoint-config.rst":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"28c8dabf5e18786b3147acf949d232373d6b7672","unresolved":false,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"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"},{"line_number":8,"context_line":"Smooth Deletes Over Time"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/glance/+spec/replace-snet-config-with-endpoint-config"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a930d6b_7b77f9d4","line":8,"updated":"2015-01-22 05:58:11.000000000","message":"This title doesn\u0027t seem to reflect what this spec is about.","commit_id":"3322acd3de92da70d751d42602555b97557117d2"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"32b0829b154d0e339a57dc0319fb10446bf4b302","unresolved":false,"context_lines":[{"line_number":5,"context_line":" http://creativecommons.org/licenses/by/3.0/legalcode"},{"line_number":6,"context_line":""},{"line_number":7,"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"},{"line_number":8,"context_line":"Smooth Deletes Over Time"},{"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"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"https://blueprints.launchpad.net/glance/+spec/replace-snet-config-with-endpoint-config"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a930d6b_9bd72e23","line":8,"in_reply_to":"1a930d6b_7b77f9d4","updated":"2015-01-22 17:01:12.000000000","message":"Copy pasta error. I\u0027ll fix.","commit_id":"3322acd3de92da70d751d42602555b97557117d2"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"28c8dabf5e18786b3147acf949d232373d6b7672","unresolved":false,"context_lines":[{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Instead of constructing a URL with a prefix from what is returned by"},{"line_number":27,"context_line":"auth, specify the URL via configuration."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Alternatives"},{"line_number":30,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a930d6b_9b090534","line":27,"updated":"2015-01-22 05:58:11.000000000","message":"might be good to say \"specify the full URL\" to make it clear this spec isn\u0027t about providing flexibility for the prefix appended to the swift publicURL in the service catalog.  Also, how much of the URL is specified in the config?  Is it just the baseUrl and not the swift account info part of the URL (which would still come from the service catalog)?","commit_id":"3322acd3de92da70d751d42602555b97557117d2"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"32b0829b154d0e339a57dc0319fb10446bf4b302","unresolved":false,"context_lines":[{"line_number":24,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"Instead of constructing a URL with a prefix from what is returned by"},{"line_number":27,"context_line":"auth, specify the URL via configuration."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"Alternatives"},{"line_number":30,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a930d6b_5bedc6ee","line":27,"in_reply_to":"1a930d6b_9b090534","updated":"2015-01-22 17:01:12.000000000","message":"Good call. I\u0027ll add detail here.","commit_id":"3322acd3de92da70d751d42602555b97557117d2"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"28c8dabf5e18786b3147acf949d232373d6b7672","unresolved":false,"context_lines":[{"line_number":114,"context_line":"Documentation Impact"},{"line_number":115,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"* Document new scrubber configuration options."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"References"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a930d6b_3b457170","line":117,"updated":"2015-01-22 05:58:11.000000000","message":"This looks like it\u0027s from a different spec.","commit_id":"3322acd3de92da70d751d42602555b97557117d2"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"32b0829b154d0e339a57dc0319fb10446bf4b302","unresolved":false,"context_lines":[{"line_number":114,"context_line":"Documentation Impact"},{"line_number":115,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":116,"context_line":""},{"line_number":117,"context_line":"* Document new scrubber configuration options."},{"line_number":118,"context_line":""},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"References"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a930d6b_fbce7282","line":117,"in_reply_to":"1a930d6b_3b457170","updated":"2015-01-22 17:01:12.000000000","message":"bah...more copy pasta","commit_id":"3322acd3de92da70d751d42602555b97557117d2"},{"author":{"_account_id":11864,"name":"Ben Roble","email":"ben.roble@rackspace.com","username":"broble"},"change_message_id":"145fddcb6527e29d46c99d9a6681b1ad16c2f648","unresolved":false,"context_lines":[{"line_number":32,"context_line":"------------"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Only use auth v2. Store and retrieve multiple internal endpoints from"},{"line_number":35,"context_line":"the catalog."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Data model impact"},{"line_number":38,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a930d6b_4436c253","line":35,"updated":"2015-01-22 19:54:30.000000000","message":"Perhaps add a one-sentence explanation as to why this alternative is not taken instead of the proposed change.","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"8999a7fc8d8b7cb503b2ced963cf793bb76fe656","unresolved":false,"context_lines":[{"line_number":32,"context_line":"------------"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"Only use auth v2. Store and retrieve multiple internal endpoints from"},{"line_number":35,"context_line":"the catalog."},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"Data model impact"},{"line_number":38,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a930d6b_53ed41b6","line":35,"in_reply_to":"1a930d6b_4436c253","updated":"2015-01-26 19:37:04.000000000","message":"Done","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11864,"name":"Ben Roble","email":"ben.roble@rackspace.com","username":"broble"},"change_message_id":"145fddcb6527e29d46c99d9a6681b1ad16c2f648","unresolved":false,"context_lines":[{"line_number":67,"context_line":"Other deployer impact"},{"line_number":68,"context_line":"---------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Configuration options will change."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Developer impact"},{"line_number":73,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a930d6b_e49a4e6d","line":70,"updated":"2015-01-22 19:54:30.000000000","message":"Think it would be good here to explicitly point out that default configs will change, so possibily \"Default configuration options will change.\"","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11864,"name":"Ben Roble","email":"ben.roble@rackspace.com","username":"broble"},"change_message_id":"d53b07d6870d5ccdb2967aec9bc1238b50627c17","unresolved":false,"context_lines":[{"line_number":67,"context_line":"Other deployer impact"},{"line_number":68,"context_line":"---------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Configuration options will change."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Developer impact"},{"line_number":73,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa81d914_400cf542","line":70,"in_reply_to":"1a930d6b_13257908","updated":"2015-01-28 20:59:53.000000000","message":"Ah, that makes sense. Still I wonder if its worth keeping backwards compatibility here, such as keep in \"swift_enable_snet\" but override it if \"swift_store_endpoint\" is set. That way, deployers who are happy with how their current setup works will continue to be happy when this change is released, and deployers who want to use this new option will be able to.\n\nThoughts on this?","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"8999a7fc8d8b7cb503b2ced963cf793bb76fe656","unresolved":false,"context_lines":[{"line_number":67,"context_line":"Other deployer impact"},{"line_number":68,"context_line":"---------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Configuration options will change."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Developer impact"},{"line_number":73,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a930d6b_13257908","line":70,"in_reply_to":"1a930d6b_e49a4e6d","updated":"2015-01-26 19:37:04.000000000","message":"Neither using snet or the endpoint config are default options, so the default behavior should not change. Only if a deployer has configured to use snet should they need to update to use the endpoint.","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"f828b4ea464df99bceb109c1fcca658299bdf1c1","unresolved":false,"context_lines":[{"line_number":67,"context_line":"Other deployer impact"},{"line_number":68,"context_line":"---------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Configuration options will change."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Developer impact"},{"line_number":73,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa81d914_f5776a39","line":70,"in_reply_to":"fa81d914_400cf542","updated":"2015-01-28 21:26:28.000000000","message":"swift_enable_snet is Rackspace only and no longer required with this change: https://github.com/openstack/glance/commit/09a67e4fb4e5c73cfa01451a7460a8c12d6f64da","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11864,"name":"Ben Roble","email":"ben.roble@rackspace.com","username":"broble"},"change_message_id":"5a4ce687be4ea53f0929a311152d3bafb7afc7e7","unresolved":false,"context_lines":[{"line_number":67,"context_line":"Other deployer impact"},{"line_number":68,"context_line":"---------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Configuration options will change."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Developer impact"},{"line_number":73,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"fa81d914_f835a1db","line":70,"in_reply_to":"fa81d914_f5776a39","updated":"2015-01-28 21:36:31.000000000","message":"Thanks for the context","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11864,"name":"Ben Roble","email":"ben.roble@rackspace.com","username":"broble"},"change_message_id":"145fddcb6527e29d46c99d9a6681b1ad16c2f648","unresolved":false,"context_lines":[{"line_number":68,"context_line":"---------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Configuration options will change."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Developer impact"},{"line_number":73,"context_line":"----------------"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a930d6b_c4f2b299","line":71,"updated":"2015-01-22 19:54:30.000000000","message":"Right now this spec is vague in reguards to exactly what config options are changing so I really want to see something like:\n\nRemoved config option: \"swift_enable_snet\"\n\nAdded config option \"swift_store_endpoint\"\nThe default value of \"swift_store_endpoint\" is None, in which case the storage url from the auth response will be used. If set, the configured endpoint will be used. Example values: \"swift_store_endpoint\" \u003d \"https://www.example.com/v1/not_a_container\"","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"8999a7fc8d8b7cb503b2ced963cf793bb76fe656","unresolved":false,"context_lines":[{"line_number":68,"context_line":"---------------------"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"Configuration options will change."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Developer impact"},{"line_number":73,"context_line":"----------------"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a930d6b_1353b96b","line":71,"in_reply_to":"1a930d6b_c4f2b299","updated":"2015-01-26 19:37:04.000000000","message":"Done","commit_id":"e168d9ceba9a18f21e1be959057934dd8dbbd5fc"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ba1f4b90bed460e22b96f1fb0be348bf291bd894","unresolved":false,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Configuration options will change:"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"- Removed config option: \"swift_enable_snet\""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"- Added config option \"swift_store_endpoint\" The default value of"},{"line_number":78,"context_line":"  \"swift_store_endpoint\" is None, in which case the storage url from the auth"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa81d914_2ffd263b","line":75,"updated":"2015-01-29 00:37:02.000000000","message":"Given the discussion in the associated code patch, I think it would be worth stating that the config option being removed (1) was \u0027false\u0027 by default, and (2) there was a comment in the config file saying \"don\u0027t change this if you are not Rackspace\".\n\nHere\u0027s the link to the api conf from Juno release:\nhttps://github.com/openstack/glance/blob/fd5a55c7f386a9d9441d5f1291ff6a92f7e6cc1b/etc/glance-api.conf#L525","commit_id":"b0d69f70f1ccd633ffbbc7c5d85d272046e59cad"},{"author":{"_account_id":11642,"name":"Jesse J. Cook","email":"jesse.cook@rackspace.com","username":"crashenx"},"change_message_id":"47891d7b85e3fb051c5b567f580ddab2bf3401cc","unresolved":false,"context_lines":[{"line_number":72,"context_line":""},{"line_number":73,"context_line":"Configuration options will change:"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"- Removed config option: \"swift_enable_snet\""},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"- Added config option \"swift_store_endpoint\" The default value of"},{"line_number":78,"context_line":"  \"swift_store_endpoint\" is None, in which case the storage url from the auth"}],"source_content_type":"text/x-rst","patch_set":3,"id":"fa81d914_15ccd7bb","line":75,"in_reply_to":"fa81d914_2ffd263b","updated":"2015-01-29 14:31:47.000000000","message":"Done","commit_id":"b0d69f70f1ccd633ffbbc7c5d85d272046e59cad"}]}