Bug 158620

Summary: Clicking "OK" without an CD in drive at CD3 causes YaST to not request CD3 but CD5
Product: [openSUSE] SUSE Linux 10.1 Reporter: Holger Sickenberg <holgi>
Component: InstallationAssignee: Holger Sickenberg <holgi>
Status: VERIFIED FIXED QA Contact: Klaus Kämpf <kkaempf>
Severity: Critical    
Priority: P5 - None CC: suse-beta
Version: Beta 8   
Target Milestone: ---   
Hardware: Other   
OS: Other   
Whiteboard:
Found By: Other Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: YaST2.tgz
Proposed patch

Description Holger Sickenberg 2006-03-16 13:22:36 UTC
Durung installation:
when YaST2 asks for CD3 and you do not enter any CD in drive but click on "OK" YaST will NOT ask for CD3 again but for CD4. Clicking "OK" again causes YaST to ask for CD5.
Comment 1 Holger Sickenberg 2006-03-16 13:23:18 UTC
Created attachment 73315 [details]
YaST2.tgz
Comment 2 Lukas Ocilka 2006-03-16 14:25:13 UTC
stano: one for you?
Comment 3 Stanislav Visnovsky 2006-03-20 14:52:46 UTC
Created attachment 73985 [details]
Proposed patch

The problem is that retry tries to attach the media
again and exception is not caught then.

I've removed trying to attach the media, but continue
in the while loop, where MediaSet should try it instead, properly catching exceptions.
Comment 4 Stanislav Visnovsky 2006-03-20 14:54:49 UTC
Fixed in libzypp rev. 2574.

Andreas, this might break the CD handling again, we need to do a bit of QA there to see if retry button works as expected in case of:

1) correct CD in drive
2) incorrect CD in drive
3) no CD in drive.
Comment 5 Stanislav Visnovsky 2006-03-21 12:15:07 UTC
*** Bug 157365 has been marked as a duplicate of this bug. ***
Comment 6 Andreas Jaeger 2006-03-22 19:19:37 UTC
Ok.
Comment 7 Andreas Jaeger 2006-03-23 12:36:59 UTC
Holgi, please test - I'll tell you once we have a build that's worth testing.
Comment 8 Andreas Jaeger 2006-03-27 18:59:49 UTC
Holgi, go ahead with Build 827 or 828!
Comment 9 Andreas Jaeger 2006-03-29 09:46:48 UTC
I assume this is fixed but like to see testing done, so let's lower severity for now.
Comment 10 Holger Sickenberg 2006-04-18 09:51:33 UTC
Works with RC1
Comment 11 Holger Sickenberg 2006-04-18 09:52:51 UTC
Verified with RC1