)]}'
{"specs/kilo/drop-namespace-packages.rst":[{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"f72aa344158462fea66557b9ba79e64556ab988a","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  from oslo_foo import bar"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"To avoid issues with distros, we should not rename the ``oslo.foo``"},{"line_number":53,"context_line":"library to ``oslo_foo``."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"To support backwards compatibility, we will provide shadow-packages"},{"line_number":56,"context_line":"for the ``oslo`` namespace so that the old import form still works for"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_5fc28534","line":53,"updated":"2014-10-17 12:50:08.000000000","message":"This might end up being confusing though... Are there rules in Python about the name of the package vs. the name of the library ? Is it acceptable to have foo.bar library deliver package foo_bar ?","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2f23aee5827e3ba03db46f33b878191dc27ab20f","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  from oslo_foo import bar"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"To avoid issues with distros, we should not rename the ``oslo.foo``"},{"line_number":53,"context_line":"library to ``oslo_foo``."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"To support backwards compatibility, we will provide shadow-packages"},{"line_number":56,"context_line":"for the ``oslo`` namespace so that the old import form still works for"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_408978c9","line":53,"in_reply_to":"9aa7fdbe_c4acbc38","updated":"2014-10-20 19:31:48.000000000","message":"Done","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"7f9a87ec01ba14f61b9d0ab9f19fc3f7fece335f","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  from oslo_foo import bar"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"To avoid issues with distros, we should not rename the ``oslo.foo``"},{"line_number":53,"context_line":"library to ``oslo_foo``."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"To support backwards compatibility, we will provide shadow-packages"},{"line_number":56,"context_line":"for the ``oslo`` namespace so that the old import form still works for"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_cadf920b","line":53,"in_reply_to":"baa201ad_5fc28534","updated":"2014-10-17 15:04:46.000000000","message":"There are several packages named `foo.py` that actually only deliver the `foo` module, so package naming is not strictly required (or entirely expected) to match package contents.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"029c66d5f122b4e9892a7fca778b7a13eb320704","unresolved":false,"context_lines":[{"line_number":50,"context_line":"  from oslo_foo import bar"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"To avoid issues with distros, we should not rename the ``oslo.foo``"},{"line_number":53,"context_line":"library to ``oslo_foo``."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"To support backwards compatibility, we will provide shadow-packages"},{"line_number":56,"context_line":"for the ``oslo`` namespace so that the old import form still works for"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_c4acbc38","line":53,"in_reply_to":"baa201ad_5fc28534","updated":"2014-10-19 01:38:37.000000000","message":"There aren\u0027t strict technical rules. I agree it could be a source of confusion, and if it proves to be trouble we could rename the packages as well later. As a first step I thought we should be as unobtrusive as we can.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"8f17150761d5bcbc829dc8737da3cc98fd053c43","unresolved":false,"context_lines":[{"line_number":56,"context_line":"for the ``oslo`` namespace so that the old import form still works for"},{"line_number":57,"context_line":"the Kilo releases of the libraries. That will give us time to update"},{"line_number":58,"context_line":"all of the applications before dropping the namespace packages"},{"line_number":59,"context_line":"entirely during the L release."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"Moving to regular packages will also let us move our unit tests inside"},{"line_number":62,"context_line":"the library, so they are delivered and installed under ``oslo_foo``,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_0011795c","line":59,"updated":"2014-10-17 16:20:14.000000000","message":"You may need to wait until the M release (or later depending on support lifecycle). Juno will still be active in the L release and will likely need the shadow packages.\n\nI highly recommend marking the shadow packages as deprecated (with warnings on import) and import everything into the shadowpackage namespace that is relevant (a simple import * from \u003cxxxx\u003e with appropriately defined __all__ in the main package).","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2f23aee5827e3ba03db46f33b878191dc27ab20f","unresolved":false,"context_lines":[{"line_number":56,"context_line":"for the ``oslo`` namespace so that the old import form still works for"},{"line_number":57,"context_line":"the Kilo releases of the libraries. That will give us time to update"},{"line_number":58,"context_line":"all of the applications before dropping the namespace packages"},{"line_number":59,"context_line":"entirely during the L release."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"Moving to regular packages will also let us move our unit tests inside"},{"line_number":62,"context_line":"the library, so they are delivered and installed under ``oslo_foo``,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_60867cbb","line":59,"in_reply_to":"9aa7fdbe_04a7a450","updated":"2014-10-20 19:31:48.000000000","message":"Done","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"029c66d5f122b4e9892a7fca778b7a13eb320704","unresolved":false,"context_lines":[{"line_number":56,"context_line":"for the ``oslo`` namespace so that the old import form still works for"},{"line_number":57,"context_line":"the Kilo releases of the libraries. That will give us time to update"},{"line_number":58,"context_line":"all of the applications before dropping the namespace packages"},{"line_number":59,"context_line":"entirely during the L release."},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"Moving to regular packages will also let us move our unit tests inside"},{"line_number":62,"context_line":"the library, so they are delivered and installed under ``oslo_foo``,"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_04a7a450","line":59,"in_reply_to":"baa201ad_0011795c","updated":"2014-10-19 01:38:37.000000000","message":"Good points, I will update to include the changes.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":12000,"name":"Ian Cordasco","email":"sigmavirus24@gmail.com","username":"sigmavirus24"},"change_message_id":"46be7bcb1713ae41bf9c51cd15a03f88524b4636","unresolved":false,"context_lines":[{"line_number":76,"context_line":"   cost of shipping a real ``oslo`` library"},{"line_number":77,"context_line":"   (https://github.com/rbtcollins/oslo) and maintaining the override"},{"line_number":78,"context_line":"   code ourselves. We don\u0027t know what other issues we might find with"},{"line_number":79,"context_line":"   namespace pacakges, though, so I\u0027m more comfortable with just"},{"line_number":80,"context_line":"   dropping them entirely."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"2. Python 3 has better native namespace package support, so we could"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_4cccb7bb","line":79,"updated":"2014-10-16 18:37:18.000000000","message":"Typo: packages","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2f23aee5827e3ba03db46f33b878191dc27ab20f","unresolved":false,"context_lines":[{"line_number":76,"context_line":"   cost of shipping a real ``oslo`` library"},{"line_number":77,"context_line":"   (https://github.com/rbtcollins/oslo) and maintaining the override"},{"line_number":78,"context_line":"   code ourselves. We don\u0027t know what other issues we might find with"},{"line_number":79,"context_line":"   namespace pacakges, though, so I\u0027m more comfortable with just"},{"line_number":80,"context_line":"   dropping them entirely."},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"2. Python 3 has better native namespace package support, so we could"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_809360f9","line":79,"in_reply_to":"baa201ad_4cccb7bb","updated":"2014-10-20 19:31:48.000000000","message":"Done","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"change_message_id":"3a0d66e244676b0165f61f473e759cfa5ef7cd52","unresolved":false,"context_lines":[{"line_number":86,"context_line":"   systems when they come up. If we choose this option, we might want"},{"line_number":87,"context_line":"   to write a script \"why_is_oslo_broken.sh\" to put in the incubator."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"The \"oslo\\_\" name prefix was selected to distinguish the libs from the"},{"line_number":90,"context_line":"non-production libraries like oslotest and oslosphinx. We could"},{"line_number":91,"context_line":"theoretially rename those as well so that all of our branded libraries"},{"line_number":92,"context_line":"have the same naming convention. I don\u0027t know if that\u0027s worth doing or"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_073d288f","line":89,"updated":"2014-10-16 19:32:12.000000000","message":"Typo? Should it be \"oslo_\"?","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2f23aee5827e3ba03db46f33b878191dc27ab20f","unresolved":false,"context_lines":[{"line_number":86,"context_line":"   systems when they come up. If we choose this option, we might want"},{"line_number":87,"context_line":"   to write a script \"why_is_oslo_broken.sh\" to put in the incubator."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"The \"oslo\\_\" name prefix was selected to distinguish the libs from the"},{"line_number":90,"context_line":"non-production libraries like oslotest and oslosphinx. We could"},{"line_number":91,"context_line":"theoretially rename those as well so that all of our branded libraries"},{"line_number":92,"context_line":"have the same naming convention. I don\u0027t know if that\u0027s worth doing or"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_00a05042","line":89,"in_reply_to":"9aa7fdbe_71a58b03","updated":"2014-10-20 19:31:48.000000000","message":"Yeah, ``foo`` marks foo as literal text and the parser ignores the fact that the literal looks like a link reference.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"5ff745ee475476bdf8715d65ca96887b9782d743","unresolved":false,"context_lines":[{"line_number":86,"context_line":"   systems when they come up. If we choose this option, we might want"},{"line_number":87,"context_line":"   to write a script \"why_is_oslo_broken.sh\" to put in the incubator."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"The \"oslo\\_\" name prefix was selected to distinguish the libs from the"},{"line_number":90,"context_line":"non-production libraries like oslotest and oslosphinx. We could"},{"line_number":91,"context_line":"theoretially rename those as well so that all of our branded libraries"},{"line_number":92,"context_line":"have the same naming convention. I don\u0027t know if that\u0027s worth doing or"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_a2cb6770","line":89,"in_reply_to":"baa201ad_073d288f","updated":"2014-10-16 22:31:15.000000000","message":"I think this is so the rst will format correctly.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"029c66d5f122b4e9892a7fca778b7a13eb320704","unresolved":false,"context_lines":[{"line_number":86,"context_line":"   systems when they come up. If we choose this option, we might want"},{"line_number":87,"context_line":"   to write a script \"why_is_oslo_broken.sh\" to put in the incubator."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"The \"oslo\\_\" name prefix was selected to distinguish the libs from the"},{"line_number":90,"context_line":"non-production libraries like oslotest and oslosphinx. We could"},{"line_number":91,"context_line":"theoretially rename those as well so that all of our branded libraries"},{"line_number":92,"context_line":"have the same naming convention. I don\u0027t know if that\u0027s worth doing or"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_223db78f","line":89,"in_reply_to":"baa201ad_073d288f","updated":"2014-10-19 01:38:37.000000000","message":"Right, in order for the reST to render properly, I have to escape the underscore to prevent \"oslo_\" from looking like I am trying to make a link.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"change_message_id":"a151c64831c0788c4632057a22a3f56a483c4416","unresolved":false,"context_lines":[{"line_number":86,"context_line":"   systems when they come up. If we choose this option, we might want"},{"line_number":87,"context_line":"   to write a script \"why_is_oslo_broken.sh\" to put in the incubator."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"The \"oslo\\_\" name prefix was selected to distinguish the libs from the"},{"line_number":90,"context_line":"non-production libraries like oslotest and oslosphinx. We could"},{"line_number":91,"context_line":"theoretially rename those as well so that all of our branded libraries"},{"line_number":92,"context_line":"have the same naming convention. I don\u0027t know if that\u0027s worth doing or"}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_71a58b03","line":89,"in_reply_to":"baa201ad_223db78f","updated":"2014-10-19 19:16:23.000000000","message":"I\u0027d thought that it being in quotes would prevent that, but I played with it and I see reST does see it as a link.   What\u0027s interesting is if you use ``oslo_`` instead it doesn\u0027t think it is a link.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"change_message_id":"3a0d66e244676b0165f61f473e759cfa5ef7cd52","unresolved":false,"context_lines":[{"line_number":119,"context_line":"Developers need to be aware of the import changes. We have a few"},{"line_number":120,"context_line":"options for communicating that widely:"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"1. Add a depreation warning to the oslo namespace packages so"},{"line_number":123,"context_line":"   importing them reports a warning."},{"line_number":124,"context_line":"2. Add a hacking rule to make importing \"from oslo.\" illegal."},{"line_number":125,"context_line":"3. Rely on reviewers to catch it."}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_e76024a7","line":122,"updated":"2014-10-16 19:32:12.000000000","message":"typo: deprecation","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2f23aee5827e3ba03db46f33b878191dc27ab20f","unresolved":false,"context_lines":[{"line_number":119,"context_line":"Developers need to be aware of the import changes. We have a few"},{"line_number":120,"context_line":"options for communicating that widely:"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"1. Add a depreation warning to the oslo namespace packages so"},{"line_number":123,"context_line":"   importing them reports a warning."},{"line_number":124,"context_line":"2. Add a hacking rule to make importing \"from oslo.\" illegal."},{"line_number":125,"context_line":"3. Rely on reviewers to catch it."}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_a09064ed","line":122,"in_reply_to":"baa201ad_e76024a7","updated":"2014-10-20 19:31:48.000000000","message":"Done","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"change_message_id":"3a0d66e244676b0165f61f473e759cfa5ef7cd52","unresolved":false,"context_lines":[{"line_number":224,"context_line":"* My WIP patch for oslo.i18n to move it out of the namespace package:"},{"line_number":225,"context_line":"  https://review.openstack.org/#/c/127323/"},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"Please add any useful references here. You are not required to have any"},{"line_number":228,"context_line":"reference. Moreover, this specification should still make sense when your"},{"line_number":229,"context_line":"references are unavailable. Examples of what you could include are:"},{"line_number":230,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_c765e0af","line":227,"updated":"2014-10-16 19:32:12.000000000","message":"Probably should remove the guidelines from the template","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2f23aee5827e3ba03db46f33b878191dc27ab20f","unresolved":false,"context_lines":[{"line_number":224,"context_line":"* My WIP patch for oslo.i18n to move it out of the namespace package:"},{"line_number":225,"context_line":"  https://review.openstack.org/#/c/127323/"},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"Please add any useful references here. You are not required to have any"},{"line_number":228,"context_line":"reference. Moreover, this specification should still make sense when your"},{"line_number":229,"context_line":"references are unavailable. Examples of what you could include are:"},{"line_number":230,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_20bc7469","line":227,"in_reply_to":"9aa7fdbe_f1b89b2c","updated":"2014-10-20 19:31:48.000000000","message":"Done","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":6601,"name":"James Carey","email":"jecarey@yerac.com","username":"jecarey"},"change_message_id":"a151c64831c0788c4632057a22a3f56a483c4416","unresolved":false,"context_lines":[{"line_number":224,"context_line":"* My WIP patch for oslo.i18n to move it out of the namespace package:"},{"line_number":225,"context_line":"  https://review.openstack.org/#/c/127323/"},{"line_number":226,"context_line":""},{"line_number":227,"context_line":"Please add any useful references here. You are not required to have any"},{"line_number":228,"context_line":"reference. Moreover, this specification should still make sense when your"},{"line_number":229,"context_line":"references are unavailable. Examples of what you could include are:"},{"line_number":230,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"9aa7fdbe_f1b89b2c","line":227,"in_reply_to":"baa201ad_c765e0af","updated":"2014-10-19 19:16:23.000000000","message":"Opps, that could be read the wrong way ... I meant leave them in the template, but remove them from here.   I like having them in the template.","commit_id":"1b39f022dd740ab904185eab7365e205c96e78fa"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"ae2a51e17520cb8cefa853cd8b99869ef0ce5f7a","unresolved":false,"context_lines":[{"line_number":29,"context_line":""},{"line_number":30,"context_line":"We also see trouble with virtualenvs configured to allow access to the"},{"line_number":31,"context_line":"global site-packages directory, as nova needs for libvirt access. If"},{"line_number":32,"context_line":"anything from the ``oslo`` namespace pacakge is installed globally, it"},{"line_number":33,"context_line":"will shadow the versions visible in the virtualenv. That would"},{"line_number":34,"context_line":"normally lead to a version error if some old version is installed"},{"line_number":35,"context_line":"globally, but in this case it can also lead to import errors because"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9aa7fdbe_b6510038","line":32,"updated":"2014-10-24 20:32:10.000000000","message":"Nit: *package","commit_id":"8492eaf4bffb7942986d7a8bacb8085705b974bd"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"4b8f55ea089e1f332143fde4ce2b1a9b2551b802","unresolved":false,"context_lines":[{"line_number":68,"context_line":"so we should do that at the same time, retaining just enough of the"},{"line_number":69,"context_line":"test suite to ensure that the public API exposed through the namespace"},{"line_number":70,"context_line":"package still works."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Alternatives"},{"line_number":73,"context_line":"------------"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_5521fb4b","line":71,"updated":"2014-11-13 08:26:14.000000000","message":"We might want to mention that new packages will still use `.` in their names to keep consistency with the existing ones.","commit_id":"8492eaf4bffb7942986d7a8bacb8085705b974bd"},{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"4b8f55ea089e1f332143fde4ce2b1a9b2551b802","unresolved":false,"context_lines":[{"line_number":68,"context_line":"so we should do that at the same time, retaining just enough of the"},{"line_number":69,"context_line":"test suite to ensure that the public API exposed through the namespace"},{"line_number":70,"context_line":"package still works."},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"Alternatives"},{"line_number":73,"context_line":"------------"},{"line_number":74,"context_line":""}],"source_content_type":"text/x-rst","patch_set":3,"id":"5a890539_751c3714","line":71,"updated":"2014-11-13 08:26:14.000000000","message":"We might want to mention that new packages will still use `.` in their names to keep consistency with the existing ones.","commit_id":"8492eaf4bffb7942986d7a8bacb8085705b974bd"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"ae2a51e17520cb8cefa853cd8b99869ef0ce5f7a","unresolved":false,"context_lines":[{"line_number":82,"context_line":"   (https://github.com/rbtcollins/oslo) and maintaining the override"},{"line_number":83,"context_line":"   code ourselves. We don\u0027t know what other issues we might find with"},{"line_number":84,"context_line":"   namespace packages, though, so I\u0027m more comfortable with just"},{"line_number":85,"context_line":"   dropping them entirely."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"2. Python 3 has better native namespace package support, so we could"},{"line_number":88,"context_line":"   review the decision when we move to Python 3."}],"source_content_type":"text/x-rst","patch_set":3,"id":"9aa7fdbe_c16c00d3","line":85,"updated":"2014-10-24 20:32:10.000000000","message":"The benefit this option has is that if we try it and it turns out to not be viable we can always fall back to dropping namespaces completely and we\u0027re in basically no worse shape than we are today.  And the best case scenario is we don\u0027t have to go through the churn of removing namespaces.","commit_id":"8492eaf4bffb7942986d7a8bacb8085705b974bd"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"ae2a51e17520cb8cefa853cd8b99869ef0ce5f7a","unresolved":false,"context_lines":[{"line_number":84,"context_line":"   namespace packages, though, so I\u0027m more comfortable with just"},{"line_number":85,"context_line":"   dropping them entirely."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"2. Python 3 has better native namespace package support, so we could"},{"line_number":88,"context_line":"   review the decision when we move to Python 3."},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"3. Do nothing, and continue to help developers debug problems on their"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9aa7fdbe_c155e0fa","line":87,"updated":"2014-10-24 20:32:10.000000000","message":"I don\u0027t know that this needs to be mentioned in the spec, but if we go down the no namespaces path I don\u0027t think we can really go back, even once we\u0027re pure Python 3.  Not only is that a long ways off in the future, but it would also be quite painful to switch back again.\n\nOf course, that\u0027s why this is an alternative and not the proposed plan. :-)","commit_id":"8492eaf4bffb7942986d7a8bacb8085705b974bd"}]}