[Pkg-ganeti-devel] Bug#674033: ganeti2: Some disk operations broken in squeeze

[Justin Azoff]

Package: ganeti2
Version: 2.1.6-1+squeeze1
Severity: important

I ran into this a while ago but forgot to follow up by reporting a bug after
fixing all the damage..

I had upgraded to ganeti2 and then tried converting some plain disk
types to drbd, and everything broke:

2011-12-30 10:39:37,045: ganeti-masterd pid=2140/JobQueue22 INFO Op 1/1: Starting opcode CLUSTER_REPAIR_DISK_SIZES
2011-12-30 10:39:37,104: ganeti-masterd pid=2140/ClientReq13 INFO Received job poll request for 4053
2011-12-30 10:39:37,241: ganeti-masterd pid=2140/JobQueue22 INFO Disk 0 of instance FOO has mismatched size, correcting: recorded 1024, actual 0
2011-12-30 10:39:37,270: ganeti-masterd pid=2140/ClientReq13 INFO Received job poll request for 4053
2011-12-30 10:39:37,332: ganeti-masterd pid=2140/JobQueue22 WARNING Disk 1 of instance FOO did not return valid size information, ignoring

Then when the disk conversion failed things like this started happening:

2011-12-30 10:40:11,046: ganeti-masterd pid=2140/JobQueue19 WARNING Could not prepare block device sda on node BAR (is_primary=False, pass=1): Error while assembling disk: Can't activate lv /dev/xenvg/0c44e017-a28f-4e56-a6bb-990ce83c2116.sda:   One or more specified logical volume(s) not found.

The fix is already in the upstream version, backporting it onto the
stable package makes everything happy:

http://git.ganeti.org/?p=ganeti.git;a=commitdiff;h=e50d88078e1dbfe3d78aa174b760aa6142f54b6c

    commit e50d88078e1dbfe3d78aa174b760aa6142f54b6c
    Author: Iustin Pop <iustin at google.com>
    Date:   Tue Feb 15 14:39:44 2011 +0100

        Fix LUClusterRepairDiskSizes and rpc result usage
        
        This LU was introduced before the RPC result conversion from .data to
        .payload, and it has managed to keep the old-style usage (how? it's
        the only LU that does so). Fix by changing to payload, and add some
        extra logging for easier diagnose.
        
        Signed-off-by: Iustin Pop <iustin at google.com>
        Reviewed-by: Stephen Shirley <diamond at google.com>
        Reviewed-by: Michael Hanselmann <hansmi at google.com>
        (cherry picked from commit 043beb38f4e10b75d0820c361c668c441c7a6980)


I only ran into this when I tried converting from plain to drbd, so it's
possible that most people will never have this problem.  2.1.x is also fairly
old at this point, but it is the version in stable..

-- System Information:
Debian Release: 6.0.5
  APT prefers stable
  APT policy: (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/2 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-- 
-- Justin