Rudder

This issue is an obstacle to the merge of initial promises: #2669

The commits were wrongly applied to branch 2.3

Nicolas PERRON wrote:

What should be initial promises for the nodes in common ?
Here are the
[...]

There are the files from the server:
[...]

I suppose that the files mandatory are:

• cfengine_stdlib.cf

• process_matching.cf

• rudder_lib.cf

• site.cf

• update.cf

Be careful, the difference is not just the files presence, but also their contents.

But what's about cf-served.cf ?

I think we decided to not run a server on the nodes until they were accepted, but I'm not sure. In fact, I'm not sure there's any point running a server on the nodes at all, at the moment...

Looking at commit 0c110ab2, I like the change for the definition of the root_server and policy_server classes, but I don't understand why you changed several "any" into "policy_server" in site.cf (the ones in cf-served.cf make sense). Could you explain please?

Jonathan CLARKE wrote:

Looking at commit 0c110ab2, I like the change for the definition of the root_server and policy_server classes, but I don't understand why you changed several "any" into "policy_server" in site.cf (the ones in cf-served.cf make sense). Could you explain please?

In the light of your comment, it appears to be an error. I thought the definitions about /var/rudder/share/${uuid}/promises/shares and /opr/rudder as peculiar to server. But I was wrong, I'll fix it immediately.

I think this task is finished, no?

Jonathan CLARKE wrote:

I think this task is finished, no?

As I said just above, the nova part is not up to date. So we can't consider this task totally done.

Nicolas PERRON wrote:

Jonathan CLARKE wrote:

I think this task is finished, no?

As I said just above, the nova part is not up to date. So we can't consider this task totally done.

Please create a separate ticket for that, in the Normation internal bugtracker (for the Windows plugin) and close this one. I don't want half-done tasks hanging around cluttering up our bugtracker.

All commits that I didn't make look good to me. Nicolas, can you review my commits please?

• your commit inserted a .swp file
• there was a mistake in the bundle sequence which inserted two commas. But I fixed it later.