Skip to content

Adding customizable defaults to kube-vip template. Add support for valueFrom in rke2_kubevip_args. - #393

Merged
MonolithProjects merged 6 commits into
lablabs:mainfrom
rumblpak:kube-vip-bgp-fixes
Jun 10, 2026
Merged

Adding customizable defaults to kube-vip template. Add support for valueFrom in rke2_kubevip_args.#393
MonolithProjects merged 6 commits into
lablabs:mainfrom
rumblpak:kube-vip-bgp-fixes

Conversation

@rumblpak

Copy link
Copy Markdown
Contributor

Description

Adding customizable defaults to the kube-vip template. This allows users that want to disable ARP mode in order to enable BGP mode ease to do so. This also adds support for using valueFrom in rke2_kubevip_args so reflected values can be included.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Small minor change not affecting the Ansible Role code (GitHub Actions Workflow, Documentation etc.)

How Has This Been Tested?

No. (I'm new to ansible development and would be happy to test if someone could link me a document on how to do so)

@MonolithProjects MonolithProjects self-assigned this May 10, 2026
@MonolithProjects MonolithProjects added the enhancement New feature or request label May 10, 2026
Comment thread templates/kube-vip/kube-vip.yml.j2
Comment thread templates/kube-vip/kube-vip.yml.j2
Comment thread templates/kube-vip/kube-vip.yml.j2
Comment thread templates/kube-vip/kube-vip.yml.j2
Comment thread templates/kube-vip/kube-vip.yml.j2
Comment thread templates/kube-vip/kube-vip.yml.j2

@rumblpak rumblpak left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Resolved all issues and merge conflicts based on current main.

@rumblpak
rumblpak requested a review from MonolithProjects May 10, 2026 20:52
Comment thread defaults/main.yml Outdated
Comment thread README.md Outdated
@rumblpak

rumblpak commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

@MonolithProjects Resolved

@MonolithProjects MonolithProjects left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM. Thanks

@MonolithProjects
MonolithProjects merged commit 4c22e4b into lablabs:main Jun 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants