[Bug 395098] New: Clicking on [Back] in Second Stage LDAP conf ( AutoMatic Conf Used) finishes YaST
https://bugzilla.novell.com/show_bug.cgi?id=395098 Summary: Clicking on [Back] in Second Stage LDAP conf (AutoMatic Conf Used) finishes YaST Product: openSUSE 11.0 Version: Beta 3plus Platform: Other OS/Version: Other Status: ASSIGNED Severity: Critical Priority: P5 - None Component: Installation AssignedTo: locilka@novell.com ReportedBy: locilka@novell.com QAContact: jsrain@novell.com CC: jsuchome@novell.com, coolo@novell.com Found By: --- Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag| |SHIP_STOPPER? To put it into a more generic bugreport, ... There are just two clients called in the second stage installation if automatic configuration is selected. They are "automatic_configuration" and "users". If users are called to create a local user, everything is fine as the [Back] button there is disabled but if user selects, for instance, to configure LDAP-auth, LDAP configuration is displayed instead and that one *has* [Back] button enabled. After clicking that button, users module returns handler to the ProductControl YCP module which calls "automatic_configuration" with $["going_back":true]. And because AC doesn't handle `back, it says: nothing to do and returns `auto. And finally, this is the point, there is no other module *before* AC so installation is closed. Current issues and how to fix them: * LDAP configuration in second stage (with AC used) has enabled [Back] button. - I'm not sure if this can be easily fixed is the same client is probably used when AC is not used too and it's hard to distinguish between these cases. On the other hand, it is always called by inst_user so it actually can handle that. * ProductControl doesn't handle the `back (`auto) correctly, if the fist client in the queue is reached, exception should be handled and definitely not by closing YaST. - Either by returning a pop-up with possibilities, the same way it does if a client fails (Retry, Back, Next, ...) but there was a text-freeze already, of course. - Or by just simply rerunning the fist client. Coolo, decision is needed, please :) I'd personally go for the simplest way: rerunning the first-in-queue client. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c1 --- Comment #1 from Lukas Ocilka <locilka@novell.com> 2008-05-28 08:49:06 MDT --- Created an attachment (id=218650) --> (https://bugzilla.novell.com/attachment.cgi?id=218650) Logs -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c2 --- Comment #2 from Lukas Ocilka <locilka@novell.com> 2008-05-28 08:49:43 MDT --- Created an attachment (id=218651) --> (https://bugzilla.novell.com/attachment.cgi?id=218651) Screenshot - LDAP configuration with [Back] enabled -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c3 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |jbohac@novell.com Status|ASSIGNED |NEEDINFO Info Provider| |coolo@novell.com --- Comment #3 from Lukas Ocilka <locilka@novell.com> 2008-05-28 08:52:35 MDT --- BTW: bug originally hit by jbohac (adding to Cc) -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User coolo@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c4 Stephan Kulow <coolo@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEEDINFO |ASSIGNED Info Provider|coolo@novell.com | Flag|SHIP_STOPPER? |SHIP_STOPPER- --- Comment #4 from Stephan Kulow <coolo@novell.com> 2008-05-28 13:02:50 MDT --- I have an even easier solution: fix it for 11.1 Alpha0 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c5 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|Critical |Major Status|ASSIGNED |RESOLVED Component|Installation |Installation Product|openSUSE 11.0 |openSUSE 11.1 Resolution| |LATER Target Milestone|--- |Alpha 0 Version|Beta 3plus |Alpha 0 --- Comment #5 from Lukas Ocilka <locilka@novell.com> 2008-05-28 13:13:18 MDT --- OK, Later -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c6 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |REOPENED Resolution|LATER | --- Comment #6 from Lukas Ocilka <locilka@novell.com> 2008-05-30 03:22:06 MDT --- Reopening, there is a safe fix possible. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c7 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |visnov@novell.com Severity|Major |Critical Status|REOPENED |NEEDINFO Info Provider| |coolo@novell.com Flag|SHIP_STOPPER- |SHIP_STOPPER? --- Comment #7 from Lukas Ocilka <locilka@novell.com> 2008-05-30 04:46:36 MDT --- The current state is: Running workflow: * AC * Users - Either do everything automatically - Or open UI with several possible clients (that are used from several other places too). We can't actually test all clients that they still work the same way if we change LDAP to disable the [Back] button (this seems to be very unsafe) as we can't test all scenarios. What we can do, is to change the AC which currently // before anything AC-related is done if (going_back) -> returns `back (`auto) (exits script) ... and finishes YaST because there is no other client to run else *** proceed with AC *** -> returns `next (`auto) If we want to be sure that rerunning the AC doesn't break anything, we can just return `next instead of `back (when going_back). That would return to the users configuration. Why this is safe? * Users are either called to write everything automatically. (non-interactive) * Or they ask user to fill up configuration and confirm writing. (interactive) After 11.0 it should be fixed as in the initial comment, for now, just return `next instead of `back. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c8 --- Comment #8 from Lukas Ocilka <locilka@novell.com> 2008-05-30 04:50:22 MDT --- Created an attachment (id=219107) --> (https://bugzilla.novell.com/attachment.cgi?id=219107) Proposed patch -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 Stephan Kulow <coolo@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Info Provider|coolo@novell.com |visnov@novell.com -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
https://bugzilla.novell.com/show_bug.cgi?id=395098 User locilka@novell.com added comment https://bugzilla.novell.com/show_bug.cgi?id=395098#c10 Lukas Ocilka <locilka@novell.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|REOPENED |RESOLVED Resolution| |FIXED --- Comment #10 from Lukas Ocilka <locilka@novell.com> 2008-06-02 04:42:12 MDT --- - Return `next when going back to the automatic configuration dialog instead of returning `auto that would finish YaST and never start it again (bnc #395098). - 2.16.49 -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug.
participants (1)
-
bugzilla_noreply＠novell.com