Closed nocturnalastro closed 1 year ago
damn inventories.
On Tue, Feb 21, 2023 at 9:12 AM Michele Costa @.***> wrote:
@.**** commented on this pull request. In roles/generate_ssh_key_pair/tasks/main.yml https://github.com/redhat-partner-solutions/crucible/pull/222#discussion_r1113121666 : +- name: "Remove temporary ssh_keys folder {{ key_pair_dir }}" file: path: "{{ key_pair_dir }}" state: absent We can do that but it could break people's inventories if they are overriding it. — Reply to this email directly, view it on GitHub https://github.com/redhat-partner-solutions/crucible/pull/222#discussion_r1113121666, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB4J47DSTUHOXIDL6NQGRIDWYTEN3ANCNFSM6AAAAAAVC4CCLI . You are receiving this because you commented.Message ID: @.***>
In roles/generate_ssh_key_pair/tasks/main.yml https://github.com/redhat-partner-solutions/crucible/pull/222#discussion_r1113121666 :
+- name: "Remove temporary ssh_keys folder {{ key_pair_dir }}" file: path: "{{ key_pair_dir }}" state: absent
+- name: "Remove temporary ssh_keys folder {{ key_pair_dir }}"
We can do that but it could break people's inventories if they are overriding it.
— Reply to this email directly, view it on GitHub https://github.com/redhat-partner-solutions/crucible/pull/222#discussion_r1113121666, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB4J47DSTUHOXIDL6NQGRIDWYTEN3ANCNFSM6AAAAAAVC4CCLI . You are receiving this because you commented.Message ID: @.***>
damn inventories.
On Tue, Feb 21, 2023 at 9:12 AM Michele Costa @.***> wrote: