)]}'
{"ansible/roles/common/tasks/pull.yml":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"7197b1735059f325d9cfe4f9581e8764e6c38e12","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    # nova-cell first (a very common case - separate computes) which sets"},{"line_number":6,"context_line":"    # this variable and makes pull action work for nova_cell twice instead"},{"line_number":7,"context_line":"    # due to how variable precedence and shadowing work in Ansible."},{"line_number":8,"context_line":"    kolla_role_name: \"common\""},{"line_number":9,"context_line":"  # NOTE(yoctozepto): and this has to be an include (and not import) to avoid"},{"line_number":10,"context_line":"  # leaking the variable above to other imports"},{"line_number":11,"context_line":"  include_role:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e1c667dc_ef608421","line":8,"updated":"2021-08-23 08:44:57.000000000","message":"Why would ansible continue to keep this in scope after the role import? That\u0027s just awful.\n\nI did a little experimenting. You can use a block with import_role to get the expected variable scoping.","commit_id":"75f2da9a7f42efcb34ae2caaf6c7dbc65cb676b6"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"59f701634baf0730996b187b6913f07349739344","unresolved":true,"context_lines":[{"line_number":5,"context_line":"    # nova-cell first (a very common case - separate computes) which sets"},{"line_number":6,"context_line":"    # this variable and makes pull action work for nova_cell twice instead"},{"line_number":7,"context_line":"    # due to how variable precedence and shadowing work in Ansible."},{"line_number":8,"context_line":"    kolla_role_name: \"common\""},{"line_number":9,"context_line":"  # NOTE(yoctozepto): and this has to be an include (and not import) to avoid"},{"line_number":10,"context_line":"  # leaking the variable above to other imports"},{"line_number":11,"context_line":"  include_role:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c7474554_b9b94092","line":8,"in_reply_to":"e1c667dc_ef608421","updated":"2021-08-23 09:11:00.000000000","message":"Ack, I\u0027ll test it too.","commit_id":"75f2da9a7f42efcb34ae2caaf6c7dbc65cb676b6"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"7197b1735059f325d9cfe4f9581e8764e6c38e12","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    # due to how variable precedence and shadowing work in Ansible."},{"line_number":8,"context_line":"    kolla_role_name: \"common\""},{"line_number":9,"context_line":"  # NOTE(yoctozepto): and this has to be an include (and not import) to avoid"},{"line_number":10,"context_line":"  # leaking the variable above to other imports"},{"line_number":11,"context_line":"  include_role:"},{"line_number":12,"context_line":"    role: service-images-pull"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"f451b1a7_cdb9d033","line":10,"updated":"2021-08-23 08:44:57.000000000","message":"something something explicit... something magic... 😊","commit_id":"75f2da9a7f42efcb34ae2caaf6c7dbc65cb676b6"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"59f701634baf0730996b187b6913f07349739344","unresolved":true,"context_lines":[{"line_number":7,"context_line":"    # due to how variable precedence and shadowing work in Ansible."},{"line_number":8,"context_line":"    kolla_role_name: \"common\""},{"line_number":9,"context_line":"  # NOTE(yoctozepto): and this has to be an include (and not import) to avoid"},{"line_number":10,"context_line":"  # leaking the variable above to other imports"},{"line_number":11,"context_line":"  include_role:"},{"line_number":12,"context_line":"    role: service-images-pull"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4f14c46f_e7894ece","line":10,"in_reply_to":"f451b1a7_cdb9d033","updated":"2021-08-23 09:11:00.000000000","message":"Well, I\u0027ve already told you there was this more-DRY vs more-explicit tradeoff. No need to tease me for Ansible being one big quirk on wheels. 😜","commit_id":"75f2da9a7f42efcb34ae2caaf6c7dbc65cb676b6"}]}