)]}'
{"specs/liberty/deprecate-bash-ramdisk.rst":[{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"5175c394fe7ff7ba44abcb9e4f25b00c49090460","unresolved":false,"context_lines":[{"line_number":29,"context_line":"Proposed change"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"We can not simple delete the code that the bash ramdisk uses, therefore"},{"line_number":33,"context_line":"we should start adding deprecation messages on the API endpoints that the"},{"line_number":34,"context_line":"bash ramdisk uses to alert the operators that it\u0027s going to be removed."},{"line_number":35,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_e1390ba5","line":32,"updated":"2015-07-23 17:07:05.000000000","message":"Nit: s/simple/simply/","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"885eaaa2986db29fc6e29803817066fa291ea840","unresolved":false,"context_lines":[{"line_number":29,"context_line":"Proposed change"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"We can not simple delete the code that the bash ramdisk uses, therefore"},{"line_number":33,"context_line":"we should start adding deprecation messages on the API endpoints that the"},{"line_number":34,"context_line":"bash ramdisk uses to alert the operators that it\u0027s going to be removed."},{"line_number":35,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_125658d2","line":32,"in_reply_to":"3a50d1a3_e1390ba5","updated":"2015-07-27 10:48:10.000000000","message":"Done","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d00ac5fd3a8976572b64cde40766705b1f7e23c6","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"We can not simple delete the code that the bash ramdisk uses, therefore"},{"line_number":33,"context_line":"we should start adding deprecation messages on the API endpoints that the"},{"line_number":34,"context_line":"bash ramdisk uses to alert the operators that it\u0027s going to be removed."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":37,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_9d016d3c","line":34,"updated":"2015-07-24 07:29:36.000000000","message":"++ for DIB element. We probably can\u0027t delete API endpoints until DIB element is *deleted* (and new version of DIB is released).","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"5175c394fe7ff7ba44abcb9e4f25b00c49090460","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"We can not simple delete the code that the bash ramdisk uses, therefore"},{"line_number":33,"context_line":"we should start adding deprecation messages on the API endpoints that the"},{"line_number":34,"context_line":"bash ramdisk uses to alert the operators that it\u0027s going to be removed."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":37,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_c1364f50","line":34,"updated":"2015-07-23 17:07:05.000000000","message":"Will operators ever actually see the response from these endpoints? It would only be logged on the console for the node, right?\n\nThinking we should log in the conductor or something. And maybe in DIB when this element is used?","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"c172d3d39dba7b8fc2e399eac8ad668168b5b402","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"We can not simple delete the code that the bash ramdisk uses, therefore"},{"line_number":33,"context_line":"we should start adding deprecation messages on the API endpoints that the"},{"line_number":34,"context_line":"bash ramdisk uses to alert the operators that it\u0027s going to be removed."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":37,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_9fd71aed","line":34,"in_reply_to":"3a50d1a3_9d016d3c","updated":"2015-07-24 10:47:47.000000000","message":"I think it could instead by logged in pass_deploy_info vendor passthru on the conductor.  This is used only by DIB today.","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"02184dbd4c4cd609189516407ce075e932160e21","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"We can not simple delete the code that the bash ramdisk uses, therefore"},{"line_number":33,"context_line":"we should start adding deprecation messages on the API endpoints that the"},{"line_number":34,"context_line":"bash ramdisk uses to alert the operators that it\u0027s going to be removed."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":37,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_afe88c17","line":34,"in_reply_to":"3a50d1a3_9d016d3c","updated":"2015-07-24 19:43:03.000000000","message":"In the mean time, we would need to update the documentation on the element to explicitly state when support for the bash element will be dropped from Ironic, since most operators won\u0027t be looking at their dib output log unless there is a failure.","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"885eaaa2986db29fc6e29803817066fa291ea840","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"We can not simple delete the code that the bash ramdisk uses, therefore"},{"line_number":33,"context_line":"we should start adding deprecation messages on the API endpoints that the"},{"line_number":34,"context_line":"bash ramdisk uses to alert the operators that it\u0027s going to be removed."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":37,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_126df897","line":34,"in_reply_to":"3a50d1a3_afe88c17","updated":"2015-07-27 10:48:10.000000000","message":"++ yeah it would be logged on the conductor as ramesh pointed out. And ++ to also log it on the DIB element saying it\u0027s deprecated","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d00ac5fd3a8976572b64cde40766705b1f7e23c6","unresolved":false,"context_lines":[{"line_number":44,"context_line":"Alternatives"},{"line_number":45,"context_line":"------------"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Continue to support the bash ramdisk for a longer time."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Data model impact"},{"line_number":50,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_bdfea936","line":47,"updated":"2015-07-24 07:29:36.000000000","message":"One more idea: can we somehow detect which ramdisk is *going* to be used? E.g. from driver_info?","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"885eaaa2986db29fc6e29803817066fa291ea840","unresolved":false,"context_lines":[{"line_number":44,"context_line":"Alternatives"},{"line_number":45,"context_line":"------------"},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"Continue to support the bash ramdisk for a longer time."},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"Data model impact"},{"line_number":50,"context_line":"-----------------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_b2e0c4fb","line":47,"in_reply_to":"3a50d1a3_bdfea936","updated":"2015-07-27 10:48:10.000000000","message":"Hmm, not that I know off. Because on the driver info we just have a reference pointing to the deploy ramdisk, and it\u0027s pretty hard to know which ramdisk is it without actually inspecting the files inside it and I don\u0027t think we want to go to that path.","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d00ac5fd3a8976572b64cde40766705b1f7e23c6","unresolved":false,"context_lines":[{"line_number":137,"context_line":"  spec review)."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"* At the end of the M release remove the code that supports the bash"},{"line_number":140,"context_line":"  ramdisk."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"* Update Devstack and tempest to use the IPA_ ramdisk instead of the"},{"line_number":143,"context_line":"  bash ramdisk."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_5dfbe547","line":140,"updated":"2015-07-24 07:29:36.000000000","message":"See above - I believe we should actually remove DIB code before we can start removing ours","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"02184dbd4c4cd609189516407ce075e932160e21","unresolved":false,"context_lines":[{"line_number":137,"context_line":"  spec review)."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"* At the end of the M release remove the code that supports the bash"},{"line_number":140,"context_line":"  ramdisk."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"* Update Devstack and tempest to use the IPA_ ramdisk instead of the"},{"line_number":143,"context_line":"  bash ramdisk."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_6fc924ac","line":140,"in_reply_to":"3a50d1a3_5dfbe547","updated":"2015-07-24 19:43:03.000000000","message":"I feel the same, additionally we should land a warning in the DIB element README as soon as possible after this spec is agreed upon to allow users to visibly see that the element is being deprecated, and that they should use the ironic-agent or a pre-built IPA image.","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"885eaaa2986db29fc6e29803817066fa291ea840","unresolved":false,"context_lines":[{"line_number":137,"context_line":"  spec review)."},{"line_number":138,"context_line":""},{"line_number":139,"context_line":"* At the end of the M release remove the code that supports the bash"},{"line_number":140,"context_line":"  ramdisk."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"* Update Devstack and tempest to use the IPA_ ramdisk instead of the"},{"line_number":143,"context_line":"  bash ramdisk."}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_b2b764f4","line":140,"in_reply_to":"3a50d1a3_6fc924ac","updated":"2015-07-27 10:48:10.000000000","message":"++\n\nTotally, yea I will update the spec","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"d00ac5fd3a8976572b64cde40766705b1f7e23c6","unresolved":false,"context_lines":[{"line_number":140,"context_line":"  ramdisk."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"* Update Devstack and tempest to use the IPA_ ramdisk instead of the"},{"line_number":143,"context_line":"  bash ramdisk."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Dependencies"},{"line_number":146,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_7df8214a","line":143,"updated":"2015-07-24 07:29:36.000000000","message":"nit: this is the first step, not the last :)","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"885eaaa2986db29fc6e29803817066fa291ea840","unresolved":false,"context_lines":[{"line_number":140,"context_line":"  ramdisk."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"* Update Devstack and tempest to use the IPA_ ramdisk instead of the"},{"line_number":143,"context_line":"  bash ramdisk."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Dependencies"},{"line_number":146,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_5296204d","line":143,"in_reply_to":"3a50d1a3_4fda287a","updated":"2015-07-27 10:48:10.000000000","message":"I will move it up","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"02184dbd4c4cd609189516407ce075e932160e21","unresolved":false,"context_lines":[{"line_number":140,"context_line":"  ramdisk."},{"line_number":141,"context_line":""},{"line_number":142,"context_line":"* Update Devstack and tempest to use the IPA_ ramdisk instead of the"},{"line_number":143,"context_line":"  bash ramdisk."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"Dependencies"},{"line_number":146,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":1,"id":"3a50d1a3_4fda287a","line":143,"in_reply_to":"3a50d1a3_7df8214a","updated":"2015-07-24 19:43:03.000000000","message":"++","commit_id":"a5d56a47a35cbbfcfad0edd633a54559b29ee022"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"56725c5b822623373a1c222dafe6778512ec0b23","unresolved":false,"context_lines":[{"line_number":33,"context_line":"we should start adding deprecation messages on the ``deploy-ironic``"},{"line_number":34,"context_line":"element from `diskimage-builder`_ and on the API endpoints that the bash"},{"line_number":35,"context_line":"ramdisk uses (``pass_deploy_info`` and ``pass_bootloader_install_info``)"},{"line_number":36,"context_line":"to alert the operators that it\u0027s going to be removed."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":39,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_b38bdc1d","line":36,"updated":"2015-07-27 14:41:28.000000000","message":"I\u0027d like this to specify \"log messages on the API endpoints\" as this still sounds like the message will be in the API response. But not going to block on it.","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"3db066cc48875d592518224e51e9659eab83ce0a","unresolved":false,"context_lines":[{"line_number":33,"context_line":"we should start adding deprecation messages on the ``deploy-ironic``"},{"line_number":34,"context_line":"element from `diskimage-builder`_ and on the API endpoints that the bash"},{"line_number":35,"context_line":"ramdisk uses (``pass_deploy_info`` and ``pass_bootloader_install_info``)"},{"line_number":36,"context_line":"to alert the operators that it\u0027s going to be removed."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":39,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_d9532938","line":36,"in_reply_to":"3a50d1a3_b38bdc1d","updated":"2015-07-27 15:31:07.000000000","message":"Oh, I thought it meant in the API response. So yeah, should be clarified.","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"63c2f66e33d92bc75613df579a9600ad631860f1","unresolved":false,"context_lines":[{"line_number":33,"context_line":"we should start adding deprecation messages on the ``deploy-ironic``"},{"line_number":34,"context_line":"element from `diskimage-builder`_ and on the API endpoints that the bash"},{"line_number":35,"context_line":"ramdisk uses (``pass_deploy_info`` and ``pass_bootloader_install_info``)"},{"line_number":36,"context_line":"to alert the operators that it\u0027s going to be removed."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":39,"context_line":"features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_ff3a85cc","line":36,"in_reply_to":"3a50d1a3_d9532938","updated":"2015-07-27 15:44:47.000000000","message":"++\n\nProbably I should say vendor passthru methods for this. Will clarify","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"3db066cc48875d592518224e51e9659eab83ce0a","unresolved":false,"context_lines":[{"line_number":35,"context_line":"ramdisk uses (``pass_deploy_info`` and ``pass_bootloader_install_info``)"},{"line_number":36,"context_line":"to alert the operators that it\u0027s going to be removed."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":39,"context_line":"features for the bash ramdisk. No new features should be added to it"},{"line_number":40,"context_line":"(like we did to include support for `local boot`_), only bug fixes will"},{"line_number":41,"context_line":"be accepted."}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_79bc556c","line":38,"updated":"2015-07-27 15:31:07.000000000","message":"s/poses/proposes/","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"63c2f66e33d92bc75613df579a9600ad631860f1","unresolved":false,"context_lines":[{"line_number":35,"context_line":"ramdisk uses (``pass_deploy_info`` and ``pass_bootloader_install_info``)"},{"line_number":36,"context_line":"to alert the operators that it\u0027s going to be removed."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Apart from the deprecation messages this spec also poses freezing the"},{"line_number":39,"context_line":"features for the bash ramdisk. No new features should be added to it"},{"line_number":40,"context_line":"(like we did to include support for `local boot`_), only bug fixes will"},{"line_number":41,"context_line":"be accepted."}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_1f3df1ba","line":38,"in_reply_to":"3a50d1a3_79bc556c","updated":"2015-07-27 15:44:47.000000000","message":"Done","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"3db066cc48875d592518224e51e9659eab83ce0a","unresolved":false,"context_lines":[{"line_number":41,"context_line":"be accepted."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Devstack and tempest jobs should also be updated to not use the bash"},{"line_number":44,"context_line":"ramdisk anymore."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"Alternatives"},{"line_number":47,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_4420909b","line":44,"updated":"2015-07-27 15:31:07.000000000","message":"You mention it in the Work Items, but I think it should be mentioned here (maybe at line 36) that support for this will be removed in M.","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"63c2f66e33d92bc75613df579a9600ad631860f1","unresolved":false,"context_lines":[{"line_number":41,"context_line":"be accepted."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"Devstack and tempest jobs should also be updated to not use the bash"},{"line_number":44,"context_line":"ramdisk anymore."},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"Alternatives"},{"line_number":47,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_9f77c19e","line":44,"in_reply_to":"3a50d1a3_4420909b","updated":"2015-07-27 15:44:47.000000000","message":"Done","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"3db066cc48875d592518224e51e9659eab83ce0a","unresolved":false,"context_lines":[{"line_number":142,"context_line":"* Stop accepting new features for the bash ramdisk (code reviews and"},{"line_number":143,"context_line":"  spec review)."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"* At the end of the M release remove the element from `diskimage-builder`"},{"line_number":146,"context_line":"  and the code that supports the bash ramdisk in Ironic."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_0442883e","line":145,"updated":"2015-07-27 15:31:07.000000000","message":"wondering why you mention \u0027at the end of\u0027 instead of just \u0027In the M release\u0027.\n\nDid you really mean to only remove this eg one month before M is released?","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"63c2f66e33d92bc75613df579a9600ad631860f1","unresolved":false,"context_lines":[{"line_number":142,"context_line":"* Stop accepting new features for the bash ramdisk (code reviews and"},{"line_number":143,"context_line":"  spec review)."},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"* At the end of the M release remove the element from `diskimage-builder`"},{"line_number":146,"context_line":"  and the code that supports the bash ramdisk in Ironic."},{"line_number":147,"context_line":""},{"line_number":148,"context_line":"Dependencies"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a50d1a3_bfb95d30","line":145,"in_reply_to":"3a50d1a3_0442883e","updated":"2015-07-27 15:44:47.000000000","message":"Not really, not sure why I put \"at the end\" it really should be \"in the M release\" as you meantioned","commit_id":"98fb28fe51fc02989eac47f7e6aae6804f85a8ab"},{"author":{"_account_id":6618,"name":"Ruby Loo","email":"opensrloo@gmail.com","username":"rloo"},"change_message_id":"4c0ff6e89e00c0a1db724da74f7e9ff3120837a7","unresolved":false,"context_lines":[{"line_number":33,"context_line":"we should start adding deprecation messages on the ``deploy-ironic``"},{"line_number":34,"context_line":"element from `diskimage-builder`_ and in the vendor passthru methods"},{"line_number":35,"context_line":"``pass_deploy_info`` and ``pass_bootloader_install_info`` which are used"},{"line_number":36,"context_line":"by the bash ramdisk to pass the deployment informantion to Ironic."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Apart from the deprecation messages this spec also proposes freezing"},{"line_number":39,"context_line":"the features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a50d1a3_7b7c42eb","line":36,"updated":"2015-07-27 18:01:37.000000000","message":"s/informantion/information/","commit_id":"ce6a2c30df23dbfdec6201fc3691af6911d30d46"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6afd8bef7d4b9b291b36d9a80901abdfd1974a05","unresolved":false,"context_lines":[{"line_number":33,"context_line":"we should start adding deprecation messages on the ``deploy-ironic``"},{"line_number":34,"context_line":"element from `diskimage-builder`_ and in the vendor passthru methods"},{"line_number":35,"context_line":"``pass_deploy_info`` and ``pass_bootloader_install_info`` which are used"},{"line_number":36,"context_line":"by the bash ramdisk to pass the deployment informantion to Ironic."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Apart from the deprecation messages this spec also proposes freezing"},{"line_number":39,"context_line":"the features for the bash ramdisk. No new features should be added to it"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a50d1a3_d6720b7c","line":36,"in_reply_to":"3a50d1a3_7b7c42eb","updated":"2015-07-27 18:12:05.000000000","message":"Done","commit_id":"ce6a2c30df23dbfdec6201fc3691af6911d30d46"},{"author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"change_message_id":"aa28cca418dd7a16b2f7ee233d10f21b075be686","unresolved":false,"context_lines":[{"line_number":111,"context_line":""},{"line_number":112,"context_line":"Deployer should start replacing the bash ramdisk with the IPA_"},{"line_number":113,"context_line":"ramdisk. There\u0027s no new configuration needed for it, it\u0027s a drop-in"},{"line_number":114,"context_line":"replacement."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Developer impact"},{"line_number":117,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a50d1a3_76f1b769","line":114,"updated":"2015-07-27 18:27:33.000000000","message":"Also, some modifications on Devstack should be done in order to use IPA by default.","commit_id":"19b0eb2c631c1223e6bc887117a86bf037082fc6"},{"author":{"_account_id":10343,"name":"Jim Rollenhagen","email":"jim@jimrollenhagen.com","username":"jimrollenhagen"},"change_message_id":"cc21027e18d2c68d6fd3f14acfa4f02232ae1240","unresolved":false,"context_lines":[{"line_number":111,"context_line":""},{"line_number":112,"context_line":"Deployer should start replacing the bash ramdisk with the IPA_"},{"line_number":113,"context_line":"ramdisk. There\u0027s no new configuration needed for it, it\u0027s a drop-in"},{"line_number":114,"context_line":"replacement."},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Developer impact"},{"line_number":117,"context_line":"----------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a50d1a3_406c52d0","line":114,"in_reply_to":"3a50d1a3_76f1b769","updated":"2015-07-30 13:38:04.000000000","message":"Yep, and that\u0027s noted. That isn\u0027t a deployer impact, though :)","commit_id":"19b0eb2c631c1223e6bc887117a86bf037082fc6"},{"author":{"_account_id":9315,"name":"ramesh","email":"rameshg87@gmail.com","username":"rameshg87"},"change_message_id":"69145ee388cf134b59767c63e0bf45a1f2b860ca","unresolved":false,"context_lines":[{"line_number":156,"context_line":"Testing"},{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unittests will be added."},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":162,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a50d1a3_b9bdd669","line":159,"updated":"2015-07-28 09:55:26.000000000","message":"Unit tests for what ? Deprecation messages ? :)","commit_id":"19b0eb2c631c1223e6bc887117a86bf037082fc6"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"e0ec7e704cfedb6e3dde41156ad8a7a0edc1e600","unresolved":false,"context_lines":[{"line_number":156,"context_line":"Testing"},{"line_number":157,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":158,"context_line":""},{"line_number":159,"context_line":"Unittests will be added."},{"line_number":160,"context_line":""},{"line_number":161,"context_line":"Upgrades and Backwards Compatibility"},{"line_number":162,"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\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a50d1a3_3498ebbf","line":159,"in_reply_to":"3a50d1a3_b9bdd669","updated":"2015-07-28 10:00:52.000000000","message":"Heh I forgot this comment from the spec I used as base","commit_id":"19b0eb2c631c1223e6bc887117a86bf037082fc6"},{"author":{"_account_id":10697,"name":"Thiago Paiva Brito","email":"thiagop@lsd.ufcg.edu.br","username":"thiagop","inactive":true},"change_message_id":"aa28cca418dd7a16b2f7ee233d10f21b075be686","unresolved":false,"context_lines":[{"line_number":167,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"The documentation should be updated to say that the bash ramdisk is"},{"line_number":170,"context_line":"deprecated and the examples should now use IPA_ instead."},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"References"},{"line_number":173,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"3a50d1a3_962fa3a0","line":170,"updated":"2015-07-27 18:27:33.000000000","message":"Developer docs should be updated also to (suppress/include) local.conf options to run devstack with IPA by default.","commit_id":"19b0eb2c631c1223e6bc887117a86bf037082fc6"}]}