Franck Bui changed bug 1012047
What Removed Added
CC   fbui@suse.com
Flags   needinfo?(igonzalezsosa@suse.com)

Comment # 4 on bug 1012047 from
(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 ?


You are receiving this mail because: