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

Make agent and server groups configurable #331

Merged
merged 2 commits into from
Jun 4, 2024
Merged

Conversation

simagick
Copy link
Contributor

@simagick simagick commented Jun 1, 2024

Changes

Inventory group names server and agent are configured through server_group and agent_group variables.

The use of hard-coded group names causes issues when managing multiple clusters within the same inventory. When the control plane is initialized each server node connects to the first member of the server group, and therefore all members of each server group must belong to the same cluster.

This change allows more flexibility in how operators manage their inventory. A configurable agent group is added for completeness.

Linked Issues

#327

roles/k3s_upgrade/tasks/main.yml Outdated Show resolved Hide resolved
@dereknola
Copy link
Member

Lint is failing for unrelated reason, I'll handle in another PR

Co-authored-by: Derek Nola <[email protected]>
Signed-off-by: Meagan Harris <[email protected]>
@dereknola dereknola merged commit 006653f into k3s-io:master Jun 4, 2024
1 check failed
steevi pushed a commit to steevi/k3s-ansible that referenced this pull request Jul 2, 2024
* Make agent and server groups configurable

Signed-off-by: Meagan Harris <[email protected]>

* Fix typo in upgrade role

Co-authored-by: Derek Nola <[email protected]>
Signed-off-by: Meagan Harris <[email protected]>

---------

Signed-off-by: Meagan Harris <[email protected]>
Signed-off-by: Meagan Harris <[email protected]>
Co-authored-by: Derek Nola <[email protected]>
andygodish pushed a commit to andygodish/k3s-ansible that referenced this pull request Aug 9, 2024
* Make agent and server groups configurable

Signed-off-by: Meagan Harris <[email protected]>

* Fix typo in upgrade role

Co-authored-by: Derek Nola <[email protected]>
Signed-off-by: Meagan Harris <[email protected]>

---------

Signed-off-by: Meagan Harris <[email protected]>
Signed-off-by: Meagan Harris <[email protected]>
Co-authored-by: Derek Nola <[email protected]>
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