)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1286925588dd311e438a9aab62a985b843d3d1d9","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To avoid calling the same tasks multiple times,"},{"line_number":10,"context_line":"this change splits out the packages component."},{"line_number":11,"context_line":"This allows us to call it from other tasks without"},{"line_number":12,"context_line":"running the entire role"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ifc38772898e864af9c76d45e87b34e4390bfd033"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ead590ad_de5a4bd2","line":11,"range":{"start_line":11,"start_character":8,"end_line":11,"end_character":26},"updated":"2022-05-13 06:58:45.000000000","message":"makes sense/lgtm but it is only being invoked in one place here. are there other patches that add move invocations?","commit_id":"49bfc0c9ef7f35bb8feb4c00859d4662e147e01d"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"7a7bc44c1c62978e8b25761f192afbb975ada011","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To avoid calling the same tasks multiple times,"},{"line_number":10,"context_line":"this change splits out the packages component."},{"line_number":11,"context_line":"This allows us to call it from other tasks without"},{"line_number":12,"context_line":"running the entire role"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ifc38772898e864af9c76d45e87b34e4390bfd033"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"a78e9450_de67b1f7","line":11,"range":{"start_line":11,"start_character":8,"end_line":11,"end_character":26},"in_reply_to":"cf5fba3a_d98bcda2","updated":"2022-05-16 06:35:09.000000000","message":"thanks I just saw and reviewed that other patch so it makes more sense now i.e. it is being invoked more than once so good to refactor like this","commit_id":"49bfc0c9ef7f35bb8feb4c00859d4662e147e01d"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"3efe80db4266d83d07deb801c198b51c39424a71","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To avoid calling the same tasks multiple times,"},{"line_number":10,"context_line":"this change splits out the packages component."},{"line_number":11,"context_line":"This allows us to call it from other tasks without"},{"line_number":12,"context_line":"running the entire role"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: Ifc38772898e864af9c76d45e87b34e4390bfd033"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"cf5fba3a_d98bcda2","line":11,"range":{"start_line":11,"start_character":8,"end_line":11,"end_character":26},"in_reply_to":"ead590ad_de5a4bd2","updated":"2022-05-15 22:58:49.000000000","message":"Hey, yeah so the problem we\u0027re actually trying to solve is: https://review.opendev.org/c/openstack/tripleo-ansible/+/841544/\n\nApparently we\u0027re already calling tripleo_bootstrap twice. So rather than call it for a third time, I thought I would break it up and just call the packages section.","commit_id":"49bfc0c9ef7f35bb8feb4c00859d4662e147e01d"}],"tripleo_ansible/roles/tripleo_bootstrap/tasks/packages.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1286925588dd311e438a9aab62a985b843d3d1d9","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Copyright 2019 Red Hat, Inc."},{"line_number":3,"context_line":"# All Rights Reserved."},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7ecb2b46_853160ea","line":2,"range":{"start_line":2,"start_character":14,"end_line":2,"end_character":16},"updated":"2022-05-13 06:58:45.000000000","message":".","commit_id":"49bfc0c9ef7f35bb8feb4c00859d4662e147e01d"},{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"826dad37f0297840060f720df0670d79809b6a50","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# Copyright 2019 Red Hat, Inc."},{"line_number":3,"context_line":"# All Rights Reserved."},{"line_number":4,"context_line":"#"},{"line_number":5,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d3d2512f_2e578a60","line":2,"range":{"start_line":2,"start_character":14,"end_line":2,"end_character":16},"in_reply_to":"7ecb2b46_853160ea","updated":"2022-05-15 23:25:53.000000000","message":"Fixed it. Thanks for highlighting.","commit_id":"49bfc0c9ef7f35bb8feb4c00859d4662e147e01d"}]}