)]}'
{"roles/baremetal/tasks/install.yml":[{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"327083ec8df5d3f09725489e43dd3c9482472c7c","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  block:"},{"line_number":5,"context_line":"    - name: Gather package facts"},{"line_number":6,"context_line":"      ansible.builtin.package_facts:"},{"line_number":7,"context_line":"        manager: auto"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    - name: Set firewall default policy"},{"line_number":10,"context_line":"      become: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"f8358fe0_c7d911bb","line":7,"updated":"2025-11-04 16:58:48.000000000","message":"LGTM, only thing is auto package_manager (?) maybe we force it based on OS ?","commit_id":"24f497ab83d3a980f8dbd56917d202a197929baa"},{"author":{"_account_id":37203,"name":"Bertrand Lanson","display_name":"Bertrand Lanson","email":"bertrand.lanson@infomaniak.com","username":"lanson","status":"Infomaniak Network SA"},"change_message_id":"9c1036d8b8f630412d336e5dd7c26ec42df2f6f8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"  block:"},{"line_number":5,"context_line":"    - name: Gather package facts"},{"line_number":6,"context_line":"      ansible.builtin.package_facts:"},{"line_number":7,"context_line":"        manager: auto"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    - name: Set firewall default policy"},{"line_number":10,"context_line":"      become: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"dca07772_52f195cc","line":7,"in_reply_to":"b2b1822f_cf067744","updated":"2025-11-05 07:54:51.000000000","message":"Yeah, the thing that I remembered was all the aliases to rpm done by all the alternative package managers, but in this case it should be fine I think.","commit_id":"24f497ab83d3a980f8dbd56917d202a197929baa"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"cd2d0dc9a84bf98cb702d27d873ffb4073f87787","unresolved":true,"context_lines":[{"line_number":4,"context_line":"  block:"},{"line_number":5,"context_line":"    - name: Gather package facts"},{"line_number":6,"context_line":"      ansible.builtin.package_facts:"},{"line_number":7,"context_line":"        manager: auto"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    - name: Set firewall default policy"},{"line_number":10,"context_line":"      become: true"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b2b1822f_cf067744","line":7,"in_reply_to":"f8358fe0_c7d911bb","updated":"2025-11-04 17:08:33.000000000","message":"Looking at https://docs.ansible.com/ansible/latest/collections/ansible/builtin/package_facts_module.html#parameter-manager we should be fine, if you\u0027ll provide some good reason - we could have a strict list with apt and rpm only (if required)","commit_id":"24f497ab83d3a980f8dbd56917d202a197929baa"}]}