)]}'
{"specs/victoria/update-refactor-tasks.rst":[{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Problem Description"},{"line_number":14,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Tripleo overcloud update run creating different stage during the"},{"line_number":17,"context_line":"update by looping over a step variable running the same playbook."},{"line_number":18,"context_line":"This creates a lot of skipped tasks for each run as only a subset of"},{"line_number":19,"context_line":"the playbook\u0027s tasks are run for every step.  This is a waste of time."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_236a31ec","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":37},"updated":"2020-07-09 09:27:37.000000000","message":"Need rewordind, something along the line of: tripleo overcloud update is: prepare, run, converge.  We focus on the run phase.  This phase is further down split into several phase some of them common with deployment....","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":13,"context_line":"Problem Description"},{"line_number":14,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Tripleo overcloud update run creating different stage during the"},{"line_number":17,"context_line":"update by looping over a step variable running the same playbook."},{"line_number":18,"context_line":"This creates a lot of skipped tasks for each run as only a subset of"},{"line_number":19,"context_line":"the playbook\u0027s tasks are run for every step.  This is a waste of time."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_c39a5512","line":16,"range":{"start_line":16,"start_character":53,"end_line":16,"end_character":54},"updated":"2020-07-09 09:27:37.000000000","message":"s","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":29,"context_line":"--------"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"Reverse the logic to match what is currently done for deployment and"},{"line_number":32,"context_line":"upgrade.  We create a playbook for each step and load them from the"},{"line_number":33,"context_line":"deploy.j2 template.  Each step playbook would have a new tasks that"},{"line_number":34,"context_line":"would name the current step so that the user know what\u0027s going on.  In"},{"line_number":35,"context_line":"case of failure we would offer the command that will allow the user to"},{"line_number":36,"context_line":"restart the update from the point of failure."}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_bea016f6","line":33,"range":{"start_line":32,"start_character":10,"end_line":33,"end_character":19},"updated":"2020-07-09 09:27:37.000000000","message":"Implementation is actually: loop over import in the first jinja rendering using a macro.  So this overall section need rewrite.","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":35,"context_line":"case of failure we would offer the command that will allow the user to"},{"line_number":36,"context_line":"restart the update from the point of failure."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Enforce a new \"strict\" policy for the collection of the steps where"},{"line_number":39,"context_line":"only tasks with step conditional are collected."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"Alternatives"},{"line_number":42,"context_line":"------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_5eaf3ae2","line":39,"range":{"start_line":38,"start_character":0,"end_line":39,"end_character":47},"updated":"2020-07-09 09:27:37.000000000","message":"This may not be related to this blueprint.  Still helpful to debug which tasks haven\u0027t a properly attached policy, but not required here.","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":59,"context_line":"---------------------"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"This would improve readability of the output.  By loading specific"},{"line_number":62,"context_line":"playbook the skipped tasks would disappear from the output."},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"Naming each step would also ease communication when error happens."},{"line_number":65,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_9eb55235","line":62,"range":{"start_line":62,"start_character":0,"end_line":62,"end_character":8},"updated":"2020-07-09 09:27:37.000000000","message":"import of tasks","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":64,"context_line":"Naming each step would also ease communication when error happens."},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"It would also offer point of recovery to the user, by being able to"},{"line_number":67,"context_line":"re-run a failed update from a specific step."},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"Performance Impact"},{"line_number":70,"context_line":"------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_9e8ef276","line":67,"range":{"start_line":67,"start_character":29,"end_line":67,"end_character":44},"updated":"2020-07-09 09:27:37.000000000","message":"Because all update would be imported any tasks would be available for an start-as-task ansible parameter (that would need to be added to the overcloud overcloud update run command)","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":70,"context_line":"------------------"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":"This would improve the overall performance of the update process as we"},{"line_number":73,"context_line":"wouldn\u0027t skip tasks anymore.  With an \"strict\" policy we would ensure"},{"line_number":74,"context_line":"no tasks are uselessly run at every step."},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"Other Deployer Impact"},{"line_number":77,"context_line":"---------------------"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_beee561f","line":74,"range":{"start_line":73,"start_character":30,"end_line":74,"end_character":41},"updated":"2020-07-09 09:27:37.000000000","message":"But it can be risky, and again not strictly required.","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":81,"context_line":"Developer Impact"},{"line_number":82,"context_line":"----------------"},{"line_number":83,"context_line":""},{"line_number":84,"context_line":"Having a tasks without an conditional step attached to it will not do"},{"line_number":85,"context_line":"anything anymore."},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"Implementation"},{"line_number":88,"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":"bf51134e_5edd7a6e","line":85,"range":{"start_line":84,"start_character":0,"end_line":85,"end_character":17},"updated":"2020-07-09 09:27:37.000000000","message":"again we may drop this and work on it on the side.","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"},{"author":{"_account_id":8297,"name":"Sofer Athlan-Guyot","email":"sathlang@redhat.com","username":"chem"},"change_message_id":"5866eb81ee1b973ffd2638e92562f1e74f34dfd1","unresolved":false,"context_lines":[{"line_number":99,"context_line":"Work Items"},{"line_number":100,"context_line":"----------"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"* Collect and fix tasks with no steps conditionals;"},{"line_number":103,"context_line":"* Implement the new strict policy in tripleo-common;"},{"line_number":104,"context_line":"* Add update to the list of per_step_tasks.  This will work as we are"},{"line_number":105,"context_line":"  still generating the all-in-one playbook, so update in CI would"}],"source_content_type":"text/x-rst","patch_set":2,"id":"bf51134e_ded22a5c","line":102,"range":{"start_line":102,"start_character":0,"end_line":102,"end_character":51},"updated":"2020-07-09 09:27:37.000000000","message":"Will do it anyway :)","commit_id":"a8da94921d5f6f03e1b9483eeafafdc87a116522"}]}