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

Fixes #14193: Remove ncf.conf usage #1407

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
1 change: 0 additions & 1 deletion Makefile
Expand Up @@ -55,7 +55,6 @@ all: rudder-templates-cli.jar test
java -jar rudder-templates-cli.jar --outext .cf --outdir initial-promises/node-server/distributePolicy/1.0/ techniques/system/distributePolicy/1.0/*.st
cp techniques/system/distributePolicy/1.0/*.cf initial-promises/node-server/distributePolicy/1.0/ || true
mkdir -p initial-promises/node-server/distributePolicy/ncf
cp techniques/system/distributePolicy/1.0/rudder-ncf-conf initial-promises/node-server/distributePolicy/ncf/ncf.conf
mkdir -p initial-promises/node-server/distributePolicy/rsyslog.conf
mv initial-promises/node-server/distributePolicy/1.0/rudder-rsyslog-root.cf initial-promises/node-server/distributePolicy/rsyslog.conf/rudder-rsyslog-root.conf
mv initial-promises/node-server/distributePolicy/1.0/rudder-rsyslog-relay.cf initial-promises/node-server/distributePolicy/rsyslog.conf/rudder-rsyslog-relay.conf
Expand Down
3 changes: 3 additions & 0 deletions techniques/system/common/1.0/promises.st
Expand Up @@ -170,6 +170,9 @@ bundle common va
"SUSE" expression => "sles";
"SuSE" expression => "sles";
"suse" expression => "sles";

# We are inside Rudder policies
"rudder_policies" => "any";

# I don't know why, but these classes are not evaluated if they are in rudder_roles
# Policy Server is a machine which delivers promises
Expand Down
5 changes: 0 additions & 5 deletions techniques/system/common/1.0/site.cf
Expand Up @@ -122,8 +122,3 @@ bundle common g
"role_rudder_cfengine_mission_portal" expression => fileexists("${rudder_base}/etc/server-roles.d/rudder-cfengine-mission-portal");
}

# defines the path to the ncf configuration file
bundle common ncf_configuration {
vars:
"ncf_configuration_file" string => "${g.rudder_ncf}/local/ncf.conf";
}
5 changes: 0 additions & 5 deletions techniques/system/distributePolicy/1.0/metadata.xml
Expand Up @@ -61,10 +61,6 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
<FILE name="apache-acl.cf">
<INCLUDED>true</INCLUDED>
</FILE>
<FILE name="rudder-ncf-conf">
<OUTPATH>distributePolicy/ncf/ncf.conf</OUTPATH>
<INCLUDED>false</INCLUDED>
</FILE>
<FILE name="send-clean.sh">
<INCLUDED>false</INCLUDED>
</FILE>
Expand Down Expand Up @@ -99,7 +95,6 @@ along with this program. If not, see <http://www.gnu.org/licenses/>.
<SECTION name="Synchronize files" component="true"/>
<SECTION name="Synchronize resources" component="true"/>
<SECTION name="Send inventories to Rudder server" component="true"/>
<SECTION name="Configure ncf" component="true"/>
<SECTION name="Configure apache ACL" component="true"/>
</SECTIONS>
</TECHNIQUE>
15 changes: 0 additions & 15 deletions techniques/system/distributePolicy/1.0/propagatePromises.cf
Expand Up @@ -64,16 +64,7 @@ bundle agent propagatePromises


files:

root_server::

"${g.rudder_ncf_origin_local}/ncf.conf"
copy_from => copy_digest_without_perms("${sys.workdir}/inputs/distributePolicy/ncf/ncf.conf"),
comment => "Enforce the ncf configuration file",
classes => classes_generic("configure_ncf_config");

(policy_server|role_rudder_relay_promises_only).!root_server.!disable_classic_promises::

"${client_data}" #that's a loop on each files in client_inputs
copy_from => remote("${server_info.policy_server}","${server_data}"),
depth_search => recurse_visible("inf"),
Expand Down Expand Up @@ -143,12 +134,6 @@ bundle agent propagatePromises
classes => if_else("nodeslist_copied", "could_not_copy_nodeslist");

methods:
root_server::
"any" usebundle => rudder_common_reports_generic("DistributePolicy", "configure_ncf_config", "${distribute_policy_common.directiveId}", "Configure ncf", "None", "Configure configuration library");

!root_server::
"any" usebundle => rudder_common_report("DistributePolicy", "result_na", "${distribute_policy_common.directiveId}", "Configure ncf", "None", "Configuration of configuration library is not necessary on non root servers");

# Success if resources are updated or not changed (kept or repaired).
# root server have only nodelist to be updated
# relays have ncf, masterfiles nodelist and shared_files folder to be updated.
Expand Down
24 changes: 0 additions & 24 deletions techniques/system/distributePolicy/1.0/rudder-ncf-conf

This file was deleted.