)]}'
{"specs/juno/graduate-oslo-middleware.rst":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"da972d4d5fcacc97a4a34959a3c208a5ce33e26d","unresolved":false,"context_lines":[{"line_number":57,"context_line":"None"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Public API"},{"line_number":60,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"To utilise the new middleware, after including oslo.middleware library,"},{"line_number":63,"context_line":"projects can declare the new middleware in wsgi pipeline as such::"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_07e179fb","line":60,"updated":"2014-08-04 20:25:07.000000000","message":"Could you make a list of the classes that need to be public? Are there any utility functions or classes that need to be exposed, too, or is it limited to just the WSGI middleware classes and the base class?","commit_id":"419284cb530557338c27a0e1788fdcc2900f9584"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"ad7d95273d4123c31c97948d459db6aa8be69a6c","unresolved":false,"context_lines":[{"line_number":57,"context_line":"None"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"Public API"},{"line_number":60,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"To utilise the new middleware, after including oslo.middleware library,"},{"line_number":63,"context_line":"projects can declare the new middleware in wsgi pipeline as such::"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_eaca5e9f","line":60,"in_reply_to":"baada198_07e179fb","updated":"2014-08-05 20:42:41.000000000","message":"Done","commit_id":"419284cb530557338c27a0e1788fdcc2900f9584"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"da972d4d5fcacc97a4a34959a3c208a5ce33e26d","unresolved":false,"context_lines":[{"line_number":63,"context_line":"projects can declare the new middleware in wsgi pipeline as such::"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    [filter:\u003cmiddleware\u003e]"},{"line_number":66,"context_line":"    paste.filter_factory \u003d oslo.middleware.\u003cmiddleware\u003e:\u003cClassName\u003e.factory"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Implementation"},{"line_number":69,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_e749e532","line":66,"updated":"2014-08-04 20:25:07.000000000","message":"If the API is the set of middleware classes, can we import them into oslo/middleware/__init__.py so we can make the implementation modules private? That would mean having\n\n  paste.filter_factory \u003d oslo.middleware:\u003cClassName\u003e.factory\n\nWe can expose the base class through its own module, since trying to import it into oslo.middleware may introduce circular dependencies.","commit_id":"419284cb530557338c27a0e1788fdcc2900f9584"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"ad7d95273d4123c31c97948d459db6aa8be69a6c","unresolved":false,"context_lines":[{"line_number":63,"context_line":"projects can declare the new middleware in wsgi pipeline as such::"},{"line_number":64,"context_line":""},{"line_number":65,"context_line":"    [filter:\u003cmiddleware\u003e]"},{"line_number":66,"context_line":"    paste.filter_factory \u003d oslo.middleware.\u003cmiddleware\u003e:\u003cClassName\u003e.factory"},{"line_number":67,"context_line":""},{"line_number":68,"context_line":"Implementation"},{"line_number":69,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_0abf92fe","line":66,"in_reply_to":"baada198_e749e532","updated":"2014-08-05 20:42:41.000000000","message":"added a note to work items","commit_id":"419284cb530557338c27a0e1788fdcc2900f9584"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"da972d4d5fcacc97a4a34959a3c208a5ce33e26d","unresolved":false,"context_lines":[{"line_number":102,"context_line":""},{"line_number":103,"context_line":"* https://wiki.openstack.org/wiki/Oslo/CreatingANewLibrary#Checklist"},{"line_number":104,"context_line":"* audit middleware will not be included. it will be included in"},{"line_number":105,"context_line":"  keystonemiddleware as part of keystone\u0027s adoption of audit"},{"line_number":106,"context_line":"* notifier middleware will not be included. it will be included in"},{"line_number":107,"context_line":"  oslo.messaging to minimise dependencies in oslo.middleware."},{"line_number":108,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baada198_4710d1f3","line":105,"updated":"2014-08-04 20:25:07.000000000","message":"I updated https://wiki.openstack.org/wiki/Oslo/GraduationStatus to reflect this change.","commit_id":"419284cb530557338c27a0e1788fdcc2900f9584"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"de8088648316fc78cdc70e7544cf363d38612ee5","unresolved":false,"context_lines":[{"line_number":1,"context_line":".."},{"line_number":2,"context_line":"  This template should be in ReSTructured text.  For help with syntax,"},{"line_number":3,"context_line":"  see http://sphinx-doc.org/rest.html"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"  To test out your formatting, build the docs using tox, or see:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_ad2267b7","line":2,"updated":"2014-08-12 14:17:03.000000000","message":"Nit: We typically remove the template boilerplate.","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"03acfaf9d54f0ec1c68f522fd33d5fd579601735","unresolved":false,"context_lines":[{"line_number":1,"context_line":".."},{"line_number":2,"context_line":"  This template should be in ReSTructured text.  For help with syntax,"},{"line_number":3,"context_line":"  see http://sphinx-doc.org/rest.html"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"  To test out your formatting, build the docs using tox, or see:"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_99931ea3","line":2,"in_reply_to":"baada198_ad2267b7","updated":"2014-08-12 16:08:37.000000000","message":"will do","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"98a1350e5465f88d99bb63ed490c8fbb84f10765","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"oslo.middleware library provides access to the following middleware::"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* CatchErrorsMiddleware"},{"line_number":65,"context_line":"* CorrelationIdMiddleware"},{"line_number":66,"context_line":"* Debug"},{"line_number":67,"context_line":"* RequestIdMiddleware"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_0c07f270","line":64,"updated":"2014-08-11 22:40:55.000000000","message":"We should talk about whether it makes sense to rename these classes to remove the redundant \"middleware\" from the class name.","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"de8088648316fc78cdc70e7544cf363d38612ee5","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"oslo.middleware library provides access to the following middleware::"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* CatchErrorsMiddleware"},{"line_number":65,"context_line":"* CorrelationIdMiddleware"},{"line_number":66,"context_line":"* Debug"},{"line_number":67,"context_line":"* RequestIdMiddleware"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_cd56bb2a","line":64,"in_reply_to":"baada198_0c07f270","updated":"2014-08-12 14:17:03.000000000","message":"I think it does.  We don\u0027t have the name collision issues we had with oslo.utils and not all of them have the Middleware suffix to begin with, so that would make the naming consistent too.","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"03acfaf9d54f0ec1c68f522fd33d5fd579601735","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"oslo.middleware library provides access to the following middleware::"},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"* CatchErrorsMiddleware"},{"line_number":65,"context_line":"* CorrelationIdMiddleware"},{"line_number":66,"context_line":"* Debug"},{"line_number":67,"context_line":"* RequestIdMiddleware"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_5976e692","line":64,"in_reply_to":"baada198_cd56bb2a","updated":"2014-08-12 16:08:37.000000000","message":"works for me. i\u0027m cool with dropping Middleware suffix","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"de8088648316fc78cdc70e7544cf363d38612ee5","unresolved":false,"context_lines":[{"line_number":118,"context_line":"Adoption Notes"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"General use of middleware remains the same. The only change in usage would be"},{"line_number":122,"context_line":"to reference middlware in oslo.middleware library rather than openstack/common"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_cda87b23","line":121,"updated":"2014-08-12 14:17:03.000000000","message":"Since this is user-facing configuration, can we provide a compatibility layer at all?","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"c7bc9ea8c42d9c0ce5767e3ad5a5d7e335302d69","unresolved":false,"context_lines":[{"line_number":118,"context_line":"Adoption Notes"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"General use of middleware remains the same. The only change in usage would be"},{"line_number":122,"context_line":"to reference middlware in oslo.middleware library rather than openstack/common"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_b4a8e721","line":121,"in_reply_to":"baada198_599fc6b2","updated":"2014-08-12 16:48:06.000000000","message":"Basically something that would map the openstack/common classes to the oslo.middleware ones so existing config files will still work once projects adopt this library.  I don\u0027t know much about how that works though.\n\nAlternatively, we could say that adopting projects should not remove the openstack/common middleware until after a deprecation period to give people a chance to update their configs.  It should be possible for the old and new classes to coexist for a time, right?","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"1c6d9a2d4a5646d57a863680139a1bed0066cba0","unresolved":false,"context_lines":[{"line_number":118,"context_line":"Adoption Notes"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"General use of middleware remains the same. The only change in usage would be"},{"line_number":122,"context_line":"to reference middlware in oslo.middleware library rather than openstack/common"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_417cef71","line":121,"in_reply_to":"baada198_65e10d2b","updated":"2014-08-13 18:29:13.000000000","message":"I\u0027m fine with coexistence.  Any bugfixes/enhancements to the lib versions will be a carrot to stop using the incubator ones. :-)","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"7d1859daaa862ac7fc8af2c47cf429e9bd80a8cc","unresolved":false,"context_lines":[{"line_number":118,"context_line":"Adoption Notes"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"General use of middleware remains the same. The only change in usage would be"},{"line_number":122,"context_line":"to reference middlware in oslo.middleware library rather than openstack/common"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_65e10d2b","line":121,"in_reply_to":"baada198_b4a8e721","updated":"2014-08-13 13:50:17.000000000","message":"yes, i don\u0027t foresee any issues with both coexisting at the same time... i could change the openstack/common version to just inherit the oslo.middleware verison... not sure that has much advantage over having them coexists unless we need to make patches to the middleware.","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"03acfaf9d54f0ec1c68f522fd33d5fd579601735","unresolved":false,"context_lines":[{"line_number":118,"context_line":"Adoption Notes"},{"line_number":119,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"General use of middleware remains the same. The only change in usage would be"},{"line_number":122,"context_line":"to reference middlware in oslo.middleware library rather than openstack/common"},{"line_number":123,"context_line":""},{"line_number":124,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baada198_599fc6b2","line":121,"in_reply_to":"baada198_cda87b23","updated":"2014-08-12 16:08:37.000000000","message":"not really sure what you mean by compatibility layer?","commit_id":"c3b3824184a4430065f18c5f10065178722d41a7"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"9706ce983282e1ba4fe520b187bf80a1f07368ec","unresolved":false,"context_lines":[{"line_number":91,"context_line":"  keystonemiddleware as part of keystone\u0027s adoption of audit"},{"line_number":92,"context_line":"* notifier middleware will not be included. it will be included in"},{"line_number":93,"context_line":"  oslo.messaging to minimise dependencies in oslo.middleware."},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Adoption Notes"},{"line_number":96,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":97,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"baada198_c1b55f1a","line":94,"updated":"2014-08-13 18:29:41.000000000","message":"Can we add a work item to wrap the incubator classes in the new deprecated decorator (https://github.com/openstack/oslo-incubator/blob/master/openstack/common/versionutils.py#L33)?  We do need to notify users that those are going away.","commit_id":"c644ef8fb0bfaaa82073f86eb3a13f2834b1d485"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"2fce124a459bc252558488306862311e3d96c003","unresolved":false,"context_lines":[{"line_number":91,"context_line":"  keystonemiddleware as part of keystone\u0027s adoption of audit"},{"line_number":92,"context_line":"* notifier middleware will not be included. it will be included in"},{"line_number":93,"context_line":"  oslo.messaging to minimise dependencies in oslo.middleware."},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"Adoption Notes"},{"line_number":96,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":97,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9ab29df4_2b04717d","line":94,"in_reply_to":"baada198_c1b55f1a","updated":"2014-08-13 20:31:43.000000000","message":"Done","commit_id":"c644ef8fb0bfaaa82073f86eb3a13f2834b1d485"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"e8ceaf3d1c3e6d129ecd7d7934ebfd2a9d261080","unresolved":false,"context_lines":[{"line_number":98,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"General use of middleware remains the same. The only change in usage would be"},{"line_number":101,"context_line":"to reference middlware in oslo.middleware library rather than openstack/common"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Dependencies"},{"line_number":104,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1abeadc6_b660f001","line":101,"updated":"2014-08-23 18:55:06.000000000","message":"There were some comments on version 3 about backwards compatibility. We need to document whatever guidance we can give developers \u0026 deployers in the body of the spec, so can you add something here based on what you and Ben worked out?","commit_id":"bd26a2c16db2b696ae4f5f105e036ca8eafd65b6"},{"author":{"_account_id":6537,"name":"gordon chung","email":"gord@live.ca","username":"chungg"},"change_message_id":"badf9d1b1cde5520881bfaad9a26734a8f875397","unresolved":false,"context_lines":[{"line_number":98,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"General use of middleware remains the same. The only change in usage would be"},{"line_number":101,"context_line":"to reference middlware in oslo.middleware library rather than openstack/common"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Dependencies"},{"line_number":104,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"1abeadc6_991f895b","line":101,"in_reply_to":"1abeadc6_b660f001","updated":"2014-08-25 13:03:48.000000000","message":"i\u0027ve added a work item to include deprecated decorator to current incubator classes above.\n\ni\u0027ll add it here as well.","commit_id":"bd26a2c16db2b696ae4f5f105e036ca8eafd65b6"}]}