)]}'
{"deployment/tripleo-packages/tripleo-packages-baremetal-puppet.yaml":[{"author":{"_account_id":26343,"name":"Jose Luis Franco","email":"jfrancoa@redhat.com","username":"jfrancoa"},"change_message_id":"b3651af02de0ec8f3a7c685714acc5fbb3c69b08","unresolved":true,"context_lines":[{"line_number":206,"context_line":"            - name: reboot to perform the upgrade"},{"line_number":207,"context_line":"              reboot:"},{"line_number":208,"context_line":"                reboot_timeout: \"{{upgrade_leapp_reboot_timeout}}\""},{"line_number":209,"context_line":"            - name: Clear gathered facts from all currently targeted hosts"},{"line_number":210,"context_line":"              meta: clear_facts"},{"line_number":211,"context_line":"            - name: Force facts refresh after OS upgrade to refresh cache."},{"line_number":212,"context_line":"              setup:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"0693271a_bd13b963","side":"PARENT","line":209,"updated":"2020-11-24 08:31:55.000000000","message":"I\u0027m not sure if this is a good idea...this was added here because when we upgraded some overcloud node the facts were not reset, so we ended up having a RHEL8 node but facts were from the node when RHEL7 was installed. So the ansible_python_interpreter variable was pointing still at python2, etc... That is why we included a facts cleaninig up and refreshing them.","commit_id":"b289ef4f4ced7db4e7641b9006e802d8dfbccff3"},{"author":{"_account_id":26343,"name":"Jose Luis Franco","email":"jfrancoa@redhat.com","username":"jfrancoa"},"change_message_id":"1275279a4344668251ed45c237ca64301ef90d28","unresolved":false,"context_lines":[{"line_number":206,"context_line":"            - name: reboot to perform the upgrade"},{"line_number":207,"context_line":"              reboot:"},{"line_number":208,"context_line":"                reboot_timeout: \"{{upgrade_leapp_reboot_timeout}}\""},{"line_number":209,"context_line":"            - name: Clear gathered facts from all currently targeted hosts"},{"line_number":210,"context_line":"              meta: clear_facts"},{"line_number":211,"context_line":"            - name: Force facts refresh after OS upgrade to refresh cache."},{"line_number":212,"context_line":"              setup:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"05ec9842_1807aebb","side":"PARENT","line":209,"in_reply_to":"0693271a_bd13b963","updated":"2020-11-24 08:53:40.000000000","message":"I was checking the CI job executed and it\u0027s true that this issue isn\u0027t present anymore... probably it has something to do with the play we now run at the beginning of echa command:\n\nPLAY[Clear cached facts]\n\nLet\u0027s leave it like your patch suggests and if something starts failing due to python incompatibilities we\u0027ll know what is it.","commit_id":"b289ef4f4ced7db4e7641b9006e802d8dfbccff3"}]}
