search

quattor / ncm-cdispd

Node Configuration Manager Configuration Dispatch Daemon
www.quattor.org
Other
2 stars 5 forks source link

unittest default statedir of ncm-cdispd and ncm-ncd #36

Closed stdweird closed 7 years ago

stdweird commented 7 years ago

so they are equal / different, but as intended.

question: what is the intended default (equal or not).

stdweird commented 7 years ago

values are currently only set in the shipped config files. can we move the defaults to the Application defaults?

jouvin commented 7 years ago

March 17 workshop: cdispd has this option only to pass it to ncd. Should probably be removed or at least its default value should be undefined.

jrha commented 7 years ago

It might be useful for some use cases (which should be documented) but the default should be to not pass it to ncm-ncd unless configured (i.e. ncm-ncd should use the value from its config).

ned21 commented 7 years ago

There are a number of other options (such as cache_root and NoAction) to ncm-cdispd which should normally never be changed. However, collectively they are useful for e.g. running a second instance that downloads a "canary" version of the profile and runs it in NoAction mode. So we should leave these options in the code for now.