Bug 468504 - preupgrade fail
Summary: preupgrade fail
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: anaconda
Version: 9
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Anaconda Maintenance Team
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-10-25 11:39 UTC by Alexey Kuznetsov
Modified: 2008-11-04 15:56 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-11-04 15:56:00 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)
error (79.28 KB, image/jpeg)
2008-10-25 11:39 UTC, Alexey Kuznetsov
no flags Details

Description Alexey Kuznetsov 2008-10-25 11:39:45 UTC
Created attachment 321500 [details]
error

i run preupgrate from f9 to f10rawhide and got attached error.

problem happens because i use swap in file located in my file system.

Comment 1 Jeremy Katz 2008-10-26 16:45:21 UTC
What does your /etc/ffstab look like?

Comment 2 Alexey Kuznetsov 2008-10-26 16:47:48 UTC
ah, sorry here is:

[axet@axet-laptop ~]$ cat /etc/fstab 
UUID=17050de4-c9bc-4938-9615-090c227ce8fb /                       ext3    defaults        1 1
tmpfs                   /dev/shm                tmpfs   defaults        0 0
devpts                  /dev/pts                devpts  gid=5,mode=620  0 0
sysfs                   /sys                    sysfs   defaults        0 0
proc                    /proc                   proc    defaults        0 0
/var/swap		swap			swap
[axet@axet-laptop ~]$ ll /var/swap 
-rw-r--r-- 1 root root 2147483648 2008-10-16 01:27 /var/swap
[axet@axet-laptop ~]$

Comment 3 Jeremy Katz 2008-10-26 17:06:28 UTC
Can you add 'updates=http://katzj.fedorapeople.org/updates-468504.cgz' to the kernel command line for preupgrade and see if it fixes things for you?


diff --git a/fsset.py b/fsset.py
index a1fdaef..f55e3cb 100644
--- a/fsset.py
+++ b/fsset.py
@@ -1974,7 +1974,7 @@ MAILADDR root
                         else:
                             continue
                     else:
-                        if anaconda.id.getUpgrade() and not (entry.getLabel() or entry.getUuid()):
+                        if anaconda.id.getUpgrade() and not (entry.getLabel() or entry.getUuid()) and entry.device.getDevice().startswith("/dev"):
                             errStr = _("Error mounting device %s as %s: "
                                        "%s\n\n"
                                        "Devices in /etc/fstab should be specified "

Comment 4 Alexey Kuznetsov 2008-10-26 17:13:01 UTC
In my opinion better to have some function who can test file for it's kind: device or file. Funtion can be more cleanly for source code, then magic if.

Do you still think your solution is good enough?

Comment 5 Jeremy Katz 2008-10-26 17:18:07 UTC
It's sufficient... and if it works, then I'll probably make it a little bit cleaner when really committing it :)  But quick and dirty is better for testing

Comment 6 Alexey Kuznetsov 2008-10-26 18:35:16 UTC
ok as you wish. i will create new vm machine for test, take a while..

Comment 7 Alexey Kuznetsov 2008-10-27 13:17:52 UTC
i can't reproduce it on my virtual machine, do you change something already in repo?

Comment 8 Chris Lumens 2008-11-04 15:56:00 UTC
This should be fixed as of the October 29th build.  Jeremy committed the fix prior to that date.


Note You need to log in before you can comment on or make changes to this bug.