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

Add role to install clatd #71

Merged
merged 5 commits into from
Sep 5, 2024
Merged

Add role to install clatd #71

merged 5 commits into from
Sep 5, 2024

Conversation

RealOrangeOne
Copy link
Member

Summary

clatd is a magical ✨ tool I don't quite understand which allows our VMs to have outbound IPv4 connectivity, without needing a dedicated address. This avoids the need to have an IPv4 address for the VM. The tool was recommended by Mythic, who also helped with the configuration.

Specifically, this is needed for the Discord bot, since Discord's API is known to be unhappy with IPv6-only connections.

I've intentionally leant on the Makefile in the project, rather than duplicating the dependencies, to allow for simpler upgrades in future.

Code review

Testing

  • applied the configuration locally
  • manually validated the new behaviour

Links

The configuration itself has already been tested on the server.

@RealOrangeOne RealOrangeOne requested a review from a team September 3, 2024 19:23
host_vars/competitorsvcs.studentrobotics.org.yml Outdated Show resolved Hide resolved
roles/clatd/tasks/main.yml Outdated Show resolved Hide resolved
roles/clatd/templates/clatd.conf Outdated Show resolved Hide resolved
roles/clatd/tasks/main.yml Outdated Show resolved Hide resolved
roles/clatd/README.md Show resolved Hide resolved
roles/clatd/tasks/main.yml Outdated Show resolved Hide resolved
Copy link
Member

@sedders123 sedders123 left a comment

Choose a reason for hiding this comment

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

Added a couple of comment suggestions with my understanding of things. I propose we merge so that the Discord bot is unblocked and add additional comments or tweaks in a follow PR as required

roles/clatd/README.md Show resolved Hide resolved
@PeterJCLaw
Copy link
Member

@RealOrangeOne was it intentional to remove the changes we'd added last night (mostly around documenting how this works)?

@RealOrangeOne
Copy link
Member Author

Yes. Much of what was there was either documenting how Ansible and clatd work, or was wrong. I incorperated the relevant changes into my commit, notably moving them to the role's README rather than keeping them in the host_vars file.

Copy link
Member

@PeterJCLaw PeterJCLaw left a comment

Choose a reason for hiding this comment

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

:shipit: Thanks for persevering with this 👍

@RealOrangeOne RealOrangeOne merged commit d143393 into main Sep 5, 2024
1 check passed
@RealOrangeOne RealOrangeOne deleted the clatd branch September 5, 2024 21:37
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.

4 participants