)]}'
{"specs/newton/cheesecake-promote-backend.rst":[{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"99d2ca6d332a7c3d6b12788870dcdc882a0ea333","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    * Ex:"},{"line_number":32,"context_line":"::"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    update services set disabled\u003d0,disabled_reason\u003dNULL,replication_status\u003d\u0027enabled\u0027,active_backend_id\u003dNULL where id\u003d3;"},{"line_number":35,"context_line":"  e. Start cinder volume service"},{"line_number":36,"context_line":"  f. Unfreeze backend"},{"line_number":37,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_181437e7","line":34,"updated":"2016-05-05 05:26:35.000000000","message":"This line should be \u003c 79 characters.","commit_id":"376ed06edb369bc4e076069f5fb8588a02075792"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"ef7c0f1e0cf6ed750c3d4e1b54d5d06e1ec00beb","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Solution A:"},{"line_number":49,"context_line":"Add a cinder-manage command to reset the active backend for a host."},{"line_number":50,"context_line":"Equivalent to \u0027update services set disabled\u003d0,disabled_reason\u003dNULL,replication_status\u003d\u0027disabled\u0027,active_backend_id\u003dNULL where host\u003d\u0027\u003cbackend-host\u003e;\u0027"},{"line_number":51,"context_line":"::"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    cinder-manage reset-active-backend \u003cbackend-host\u003e"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"which can then be followed with::"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_17bff4d0","line":53,"range":{"start_line":50,"start_character":0,"end_line":53,"end_character":53},"updated":"2016-05-04 20:55:53.000000000","message":"Dunno how much it matters, but should we maybe leave disabled alone? Seems like if it was disabled and the admin wants to put it back it might be nice to have it stay that way until they explicitly enable it again?","commit_id":"376ed06edb369bc4e076069f5fb8588a02075792"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"35701c97da8dba9b3147cf39f87c8e0f36a7193f","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Solution A:"},{"line_number":49,"context_line":"Add a cinder-manage command to reset the active backend for a host."},{"line_number":50,"context_line":"Equivalent to \u0027update services set disabled\u003d0,disabled_reason\u003dNULL,replication_status\u003d\u0027disabled\u0027,active_backend_id\u003dNULL where host\u003d\u0027\u003cbackend-host\u003e;\u0027"},{"line_number":51,"context_line":"::"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    cinder-manage reset-active-backend \u003cbackend-host\u003e"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"which can then be followed with::"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_7c6a5f1b","line":53,"range":{"start_line":50,"start_character":0,"end_line":53,"end_character":53},"in_reply_to":"1a122d0e_17bff4d0","updated":"2016-05-05 19:00:43.000000000","message":"Yeah this is what i\u0027m proposing iiuc. That\u0027s why I have the next step (which apparently never got impl but is in the cheesecake spec)","commit_id":"376ed06edb369bc4e076069f5fb8588a02075792"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"99d2ca6d332a7c3d6b12788870dcdc882a0ea333","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"which can then be followed with::"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    cinder replication-enable \u003cbackend-name\u003e"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"to enable replication to C."},{"line_number":60,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_f895fb3f","line":57,"updated":"2016-05-05 05:26:35.000000000","message":"We don\u0027t have this in Cheesecake. Are you planning to add this? I don\u0027t see this in the work item.","commit_id":"376ed06edb369bc4e076069f5fb8588a02075792"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"35701c97da8dba9b3147cf39f87c8e0f36a7193f","unresolved":false,"context_lines":[{"line_number":54,"context_line":""},{"line_number":55,"context_line":"which can then be followed with::"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    cinder replication-enable \u003cbackend-name\u003e"},{"line_number":58,"context_line":""},{"line_number":59,"context_line":"to enable replication to C."},{"line_number":60,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_9cdbfbec","line":57,"in_reply_to":"1a122d0e_f895fb3f","updated":"2016-05-05 19:00:43.000000000","message":":( darn, this was in the cheesecake spec but you are right, it\u0027s not there. I do think these should be separate steps so yes, we should have this as a work item.","commit_id":"376ed06edb369bc4e076069f5fb8588a02075792"},{"author":{"_account_id":12924,"name":"Patrick East","email":"east.patrick@gmail.com","username":"patrick.east"},"change_message_id":"ef7c0f1e0cf6ed750c3d4e1b54d5d06e1ec00beb","unresolved":false,"context_lines":[{"line_number":113,"context_line":"Assignee(s)"},{"line_number":114,"context_line":"-----------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Primary assignee:"},{"line_number":117,"context_line":"  ?"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Work Items"},{"line_number":120,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_97f664d5","line":117,"range":{"start_line":116,"start_character":0,"end_line":117,"end_character":3},"updated":"2016-05-04 20:55:53.000000000","message":"I can help out with this if needed.","commit_id":"376ed06edb369bc4e076069f5fb8588a02075792"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"35701c97da8dba9b3147cf39f87c8e0f36a7193f","unresolved":false,"context_lines":[{"line_number":113,"context_line":"Assignee(s)"},{"line_number":114,"context_line":"-----------"},{"line_number":115,"context_line":""},{"line_number":116,"context_line":"Primary assignee:"},{"line_number":117,"context_line":"  ?"},{"line_number":118,"context_line":""},{"line_number":119,"context_line":"Work Items"},{"line_number":120,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a122d0e_fc994f20","line":117,"range":{"start_line":116,"start_character":0,"end_line":117,"end_character":3},"in_reply_to":"1a122d0e_97f664d5","updated":"2016-05-05 19:00:43.000000000","message":"careful what you wish for :)","commit_id":"376ed06edb369bc4e076069f5fb8588a02075792"},{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"2308ebe5e96ef390d79ea4f6de29940954ae63c2","unresolved":false,"context_lines":[{"line_number":25,"context_line":"4. Promote Secondary Backend"},{"line_number":26,"context_line":"  a. Freeze backend to prevent manage operations"},{"line_number":27,"context_line":"  b. Stop cinder volume service"},{"line_number":28,"context_line":"  c. Update cinder.conf to have backend A replaced with B and B with C"},{"line_number":29,"context_line":"  d. *Hack db to set backend to no longer be in \u0027failed-over\u0027 state*"},{"line_number":30,"context_line":"    * This is the step this spec is concerned with"},{"line_number":31,"context_line":"    * Example:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dab17558_cf77d99d","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":70},"updated":"2016-05-07 17:30:36.000000000","message":"Seems like this step is still needed in the proposed solution. Or will these commands update cinder.conf?","commit_id":"2fb5fbe8e2f48d5ebc4efdf1f59820f2ca05fff7"},{"author":{"_account_id":10213,"name":"Alon Marx","email":"alonma@il.ibm.com","username":"alonmarx"},"change_message_id":"45635fb87b1d42e08a8c32c5fbd84a263b13059e","unresolved":false,"context_lines":[{"line_number":25,"context_line":"4. Promote Secondary Backend"},{"line_number":26,"context_line":"  a. Freeze backend to prevent manage operations"},{"line_number":27,"context_line":"  b. Stop cinder volume service"},{"line_number":28,"context_line":"  c. Update cinder.conf to have backend A replaced with B and B with C"},{"line_number":29,"context_line":"  d. *Hack db to set backend to no longer be in \u0027failed-over\u0027 state*"},{"line_number":30,"context_line":"    * This is the step this spec is concerned with"},{"line_number":31,"context_line":"    * Example:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dab17558_a4fff365","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":70},"in_reply_to":"dab17558_489583dc","updated":"2016-05-09 17:02:07.000000000","message":"I agree that the service shouldn\u0027t edit the conf file. But we should address this and see that there is no conflict. Maybe just documenting the process, including updating cinder.conf is enough for now.","commit_id":"2fb5fbe8e2f48d5ebc4efdf1f59820f2ca05fff7"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"8a9c998b06ce1defc9a5f79fac8b66c05da4ab21","unresolved":false,"context_lines":[{"line_number":25,"context_line":"4. Promote Secondary Backend"},{"line_number":26,"context_line":"  a. Freeze backend to prevent manage operations"},{"line_number":27,"context_line":"  b. Stop cinder volume service"},{"line_number":28,"context_line":"  c. Update cinder.conf to have backend A replaced with B and B with C"},{"line_number":29,"context_line":"  d. *Hack db to set backend to no longer be in \u0027failed-over\u0027 state*"},{"line_number":30,"context_line":"    * This is the step this spec is concerned with"},{"line_number":31,"context_line":"    * Example:"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dab17558_489583dc","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":70},"in_reply_to":"dab17558_cf77d99d","updated":"2016-05-09 14:18:39.000000000","message":"I\u0027m not sure we can improve this step. The service can\u0027t edit it\u0027s conf file (folks have automation that would compete with that). Unless anyone has a good idea on how to remove this part then we have to keep it manual.","commit_id":"2fb5fbe8e2f48d5ebc4efdf1f59820f2ca05fff7"},{"author":{"_account_id":6491,"name":"xing-yang","email":"xingyang105@gmail.com","username":"xing-yang"},"change_message_id":"65eb79c651cf652131e88a7bf241aca4ab110ee1","unresolved":false,"context_lines":[{"line_number":132,"context_line":"-----------"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"Primary assignee:"},{"line_number":135,"context_line":"  ?"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Work Items"},{"line_number":138,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dab17558_29821801","line":135,"updated":"2016-05-06 19:11:04.000000000","message":"Are you only writing spec but not code? Ask Ben what he thinks:).","commit_id":"2fb5fbe8e2f48d5ebc4efdf1f59820f2ca05fff7"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"8a9c998b06ce1defc9a5f79fac8b66c05da4ab21","unresolved":false,"context_lines":[{"line_number":132,"context_line":"-----------"},{"line_number":133,"context_line":""},{"line_number":134,"context_line":"Primary assignee:"},{"line_number":135,"context_line":"  ?"},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Work Items"},{"line_number":138,"context_line":"----------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"dab17558_c83cd3e4","line":135,"in_reply_to":"dab17558_29821801","updated":"2016-05-09 14:18:39.000000000","message":"lol \u003e.\u003c","commit_id":"2fb5fbe8e2f48d5ebc4efdf1f59820f2ca05fff7"}]}