Improvement/upstream updates

[jsf9k]

🗣 Description

This pull request pulls in upstream changes from skeleton-ansible-role.

💭 Motivation and Context

@felddy is making some changes to this Ansible role, so it makes sense to get it up to date first.

🧪 Testing

All pre-commit linters and molecule tests pass.

🚥 Types of Changes

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (causes existing functionality to change)

✅ Checklist

• [x] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [x] I have read the CONTRIBUTING document.
• [ ] I have added tests to cover my changes.
• [x] All new and existing tests passed.

[felddy]

Just noticed the README still mentions the build user foo.

[jsf9k]

Just noticed the README still mentions the build user foo.

See commit 8555322.

[jsf9k]

@felddy, is the text in README.md about manually setting up NAT still accurate? I haven't been doing that when I have redeployed the VPN instance.

[felddy]

@felddy, is the text in README.md about manually setting up NAT still accurate? I haven't been doing that when I have redeployed the VPN instance.

It is still accurate but also handled for us in cloudinit. See: https://github.com/cisagov/openvpn-server-tf-module/pull/21