Bug 807507

Summary: yast2-nis-client doesn't handle systemd service files propery
Product: [openSUSE] openSUSE 12.3 Reporter: Ludwig Nussel <lnussel>
Component: YaST2Assignee: Lukas Ocilka <locilka>
Status: RESOLVED FIXED QA Contact: Jiri Srain <jsrain>
Severity: Major    
Priority: P5 - None CC: heiko.rommel, jsuchome, roger.oberholtzer, tgoettlicher
Version: RC 2   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: --- Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: patch

Description Ludwig Nussel 2013-03-05 14:11:31 UTC
yast nis doesn't handle ypbind and autofs scripts properly as it still reads sysv init script info to determine whether the scripts are enabled.
Comment 1 Ludwig Nussel 2013-03-05 14:12:09 UTC
Created attachment 528198 [details]
patch
Comment 2 Ludwig Nussel 2013-03-05 14:12:30 UTC
small correction, the bug is in yast-yast2 Service.ycp
Comment 3 Thomas Göttlicher 2013-03-06 09:10:47 UTC
Ludwig, thanks for the patch.

Pull request is: https://github.com/yast/yast-yast2/pull/52
Comment 4 Thomas Fehr 2013-03-12 11:16:41 UTC
Patch seems to be added to repo setting to fixed.
Comment 5 Ludwig Nussel 2013-03-12 12:23:39 UTC
I don't see an entry from the bug bot. Did anyone submit an actual update for 12.3?
Comment 6 Thomas Fehr 2013-03-12 12:51:17 UTC
No but the fix will show up in upcoming openSUSE 12.4
Comment 7 Ludwig Nussel 2013-03-12 13:43:13 UTC
the fix is needed in 12.3
Comment 8 Thomas Fehr 2013-03-12 13:48:00 UTC
Reassigned to Lukas since there is not maintainer of package yast2.rpm
Comment 9 Ludwig Nussel 2013-03-15 14:02:15 UTC
The pull request above is incomplete btw. It misses the actual fix.

The Info function needs to be removed in Factory. The warning is just cosmetics.
Comment 10 Ludwig Nussel 2013-03-15 14:46:35 UTC
forget about the first sentence in comment#9, github web interface fooled me.
Comment 11 Jiří Suchomel 2013-03-15 15:14:40 UTC
Should we release update of yast2.rpm for 12.3?
Comment 12 Jiří Suchomel 2013-03-15 15:38:57 UTC
Ludwing, does it actually make sense to use Service::Info or Service::FullInfo now?

They are used at several places from other YaST modules...
Comment 13 Jiří Suchomel 2013-03-15 16:13:06 UTC
Package for testing:

https://build.opensuse.org/package/show?package=yast2&project=home%3Ajsuchome%3A12.3
Comment 14 Ludwig Nussel 2013-03-18 12:22:37 UTC
*** Bug 804894 has been marked as a duplicate of this bug. ***
Comment 15 Jiří Suchomel 2013-03-18 12:39:48 UTC
> osc maintenancerequest home:jsuchome:12.3 yast2 openSUSE:12.3
Using target project 'openSUSE:Maintenance'
159793
Comment 16 Benjamin Brunner 2013-03-18 13:36:04 UTC
Thanks for your submission. I started an update for openSUSE 12.3. See openSUSE:Maintenance:1461.
Comment 17 Jiří Suchomel 2013-03-18 13:44:55 UTC
So I think we can close the bug
Comment 18 Swamp Workflow Management 2013-03-25 11:11:56 UTC
openSUSE-RU-2013:0533-1: An update that has one recommended fix can now be installed.

Category: recommended (moderate)
Bug References: 807507
CVE References: 
Sources used:
openSUSE 12.3 (src):    yast2-2.23.23-1.4.1
Comment 19 Bernhard Wiedemann 2013-05-14 09:00:09 UTC
This is an autogenerated message for OBS integration:
This bug (807507) was mentioned in
https://build.opensuse.org/request/show/175523 Factory / yast2-tftp-server
Comment 20 Lukas Ocilka 2013-06-11 07:09:07 UTC
*** Bug 808317 has been marked as a duplicate of this bug. ***