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

Setup controller #69

Merged
merged 10 commits into from
Jul 3, 2024
Merged

Setup controller #69

merged 10 commits into from
Jul 3, 2024

Conversation

hellais
Copy link
Member

@hellais hellais commented Jul 3, 2024

This contains the local playbook to be executed on the controller host to setup all the user accounts and install all the required packages.

The ssh_users role we should then declare it in the meta as a dep for any other playbook we run on remote hosts so we can always ensure everyone has access to hosts they should have access to.

This is the last piece needed to close: #67

@hellais hellais requested a review from DecFox July 3, 2024 09:58
Copy link

github-actions bot commented Jul 3, 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 @hellais
Action pull_request
Working Directory
Workflow .github/workflows/check_ansible.yml
Last updated Wed, 03 Jul 2024 10:14:31 GMT

@hellais
Copy link
Member Author

hellais commented Jul 3, 2024

I realized now that I am missing 2 critical pieces:

  • Add support for passwordless sudo for all admin users
  • Clone the devops repo in a directory that is readable by all admin users instead of the ubuntu home

@hellais
Copy link
Member Author

hellais commented Jul 3, 2024

Ok it's now done.

Copy link
Contributor

@DecFox DecFox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@hellais
Copy link
Member Author

hellais commented Jul 3, 2024

This is deployed, so I am merging.

@hellais hellais merged commit ccd60ac into main Jul 3, 2024
2 checks passed
@hellais hellais deleted the setup-controller branch July 3, 2024 12:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a control node for running ansible
3 participants