Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makes drop-in functionality configurable by the user #200

Merged
merged 1 commit into from
Aug 29, 2022
Merged

Makes drop-in functionality configurable by the user #200

merged 1 commit into from
Aug 29, 2022

Conversation

nkakouros
Copy link
Contributor

This is based off of #199.

This PR simplifies the logic behind the drop-in config files and also allows the user to use drop-in configs even if the distribution does not support it out of the box.

@mattwillsher
Copy link
Member

@nkakouros lint should be fixed on master I believe, so please merge/rebash that in to the test work.

This change supercedes #199 ?

@nkakouros nkakouros changed the title Fixes un-overrideable public api variables Makes drop-in functionality configurable by the user Aug 18, 2022
@nkakouros
Copy link
Contributor Author

@mattwillsher Sorry, I was too sleepless to read properly, this is supposed to be a continuation of #199. I updated the title. I will rebase both PRs now.

templates/sshd_config.j2 Outdated Show resolved Hide resolved
templates/sshd_config.j2 Outdated Show resolved Hide resolved
tests/tasks/restore.yml Outdated Show resolved Hide resolved
tasks/install_config.yml Show resolved Hide resolved
tasks/install_config.yml Outdated Show resolved Hide resolved
tests/tests_include_present.yml Outdated Show resolved Hide resolved
tests/tests_duplicate_role.yml Outdated Show resolved Hide resolved
vars/main.yml Show resolved Hide resolved
tasks/install_config.yml Outdated Show resolved Hide resolved
@Jakuje
Copy link
Collaborator

Jakuje commented Aug 29, 2022

Thank you for the patience! It looks good now.

@mattwillsher mattwillsher merged commit 1731328 into willshersystems:master Aug 29, 2022
@nkakouros nkakouros deleted the drop-in branch August 29, 2022 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants