ncm-chkconfig: add chkconfig function to enable/disable services #1155
Conversation
|
i'm really fed up with the usual way of enabling services. |
stdweird
force-pushed
the
chkconfig_default_enable
branch
from
05a4f37 to
479dc3c
Compare
| @{Enables (on + startstop=true) all arguments as services. | ||
| If argument starts with a -, it is disabled (off + startstop=true). | ||
| Use as e.g. | ||
| "/software/components" = chkconfig("service1", "-service2"); |
There was a problem hiding this comment.
Why is it necessary to escape the = ?
There was a problem hiding this comment.
pan bug quattor/pan#127
| if (m[1] == '-') { | ||
| what = 'off'; | ||
| }; | ||
| SELF['chkconfig']['service'][escape(m[2])] = dict(what, '', 'startstop', true); |
There was a problem hiding this comment.
Can you operate a level lower in the tree? This will cause an error if any part of /software/components is marked final since the compiler cannot tell that you are only modifying the chkconfig subtree.
There was a problem hiding this comment.
i don't understand what you mean here. but if you want
"/software/components/chkconfig" = chkconfig(...);we need a new function name. i don't like redundancy in names
There was a problem hiding this comment.
i opened quattor/pan#164
There was a problem hiding this comment.
I doubt the PAN bug is going to get fixed in time for 17.8 so how do we proceed? I can't think of a better name for the function than chkconfig()
|
As discussed at workshop, fixing quattor/pan#166 solves this in a better way so closing won't fix. |
No description provided.