Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.

Bug 722963

Summary: TypeError: %d format: a number is required, not str
Product: [Fedora] Fedora Reporter: John Dulaney <jdulaney>
Component: anacondaAssignee: David Lehman <dlehman>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: unspecified    
Version: rawhideCC: anaconda-maint-list, athmanem, awilliam, bwalker, jdulaney, jonathan, tflink, vanmeeuwen+fedora
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard: anaconda_trace_hash:1d52b7902d8c7a2b9c9dac26308143f9cc0be88dcf0d2083d7f4050106744f7a AcceptedBlocker
Fixed In Version: anaconda-16.15-1 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-10-14 20:11:35 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:
Bug Depends On:    
Bug Blocks: 713568    
Attachments:
Description Flags
Attached traceback automatically from anaconda. none

Description John Dulaney 2011-07-18 15:47:32 UTC
The following was filed automatically by anaconda:
anaconda 16.12 exception report
Traceback (most recent call first):
  File "/usr/lib64/python2.7/site-packages/pyanaconda/storage/partitioning.py", line 1077, in allocatePartitions
    _part.format.maxSize))
  File "/usr/lib64/python2.7/site-packages/pyanaconda/storage/partitioning.py", line 921, in doPartitioning
    allocatePartitions(storage, disks, partitions, free, bootloader=bootloader)
  File "/usr/lib64/python2.7/site-packages/pyanaconda/iw/partition_gui.py", line 1549, in refresh
    doPartitioning(self.storage)
  File "/usr/lib64/python2.7/site-packages/pyanaconda/iw/partition_gui.py", line 1648, in editPartition
    if self.refresh(justRedraw=not actions):
  File "/usr/lib64/python2.7/site-packages/pyanaconda/iw/partition_gui.py", line 1606, in editCB
    self.editPartition(device)
TypeError: %d format: a number is required, not str

Comment 1 John Dulaney 2011-07-18 15:47:41 UTC
Created attachment 513641 [details]
Attached traceback automatically from anaconda.

Comment 2 Adam Williamson 2011-07-18 17:43:13 UTC
can you provide a bit more information about the context in which you encountered this, and which release criterion it infringes? thanks!

Comment 3 David Lehman 2011-07-18 19:11:09 UTC
This occurs when the user tries to create a new partition that is not within the size limits for the type of formatting selected. In this case the formatting is BIOS Boot, which must be between 512KB and 2MB in size. I have posted a simple patch for review.

Comment 4 John Dulaney 2011-07-18 20:06:51 UTC
@Adam, I'm going to say 7 because it kills the install.  It is also a very easily reproachable bug, which means that *random user* could cause it.

@David, This is what happens when I am not quite paying full attention to what I'm doing.  However, it needs to not cause Anaconda to crash.

Comment 5 Tim Flink 2011-07-21 15:15:47 UTC
I'm not sure that this hits any of the alpha release criteria. I think that it definitely hits the final criterion:

The installer must be able to create and install to any workable partition layout using any file system offered in a default installer configuration, LVM, software, hardware or BIOS RAID, or combination of the above

The closest alpha criterion that I'm seeing (assuming that the default partition layout doesn't hit this) would be:

The installer must be able to complete an installation using the entire disk, existing free space, or existing Linux partitions methods, with or without encryption enabled 

But I do think that using that criterion is a bit of a stretch.

Unless I'm missing something here, I'm thinking -1 alpha blocker and +1 final blocker. It sounds like a fix will be in shortly, though and this will be closed well before final.

Comment 6 Adam Williamson 2011-07-21 16:16:14 UTC
If this requires a custom layout, yes, it's not Alpha. The criterion you cite is pretty straightforward: those are actual options in the installer. So it's easy enough to check if any of them hits this bug. Just try doing and install and using those options - *not* the custom layout option.

Comment 7 Tim Flink 2011-07-22 20:44:40 UTC
Discussed in the 2011-07-22 blocker bug review meeting. We agreed that this does not hit any of the alpha release criteria and is rejected as an alpha blocker.

However, it does hit the final release criterion:

The installer must be able to create and install to any workable partition
layout using any file system offered in a default installer configuration, LVM,
software, hardware or BIOS RAID, or combination of the above

As such, it has been accepted as a final blocker.

Comment 8 David Lehman 2011-08-03 02:23:51 UTC
*** Bug 727688 has been marked as a duplicate of this bug. ***

Comment 9 Tim Flink 2011-09-30 20:50:09 UTC
It's been a while since this bug was reported and there have been some changes in anaconda since then.

Can you still reproduce this with beta?

Comment 10 John Dulaney 2011-09-30 21:34:43 UTC
I'll try when I get home.

Comment 11 Adam Williamson 2011-10-14 18:52:37 UTC
Discussed at 2011-10-14 blocker review. John, can you please confirm this one so we can close it? Thanks!

Comment 12 John Dulaney 2011-10-14 20:11:35 UTC
Oops, forgot to do this.

I can confirm that it is fixed.  Going ahead and closing.