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

Initial work on hiera-ifying and Puppet 8 support for standalone etcd #66

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

ananace
Copy link
Member

@ananace ananace commented Jul 10, 2023

The code is remarkably ugly, but it should allow etcd to still be installed both as part of the control-plane as well as on standalone nodes - and still participate in the same auto-configured cluster through PuppetDB.

Probably quite closely related with #61

Copy link
Sponsor Member

@rwaffen rwaffen left a comment

Choose a reason for hiding this comment

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

except my one question, LGTM.

manifests/server/etcd.pp Outdated Show resolved Hide resolved
Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

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

I am always concerned by hiera usage beyond system-dependent default values for automatic parameter lookup 🤔.

data/common.yaml Outdated Show resolved Hide resolved
REFERENCE.md Outdated Show resolved Hide resolved
manifests/server/etcd.pp Outdated Show resolved Hide resolved
@ananace ananace requested review from smortex and rwaffen July 14, 2023 14:09
The code is ugly, but should mean that etcd can still be installed both as
part of the control-plane as well as on standalone nodes
For ease of understanding, it also accepts control-plane to describe a
K8s server - since that's the name that's used internally in Kubernetes
to describe the function of such a node.
data/common.yaml Outdated Show resolved Hide resolved
manifests/init.pp Outdated Show resolved Hide resolved
manifests/node.pp Outdated Show resolved Hide resolved
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