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

feat: use galaxy module to configure sshd #78

Merged
merged 9 commits into from
Jul 24, 2024
Merged

feat: use galaxy module to configure sshd #78

merged 9 commits into from
Jul 24, 2024

Conversation

DecFox
Copy link
Contributor

@DecFox DecFox commented Jul 21, 2024

This is part of the work being done in #27.

This diff adds support for using the ansible galaxy module ((ansible-sshd){https://github.com/willshersystems/ansible-sshd}) to configure sshd on the ansible-controller and other hosts (all hosts using the ssh_users role)

Copy link

github-actions bot commented Jul 21, 2024

Ansible Run Output 🤖

Ansible Playbook Recap 🔍



Ansible playbook output 📖success

Show Execution

$ ansible-playbook playbook.yml --check --diff -i ../tf/modules/ansible_inventory/inventories/inventory-dev.ini
[WARNING]: provided hosts list is empty, only localhost is available. Note that
the implicit localhost does not match 'all'
[WARNING]: Could not match supplied host pattern, ignoring: clickhouse_servers
[WARNING]: Could not match supplied host pattern, ignoring: monitoring.ooni.org

PLAY [ClickHouse servers] ******************************************************
skipping: no hosts matched

PLAY [Update monitoring config] ************************************************
skipping: no hosts matched

PLAY RECAP *********************************************************************

Pusher @DecFox
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Sun, 21 Jul 2024 13:37:46 GMT

@DecFox
Copy link
Contributor Author

DecFox commented Jul 21, 2024

The diff till commit 831068f has been applied on backend-hel.ooni.org and ams-slack-1.ooni.org.

@DecFox DecFox marked this pull request as ready for review July 22, 2024 06:32
@hellais
Copy link
Member

hellais commented Jul 24, 2024

I will move forward with deploying this onto more hosts and land it into main. Thanks for working on it!

@hellais hellais merged commit d820d3c into main Jul 24, 2024
2 checks passed
@hellais hellais deleted the ssh-users branch July 24, 2024 14:04
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.

2 participants