search

quattor / ncm-cdispd

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

add unitfile for ncm-cdispd, remove initscripts/chkconfig support #61

Open stdweird opened 5 months ago

stdweird commented 5 months ago

Add unitfile for ncm-cdispd.

Possible issue: the configfile options "--quiet/-D" should not be set; and pidfile and logfile options are ignored

jouvin commented 4 months ago

@stdweird I open a PR on your PR to remove references to chkconfig that requires to define chkconfig as dependency for ncm-cdispd, as it is a source of problem on EL9. This breaks the compatibility with EL6 but it is also the case in cdp-listend (https://github.com/quattor/cdp-listend/pull/33).

In addition to the potential chicken&egg problem with chkconfig packages if it is not installed first, these changes fix the boot ordering issue between ncm-cdispd and NetworkManager (see https://github.com/quattor/configuration-modules-core/issues/1671). The new version has been tested successfully at IJCLab (EL9).

I let you fix the unit test problems that are not introduced by my changes.

jouvin commented 4 months ago

@stdweird please could you merge my PR to your branch so that we have a branch with everything. This is really required for EL9 support.

ned21 commented 4 months ago

Not sure it's completely accurate but adding the backwards incompatible label to highlight in the release notes that this drops EL6 support.

jouvin commented 1 month ago

@stdweird can we fix the unit tests? I've the feeling the error is caused by your changes but I have not checked carefully... :-)