Skip to content

Conversation

wtripp180901
Copy link
Contributor

@wtripp180901 wtripp180901 commented Sep 19, 2024

NB: currently doesn't support multiple control nodes (needs support for multiple k3s servers, some stuff to do that here

Install k3s cluster on nodes with control plane on control node. Adds a k3s role to run at bootstrap which installs k3s, k9s and helm as well as an ansible-init playbook to start each node's k3s service as an agent or server depending on metadata specified in terraform.

@wtripp180901 wtripp180901 requested a review from sjpb September 19, 2024 09:06
@wtripp180901 wtripp180901 requested a review from a team as a code owner September 19, 2024 09:06
@sjpb
Copy link
Collaborator

sjpb commented Sep 19, 2024

@wtripp180901 can you review and merge this into this PR if you're happy pls? Then update the 1st comment above appropriately.

@wtripp180901
Copy link
Contributor Author

@wtripp180901
Copy link
Contributor Author

^ that merge is just updating my local branch with new password role, current images should be fine to merge if no more changes

@wtripp180901 wtripp180901 requested a review from sjpb November 5, 2024 12:52
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

Lets minimise changes to the caas / cluster_infra TF templates, then call it done.

@wtripp180901
Copy link
Contributor Author

wtripp180901 commented Nov 12, 2024

@wtripp180901
Copy link
Contributor Author

@wtripp180901 wtripp180901 requested a review from sjpb November 18, 2024 11:46
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

LGTM but I have a couple of minor amends you can just accept if you are happy, no need to rebuild

Co-authored-by: Steve Brasier <[email protected]>
@wtripp180901 wtripp180901 requested a review from sjpb November 19, 2024 09:30
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

LGTM

@wtripp180901 wtripp180901 merged commit 92f5115 into main Nov 19, 2024
5 checks passed
@wtripp180901 wtripp180901 deleted the feature/k3s-ansible-init branch November 19, 2024 09: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.

2 participants