)]}'
{"specs/kilo/extract-brick.rst":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"7543c91b49c91b9c7022bda88f9d9d05604f5876","unresolved":false,"context_lines":[{"line_number":22,"context_line":"The idea of \u0027brick\u0027 was created back in the Havana timeframe.  All along"},{"line_number":23,"context_line":"it was meant to be a standalone library that Cinder, Nova and any other"},{"line_number":24,"context_line":"project in OpenStack could use.  Currently brick lives in a directory"},{"line_number":25,"context_line":"inside of Cinder, which means that only Cinder can use it.   "},{"line_number":26,"context_line":""},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Proposed change"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_5381abf9","line":25,"updated":"2014-10-17 02:50:39.000000000","message":"Trailing white spaces.","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"7543c91b49c91b9c7022bda88f9d9d05604f5876","unresolved":false,"context_lines":[{"line_number":37,"context_line":"* Add brick to cinder\u0027s requirements.txt"},{"line_number":38,"context_line":"* Remove the existing cinder/brick directory"},{"line_number":39,"context_line":"* Everywhere in Cinder that uses cinder/brick will need to be modified to use"},{"line_number":40,"context_line":"  the new pypi library."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Alternatives"},{"line_number":43,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_7384ef0a","line":40,"updated":"2014-10-17 02:50:39.000000000","message":"You\u0027ll need a Nova-spec too?","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"3645ba2f37aa3c8749b17e78e355ade2fef2b1ba","unresolved":false,"context_lines":[{"line_number":37,"context_line":"* Add brick to cinder\u0027s requirements.txt"},{"line_number":38,"context_line":"* Remove the existing cinder/brick directory"},{"line_number":39,"context_line":"* Everywhere in Cinder that uses cinder/brick will need to be modified to use"},{"line_number":40,"context_line":"  the new pypi library."},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"Alternatives"},{"line_number":43,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_5bda9278","line":40,"in_reply_to":"baa201ad_7384ef0a","updated":"2014-10-17 16:29:17.000000000","message":"Step 1 is to just get Cinder to use the library.  Once that\u0027s in place, then yes, Nova will be next and it will need a separate spec.","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"7543c91b49c91b9c7022bda88f9d9d05604f5876","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Any security related issues that impact brick may impact anything in Cinder"},{"line_number":63,"context_line":"that would use the new library.   Brick will be placed into stackforge for"},{"line_number":64,"context_line":"code reviews, so anyone can contribute fixes."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Notifications impact"},{"line_number":67,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_b379f730","line":64,"updated":"2014-10-17 02:50:39.000000000","message":"How often will a new brick library be released?  Does it follow any release schedule?","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"3645ba2f37aa3c8749b17e78e355ade2fef2b1ba","unresolved":false,"context_lines":[{"line_number":61,"context_line":""},{"line_number":62,"context_line":"Any security related issues that impact brick may impact anything in Cinder"},{"line_number":63,"context_line":"that would use the new library.   Brick will be placed into stackforge for"},{"line_number":64,"context_line":"code reviews, so anyone can contribute fixes."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"Notifications impact"},{"line_number":67,"context_line":"--------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_5bc1b27e","line":64,"in_reply_to":"baa201ad_b379f730","updated":"2014-10-17 16:29:17.000000000","message":"As often as Cinder will need it until things stabilize.   Then I\u0027d probably schedule a release every OpenStack milestone as needed.","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"7543c91b49c91b9c7022bda88f9d9d05604f5876","unresolved":false,"context_lines":[{"line_number":98,"context_line":"The downside to this is that If there are new features, APIs or interface"},{"line_number":99,"context_line":"changes in the brick library, a release will have to be published in order"},{"line_number":100,"context_line":"for Cinder to get those changes.  "},{"line_number":101,"context_line":""},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"Implementation"},{"line_number":104,"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":"baa201ad_5398cb69","line":101,"updated":"2014-10-17 02:50:39.000000000","message":"Ok, I\u0027ve already asked about the release schedule.  Will it be released whenever it is needed by Cinder or Nova?","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"7543c91b49c91b9c7022bda88f9d9d05604f5876","unresolved":false,"context_lines":[{"line_number":124,"context_line":"  exists in pypi."},{"line_number":125,"context_line":"* Then a patch to Cinder can be made to add it to the requirements.txt"},{"line_number":126,"context_line":"* The another patch to follow that removes brick/cinder and adjusts all of"},{"line_number":127,"context_line":"  Cinder to use the new library."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_13b9e3cb","line":127,"updated":"2014-10-17 02:50:39.000000000","message":"And a similar patch goes to Nova.","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"7543c91b49c91b9c7022bda88f9d9d05604f5876","unresolved":false,"context_lines":[{"line_number":136,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":137,"context_line":""},{"line_number":138,"context_line":"All of the existing unit tests in cinder for brick can remain to ensure that it"},{"line_number":139,"context_line":"works as advertised.   The existing Cinder brick unit tests were migrated to"},{"line_number":140,"context_line":"the new project and run there as well."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_b3c73744","line":139,"updated":"2014-10-17 02:50:39.000000000","message":"were migrated -\u003e will be migrated","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"7543c91b49c91b9c7022bda88f9d9d05604f5876","unresolved":false,"context_lines":[{"line_number":143,"context_line":"Documentation Impact"},{"line_number":144,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":145,"context_line":""},{"line_number":146,"context_line":"None."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"References"}],"source_content_type":"text/x-rst","patch_set":2,"id":"baa201ad_93d31303","line":146,"updated":"2014-10-17 02:50:39.000000000","message":"I think this is an important change that needs to be documented somewhere.","commit_id":"31ed532dbbb1f620ae323962ae8d36afcaf4b6f6"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f40a7a8d38c2e0164abcd770388d3355d561b35c","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Then a patch to Cinder can be made to add it to the requirements.txt"},{"line_number":126,"context_line":"* The another patch to follow that removes brick/cinder and adjusts all of"},{"line_number":127,"context_line":"  Cinder to use the new library."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Dependencies"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baa201ad_bb629e3b","line":128,"updated":"2014-10-17 16:44:36.000000000","message":"Please ensure that code changes going into Brick between now and removing Brick from Cinder get synched over fully.","commit_id":"cb21207698bdb2d395fe7e1f42673bc3eadb2663"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d2fab90aa055a33a9ba025239bc79123cf34bb42","unresolved":false,"context_lines":[{"line_number":125,"context_line":"* Then a patch to Cinder can be made to add it to the requirements.txt"},{"line_number":126,"context_line":"* The another patch to follow that removes brick/cinder and adjusts all of"},{"line_number":127,"context_line":"  Cinder to use the new library."},{"line_number":128,"context_line":""},{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Dependencies"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9aa7fdbe_8fa2d598","line":128,"in_reply_to":"baa201ad_bb629e3b","updated":"2014-10-17 21:37:33.000000000","message":"I\u0027ve been trying my best to track these.","commit_id":"cb21207698bdb2d395fe7e1f42673bc3eadb2663"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"f40a7a8d38c2e0164abcd770388d3355d561b35c","unresolved":false,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Dependencies"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"None"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":3,"id":"baa201ad_bb145ea0","line":132,"updated":"2014-10-17 16:44:36.000000000","message":"I suppose this section is for dependencies for the effort and not the code itself, but it might be good to note in here somewhere that the library will still depend on openstack libraries such as *utils from oslo.incubator and oslo.i18n.","commit_id":"cb21207698bdb2d395fe7e1f42673bc3eadb2663"},{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d2fab90aa055a33a9ba025239bc79123cf34bb42","unresolved":false,"context_lines":[{"line_number":129,"context_line":""},{"line_number":130,"context_line":"Dependencies"},{"line_number":131,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":132,"context_line":""},{"line_number":133,"context_line":"None"},{"line_number":134,"context_line":""},{"line_number":135,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":3,"id":"9aa7fdbe_6f9e9158","line":132,"in_reply_to":"baa201ad_bb145ea0","updated":"2014-10-17 21:37:33.000000000","message":"The library has a requirements.txt and test-requirements.txt file just as Cinder does with it\u0027s deps.","commit_id":"cb21207698bdb2d395fe7e1f42673bc3eadb2663"}]}