Bug#497110: boot loader installation failed when dmraid=true

2008-09-13 Thread Frans Pop
# Partly undo previous reshuffle as we want keep the follow-ups to #497110 # on the debian-boot list reopen 497110 reassign 497110 partman-dmraid retitle 497110 boot loader installation failed when dmraid=true severity 497110 serious block 497110 by 498838 # This is now a duplicate, so close close

Bug#497110: boot loader installation failed when dmraid=true

2008-09-13 Thread Frans Pop
# Reopen as the BR was not assigned to parted # and other packages need updating too unblock 497110 by 498838 clone 497110 -1 -2 -3 reassign 497110 parted retitle 497110 Improved support for partitioning dmraid devices severity 497110 wishlist block -1 by 498838 reopen -1 reopen -2 reopen -3 reassi

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Giuseppe Iuculano
Frans Pop ha scritto: > Best thing is probably to add a small helper function to check for > multipath: > > is_dm_multipath() { > dmtype=$(dm_table $device) > [ "$dmtype" = multipath ] || return 1 > } > > and do the following: > device=$(cat $dev/device) > >

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Frans Pop
On Saturday 06 September 2008, Giuseppe Iuculano wrote: > ++#ifdef ENABLE_DEVICE_MAPPER > ++static int _is_dmraid_device (char* devpath); > ++static int _is_dmraid_major(char* devpath); > ++#endif In my cleanup I also added a space after the function name in the 3rd line here (minor consistency i

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Frans Pop
On Saturday 06 September 2008, Giuseppe Iuculano wrote: > Frans Pop ha scritto: > > This should be done by a test _inside_ the if statement and not by > > adding a condition to the if statement: > > > > Also, the comment just above that code should be updated! > > Done. Oops, sorry. The idea is OK

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Otavio Salvador
Giuseppe Iuculano <[EMAIL PROTECTED]> writes: >> P.S. Are you subscribed to the d-boot list or should we keep you CCed? > > > No, I'm not subscribed, please keep me CCed. Please subscribe! You are doing a very good job helping d-i team to get this sorted out and would be awesome to count on you f

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Giuseppe Iuculano
Otavio Salvador ha scritto: > Looks fine however there's a wrong indentation on the static vars to > hold the uuid values. Please take a look and resend. I'll then use > your patch to commit on parted git and prepare an upload. Fixed, previous was not cleaned-up, sorry. Giuseppe diff -u parted-

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Giuseppe Iuculano
Otavio Salvador ha scritto: > Looks fine however there's a wrong indentation on the static vars to > hold the uuid values. Please take a look and resend. I'll then use > your patch to commit on parted git and prepare an upload. Fixed. Giuseppe. reverted: --- parted-1.8.8.git.2008.03.24/.gitigno

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Giuseppe Iuculano
Frans Pop ha scritto: > This should be done by a test _inside_ the if statement and not by adding > a condition to the if statement: > > if echo $device | grep -q "^/dev/mapper/"; then > [ -f "$dev/sataraid" ] || continue > > dmtype=$(dm_table $device) >

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Giuseppe Iuculano
Frans Pop ha scritto: > On Saturday 06 September 2008, Frans Pop wrote: >> At first glance the patch also needs some cleanup. The .gitignore >> changes look like they are just noise and the actual changes are not >> really consistent regarding whitespace for example. > > Here's a cleaned-up versio

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Frans Pop
On Saturday 06 September 2008, Giuseppe Iuculano wrote: > I merged all relative dmraid Ubuntu changes in: Yeah, this really looks like another great example of Ubuntu "giving back" to Debian. > devmapper This looks trivial, but will definitely need an OK from the maintainer and a freeze except

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Otavio Salvador
Giuseppe Iuculano <[EMAIL PROTECTED]> writes: > I merged all relative dmraid Ubuntu changes in: > > devmapper Looks to be safe but requires an OK by maintainer and RT. > parted Looks fine however there's a wrong indentation on the static vars to hold the uuid values. Please take a look and rese

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Frans Pop
On Saturday 06 September 2008, Frans Pop wrote: > At first glance the patch also needs some cleanup. The .gitignore > changes look like they are just noise and the actual changes are not > really consistent regarding whitespace for example. Here's a cleaned-up version. diff -u parted-1.8.8.git.20

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Giuseppe Iuculano
Otavio Salvador ha scritto: > Yes. That is a great news. Sad that Ubuntu hasn't communicated with us > to get this fixed on Debian. I'll take a look and prepare a parted > upload with it. > I merged all relative dmraid Ubuntu changes in: devmapper os-prober parted partman-auto partman-base par

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Otavio Salvador
Giuseppe Iuculano <[EMAIL PROTECTED]> writes: > Otavio Salvador ha scritto: > >> We have two possible way to fix that. Change parted to use previous > > I think we have a third possible way, merge some good work from Ubuntu. > > For example, attached debdiff is an attempt to fix (p) naming issue (

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Frans Pop
On Saturday 06 September 2008, Otavio Salvador wrote: > That message looks to be the "Frans mode" to say: "I believe we should > avoid this change at this release stage and try to revert parted > change, leaving the dmraid change for post lenny". No, what I meant was: proposing this change without

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Frans Pop
On Saturday 06 September 2008, Giuseppe Iuculano wrote: > Otavio Salvador ha scritto: > > We have two possible way to fix that. Change parted to use previous > > I think we have a third possible way, merge some good work from Ubuntu. > > For example, attached debdiff is an attempt to fix (p) naming

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Giuseppe Iuculano
Otavio Salvador ha scritto: > We have two possible way to fix that. Change parted to use previous I think we have a third possible way, merge some good work from Ubuntu. For example, attached debdiff is an attempt to fix (p) naming issue (but not fixes OOM, I must make /lib/partman/active_partit

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Otavio Salvador
Luk Claes <[EMAIL PROTECTED]> writes: > Frans Pop wrote: >> On Saturday 06 September 2008, Otavio Salvador wrote: >>> I've done a look on dmraid and it looks simple to change. The attached >>> patch should do that. >> >> Oh, wow! Yes! >> >> Let's make a change that's really dmraid upstream's bus

Bug#497110: boot loader installation failed when dmraid=true

2008-09-06 Thread Luk Claes
Frans Pop wrote: > On Saturday 06 September 2008, Otavio Salvador wrote: >> Frans Pop <[EMAIL PROTECTED]> writes: >>> On Friday 05 September 2008, Otavio Salvador wrote: It is indeed a change on libparted side however this is doing right using the p here. Would be too hard to change partm

Bug#497110: boot loader installation failed when dmraid=true

2008-09-05 Thread Frans Pop
On Saturday 06 September 2008, Otavio Salvador wrote: > Frans Pop <[EMAIL PROTECTED]> writes: > > On Friday 05 September 2008, Otavio Salvador wrote: > >> It is indeed a change on libparted side however this is doing right > >> using the p here. Would be too hard to change partman-dmraid to use > >

Bug#497110: boot loader installation failed when dmraid=true

2008-09-05 Thread Otavio Salvador
Frans Pop <[EMAIL PROTECTED]> writes: > On Friday 05 September 2008, Otavio Salvador wrote: >> It is indeed a change on libparted side however this is doing right >> using the p here. Would be too hard to change partman-dmraid to use >> it? > > The problem is not really partman-dmraid. The problem

Bug#497110: boot loader installation failed when dmraid=true

2008-09-05 Thread Frans Pop
On Friday 05 September 2008, Otavio Salvador wrote: > It is indeed a change on libparted side however this is doing right > using the p here. Would be too hard to change partman-dmraid to use > it? The problem is not really partman-dmraid. The problem is that dmraid itself also does NOT use or ex

Bug#497110: boot loader installation failed when dmraid=true

2008-09-05 Thread Otavio Salvador
Frans Pop <[EMAIL PROTECTED]> writes: > I think I have done what I can here to identify the issues. From here on > it is up to Jérémy and Otavio to actually fix the bugs that have been > identified. > I really do not know at this point whether that means reverting some change > in libparted, or fi

Bug#497110: boot loader installation failed when dmraid=true

2008-09-05 Thread Frans Pop
On Friday 05 September 2008, Giuseppe Iuculano wrote: > This is the ls -la after the first "Write changes in SATA RAID > partitioning to disk" # ls -la /dev/mapper/ > drwxr-xr-x2 root root 160 Sep 5 08:33 . > drwxr-xr-x 10 root root 2840 Sep 5 08:33 .. > crw-rw

Bug#497110: boot loader installation failed when dmraid=true

2008-09-05 Thread Giuseppe Iuculano
Frans Pop ha scritto: > Giuseppe: if you want to work around this bug to test dmraid support, you > can do so by changing /lib/partman/active_partition/25divider/choices > before you start partman: just make that script 'exit 0'. > This would allow you to provide further details on the "p" in creat

Bug#497110: boot loader installation failed when dmraid=true

2008-09-04 Thread Frans Pop
On Thursday 04 September 2008, Frans Pop wrote: > - partman does not build the fields correctly when the first (valid) > option is a divider Right. This problem is in partman-base/lib/base.sh in debconf_select(). This has: descriptions="${descriptions:+${descriptions}, }$( echo "${x#

Bug#497110: boot loader installation failed when dmraid=true

2008-09-04 Thread Frans Pop
On Tuesday 02 September 2008, Giuseppe Iuculano wrote: > Done, here the logs > http://sd6.iuculano.it/dmraid-testing/log3.tar.gz The last thing the log shows is that the "active_partition" dialog is being prepared, which has the following "keys": 25divider__divider, 65toggle_bootable_

Bug#497110: boot loader installation failed when dmraid=true

2008-09-02 Thread Giuseppe Iuculano
Frans Pop ha scritto: > Unfortunately these don't help very much. What I think we need here is a > full debug log for partman to see what was running at the time of the > OOM. I suspect you somehow get into a loop. > > Please make sure you have started your ssh sessions _before_ starting > par

Bug#497110: boot loader installation failed when dmraid=true

2008-09-01 Thread Jérémy Bobbio
On Sat, Aug 30, 2008 at 11:54:45AM +0200, Frans Pop wrote: > On Saturday 30 August 2008, Giuseppe Iuculano wrote: > > Comments/Problems: > > I used a set of fakeraid qcow2 format disk images > > (http://www.themuso.id.au/ubuntu/dmraid-empty-disks.tar.bz2), but boot > > loader installation failed. >

Bug#497110: boot loader installation failed when dmraid=true

2008-08-31 Thread Frans Pop
On Saturday 30 August 2008, Giuseppe Iuculano wrote: > - First test > > Partition disk: > http://sd6.iuculano.it/dmraid-testing/partman1-1.png > > Choose free space and create a new partition: > http://sd6.iuculano.it/dmraid-testing/partman1-2.png > http://sd6.iuculano.it/dmraid-testing/partman1-3.

Bug#497110: boot loader installation failed when dmraid=true

2008-08-30 Thread Giuseppe Iuculano
Frans Pop ha scritto: > But another possibility is that you are just not following the correct > procedure. Support for dmraid is far from perfect and things will *only* > work if you follow the documented procedure. > > How did you do the partitioning? Did you *exactly* follow the instructions >

Bug#497110: boot loader installation failed when dmraid=true

2008-08-30 Thread Frans Pop
On Saturday 30 August 2008, Giuseppe Iuculano wrote: > after partitioning: > > ~ # ls -la /dev/mapper > drwxr-xr-x2 root root 140 Aug 30 13:17 . > drwxr-xr-x 10 root root 2820 Aug 30 13:17 .. > crw-rw1 root root 10, 60 Aug 30 13:11 control > brw-

Bug#497110: boot loader installation failed when dmraid=true

2008-08-30 Thread Frans Pop
(No need to CC me on replies.) On Saturday 30 August 2008, Giuseppe Iuculano wrote: > Frans Pop ha scritto: > > First thing that needs to be determined is what exactly has changed > > to add this "p". Is it a change in dmraid, or in libparted, or > > elsewhere? When we know that we can check where

Bug#497110: boot loader installation failed when dmraid=true

2008-08-30 Thread Giuseppe Iuculano
Frans Pop ha scritto: > First thing that needs to be determined is what exactly has changed to add > this "p". Is it a change in dmraid, or in libparted, or elsewhere? > When we know that we can check where changes are needed. > Do you have any idea yourself where this "p" comes from? I don't kno

Bug#497110: boot loader installation failed when dmraid=true

2008-08-30 Thread Frans Pop
reassign 497110 partman-dmraid severity 497110 serious thanks On Saturday 30 August 2008, Giuseppe Iuculano wrote: > Comments/Problems: > I used a set of fakeraid qcow2 format disk images > (http://www.themuso.id.au/ubuntu/dmraid-empty-disks.tar.bz2), but boot > loader installation failed. > It se

Bug#497110: boot loader installation failed when dmraid=true

2008-08-29 Thread Giuseppe Iuculano
Package: installation-reports Severity: normal -- Package-specific info: Boot method: CD Image version: http://cdimage.debian.org/cdimage/daily-builds/daily/20080827-1/i386/iso-cd/debian-testing-i386-businesscard.iso This build finished at Wed Aug 27 07:45:39 UTC 2008. Date: Wed Aug 27 07:45: