[Bug 938951] New: YaST installer tries to enable ntp even if it is not installed.
http://bugzilla.suse.com/show_bug.cgi?id=938951 Bug ID: 938951 Summary: YaST installer tries to enable ntp even if it is not installed. Classification: openSUSE Product: openSUSE Factory Version: 201505* Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: Installation Assignee: yast2-maintainers@suse.de Reporter: cwh@suse.com QA Contact: jsrain@suse.com Found By: --- Blocker: --- Created attachment 641569 --> http://bugzilla.suse.com/attachment.cgi?id=641569&action=edit NTP config error When enabling the option not to install recommended packages, an error pops up at the end of the package installation: Cannot adjust NTP service. (See screenshot) Checking t he logs shows, that ntp is not installed although in the installation workflow the enablement of NTP is a hardcoded step in the workflow. Since a user might want to not install NTP we should consider to check if ntp is there before we try to enable it. OR Require NTP to be installed even in the minimal pattern. A requirement for the yast2 package might not be sufficient since maybe not even yast2 is installed. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=938951 http://bugzilla.suse.com/show_bug.cgi?id=938951#c1 Martin Vidner <mvidner@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |mvidner@suse.com --- Comment #1 from Martin Vidner <mvidner@suse.com> --- As long as NTP is hardcoded in the workflow (where?), we should simply install the needed packages. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=938951 http://bugzilla.suse.com/show_bug.cgi?id=938951#c2 --- Comment #2 from Christopher Hofmann <cwh@suse.com> --- Just tested: Even if I disable saving NTP config and starting NTP as a service or even if I completely disable NTP during insatallation workflow the error about the nonexisting NTP occurs. Means the solution is: If user wants NTP in the installed system enabled (it's currently default) NTP should be installed. If not, yast should not try to enable NTP. -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=938951 http://bugzilla.suse.com/show_bug.cgi?id=938951#c3 --- Comment #3 from Lukas Ocilka <locilka@suse.com> --- (In reply to Christopher Hofmann from comment #2)
Just tested:
Yes, I agree -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=938951 http://bugzilla.suse.com/show_bug.cgi?id=938951#c4 --- Comment #4 from Lukas Ocilka <locilka@suse.com> --- Please use https://github.com/yast/yast-yast2/blob/master/library/general/src/modules/P... and make sure that both cases are handled: 1. User selects NTP to be used -> set required packages 2. User selects not to be used -> empty required packages -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=938951 http://bugzilla.suse.com/show_bug.cgi?id=938951#c5 --- Comment #5 from Gabriele Mohr <gs@suse.com> --- Trello PBI https://trello.com/c/L3fgn9sw/289-5-12-sp1-m-318099-install-the-system-witho.... -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=938951 http://bugzilla.suse.com/show_bug.cgi?id=938951#c6 Martin Vidner <mvidner@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |IN_PROGRESS Assignee|yast-internal@suse.de |mvidner@suse.com --- Comment #6 from Martin Vidner <mvidner@suse.com> --- A fix in review: https://github.com/yast/yast-country/pull/55 (and https://github.com/yast/yast-ntp-client/pull/33 , which can go later) -- You are receiving this mail because: You are on the CC list for the bug.
http://bugzilla.suse.com/show_bug.cgi?id=938951 http://bugzilla.suse.com/show_bug.cgi?id=938951#c7 Martin Vidner <mvidner@suse.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|IN_PROGRESS |RESOLVED Resolution|--- |FIXED --- Comment #7 from Martin Vidner <mvidner@suse.com> --- I used a smaller fix in the end: Git: https://github.com/yast/yast-ntp-client/pull/34 OBS: https://build.opensuse.org/request/show/321244 RPM: yast2-ntp-client-3.1.17 -- You are receiving this mail because: You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com