Skip to content

feat: allow to set node_terminus variable#995

Open
aba-lrettler wants to merge 7 commits into
theforeman:masterfrom
aba-lrettler:feature/server_terminus_variable
Open

feat: allow to set node_terminus variable#995
aba-lrettler wants to merge 7 commits into
theforeman:masterfrom
aba-lrettler:feature/server_terminus_variable

Conversation

@aba-lrettler

@aba-lrettler aba-lrettler commented Jul 20, 2026

Copy link
Copy Markdown

Allow to override default value of [server]/node_terminus in puppet.conf.
Useful in case of standalone (non-Foreman) Puppet Server installs without an ENC.

Copy of: #921
Rebased on to master

d1nuc0m and others added 3 commits July 20, 2026 14:18
Allow to override default value of [server]/node_terminus in puppet.conf
Useful in case of standalone (non-Foreman) Puppet Server installs
without an ENC.
Tests are updated as now without Foreman, node terminus is explicitly set
to the default 'plain' value

Reference
https://www.puppet.com/docs/puppet/8/configuration.html#node-terminus
Co-authored-by: Ewoud Kohl van Wijngaarden <ewoud@kohlvanwijngaarden.nl>
node_terminus should default to exec and write nothing if set to plain
Comment thread manifests/server/config.pp Outdated
enc_path => $server_external_nodes,
}
} else {
class { 'puppet::server::enc':

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this leads to duplicate class declaration

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@bastelfreak fixed.
All tests now passes

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.

3 participants