Skip to content
This repository was archived by the owner on Jan 5, 2025. It is now read-only.

Update role using community.crypto #1

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

leophys
Copy link

@leophys leophys commented Jan 24, 2021

Hi! Thank you for this role!
I update it using the ansible community.crypto collection. The logic should be the same and in the original spirit of the role you wrote. I just changed minor things:

  • added an (optional) task to populate the /etc/hosts file with the nodes in the inventory
  • added a Vagrantfile to test this role locally
  • made so that the certificates are copied to the remote machines in a configurable path that defaults to /etc/ssl/local/{certs,private}
  • changed all the variables name to have a pseudo-namespace gen_tls_, to ease the use of this role together with others

Let me know what you think of it.

The certificate form of the CA has to be crt to be sure it is manageable
by the Debian update-ca-certificates executable.
Also, added option to force the copy of the certificates, also if the
local files did not change.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants