Bug 248085

Summary: liveinst unnecessarily formats rootfs
Product: [Fedora] Fedora Reporter: Jane Dogalt <jdogalt>
Component: anacondaAssignee: Jeremy Katz <katzj>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: low    
Version: 7CC: tao
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2007-07-16 19:54:22 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
INCOMPLETE patch to remove unnecessary format of rootfs in livecd backend
none
patch to remove unnecessary format of rootfs in livecd backend none

Description Jane Dogalt 2007-07-13 05:15:59 UTC 
Description of problem:

During a typical f7livecd install to hard drive, a user would choose a
destination / volume, and choose to format it as ext3.

Anaconda then proceeds to format it, before making that format irrelevant
when it subsequently copies the live ext3 image from live media to the
destination / volume, and then resize2fs it to the full size.

Thus the livecd install process takes much longer than it needs to, depending on
 the size of the volume. 


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:
Comment 1 Jane Dogalt 2007-07-13 10:27:13 UTC 
Created attachment 159146 [details]
INCOMPLETE patch to remove unnecessary format of rootfs in livecd backend

This patch, appears after the most rudimentary of testing, to fix the problem
that caused me to file the bug.  Please review and comment.
Comment 2 Jane Dogalt 2007-07-13 13:08:51 UTC 
Created attachment 159170 [details]
patch to remove unnecessary format of rootfs in livecd backend

Ok, so I jumped the gun on the earlier patch.  This one actually works.  It may
well break a million other things, but at least it does what it's supposed to.

I'd be more than happy fix any coding style or other suggestions.  I'm
definitely not a seasoned python programmer.
Comment 3 Jeremy Katz 2007-07-16 19:54:22 UTC 
I had looked at this prior to F7, but had been ending up with things more
invasive.  While not entirely elegant, it definitely wins on the simple count
and thus I'll apply it for now and then we'll relook at the question as we
rework partitioning.