[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#389570: marked as done (Incomplete patch to add dmraid support)

Your message dated Thu, 05 Jul 2007 22:47:10 +0000
with message-id <E1I6a6Q-0005MD-AQ@ries.debian.org>
and subject line Bug#389570: fixed in partman-base 109
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)
--- Begin Message --- 
Package: partman-base
Version: 94
Severity: wishlist

Hi!

I have tried to improve dmraid support in d-i during the last few days.
What has been done until now:
  * improve the dmraid package [1] to fix #374480 and #375413,
  * add dmraid support in disk-detect (see #338719).

Attached is a patch that is a small improvement of partman-base support
for dmraid.  It only modify definitions.sh:humandev() to add support for
"mirror" devmapper devices.

Other needed changes in partman-base seems mostly related to device
detection (in init.d/parted) where disks under dmraid should be
filtered out of the listed devices.

Other missing changes in the debian-installer are:
  * debian-installer-utils/list-devices (and maybe udevinfo)
      Detection for disks and partitions should be added + filtering of
      devices under dmraid.
  * grub-installer (and possibly lilo-installer).

[1] http://lists.alioth.debian.org/pipermail/utnubu-maintainers/2006-September/000318.html

Regards,
-- 
Jérémy

Index: debian/partman-base.templates
===================================================================
--- debian/partman-base.templates	(revision 40924)
+++ debian/partman-base.templates	(working copy)
@@ -285,6 +285,10 @@
 Type: text
 _Description: Encrypted volume (%s)
 
+Template: partman/text/dmraid_volume
+Type: text
+_Description: Software RAID %s (%s)
+
 Template: partman/text/lvm_lv
 Type: text
 _Description: LVM VG %s, LV %s
Index: definitions.sh
===================================================================
--- definitions.sh	(revision 40924)
+++ definitions.sh	(working copy)
@@ -703,6 +703,13 @@
 	        mapping=${1#/dev/mapper/}
 	        db_metaget partman/text/dmcrypt_volume description
 	        printf "$RET" $mapping
+	    elif [ $type = mirror ]; then
+	        device=${1#/dev/mapper/}
+		name=$(echo "$device" | cut -d_ -f2)
+		desc=$(dmraid -s -c -c "$device")
+		type=$(echo "$desc" | cut -d: -f4)
+	        db_metaget partman/text/dmraid_volume description
+		printf "$RET" $name $type
 	    else
 	        # LVM2 devices are found as /dev/mapper/<vg>-<lv>.  If the vg
 	        # or lv contains a dash, the dash is replaced by two dashes.

Attachment: signature.asc
Description: Digital signature


--- End Message ---
--- Begin Message --- 
Source: partman-base
Source-Version: 109

We believe that the bug you reported is fixed in the latest version of
partman-base, which is due to be installed in the Debian FTP archive:

partman-base_109.dsc
  to pool/main/p/partman-base/partman-base_109.dsc
partman-base_109.tar.gz
  to pool/main/p/partman-base/partman-base_109.tar.gz
partman-base_109_amd64.udeb
  to pool/main/p/partman-base/partman-base_109_amd64.udeb
partman-utils_109_amd64.udeb
  to pool/main/p/partman-base/partman-utils_109_amd64.udeb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 389570@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Frans Pop <fjp@debian.org> (supplier of updated partman-base package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Fri, 06 Jul 2007 00:22:23 +0200
Source: partman-base
Binary: partman-base partman-utils
Architecture: source amd64
Version: 109
Distribution: unstable
Urgency: low
Maintainer: Debian Install System Team <debian-boot@lists.debian.org>
Changed-By: Frans Pop <fjp@debian.org>
Description: 
 partman-base - Partition the storage devices (partman) (udeb)
 partman-utils - Utilities related to partitioning (udeb)
Closes: 389570
Changes: 
 partman-base (109) unstable; urgency=low
 .
   [ Otavio Salvador ]
   * Remove partmap on clean.
   * Fix a compilation warning on partmap.c.
 .
   [ Colin Watson ]
   * Use MS-DOS label by default on PS3 systems.
 .
   [ Frans Pop ]
   * Add support for Serial ATA RAID (closes: #389570):
     - recognize and display dmraid devices
     - skip physical disks that are part of a SATA RAID disk
     Thanks to Jérémy Bobbio whose patch was used in the changes.
 .
   [ Updated translations ]
   * Bulgarian (bg.po) by Damyan Ivanov
   * Czech (cs.po) by Miroslav Kure
   * Basque (eu.po) by Piarres Beobide
   * French (fr.po) by Christian Perrier
   * Galician (gl.po) by Jacobo Tarrio
   * Japanese (ja.po) by Kenshi Muto
   * Dutch (nl.po) by Frans Pop
   * Punjabi (Gurmukhi) (pa.po) by A S Alam
   * Portuguese (pt.po) by Miguel Figueiredo
   * Russian (ru.po) by Yuri Kozlov
   * Ukrainian (uk.po)
   * Vietnamese (vi.po) by Clytie Siddall
   * Simplified Chinese (zh_CN.po) by Ming Hua
Files: 
 24f7f5c895c4fd370b01f4abaedd0553 782 debian-installer standard partman-base_109.dsc
 8c99a9d1ae2267142d5c6592f62c7de0 177691 debian-installer standard partman-base_109.tar.gz
 648d99a097c0869bd4e998b3d2e18b43 163366 debian-installer standard partman-base_109_amd64.udeb
 14fbcd5ee0e363f158487746ad0cc147 3536 debian-installer extra partman-utils_109_amd64.udeb
Package-Type: udeb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)

iD8DBQFGjXJigm/Kwh6ICoQRAlfSAJ948221ONnFr60qP6fxpmFMfo9qTACbBrrm
GJN8tTKHFAjmoCj4Py47Xjc=
=WAtO
-----END PGP SIGNATURE-----

--- End Message ---
Reply to: