What | Removed | Added |
---|---|---|
CC | fbui@suse.com | |
Flags | needinfo?(igonzalezsosa@suse.com) |
(In reply to Imobach Gonzalez Sosa from comment #2) > > Pay attention to this line: "loaded (/etc/init.d/named; bad; vendor preset: > disabled)". So instead of "enabled" or "disabled", we get a "bad". > That's indeed a shortcoming that has been fixed by upstream commit a3c4eb07106b29, which is included in v230+. This means that the pending systemd update (v232) for factory will "fix" the description of the unit state ("bad"). But the change won't fix this particular bug, see below. @Imobach, it seems that yast is parsing the output and of "systemctl status" and expect to either find "enabled" or "disabled". If it's the case, this doesn't sound correct as: 1. there's no guaranty that the output of "systemctl status" will be stable. For example with v232 (which has the "fix") you'll see "generated" (instead of "bad"). 2. systemd provides a real (D-BUS) API to query the state of a unit file. 3. if 2. is not an option you still might rely on "systemctl is-enabled", which seems more appropriate and has at least a well defined behavior. So IMHO, a better fix would be to change yast to do the right thing. WDYT ?